diff mbox

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

Message ID 56A4A837.5030509@gmail.com (mailing list archive)
State Not Applicable
Delegated to: Johannes Berg
Headers show

Commit Message

Hubert Tarasiuk Jan. 24, 2016, 10:32 a.m. UTC
Attached is the patch for bug 110551.

W dniu 24.01.2016 o 11:16, bugzilla-daemon@bugzilla.kernel.org pisze:
> https://bugzilla.kernel.org/show_bug.cgi?id=110551
> 
> --- Comment #13 from Emmanuel Grumbach <emmanuel.grumbach@intel.com> ---
> https://bugzilla.kernel.org/attachment.cgi?id=201271
> 
> This one seems to have a fix for the module param as well.
> I would much prefer you to send the patch to the wireless mailing list rather
> than attaching it to this bugzilla.
>

Comments

Emmanuel Grumbach Jan. 24, 2016, 11:45 a.m. UTC | #1
On Sun, Jan 24, 2016 at 12:32 PM, Hubert Tarasiuk
<hubert.tarasiuk@gmail.com> wrote:
> Attached is the patch for bug 110551.
>
> W dniu 24.01.2016 o 11:16, bugzilla-daemon@bugzilla.kernel.org pisze:
>> https://bugzilla.kernel.org/show_bug.cgi?id=110551
>>
>> --- Comment #13 from Emmanuel Grumbach <emmanuel.grumbach@intel.com> ---
>> https://bugzilla.kernel.org/attachment.cgi?id=201271
>>
>> This one seems to have a fix for the module param as well.
>> I would much prefer you to send the patch to the wireless mailing list rather
>> than attaching it to this bugzilla.
>>

The better way is send the patches with git-send-email tool.
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

From 40e783a5464e43722d87b5cb0c3d3899f3f7417e Mon Sep 17 00:00:00 2001
From: Hubert Tarasiuk <hubert.tarasiuk@gmail.com>
Date: Mon, 4 Jan 2016 23:45:18 +0100
Subject: [PATCH] iwlwifi: dvm: handle zero brightness for wifi LED

This fixes bug 110551 with regard to sysfs.

Misleading behaviour when 'led_mode=3' is passed to iwlwifi is still
to be fixed (either by changing the option description or by making
the LED permanently dark in such case).

Signed-off-by: Hubert Tarasiuk <hubert.tarasiuk@gmail.com>
---
 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.7.0