Re: [PATCH 0/2] ARM: dts: omap3-overo: Add pwm-leds and audio support
From: Philippe Rétornaz <hidden>
Date: 2013-02-06 14:26:43
Also in:
linux-arm-kernel, linux-omap
From: Philippe Rétornaz <hidden>
Date: 2013-02-06 14:26:43
Also in:
linux-arm-kernel, linux-omap
Hi all ! Le 06/02/2013 14:30, Benoit Cousson a écrit :
Salut Florian, On 02/04/2013 10:14 AM, Florian Vaussard wrote:quoted
Hello Benoit, On 01/24/2013 01:21 PM, Benoit Cousson wrote:quoted
+ Peter who did the original PWM Hi Florian, On 01/23/2013 06:56 PM, Florian Vaussard wrote:quoted
Hello Benoit, This patchset adds some new DT supports to the Overo products. The first patch converts the PMIC LEDB output to use the pwm-leds, newly merged in your branch for_3.9/dts. The second patch adds the audio support.Excellent, that looks very good to me, but I'd like to get the feedback from Peter before merging it.So a patch is being merged to handle triggers in the case of pwm leds [1]. When done, we will be able to add back the default trigger. Do you want to wait on it to merge this series?What kind of dependency do we have between these two series? I mean what will happen if the DTS is merged before the pwm subsystem? If that does not generate any regression / crash, then it is OK, if not, we should take care of the order.
Florian is on holiday the next two weeks. AFAIK if the PWM/leds subsystem patches are not merged before this one then the kernel will panic if a trigger is used to change the led status. Regards, Philippe