[U-Boot] [PATCH 06/14] powerpc: Remove unneccessary #ifdefs in reginfo

Tom Rini trini at konsulko.com
Wed Jul 12 19:56:59 UTC 2017


On Wed, Jul 12, 2017 at 04:53:26PM +0200, Wolfgang Denk wrote:
> Dear Christophe,
> 
> In message <194b25e4-81fc-52cf-aeef-61ce6f4679a7 at c-s.fr> you wrote:
> > 
> > Oh ? Ok. I thought it would be a possible name because for instance in 
> > the Linux Kernel, the watchdog driver is named that way and used also 
> > for the 8xx and so was also the SPI driver before its name was change to 
> > fsl_spi.
> 
> There are always bad examples ;-)
> 
> > Isn't the 8xx an 81xx indeed ?
> 
> I have never seen this naming used; google does not know about it
> either.
> 
> > Any suggestion for a good name ? Would fsl_reginfo() be a good name ?
> 
> FSL does not even exist any more...
> 
> 
> Soory, I was never good in naming things...

FSL doesn't exist, and I'm terrible at naming things as well, but I
think fsl_reginfo makes enough namespace sense, we have a lot of fsl_xxx
in arch/powerpc and qcom_ might start confusing folks more than helping
:)

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170712/0ec30ffb/attachment.sig>


More information about the U-Boot mailing list