[U-Boot] [PATCH 1/2] 83xx: Replace CONFIG_MPC83XX with CONFIG_MPC83xx

Kim Phillips kim.phillips at freescale.com
Sat Jun 6 02:44:39 CEST 2009


On Sun, 24 May 2009 22:13:27 +0200
Wolfgang Denk <wd at denx.de> wrote:

> In message <1243192952.703.7.camel at 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


More information about the U-Boot mailing list