On Thu, May 16, 2013 at 10:35 AM, Dan Murphy <dmurphy@xxxxxx> wrote: > I am not sure you really want to do this. > If I make the pinctrl part of the led structure then the only way the gpio_wk7 on a1-a3 to be configured is when > the CONFIG_LEDS_GPIO flag is set. > > Do you really want that dependency? You did say it was a key fix > At least this way the pins are configured regardless of that flag. That is better as the system will be left in the pinmux configuration handed over from bootloader. The point being, muxing up pins even when not needed(config switched off) has no real benefit - in this case albeit, the default mux was causing a bug. pinctrl IMHO should be considered as any other resource, if it is not mandatory for boot, and needed only for a device functionality when probed, it should done there only. just my 2 cents. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html