This is the mail archive of the ecos-patches@sources.redhat.com mailing list for the eCos project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: New port for ARM Industrial Module AIM 711 - Checked by AntiVirDEM


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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]