[PATCH v4] ARM:dts:omap4-panda: Update the LED support for the panda DTS

Nishanth Menon nm at ti.com
Thu May 16 14:18:10 EDT 2013


On Thu, May 16, 2013 at 10:35 AM, Dan Murphy <dmurphy at ti.com> 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.



More information about the linux-arm-kernel mailing list