From:   Florian Eckert <fe@dev.tdt.de>
To:     u.kleine-koenig@pengutronix.de, gregkh@linuxfoundation.org,
        jirislaby@kernel.org, pavel@ucw.cz, lee@kernel.org
Cc:     linux-kernel@vger.kernel.org, linux-leds@vger.kernel.org,
        Eckert.Florian@googlemail.com
Subject: [PATCH v8 3/3] ledtrig-tty: call correct brightness set function
Date:   Mon,  6 Mar 2023 10:41:13 +0100
Message-ID: <20230306094113.273988-4-fe@dev.tdt.de>
In-Reply-To: <20230306094113.273988-1-fe@dev.tdt.de>
References: <20230306094113.273988-1-fe@dev.tdt.de>
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

The problem is, however, that with LEDs that are connected directly to
the SOC via a GPIO, the LED cannot be switched on or off.
Because the 'leds-gpio' device driver does not need a delayed work.

If we look at the callstack for the function 'led_set_brightness_sync',
then at the end it is checked whether the LED device driver has the
function 'brightness_set_blocking' implemented. This is not the case for
the 'leds-gpio' device.

The function 'brightness_set_blocking' returns LED devices -ENOTSUPP.
This is not evaluated in 'ledtrig-tty' trigger.

In order for the trigger to also work with LED that does not have the
'brightness_set_blocking' function implemented, we must therefore use the
internal API of the LED subsystem and use the function
'led_set_brightness_nosleep'.

Signed-off-by: Florian Eckert <fe@dev.tdt.de>
---
 drivers/leds/trigger/ledtrig-tty.c | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/drivers/leds/trigger/ledtrig-tty.c b/drivers/leds/trigger/le=
dtrig-tty.c
index d3407830fa69..132e809898a7 100644
--- a/drivers/leds/trigger/ledtrig-tty.c
+++ b/drivers/leds/trigger/ledtrig-tty.c
@@ -7,6 +7,8 @@
 #include <linux/tty.h>
 #include <uapi/linux/serial.h>
=20
+#include "../leds.h"
+
 enum tty_led_mode {
 	TTY_LED_RXTX,
 	TTY_LED_CTS,
@@ -164,9 +166,9 @@ static void ledtrig_tty_flags(struct ledtrig_tty_data=
 *trigger_data, unsigned in
=20
 	status =3D tty_get_mget(trigger_data->tty);
 	if (status & flag)
-		led_set_brightness_sync(trigger_data->led_cdev, LED_ON);
+		led_set_brightness_nosleep(trigger_data->led_cdev, LED_ON);
 	else
-		led_set_brightness_sync(trigger_data->led_cdev, LED_OFF);
+		led_set_brightness_nosleep(trigger_data->led_cdev, LED_OFF);
 }
=20
 static void ledtrig_tty_work(struct work_struct *work)
@@ -229,12 +231,12 @@ static void ledtrig_tty_work(struct work_struct *wo=
rk)
 		}
=20
 		if (icount.rx !=3D trigger_data->rx || icount.tx !=3D trigger_data->tx=
) {
-			led_set_brightness_sync(trigger_data->led_cdev, LED_ON);
+			led_set_brightness_nosleep(trigger_data->led_cdev, LED_ON);
=20
 			trigger_data->rx =3D icount.rx;
 			trigger_data->tx =3D icount.tx;
 		} else {
-			led_set_brightness_sync(trigger_data->led_cdev, LED_OFF);
+			led_set_brightness_nosleep(trigger_data->led_cdev, LED_OFF);
 		}
 		break;
 	}
--=20
2.30.2