On Fri, Sep 19, 2008 at 05:11:15PM +0200, Peter Korsgaard wrote:

quoted

quoted

quoted

quoted

quoted

"Anton" == Anton Vorontsov [off-list ref] writes:

Hi,

 >> It seems basically the same as my patch (gpio driver for
 >> mpc831x/mpc834x/mpc837x with OF bindings) from September 5th except
 >> you don't have any documentation

 Anton> Well, do we really need documentation for every trivial gpio
 Anton> controller? The bindings are the same, only different compatible
 Anton> entries...

Well, we need some way of knowing that atleast. Having people grepping
through arch/powerpc/boot/dts/ (and knowing that they should look for
8349 even though they might use 8313 or 8610) doesn't seem optimal to
me - I take it that's why we created
Documentation/powerpc/dts-bindings in the first place?

Ok.

 Anton> If we do need the documentation for this, I can readily include it.

 >> and the Kconfig doesn't depend on
 >> 831x/834x/837x.

 Anton> This is purposely. We also need support for 8610, and maybe
 Anton> later we'll find another chip with the same unit. So, to not touch
 Anton> the Kconfig for every new chip I just made it PPC32-wide. Other
 Anton> option is to depend on FSL_SOC, but the driver really does not
 Anton> depend on any fsl_soc stuff...

Adding another symbol to the Kconfig once it is verified that a new
SoC is compatible doesn't seem like a big deal - Figuring out all the
knobs we already have is, without having options for stuff that is
known to be irrelevant for the SoC.

The other 83xx specific drivers also depend on PPC_83xx.

Lets wait for Kumar's comments. We've already had a PPC_* mess
for the USB_EHCI_FSL symbol. What I've learned from it, is that
huge PPC_* list isn't perfect either.

See this discussion:

http://www.mail-archive.com/linux-usb@vger.kernel.org/msg01926.html

And this one:

http://fixunix.com/kernel/397972-patch-1-4-spi-powerpc-spi_mpc83xx-handles-freescale-mpc8610-well.html

 >> Why do you need this when you have the lock?

 Anton> See this discussion:

 Anton> http://www.mail-archive.com/linuxppc-dev@ozlabs.org/msg12299.html

Ahh, for open drain outputs. gpiolib doesn't actually provide an
interface to put the gpios in that mode, but it could ofcause have
been set up in the bootloader. I'll fix that and send an updated
patch.

Pardon? Who will send an updated patch, and what patch exactly? ;-)

-- 
Anton Vorontsov
email: cbouatmailru@gmail.com
irc://irc.freenode.net/bd2