[U-Boot] [PATCH 3/5] ppc4xx: Move gpio.h to ppc4xx-gpio.h since its ppc4xx specific
Stefan Roese
sr at denx.de
Thu Sep 16 15:47:00 CEST 2010
Hi Detlev,
On Thursday 16 September 2010 15:00:05 Detlev Zundel wrote:
> > Please note that I plan to stay with this "asm/ppc4xx-gpio.h" name
> > instead of "asm/gpio.h", since there are many other PowerPC variants out
> > there.
>
> I have to admit that I don't like this.
>
> Hm, while you are doing something here, why not follow the Blackfin
> example which I like a lot and move towards the Linux gpio interface
> (see end of ./arch/blackfin/include/asm/gpio.h). It will be just lovely
> to only generically request a gpio in common driver code.
>
> Going down this route, code using gpios will only include "asm/gpio.h"
> and use the generic functions. Doesn't that sound like a worthwhile
> target? ;)
Sure. I'm all for this idea, all platforms implementing this common GPIO
framework. The current 4xx GPIO implementation out-dates the Linux framework,
since I added this quite some time ago. Perhaps I'll find some time in the
future to move to this common platform for PPC4xx as well. I won't complain if
somebody beats me at though. ;)
Back to the subject: This patch right now was simply meant as part of a header
cleanup patch, moving a 4xx specific header with stuff like this:
/* GPIO controller */
struct ppc4xx_gpio {
u32 or; /* Output Control */
u32 tcr; /* Tri-State Control */
...
to a "better" name. Even though the common gpio framework should be the "final
target", I think it makes sense to make this header name move right now.
Cheers,
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: office at denx.de
More information about the U-Boot
mailing list