Re: [PATCH 2/5] leds: leds-mc13783: Use LED core PM functions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Dec 16, 2013 at 12:06 PM, Alexander Shiyan <shc_work@xxxxxxx> wrote:
>> On Sat, Dec 14, 2013 at 3:00 AM, Alexander Shiyan <shc_work@xxxxxxx> wrote:
>> > Signed-off-by: Alexander Shiyan <shc_work@xxxxxxx>
>> > ---
>> >  drivers/leds/leds-mc13783.c | 1 +
>> >  1 file changed, 1 insertion(+)
>> >
>> > diff --git a/drivers/leds/leds-mc13783.c b/drivers/leds/leds-mc13783.c
>> > index e61bdb5..670b936 100644
>> > --- a/drivers/leds/leds-mc13783.c
>> > +++ b/drivers/leds/leds-mc13783.c
>> > @@ -184,6 +184,7 @@ static int __init mc13xxx_led_probe(struct platform_device *pdev)
>> >                 leds->led[i].leds = leds;
>> >                 leds->led[i].cdev.name = name;
>> >                 leds->led[i].cdev.default_trigger = trig;
>> > +               leds->led[i].cdev.flags |= LED_CORE_SUSPENDRESUME;
>>
>> Looks good, I will merge that later.
>
> Now I see useless "|" (OR) in assignment. Can you drop it during merge?
>
Sure, please update this patchset. I will review it later.

-Bryan
--
To unsubscribe from this list: send the line "unsubscribe linux-leds" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux