|
 |
8604c91 |
From 0dbc2bc96b1ec741bdd43451c286ccd45da3310b Mon Sep 17 00:00:00 2001
|
|
|
8604c91 |
From: Josh Boyer <jwboyer@redhat.com>
|
|
|
8604c91 |
Date: Wed, 2 Nov 2011 14:31:59 -0400
|
|
|
8604c91 |
Subject: [PATCH] sony-laptop: Enable keyboard backlight by default
|
|
|
8604c91 |
|
|
|
8604c91 |
When the keyboard backlight support was originally added, the commit said
|
|
|
8604c91 |
to default it to on with a 10 second timeout. That actually wasn't the
|
|
|
8604c91 |
case, as the default value is commented out for the kbd_backlight parameter.
|
|
|
8604c91 |
Because it is a static variable, it gets set to 0 by default without some
|
|
|
8604c91 |
other form of initialization.
|
|
|
8604c91 |
|
|
|
8604c91 |
However, it seems the function to set the value wasn't actually called
|
|
|
8604c91 |
immediately, so whatever state the keyboard was in initially would remain.
|
|
|
8604c91 |
Then commit df410d522410e67660 was introduced during the 2.6.39 timeframe to
|
|
|
8604c91 |
immediately set whatever value was present (as well as attempt to
|
|
|
8604c91 |
restore/reset the state on module removal or resume). That seems to have
|
|
|
8604c91 |
now forced the light off immediately when the module is loaded unless
|
|
|
8604c91 |
the option kbd_backlight=1 is specified.
|
|
|
8604c91 |
|
|
|
8604c91 |
Let's enable it by default again (for the first time). This should solve
|
|
|
8604c91 |
https://bugzilla.redhat.com/show_bug.cgi?id=728478
|
|
|
8604c91 |
|
|
|
8604c91 |
Acked-by: Mattia Dongili <malattia@linux.it>
|
|
|
8604c91 |
Signed-off-by: Josh Boyer <jwboyer@redhat.com>
|
|
|
8604c91 |
---
|
|
|
8604c91 |
drivers/platform/x86/sony-laptop.c | 2 +-
|
|
|
8604c91 |
1 files changed, 1 insertions(+), 1 deletions(-)
|
|
|
8604c91 |
|
|
|
8604c91 |
diff --git a/drivers/platform/x86/sony-laptop.c b/drivers/platform/x86/sony-laptop.c
|
|
|
8604c91 |
index c006dee..40c4705 100644
|
|
|
8604c91 |
--- a/drivers/platform/x86/sony-laptop.c
|
|
|
8604c91 |
+++ b/drivers/platform/x86/sony-laptop.c
|
|
|
8604c91 |
@@ -127,7 +127,7 @@ MODULE_PARM_DESC(minor,
|
|
|
8604c91 |
"default is -1 (automatic)");
|
|
|
8604c91 |
#endif
|
|
|
8604c91 |
|
|
|
8604c91 |
-static int kbd_backlight; /* = 1 */
|
|
|
8604c91 |
+static int kbd_backlight = 1;
|
|
|
8604c91 |
module_param(kbd_backlight, int, 0444);
|
|
|
8604c91 |
MODULE_PARM_DESC(kbd_backlight,
|
|
|
8604c91 |
"set this to 0 to disable keyboard backlight, "
|
|
|
8604c91 |
--
|
|
|
8604c91 |
1.7.7.6
|
|
|
8604c91 |
|