This is the mail archive of the
ecos-patches@sources.redhat.com
mailing list for the eCos project.
Re: New port for ARM Industrial Module AIM 711 - Checked by AntiVir DEM
- From: Roland Caßebohm <roland dot cassebohm at VisionSystems dot de>
- To: "John Dallaway" <jld at ecoscentric dot com>
- Cc: <ecos-patches at ecos dot sourceware dot org>
- Date: Tue, 27 Apr 2004 16:05:38 +0200
- Subject: Re: New port for ARM Industrial Module AIM 711 - Checked by AntiVir DEM
- References: <200403031629.40658.roland.cassebohm@visionsystems.de>
Hi,
instead of using this commands in the target description of
the AIM 711:
- enable CYGHWR_HAL_ARM_BIGENDIAN
- disable CYGPKG_DEVS_ETH_ARM_KS32C5000_PHY_LXT970
- enable CYGPKG_DEVS_ETH_ARM_KS32C5000_PHY_RTL8201
- set_value CYGPKG_DEVS_ETH_ARM_KS32C5000_PHYADDR 2
I have implemented "requires" contraints in the cdl file of
the hal package. But for ethernet I have made an extra
package "devs/eth/arm/aim711" which then "requires"
CYGPKG_DEVS_ETH_ARM_KS32C5000_PHY_RTL8201 == 1
and CYGPKG_DEVS_ETH_ARM_KS32C5000_PHYADDR == 2,
because if I do this in the hal package I get a conflict if
ethernet support is disabled.
Normaly platforms (snds) that use the
CYGPKG_DEVS_ETH_ARM_KS32C5000
don't have its own ethernet package.
Is that OK?
Roland
Am Mittwoch, 3. März 2004 16:29 schrieb Roland Caßebohm:
> On Mittwoch, 3. März 2004 14:48, John Dallaway wrote:
> > Hi Roland
> >
> > Roland Caßebohm wrote:
> > >>The "enable", "disable" and "set_value" commands
> > >> should be implemented as "requires" contraints in
> > >> CYGPKG_HAL_ARM_AIM711 rather than in ecos.db. It is
> > >> best to avoid these commands wherever possible since
> > >> their implementation is not elegant. For example,
> > >> the corresponding CDL options will be given a "user
> > >> value" rather than an "inferred value".
> > >>
> > >>I will leave comments on the run-time aspects to
> > >> other maintainers.
> > >
> > > How is the best way to change this? Maybe write them
> > > as "required" in the cdl file of the AIM 711 hal?
> >
> > Exactly. The correct values for these CDL options will
> > them be inferred when the target is selected.
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>
> OK, I will make an additional patch which correct this.
> Or is it better to send the whole package new while it is
> not in CVS?
>
> Roland
--
___________________________________________________
VS Vision Systems GmbH, Industrial Image Processing
Dipl.-Ing. Roland Caßebohm
Aspelohe 27A, D-22848 Norderstedt, Germany
Mail: roland.cassebohm@visionsystems.de
http://www.visionsystems.de
___________________________________________________