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 AntiVirDEM
- From: Jonathan Larmour <jifl at eCosCentric dot com>
- To: Roland Caßebohm <roland dot cassebohm at VisionSystems dot de>
- Cc: ecos-patches at ecos dot sourceware dot org
- Date: Thu, 29 Apr 2004 04:14:36 +0100
- Subject: Re: New port for ARM Industrial Module AIM 711 - Checked by AntiVirDEM
- References: <200403031629.40658.roland.cassebohm@visionsystems.de> <200404271605.38706.roland.cassebohm@visionsystems.de>
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) }
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.
Jifl
--
eCosCentric http://www.eCosCentric.com/ The eCos and RedBoot experts
--["No sense being pessimistic, it wouldn't work anyway"]-- Opinions==mine