On Mon, Feb 21, 2011 at 12:44 PM, Olof Johansson <olof@xxxxxxxxx> wrote: > Hi, > > On Mon, Feb 21, 2011 at 11:40 AM, Erik Gilling <konkers@xxxxxxxxxxx> wrote: > >>> This should remove the need for calling these from specific drivers and >>> thus reduce tegra-specific code from them. >> >> It was never my intention that drives call gpio_enable/disable, just board >> files. > > True, but it seems even then that it ends up in various parts of the > per-board files. Having it in one central location (for the base > config) seems like a good idea. sounds good to me. >> Gpio subsystem init and configurations should be left separate. > > I.e. not part of the pinmux file? Sure, can do. I meant that tegra_gpio_init should still be a postcore_initcall. That way GPIOs work even if you don't use the tegra_gpio_config helper. Also, I'd take the __init off tegra_gpio_config. The function could be useful in some suspend/resume chains where the board designer hates you. -Erik -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html