[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20210219133307.4840-1-u.kleine-koenig@pengutronix.de>
Date: Fri, 19 Feb 2021 14:33:05 +0100
From: Uwe Kleine-König
<u.kleine-koenig@...gutronix.de>
To: Pavel Machek <pavel@....cz>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc: kernel@...gutronix.de, linux-leds@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-serial@...r.kernel.org
Subject: [PATCH v2 0/2] Fixes for the tty ledtrigger
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@....cz>
Fixes: fd4a641ac88f ("leds: trigger: implement a tty trigger")
+ Acked-by: Pavel Machek <pavel@....cz>
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@...gutronix.de>
## 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@....cz>
Fixes: fd4a641ac88f ("leds: trigger: implement a tty trigger")
+ Acked-by: Pavel Machek <pavel@....cz>
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@...gutronix.de>
## drivers/leds/trigger/ledtrig-tty.c ##
--
2.29.2
Powered by blists - more mailing lists