Re: [PATCH v2 3/4] trigger: ledtrig-tty: move variable definition to the top

From: Florian Eckert
Date: Wed Oct 04 2023 - 02:37:55 EST


On 2023-10-03 07:00, Jiri Slaby wrote:
On 02. 10. 23, 16:05, Lee Jones wrote:
On Thu, 28 Sep 2023, Florian Eckert wrote:

The Intel build robot has complained about this. Hence move the commit
of the variable definition to the beginning of the function.

Please copy the robot's error message into the commit message.

For a v3 patch-set I will add the error message from build robot.

Build robot output of my v1 change:
https://lore.kernel.org/linux-leds/20230926093607.59536-1-fe@xxxxxxxxxx/T/#m777371c5de8fadc505a833139b8ae69ac7fa8dab

I decided to move the variable definition with a separate commit
to the top of the function, to make the build robot happy. After that
I made my changes for v2 to the ledtrig-tty to add the feature.

Ah, lkp, then also the Closes: line as it suggests.

Sorry I do not understand your statement

Reported-by: kernel test robot <lkp@xxxxxxxxx>
Signed-off-by: Florian Eckert <fe@xxxxxxxxxx>
---
drivers/leds/trigger/ledtrig-tty.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/leds/trigger/ledtrig-tty.c b/drivers/leds/trigger/ledtrig-tty.c
index 8ae0d2d284af..1c6fadf0b856 100644
--- a/drivers/leds/trigger/ledtrig-tty.c
+++ b/drivers/leds/trigger/ledtrig-tty.c
@@ -82,6 +82,7 @@ static void ledtrig_tty_work(struct work_struct *work)
{
struct ledtrig_tty_data *trigger_data =
container_of(work, struct ledtrig_tty_data, dwork.work);
+ unsigned long interval = LEDTRIG_TTY_INTERVAL;
struct serial_icounter_struct icount;
int ret;
@@ -124,8 +125,6 @@ static void ledtrig_tty_work(struct work_struct *work)
if (icount.rx != trigger_data->rx ||
icount.tx != trigger_data->tx) {
- unsigned long interval = LEDTRIG_TTY_INTERVAL;
-
led_blink_set_oneshot(trigger_data->led_cdev, &interval,
&interval, 0);
-- 2.30.2