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 DE
- From: Roland Caßebohm <roland dot cassebohm at VisionSystems dot de>
- To: Jonathan Larmour <jifl at eCosCentric dot com>
- Cc: ecos-patches at ecos dot sourceware dot org
- Date: Thu, 29 Apr 2004 10:26:23 +0200
- Subject: Re: New port for ARM Industrial Module AIM 711 - Checked by AntiVir DE
- References: <200403031629.40658.roland.cassebohm@visionsystems.de> <200404271605.38706.roland.cassebohm@visionsystems.de> <4090731C.2060408@eCosCentric.com>
Am Donnerstag, 29. April 2004 05:14 schrieb Jonathan
Larmour:
> Roland Caßebohm wrote:
> > 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?
>
> You mean you have created a separate HAL package just for
> those options? If so, then there's no need. CDL has a way
> of doing what you want in the HAL. I believe this should
> do it:
> requires { is_active(CYGPKG_DEVS_ETH_ARM_KS32C5000)
> implies !CYGPKG_DEVS_ETH_ARM_KS32C5000_PHY_LXT970 }
> requires { is_active(CYGPKG_DEVS_ETH_ARM_KS32C5000)
> implies CYGPKG_DEVS_ETH_ARM_KS32C5000_PHY_RTL8201 }
> requires { is_active(CYGPKG_DEVS_ETH_ARM_KS32C5000)
> implies (CYGPKG_DEVS_ETH_ARM_KS32C5000_PHYADDR == 2) }
Yes it works well.
>
>
> Since Andrew's away right now, let me know when you have
> a new version to look at and I will review it (and
> hopefully commit it :-)) for you.
Yes I think I have every thing together to make a new
version.
Roland
--
___________________________________________________
VS Vision Systems GmbH, Industrial Image Processing
Dipl.-Ing. Roland Caßebohm
Aspelohe 27A, D-22848 Norderstedt, Germany
http://www.visionsystems.de
___________________________________________________