[Bug 110551] iwlwifi: dvm: cannot turn off wifi LED

[Hubert Tarasiuk <hubert.tarasiuk@xxxxxxxxx>]

Re-sending as plain-text.


--- Treść przekazanej wiadomości ---
Temat: 	[Bug 110551] iwlwifi: dvm: cannot turn off wifi LED
Data: 	Wed, 13 Jan 2016 23:31:28 +0100
Nadawca: 	Hubert Tarasiuk <hubert.tarasiuk@xxxxxxxxx>
Adresat: 	emmanuel.grumbach@xxxxxxxxx
Kopia: 	linux-wireless@xxxxxxxxxxxxxxx, bugzilla-daemon@xxxxxxxxxxxxxxxxxxx



Hi Emmanuel,

Actually to properly handle led_mode=3 option is not as easy as it seems.
Even if I initially turn of the LED in case of led_mode=3, it still gets
turned on after doing
# ifconfig wlan0 down && ifconfig wlan0 up

Maybe if I have more time later I can investigate further on how to
avoid this behaviour. (the LED trigger is set to none according to
sysfs) Or you have an idea on how to stop this behaviour?

Otherwise I would suggest that you apply the patch that I posted on
bugzilla (attached) that fixes sysfs handling. It is then easy to keep
the LED on or off as desired using simple shell scripts.

Thank you,
Hubert

On Tue, Jan 12, 2016 at 7:58 AM, <bugzilla-daemon@xxxxxxxxxxxxxxxxxxx
<mailto:bugzilla-daemon@xxxxxxxxxxxxxxxxxxx>>Â wrote:

    https://bugzilla.kernel.org/show_bug.cgi?id=110551

    --- Comment #1 from Emmanuel Grumbach <emmanuel.grumbach@xxxxxxxxx
    <mailto:emmanuel.grumbach@xxxxxxxxx>> ---
    Your bug is fine, but I'd prefer if you'd send the patch :)

    you can send it to me and CCÂ linux-wireless@xxxxxxxxxxxxxxx
    <mailto:linux-wireless@xxxxxxxxxxxxxxx>

    thanks.

    --
    You are receiving this mail because:
    You are on the CC list for the bug.
    You reported the bug.



From 132afdea768bea0a5dc0f46417b261138b9b0286 Mon Sep 17 00:00:00 2001
From: Hubert Tarasiuk <hubert.tarasiuk@xxxxxxxxx>
Date: Mon, 4 Jan 2016 23:45:18 +0100
Subject: [PATCH] iwlwifi: allow led turn off in iwl dvm module

---
 drivers/net/wireless/iwlwifi/dvm/led.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/net/wireless/iwlwifi/dvm/led.c b/drivers/net/wireless/iwlwifi/dvm/led.c
index ca4d669..218617a 100644
--- a/drivers/net/wireless/iwlwifi/dvm/led.c
+++ b/drivers/net/wireless/iwlwifi/dvm/led.c
@@ -154,11 +154,14 @@ static void iwl_led_brightness_set(struct led_classdev *led_cdev,
 {
 	struct iwl_priv *priv = container_of(led_cdev, struct iwl_priv, led);
 	unsigned long on = 0;
+	unsigned long off = 0;
 
 	if (brightness > 0)
 		on = IWL_LED_SOLID;
+	else
+		off = IWL_LED_SOLID;
 
-	iwl_led_cmd(priv, on, 0);
+	iwl_led_cmd(priv, on, off);
 }
 
 static int iwl_led_blink_set(struct led_classdev *led_cdev,
-- 
2.6.4

Attachment: signature.asc
Description: OpenPGP digital signature