Hi Linus, On 11/17/2012 09:16 PM, Linus Walleij wrote: > On Tue, Nov 13, 2012 at 10:35 AM, Peter Ujfalusi <peter.ujfalusi@xxxxxx> wrote: > >> Avoid using the TWL4030_MODULE_PWMA/B as module ID. The LEDEN, PWMA ON/OFF >> and PWMB ON/OFF registers are in a continuous range starting from LED base. >> This is going to be helpful for further cleanup in the twl stack. >> >> No functional changes. >> >> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@xxxxxx> > > OK if you say so. > > Patch applied! Thanks, I was actually tempted to remove the whole LED (PWM) thing from the gpio-twl4030 driver. It was a big surprise to me to see something like this in there. It turns out that on BeagleBoard the USB host enable signal is connected to LEDA (PWMA) of twl4030... It is an enable signal. Seriously. So what we do here is either configure the PWMs as full on, or turn it off. I'm not sure how common is this practice (PWM to drive enable signal) but if we have more than one board out there with this type of connection we might be better to add a 'gpio-pwm.c' driver to handle them. Either way this is wrong IMHO to handle the LEDA/B via the gpio-twl4030 driver. -- Péter -- 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
