[PATCH v8 3/3] leds/powernv: Add driver for PowerNV platform
hegdevasant at linux.vnet.ibm.com
Tue Jul 28 23:40:46 AEST 2015
On 07/27/2015 03:20 PM, Jacek Anaszewski wrote:
> Hi Vasant,
> On 27.07.2015 05:41, Vasant Hegde wrote:
>> On 07/27/2015 03:11 AM, Jacek Anaszewski wrote:
>>> Hi Vasant,
>> Hi Jacek,
>>> Two trivial details left. Please find them below.
>> Thanks for the review/Ack. I'll fix below issues and resend patchset.
>> I will ask Benh/Michael to take this patchset. But this patchset is depending
>> on your core changes. Can you confirm that you are pushing that patchset in next
>> merge window?
> Without my core changes your driver won't work with led triggers, but
> AFAIR this use case is not relevant for your LEDs? Eventually, we could
> produce a patch set adding support for LED triggers if it will be clear
> that LED core changes will not be merged in the upcoming merge window.
IIUC current LED code doesn't allow me to sleep (without driver specific
workqueue). And powernv_led_set() call will sleep. Hence I think it won't work.
I did a quick test without your patch. It doesn't seems to be working.
Alternatively we can revert the changes (add driver specific workqueue now) and
later when your changes goes to upstream, I can fix my code.
More information about the Linuxppc-dev