[PATCH v2 0/2] Fixes for the tty ledtrigger

From: Uwe Kleine-König
Date: Fri Feb 19 2021 - 08:33:59 EST


Hello,

this drops the unused label that the kernel test robot found. Thanks for
catching that.

I also added Pavel's Ack.

Best regards
Uwe

Uwe Kleine-König (2):
leds: trigger: Fix error path to not unlock the unlocked mutex
leds: trigger/tty: Use led_set_brightness_sync() from workqueue

drivers/leds/trigger/ledtrig-tty.c | 11 ++++-------
1 file changed, 4 insertions(+), 7 deletions(-)

Range-diff against v1:
1: 7fc10ce8eb8b ! 1: ed39ad41cd91 leds: trigger: Fix error path to not unlock the unlocked mutex
@@ Commit message

Reported-by: Pavel Machek <pavel@xxxxxx>
Fixes: fd4a641ac88f ("leds: trigger: implement a tty trigger")
+ Acked-by: Pavel Machek <pavel@xxxxxx>
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx>

## drivers/leds/trigger/ledtrig-tty.c ##
@@ drivers/leds/trigger/ledtrig-tty.c: static ssize_t ttyname_store(struct device *
} else {
ttyname = NULL;
}
+@@ drivers/leds/trigger/ledtrig-tty.c: static ssize_t ttyname_store(struct device *dev,
+
+ trigger_data->ttyname = ttyname;
+
+-out_unlock:
+ mutex_unlock(&trigger_data->mutex);
+
+ if (ttyname && !running)
2: fe3d28f4a786 ! 2: a812318f4cfc leds: trigger/tty: Use led_set_brightness_sync() from workqueue
@@ Commit message

Reported-by: Pavel Machek <pavel@xxxxxx>
Fixes: fd4a641ac88f ("leds: trigger: implement a tty trigger")
+ Acked-by: Pavel Machek <pavel@xxxxxx>
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx>

## drivers/leds/trigger/ledtrig-tty.c ##
--
2.29.2