New port for ARM Industrial Module AIM 711 - Checked by AntiVir DEM

[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) }


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