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

2017-07-13 Thread Christophe LEROY
Le 12/07/2017 à 21:56, Tom Rini a écrit : On Wed, Jul 12, 2017 at 04:53:26PM +0200, Wolfgang Denk wrote: Dear Christophe, In message <194b25e4-81fc-52cf-aeef-61ce6f467...@c-s.fr> you wrote: Oh ? Ok. I thought it would be a possible name because for instance in the Linux Kernel, the watchdog

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

2017-07-12 Thread Tom Rini
On Wed, Jul 12, 2017 at 04:53:26PM +0200, Wolfgang Denk wrote: > Dear Christophe, > > In message <194b25e4-81fc-52cf-aeef-61ce6f467...@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

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

2017-07-12 Thread Wolfgang Denk
Dear Christophe, In message <194b25e4-81fc-52cf-aeef-61ce6f467...@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 wa

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

2017-07-12 Thread Christophe LEROY
Dear Wolfgang, Le 12/07/2017 à 15:07, Wolfgang Denk a écrit : Dear Christophe Leroy, In message you wrote: reginfo command is calling mpc8xx_reginfo(), mpc85xx_reginfo() or mpc86xx_reginfo() based on CONFIG_ symbol. As those 3 functions can't me defined at the same time, let's rename them m

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

2017-07-12 Thread Wolfgang Denk
Dear Christophe Leroy, In message you wrote: > reginfo command is calling mpc8xx_reginfo(), mpc85xx_reginfo() > or mpc86xx_reginfo() based on CONFIG_ symbol. > As those 3 functions can't me defined at the same time, let's > rename them mpc8xxx_reginfo() to avoid the #ifdefs This is indeed a te

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

2017-07-12 Thread Christophe Leroy
reginfo command is calling mpc8xx_reginfo(), mpc85xx_reginfo() or mpc86xx_reginfo() based on CONFIG_ symbol. As those 3 functions can't me defined at the same time, let's rename them mpc8xxx_reginfo() to avoid the #ifdefs Lets all remove the #ifdefs around the U_BOOT_CMD as this file is only compi