On Tuesday 05 February 2008, Stefan Roese wrote: > > Therefore I concluded that it is very PPC405GPr specific. But I did not > > check all other PPC405 variants. > > Yes, you seem to be right here. This *is* 405GPr specific. But I don't > think it is worth adding this CONFIG_405GPr. Right now by setting > CONFIG_405GP in your board config file, it should work correctly on boards > with 405GP *and* 405GPr. We shouldn't loose this "feature". So please add > this define if CONFIG_405GP is defined. And add a comment that it is only > valid for 405GPr.
And please use the upper case name from the users manual: CPC0_EIRR. I know this is not done consistently but I we should use these names for newer definitions since it makes it easier to match those defines with the manuals. Thanks. Best regards, Stefan ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] ===================================================================== ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ U-Boot-Users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/u-boot-users
