On Sun, 24 May 2009 22:13:27 +0200 Wolfgang Denk wd@denx.de wrote:
In message 1243192952.703.7.camel@ptyser-laptop you wrote:
CodingStyle Ch.12 suggests that it's the other fsl arches that are in violation here, not 83XX:
Agreed...
Either way is fine with me, as long as things are consistent. It looked like most Freescale CPUs in U-Boot as well as Linux used the lowercase convention for CONFIG_<cpu type> so I thought: a. Most likely someone made a conscious decision to use lowercase b. It'd be much easier to change just 83xx than all other arches:)
If others would prefer standardizing on uppercase, I can submit new patches for 85xx, 5xxx, 86xx, etc. This would be still be inconsistent with Linux though, which is a bit annoying.
It seems Linux uses 8?xx with very few exceptions (CONFIG_SND_VIA82XX* and CONFIG_EDAC_MPC85XX), so let's do the same here.
applied 1-2 to u-boot-mpc83xx's next branch.
reluctantly, that is. I just don't have the time to fix it myself right now.
Kim