* [patch 1/8] platform: samsung_laptop: add dmi information for Samsung R700 laptops
2011-09-20 16:17 [patch 0/8] Samsung Laptop driver patches - resend Greg KH
@ 2011-09-20 16:16 ` Greg KH
2011-09-20 16:16 ` [patch 2/8] Platform: samsung_laptop: add support for X520 machines Greg KH
` (7 subsequent siblings)
8 siblings, 0 replies; 14+ messages in thread
From: Greg KH @ 2011-09-20 16:16 UTC (permalink / raw)
To: Matthew Garrett
Cc: platform-driver-x86, linux-kernel, greg, Stefan Beller, stable
From: Stefan Beller <stefanbeller@googlemail.com>
My DMI model is this:
>dmesg |grep DMI
[ 0.000000] DMI present.
[ 0.000000] DMI: SAMSUNG ELECTRONICS CO., LTD. SR700/SR700, BIOS
04SR 02/20/2008
adding dmi information of Samsung R700 laptops
This adds the dmi information of Samsungs R700 laptops.
Signed-off-by: Stefan Beller <stefanbeller@googlemail.com>
Cc: stable <stable@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
---
drivers/platform/x86/samsung-laptop.c | 9 +++++++++
1 file changed, 9 insertions(+)
--- a/drivers/platform/x86/samsung-laptop.c
+++ b/drivers/platform/x86/samsung-laptop.c
@@ -641,6 +641,15 @@ static struct dmi_system_id __initdata s
.callback = dmi_check_cb,
},
{
+ .ident = "R700",
+ .matches = {
+ DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
+ DMI_MATCH(DMI_PRODUCT_NAME, "SR700"),
+ DMI_MATCH(DMI_BOARD_NAME, "SR700"),
+ },
+ .callback = dmi_check_cb,
+ },
+ {
.ident = "R530/R730",
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
^ permalink raw reply [flat|nested] 14+ messages in thread* [patch 2/8] Platform: samsung_laptop: add support for X520 machines.
2011-09-20 16:17 [patch 0/8] Samsung Laptop driver patches - resend Greg KH
2011-09-20 16:16 ` [patch 1/8] platform: samsung_laptop: add dmi information for Samsung R700 laptops Greg KH
@ 2011-09-20 16:16 ` Greg KH
2011-09-20 16:16 ` [patch 3/8] Platform: samsung_laptop: samsung backlight for R528/R728 Greg KH
` (6 subsequent siblings)
8 siblings, 0 replies; 14+ messages in thread
From: Greg KH @ 2011-09-20 16:16 UTC (permalink / raw)
To: Matthew Garrett; +Cc: platform-driver-x86, linux-kernel, greg, stable
From: Tommaso Massimi <tmassimi@gmail.com>
my samsung laptop would be very happy if you add
these lines to the file drivers/platform/x86/samsung-laptop.c
Cc: stable <stable@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
---
drivers/platform/x86/samsung-laptop.c | 9 +++++++++
1 file changed, 9 insertions(+)
--- a/drivers/platform/x86/samsung-laptop.c
+++ b/drivers/platform/x86/samsung-laptop.c
@@ -695,6 +695,15 @@ static struct dmi_system_id __initdata s
},
.callback = dmi_check_cb,
},
+ {
+ .ident = "X520",
+ .matches = {
+ DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
+ DMI_MATCH(DMI_PRODUCT_NAME, "X520"),
+ DMI_MATCH(DMI_BOARD_NAME, "X520"),
+ },
+ .callback = dmi_check_cb,
+ },
{ },
};
MODULE_DEVICE_TABLE(dmi, samsung_dmi_table);
^ permalink raw reply [flat|nested] 14+ messages in thread* [patch 3/8] Platform: samsung_laptop: samsung backlight for R528/R728
2011-09-20 16:17 [patch 0/8] Samsung Laptop driver patches - resend Greg KH
2011-09-20 16:16 ` [patch 1/8] platform: samsung_laptop: add dmi information for Samsung R700 laptops Greg KH
2011-09-20 16:16 ` [patch 2/8] Platform: samsung_laptop: add support for X520 machines Greg KH
@ 2011-09-20 16:16 ` Greg KH
2011-09-20 16:16 ` [patch 4/8] platform: samsung_laptop: fix samsung brightness min/max calculations Greg KH
` (5 subsequent siblings)
8 siblings, 0 replies; 14+ messages in thread
From: Greg KH @ 2011-09-20 16:16 UTC (permalink / raw)
To: Matthew Garrett
Cc: platform-driver-x86, linux-kernel, greg, Smelov Andrey, stable
From: Smelov Andrey <xor29a@bk.ru>
patch works for me, but I need to add "acpi_backlight=vendor" to kernel
params
Signed-off-by: Smelov Andrey <xor29a@bk.ru>
Cc: stable <stable@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
---
drivers/platform/x86/samsung-laptop.c | 9 +++++++++
1 file changed, 9 insertions(+)
--- a/drivers/platform/x86/samsung-laptop.c
+++ b/drivers/platform/x86/samsung-laptop.c
@@ -704,6 +704,15 @@ static struct dmi_system_id __initdata s
},
.callback = dmi_check_cb,
},
+ {
+ .ident = "R528/R728",
+ .matches = {
+ DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
+ DMI_MATCH(DMI_PRODUCT_NAME, "R528/R728"),
+ DMI_MATCH(DMI_BOARD_NAME, "R528/R728"),
+ },
+ .callback = dmi_check_cb,
+ },
{ },
};
MODULE_DEVICE_TABLE(dmi, samsung_dmi_table);
^ permalink raw reply [flat|nested] 14+ messages in thread* [patch 4/8] platform: samsung_laptop: fix samsung brightness min/max calculations
2011-09-20 16:17 [patch 0/8] Samsung Laptop driver patches - resend Greg KH
` (2 preceding siblings ...)
2011-09-20 16:16 ` [patch 3/8] Platform: samsung_laptop: samsung backlight for R528/R728 Greg KH
@ 2011-09-20 16:16 ` Greg KH
2011-09-20 16:16 ` [patch 5/8] Platform: Fix error path in samsung-laptop init Greg KH
` (4 subsequent siblings)
8 siblings, 0 replies; 14+ messages in thread
From: Greg KH @ 2011-09-20 16:16 UTC (permalink / raw)
To: Matthew Garrett, Greg KH
Cc: platform-driver-x86, linux-kernel, Jason Stubbs, stable
From: Jason Stubbs <jasonbstubbs@gmail.com>
The min_brightness value of the sabi_config is incorrectly used in brightness
calculations. For the config where min_brightness = 1 and max_brightness = 8,
the user visible range should be 0 to 7 with hardware being set in the range
of 1 to 8. What is actually happening is that the user visible range is 0 to
8 with hardware being set in the range of -1 to 7.
This patch fixes the above issue as well as a miscalculation that would occur
in the case of min_brightness > 1.
Signed-off-by: Jason Stubbs <jasonbstubbs@gmail.com>
Cc: stable <stable@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
---
drivers/platform/x86/samsung-laptop.c | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
--- a/drivers/platform/x86/samsung-laptop.c
+++ b/drivers/platform/x86/samsung-laptop.c
@@ -370,15 +370,17 @@ static u8 read_brightness(void)
&sretval);
if (!retval) {
user_brightness = sretval.retval[0];
- if (user_brightness != 0)
+ if (user_brightness > sabi_config->min_brightness)
user_brightness -= sabi_config->min_brightness;
+ else
+ user_brightness = 0;
}
return user_brightness;
}
static void set_brightness(u8 user_brightness)
{
- u8 user_level = user_brightness - sabi_config->min_brightness;
+ u8 user_level = user_brightness + sabi_config->min_brightness;
sabi_set_command(sabi_config->commands.set_brightness, user_level);
}
@@ -829,7 +831,8 @@ static int __init samsung_init(void)
/* create a backlight device to talk to this one */
memset(&props, 0, sizeof(struct backlight_properties));
props.type = BACKLIGHT_PLATFORM;
- props.max_brightness = sabi_config->max_brightness;
+ props.max_brightness = sabi_config->max_brightness -
+ sabi_config->min_brightness;
backlight_device = backlight_device_register("samsung", &sdev->dev,
NULL, &backlight_ops,
&props);
^ permalink raw reply [flat|nested] 14+ messages in thread* [patch 5/8] Platform: Fix error path in samsung-laptop init
2011-09-20 16:17 [patch 0/8] Samsung Laptop driver patches - resend Greg KH
` (3 preceding siblings ...)
2011-09-20 16:16 ` [patch 4/8] platform: samsung_laptop: fix samsung brightness min/max calculations Greg KH
@ 2011-09-20 16:16 ` Greg KH
2011-09-20 16:16 ` [patch 6/8] Platform: Brightness quirk for samsung laptop driver Greg KH
` (3 subsequent siblings)
8 siblings, 0 replies; 14+ messages in thread
From: Greg KH @ 2011-09-20 16:16 UTC (permalink / raw)
To: Matthew Garrett, platform-driver-x86
Cc: linux-kernel, greg, David Herrmann, stable
From: David Herrmann <dh.herrmann@googlemail.com>
samsung_init() should not return success if not all devices are initialized.
Otherwise, samsung_exit() will dereference sdev NULL pointers and others.
Signed-off-by: David Herrmann <dh.herrmann@googlemail.com>
Cc: stable <stable@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
---
drivers/platform/x86/samsung-laptop.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
--- a/drivers/platform/x86/samsung-laptop.c
+++ b/drivers/platform/x86/samsung-laptop.c
@@ -799,7 +799,7 @@ static int __init samsung_init(void)
sabi_iface = ioremap_nocache(ifaceP, 16);
if (!sabi_iface) {
pr_err("Can't remap %x\n", ifaceP);
- goto exit;
+ goto error_no_signature;
}
if (debug) {
printk(KERN_DEBUG "ifaceP = 0x%08x\n", ifaceP);
@@ -851,7 +851,6 @@ static int __init samsung_init(void)
if (retval)
goto error_file_create;
-exit:
return 0;
error_file_create:
^ permalink raw reply [flat|nested] 14+ messages in thread* [patch 6/8] Platform: Brightness quirk for samsung laptop driver
2011-09-20 16:17 [patch 0/8] Samsung Laptop driver patches - resend Greg KH
` (4 preceding siblings ...)
2011-09-20 16:16 ` [patch 5/8] Platform: Fix error path in samsung-laptop init Greg KH
@ 2011-09-20 16:16 ` Greg KH
2011-09-20 16:16 ` [patch 7/8] Platform: Samsung laptop DMI info for NC210/NC110 Greg KH
` (2 subsequent siblings)
8 siblings, 0 replies; 14+ messages in thread
From: Greg KH @ 2011-09-20 16:16 UTC (permalink / raw)
To: Matthew Garrett, Greg KH
Cc: platform-driver-x86, linux-kernel, David Herrmann, Jason Stubbs
From: Jason Stubbs <jasonbstubbs@gmail.com>
On some Samsung laptops the brightness regulation works slightly different.
All SABI commands except for set_brightness work as expected. The behaviour
of set_brightness is as follows:
- Setting a new brightness will only step one level toward the new brightness
level. For example, setting a level of 5 when the current level is 2 will
result in a brightness level of 3.
- A spurious KEY_BRIGHTNESS_UP or KEY_BRIGHTNESS_DOWN event is also generated
along with the change in brightness.
- Neither of the above two issues occur when changing from/to brightness
level 0.
This patch adds detection and a non-intrusive workaround for the above issues.
Signed-off-by: Jason Stubbs <jasonbstubbs@gmail.com>
Tested-by: David Herrmann <dh.herrmann@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
---
drivers/platform/x86/samsung-laptop.c | 43 ++++++++++++++++++++++++++++++++++
1 file changed, 43 insertions(+)
--- a/drivers/platform/x86/samsung-laptop.c
+++ b/drivers/platform/x86/samsung-laptop.c
@@ -226,6 +226,7 @@ static struct backlight_device *backligh
static struct mutex sabi_mutex;
static struct platform_device *sdev;
static struct rfkill *rfk;
+static bool has_stepping_quirk;
static int force;
module_param(force, bool, 0);
@@ -382,6 +383,17 @@ static void set_brightness(u8 user_brigh
{
u8 user_level = user_brightness + sabi_config->min_brightness;
+ if (has_stepping_quirk && user_level != 0) {
+ /*
+ * short circuit if the specified level is what's already set
+ * to prevent the screen from flickering needlessly
+ */
+ if (user_brightness == read_brightness())
+ return;
+
+ sabi_set_command(sabi_config->commands.set_brightness, 0);
+ }
+
sabi_set_command(sabi_config->commands.set_brightness, user_level);
}
@@ -390,6 +402,34 @@ static int get_brightness(struct backlig
return (int)read_brightness();
}
+static void check_for_stepping_quirk(void)
+{
+ u8 initial_level = read_brightness();
+ u8 check_level;
+
+ /*
+ * Some laptops exhibit the strange behaviour of stepping toward
+ * (rather than setting) the brightness except when changing to/from
+ * brightness level 0. This behaviour is checked for here and worked
+ * around in set_brightness.
+ */
+
+ if (initial_level <= 2)
+ check_level = initial_level + 2;
+ else
+ check_level = initial_level - 2;
+
+ has_stepping_quirk = false;
+ set_brightness(check_level);
+
+ if (read_brightness() != check_level) {
+ has_stepping_quirk = true;
+ pr_info("enabled workaround for brightness stepping quirk\n");
+ }
+
+ set_brightness(initial_level);
+}
+
static int update_status(struct backlight_device *bd)
{
set_brightness(bd->props.brightness);
@@ -823,6 +863,9 @@ static int __init samsung_init(void)
}
}
+ /* Check for stepping quirk */
+ check_for_stepping_quirk();
+
/* knock up a platform device to hang stuff off of */
sdev = platform_device_register_simple("samsung", -1, NULL, 0);
if (IS_ERR(sdev))
^ permalink raw reply [flat|nested] 14+ messages in thread* [patch 7/8] Platform: Samsung laptop DMI info for NC210/NC110
2011-09-20 16:17 [patch 0/8] Samsung Laptop driver patches - resend Greg KH
` (5 preceding siblings ...)
2011-09-20 16:16 ` [patch 6/8] Platform: Brightness quirk for samsung laptop driver Greg KH
@ 2011-09-20 16:16 ` Greg KH
2011-09-20 16:16 ` [patch 8/8] Platform: fix samsung-laptop DMI identification for N220 model Greg KH
2011-11-06 22:54 ` [patch 0/8] Samsung Laptop driver patches - resend Corentin Chary
8 siblings, 0 replies; 14+ messages in thread
From: Greg KH @ 2011-09-20 16:16 UTC (permalink / raw)
To: Matthew Garrett, Greg KH
Cc: platform-driver-x86, linux-kernel, David Herrmann, Jason Stubbs
From: Jason Stubbs <jasonbstubbs@gmail.com>
This patch just adds the DMI info for the samsung laptop driver to work with
the NC210/NC110. It needs the brightness quirk patch for proper support.
Signed-off-by: Jason Stubbs <jasonbstubbs@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
---
drivers/platform/x86/samsung-laptop.c | 9 +++++++++
1 file changed, 9 insertions(+)
--- a/drivers/platform/x86/samsung-laptop.c
+++ b/drivers/platform/x86/samsung-laptop.c
@@ -755,6 +755,15 @@ static struct dmi_system_id __initdata s
},
.callback = dmi_check_cb,
},
+ {
+ .ident = "NC210/NC110",
+ .matches = {
+ DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
+ DMI_MATCH(DMI_PRODUCT_NAME, "NC210/NC110"),
+ DMI_MATCH(DMI_BOARD_NAME, "NC210/NC110"),
+ },
+ .callback = dmi_check_cb,
+ },
{ },
};
MODULE_DEVICE_TABLE(dmi, samsung_dmi_table);
^ permalink raw reply [flat|nested] 14+ messages in thread* [patch 8/8] Platform: fix samsung-laptop DMI identification for N220 model
2011-09-20 16:17 [patch 0/8] Samsung Laptop driver patches - resend Greg KH
` (6 preceding siblings ...)
2011-09-20 16:16 ` [patch 7/8] Platform: Samsung laptop DMI info for NC210/NC110 Greg KH
@ 2011-09-20 16:16 ` Greg KH
2011-11-06 22:54 ` [patch 0/8] Samsung Laptop driver patches - resend Corentin Chary
8 siblings, 0 replies; 14+ messages in thread
From: Greg KH @ 2011-09-20 16:16 UTC (permalink / raw)
To: Matthew Garrett
Cc: platform-driver-x86, linux-kernel, greg, Raul Gutierrez Segales
From: Raul Gutierrez Segales <rgs@collabora.co.uk>
This is a follow-up for commit 78a7539b, which didn't cover the
Samsung N220 laptop. With this backlight brightness works nicely
on the N220 netbook.
Signed-off-by: Raul Gutierrez Segales <rgs@collabora.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
---
drivers/platform/x86/samsung-laptop.c | 10 ++++++++++
1 file changed, 10 insertions(+)
--- a/drivers/platform/x86/samsung-laptop.c
+++ b/drivers/platform/x86/samsung-laptop.c
@@ -663,6 +663,16 @@ static struct dmi_system_id __initdata s
.callback = dmi_check_cb,
},
{
+ .ident = "N220",
+ .matches = {
+ DMI_MATCH(DMI_SYS_VENDOR,
+ "SAMSUNG ELECTRONICS CO., LTD."),
+ DMI_MATCH(DMI_PRODUCT_NAME, "N220"),
+ DMI_MATCH(DMI_BOARD_NAME, "N220"),
+ },
+ .callback = dmi_check_cb,
+ },
+ {
.ident = "N150/N210/N220/N230",
.matches = {
DMI_MATCH(DMI_SYS_VENDOR,
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [patch 0/8] Samsung Laptop driver patches - resend
2011-09-20 16:17 [patch 0/8] Samsung Laptop driver patches - resend Greg KH
` (7 preceding siblings ...)
2011-09-20 16:16 ` [patch 8/8] Platform: fix samsung-laptop DMI identification for N220 model Greg KH
@ 2011-11-06 22:54 ` Corentin Chary
2011-11-07 18:12 ` Greg KH
8 siblings, 1 reply; 14+ messages in thread
From: Corentin Chary @ 2011-11-06 22:54 UTC (permalink / raw)
To: Greg KH; +Cc: Matthew Garrett, platform-driver-x86, linux-kernel, greg
On Tue, Sep 20, 2011 at 6:17 PM, Greg KH <gregkh@suse.de> wrote:
> Hi Matthew,
>
> Here is a resend of the samsung-laptop patches, with the correct
> authorship info in it. Sorry about the last batch, 'quilt mail' strips
> them off when sending them out, despite the information being in the
> original patch.
>
> The patches are against Linus's current tree, but should apply just fine
> to linux-next as well.
>
> If there are any problems with them, please let me know.
>
> Oh, and if anyone wants to take over maintainership of this driver,
> please let me know. I no longer have the hardware to be able to test
> any changes for it, which is why this took so long to get pushed out to
> you.
>
> thanks,
>
> greg k-h
Hi Greg,
Just got a 90X3A from Intel and I'll try to take a look at samsung-laptop
since some features are missing (keyboard backlight for example).
Do you have any doc on the vendor "interface" (other than just reading
the code :) ) ?
Thanks,
--
Corentin Chary
http://xf.iksaif.net
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [patch 0/8] Samsung Laptop driver patches - resend
2011-11-06 22:54 ` [patch 0/8] Samsung Laptop driver patches - resend Corentin Chary
@ 2011-11-07 18:12 ` Greg KH
2011-11-07 22:57 ` Corentin Chary
0 siblings, 1 reply; 14+ messages in thread
From: Greg KH @ 2011-11-07 18:12 UTC (permalink / raw)
To: Corentin Chary; +Cc: Matthew Garrett, platform-driver-x86, linux-kernel, greg
On Sun, Nov 06, 2011 at 11:54:39PM +0100, Corentin Chary wrote:
> Just got a 90X3A from Intel and I'll try to take a look at samsung-laptop
> since some features are missing (keyboard backlight for example).
Try adding your device to this driver. Or, watch out, as newer Samsung
devices support backlight control "properly" through acpi, which is
good, and this driver is not needed by them at all.
> Do you have any doc on the vendor "interface" (other than just reading
> the code :) ) ?
I do not have any documentation for the interface, sorry. The code
should be self-explanatory.
greg k-h
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [patch 0/8] Samsung Laptop driver patches - resend
2011-11-07 18:12 ` Greg KH
@ 2011-11-07 22:57 ` Corentin Chary
2011-11-13 9:52 ` Corentin Chary
0 siblings, 1 reply; 14+ messages in thread
From: Corentin Chary @ 2011-11-07 22:57 UTC (permalink / raw)
To: Greg KH; +Cc: Matthew Garrett, platform-driver-x86, linux-kernel, greg
On Mon, Nov 7, 2011 at 7:12 PM, Greg KH <gregkh@suse.de> wrote:
> On Sun, Nov 06, 2011 at 11:54:39PM +0100, Corentin Chary wrote:
>> Just got a 90X3A from Intel and I'll try to take a look at samsung-laptop
>> since some features are missing (keyboard backlight for example).
>
> Try adding your device to this driver. Or, watch out, as newer Samsung
> devices support backlight control "properly" through acpi, which is
> good, and this driver is not needed by them at all.
I'm speaking about *keyboard* backlight here, not screen backlight
(which is already available using both the acpi driver and the intel
graphic driver). There is also a setting to enable USB power while
sleeping without having to change it in the BIOS, and it's probably
possible to control the sound and wlan led.
>> Do you have any doc on the vendor "interface" (other than just reading
>> the code :) ) ?
>
> I do not have any documentation for the interface, sorry. The code
> should be self-explanatory.
Arg... I can understand the code, I can easilly write ACPI and WMI
drivers, but the "SABI" thing will be harder to reverse engineer :).
Thanks,
--
Corentin Chary
http://xf.iksaif.net
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [patch 0/8] Samsung Laptop driver patches - resend
2011-11-07 22:57 ` Corentin Chary
@ 2011-11-13 9:52 ` Corentin Chary
2011-11-13 15:58 ` Greg KH
0 siblings, 1 reply; 14+ messages in thread
From: Corentin Chary @ 2011-11-13 9:52 UTC (permalink / raw)
To: Greg KH; +Cc: Matthew Garrett, platform-driver-x86, linux-kernel, greg
On Mon, Nov 7, 2011 at 11:57 PM, Corentin Chary
<corentin.chary@gmail.com> wrote:
> On Mon, Nov 7, 2011 at 7:12 PM, Greg KH <gregkh@suse.de> wrote:
>> On Sun, Nov 06, 2011 at 11:54:39PM +0100, Corentin Chary wrote:
>>> Just got a 90X3A from Intel and I'll try to take a look at samsung-laptop
>>> since some features are missing (keyboard backlight for example).
>>
>> Try adding your device to this driver. Or, watch out, as newer Samsung
>> devices support backlight control "properly" through acpi, which is
>> good, and this driver is not needed by them at all.
>
> I'm speaking about *keyboard* backlight here, not screen backlight
> (which is already available using both the acpi driver and the intel
> graphic driver). There is also a setting to enable USB power while
> sleeping without having to change it in the BIOS, and it's probably
> possible to control the sound and wlan led.
>
>>> Do you have any doc on the vendor "interface" (other than just reading
>>> the code :) ) ?
>>
>> I do not have any documentation for the interface, sorry. The code
>> should be self-explanatory.
>
> Arg... I can understand the code, I can easilly write ACPI and WMI
> drivers, but the "SABI" thing will be harder to reverse engineer :).
It was in fact very fun to find out new commands, and I found interesting stuff.
However, I have a quick question: what does "wireless button" control
exactly ? Because my laptop (SwSmi@ one), seems to have a totally
different command to control rfkill (both for bluetooth and wlan). My
guess is that command do the same thing that happens when you press
the Fn+F12 key, but if it's really that I'd prefer use the "true"
rfkill interface with differencied control for bluetooth and wlan.
I have found other interesting information including:
- you can pass up to 11 byte of data for each command (but I didn't
found any command using more than 6)
- there is a way to find the model name and bios version near the signature
- wlan and bluetooth control
- USB charge control
- Keyboard backlight control
- Battery life extender control
I'll try to clean that up and send some patches in one week or two.
--
Corentin Chary
http://xf.iksaif.net
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [patch 0/8] Samsung Laptop driver patches - resend
2011-11-13 9:52 ` Corentin Chary
@ 2011-11-13 15:58 ` Greg KH
0 siblings, 0 replies; 14+ messages in thread
From: Greg KH @ 2011-11-13 15:58 UTC (permalink / raw)
To: Corentin Chary; +Cc: Matthew Garrett, platform-driver-x86, linux-kernel, greg
On Sun, Nov 13, 2011 at 10:52:29AM +0100, Corentin Chary wrote:
> However, I have a quick question: what does "wireless button" control
> exactly ? Because my laptop (SwSmi@ one), seems to have a totally
> different command to control rfkill (both for bluetooth and wlan). My
> guess is that command do the same thing that happens when you press
> the Fn+F12 key, but if it's really that I'd prefer use the "true"
> rfkill interface with differencied control for bluetooth and wlan.
I think it controlled the wireless on/off on some machines, others might
be different. And it should be using rfkill, right?
greg k-h
^ permalink raw reply [flat|nested] 14+ messages in thread