Re: [PATCH 2/2] leds: trigger/tty: Use led_set_brightness_sync() from workqueue

From: Pavel Machek
Date: Fri Feb 19 2021 - 05:01:37 EST


On Fri 2021-02-19 10:52:54, Uwe Kleine-König wrote:
> On Fri, Feb 19, 2021 at 10:46:32AM +0100, Pavel Machek wrote:
> > On Fri 2021-02-19 09:29:55, Uwe Kleine-König wrote:
> > > led_set_brightness() involves scheduling a workqueue. As here the led's
> > > brightness setting is done in context of the trigger's workqueue this is
> > > unjustified overhead and it's more sensible to use
> > > led_set_brightness_sync().
> >
> > Acked-by: Pavel Machek <pavel@xxxxxx>
>
> Who is expected to pick these two up? I assume Greg as the commit that
> is fixed here is only in his try for now?

I can't reasonably pick them up.

Pavel
--
http://www.livejournal.com/~pavelmachek

Attachment: signature.asc
Description: PGP signature