Sony laptop: Some Sony Vaia laptops do not enable wwan power by default.
Subsystems (2)
PATCH 1/1 Sony laptop: Some Sony Vaia laptops do not enable wwan power by default.
Mattia,
Does this patch makes sense for just the Vaio? I've had positive feedback from the tester in the bug report.
rtg
-----------------
Quoted 1 line
From
083972abb1d5e84845927b3925f077953c207b82Mon Sep 17 00:00:00 2001
From: Tim Gardner <[email protected]>
Date: Tue, 21 Apr 2009 11:09:10 -0600
Subject: [PATCH] Sony laptop: Some Sony Vaio laptops do not enable wwan power by default.
Added quirk to enable wwan power based on DMI information already present in the module. It appears that wwan is not powered up from a cold boot.
--- drivers/misc/sony-laptop.c | 24 +++++++++++++++++++++++- 1 files changed, 23 insertions(+), 1 deletions(-)
drivers/misc/sony-laptop.c +23 -1
@@ -1248,10 +1248,13 @@ struct device_ctrl {u16 evport_offset;u8 has_camera;u8 has_bluetooth; -u8 has_wwan;struct sonypi_eventtypes *event_types; }; +struct sony_pic_quirk_entry { +u8 set_wwan_power; +}; +struct sony_pic_dev {struct device_ctrl *control;struct acpi_device *acpi_dev;
@@ -1260,6 +1263,7 @@ struct sony_pic_dev {struct list_head interrupts;struct list_head ioports;struct mutex lock; +struct sony_pic_quirk_entry *quirks;u8 camera_power;u8 bluetooth_power;u8 wwan_power;
@@ -2685,6 +2689,12 @@ static int sony_pic_add(struct acpi_device *device)if (result) goto err_remove_pf; +if (spic_dev.quirks && spic_dev.quirks->set_wwan_power) { +/* +* Power isn't enabled by default. +*/ +sony_pic_set_wwanpower (1 ); + }return 0 ; err_remove_pf:
@@ -2755,6 +2765,16 @@ static struct acpi_driver sony_pic_driver = { }, }; +static struct sony_pic_quirk_entry sony_pic_vaio_vgn = { + .set_wwan_power =1 , +}; + +static int dmi_matched (const struct dmi_system_id *dmi) +{ + spic_dev.quirks = dmi->driver_data; +return 0 ; +} +static struct dmi_system_id __initdata sonypi_dmi_table[] = { { .ident ="Sony Vaio" ,
@@ -2769,6 +2789,8 @@ static struct dmi_system_id __initdata sonypi_dmi_table[] = { DMI_MATCH(DMI_SYS_VENDOR, "Sony Corporation"), DMI_MATCH(DMI_PRODUCT_NAME, "VGN-"), }, + .callback = dmi_matched, + .driver_data = &sony_pic_vaio_vgn, }, { } };
Signature · 1 line
1.5.6.3
Quoted 4 lines · Tim Gardner
Mattia,
Does this patch makes sense for just the Vaio? I've had positive feedback from the tester in the bug report.
It'd make more sense to change the wwan power option to use rfkill. That way it'll be powered up at appropriate times.
Signature · 1 line
Matthew Garrett | [email protected]
Quoted 9 lines · Matthew Garrett
Quoted 4 lines · Tim Gardner
Mattia,
Does this patch makes sense for just the Vaio? I've had positive feedback from the tester in the bug report.
It'd make more sense to change the wwan power option to use rfkill. That way it'll be powered up at appropriate times.
The laptop model that doesn't power up wwan on cold boot contains a "Sony Programmable IO Control Driver" ("VGN-" according to the DMI info). It appears that only laptops with a "Sony Notebook Control Driver" enjoy rfkill goodness. I don't know for sure if those models fail to power up wwan when rfkill is disabled.
rtg
Signature · 2 lines
Tim Gardner [email protected] www.tpi.com OR 503-601-0234 x102 MT 406-443-5357
Quoted 5 lines · Tim Gardner
The laptop model that doesn't power up wwan on cold boot contains a "Sony Programmable IO Control Driver" ("VGN-" according to the DMI info). It appears that only laptops with a "Sony Notebook Control Driver" enjoy rfkill goodness. I don't know for sure if those models fail to power up wwan when rfkill is disabled.
Right. It'd need porting to the rfkill infrastructure, rather than just being a sysfs attribute.
Signature · 1 line
Matthew Garrett | [email protected]
Quoted 10 lines · Matthew Garrett
Quoted 5 lines · Tim Gardner
The laptop model that doesn't power up wwan on cold boot contains a "Sony Programmable IO Control Driver" ("VGN-" according to the DMI info). It appears that only laptops with a "Sony Notebook Control Driver" enjoy rfkill goodness. I don't know for sure if those models fail to power up wwan when rfkill is disabled.
Right. It'd need porting to the rfkill infrastructure, rather than just being a sysfs attribute.
Agreed, the only thing I'm missing is how to read the initial device state (same for bluetooth and wifi).
I've been throwing some values at the SPIC device with much success for now. Tracing in windows is a big no-no as it dies on me as soon as I start any tracing utility.
I'm open for suggestions.
Signature · 2 lines
mattia :wq!
Quoted 12 lines · Tim Gardner
Mattia,
Does this patch makes sense for just the Vaio? I've had positive feedback from the tester in the bug report.
rtg
-----------------
From083972abb1d5e84845927b3925f077953c207b82Mon Sep 17 00:00:00 2001
From: Tim Gardner <[email protected]>
Date: Tue, 21 Apr 2009 11:09:10 -0600
Subject: [PATCH] Sony laptop: Some Sony Vaio laptops do not enable wwan power by default.
Quoted 4 lines
Added quirk to enable wwan power based on DMI information already present in the module. It appears that wwan is not powered up from a cold boot.
The bug reporter says that this happens when booting from windows to linux, is it still true when manually enabling the device in linux and then rebooting (into linux again)?
Quoted 63 lines
Signed-off-by Tim Gardner <[email protected]>--- drivers/misc/sony-laptop.c | 24 +++++++++++++++++++++++- 1 files changed, 23 insertions(+), 1 deletions(-)drivers/misc/sony-laptop.c +21
-1@@ -1248,10 +1248,13 @@ struct device_ctrl {u16 evport_offset;u8 has_camera;u8 has_bluetooth; -u8 has_wwan;struct sonypi_eventtypes *event_types; }; +struct sony_pic_quirk_entry { +u8 set_wwan_power; +}; +struct sony_pic_dev {struct device_ctrl *control;struct acpi_device *acpi_dev;@@ -1260,6 +1263,7 @@ struct sony_pic_dev {struct list_head interrupts;struct list_head ioports;struct mutex lock; +struct sony_pic_quirk_entry *quirks;u8 camera_power;u8 bluetooth_power;u8 wwan_power;@@ -2685,6 +2689,12 @@ static int sony_pic_add(struct acpi_device *device)if (result) goto err_remove_pf; +if (spic_dev.quirks && spic_dev.quirks->set_wwan_power) { +/* +* Power isn't enabled by default. +*/ +sony_pic_set_wwanpower (1 ); + }return 0 ; err_remove_pf:@@ -2755,6 +2765,16 @@ static struct acpi_driver sony_pic_driver = { }, }; +static struct sony_pic_quirk_entry sony_pic_vaio_vgn = { + .set_wwan_power =1 , +}; + +static int dmi_matched (const struct dmi_system_id *dmi) +{ + spic_dev.quirks = dmi->driver_data; +return 0 ; +} +static struct dmi_system_id __initdata sonypi_dmi_table[] = { { .ident ="Sony Vaio" ,
drivers/misc/sony-laptop.c static struct dmi_system_id __initdata sonypi_dmi_table[] = { · line 2793
Quoted 5 lines
drivers/misc/sony-laptop.c +2
-0@@ -2769,6 +2789,8 @@ static struct dmi_system_id __initdata sonypi_dmi_table[] = {DMI_MATCH (DMI_SYS_VENDOR ,"Sony Corporation" ), DMI_MATCH(DMI_PRODUCT_NAME, "VGN-"), }, + .callback = dmi_matched, + .driver_data = &sony_pic_vaio_vgn,
This is like just enabling it on all type3 (and some type2) vaios. Might as well call your quirk unconditionally for all models.
Honestly I'd rather work towards the rfkill implementation as Matthew suggests.
Signature · 2 lines
mattia :wq!
Quoted 13 lines · Mattia Dongili
Quoted 12 lines · Tim Gardner
Mattia,
Does this patch makes sense for just the Vaio? I've had positive feedback from the tester in the bug report.
rtg
-----------------
From083972abb1d5e84845927b3925f077953c207b82Mon Sep 17 00:00:00 2001
From: Tim Gardner <[email protected]>
Date: Tue, 21 Apr 2009 11:09:10 -0600
Subject: [PATCH] Sony laptop: Some Sony Vaio laptops do not enable wwan power by default.
Quoted 9 lines
Quoted 4 lines
Added quirk to enable wwan power based on DMI information already present in the module. It appears that wwan is not powered up from a cold boot.
The bug reporter says that this happens when booting from windows to linux, is it still true when manually enabling the device in linux and then rebooting (into linux again)?
I can probably find out, though I'm not sure what impact it would have on the driver. Does this laptop have a physical rfkill slider switch from which you can somehow query initial state via ACPI or GPIO? If not, then I assume the desired behavior is to just enable wwan and bluetooth upon every boot ?
Quoted 72 lines
Quoted 63 lines
Signed-off-by Tim Gardner <[email protected]>--- drivers/misc/sony-laptop.c | 24 +++++++++++++++++++++++- 1 files changed, 23 insertions(+), 1 deletions(-)drivers/misc/sony-laptop.c +21
-1@@ -1248,10 +1248,13 @@ struct device_ctrl {u16 evport_offset;u8 has_camera;u8 has_bluetooth; -u8 has_wwan;struct sonypi_eventtypes *event_types; }; +struct sony_pic_quirk_entry { +u8 set_wwan_power; +}; +struct sony_pic_dev {struct device_ctrl *control;struct acpi_device *acpi_dev;@@ -1260,6 +1263,7 @@ struct sony_pic_dev {struct list_head interrupts;struct list_head ioports;struct mutex lock; +struct sony_pic_quirk_entry *quirks;u8 camera_power;u8 bluetooth_power;u8 wwan_power;@@ -2685,6 +2689,12 @@ static int sony_pic_add(struct acpi_device *device)if (result) goto err_remove_pf; +if (spic_dev.quirks && spic_dev.quirks->set_wwan_power) { +/* +* Power isn't enabled by default. +*/ +sony_pic_set_wwanpower (1 ); + }return 0 ; err_remove_pf:@@ -2755,6 +2765,16 @@ static struct acpi_driver sony_pic_driver = { }, }; +static struct sony_pic_quirk_entry sony_pic_vaio_vgn = { + .set_wwan_power =1 , +}; + +static int dmi_matched (const struct dmi_system_id *dmi) +{ + spic_dev.quirks = dmi->driver_data; +return 0 ; +} +static struct dmi_system_id __initdata sonypi_dmi_table[] = { { .ident ="Sony Vaio" ,drivers/misc/sony-laptop.c static struct dmi_system_id __initdata sonypi_dmi_table[] = { · line 2793
Quoted 5 lines
drivers/misc/sony-laptop.c +2
-0@@ -2769,6 +2789,8 @@ static struct dmi_system_id __initdata sonypi_dmi_table[] = {DMI_MATCH (DMI_SYS_VENDOR ,"Sony Corporation" ), DMI_MATCH(DMI_PRODUCT_NAME, "VGN-"), }, + .callback = dmi_matched, + .driver_data = &sony_pic_vaio_vgn,This is like just enabling it on all type3 (and some type2) vaios. Might as well call your quirk unconditionally for all models.
Quoted 2 lines
Honestly I'd rather work towards the rfkill implementation as Matthew suggests.
Agreed.
rtg
Signature · 2 lines
Tim Gardner [email protected] www.tpi.com OR 503-601-0234 x102 MT 406-443-5357