Mattia,
Does this patch makes sense for just the Vaio? I've had positive feedback from
the tester in the bug report.
rtg
-----------------
>From 083972abb1d5e84845927b3925f077953c207b82 Mon 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.
Bug: https://bugs.launchpad.net/bugs/364678
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.
Signed-off-by: Tim Gardner <[email protected]>
---
drivers/misc/sony-laptop.c | 24 +++++++++++++++++++++++-
1 files changed, 23 insertions(+), 1 deletions(-)
diff --git a/drivers/misc/sony-laptop.c b/drivers/misc/sony-laptop.c
index 7e73acc..eab7011 100644
--- a/drivers/misc/sony-laptop.c
+++ b/drivers/misc/sony-laptop.c
@@ -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,
},
{ }
};
--
1.5.6.3
On Wed, Apr 22, 2009 at 12:53:51PM -0600, Tim Gardner wrote:
> 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.
--
Matthew Garrett | [email protected]
Matthew Garrett wrote:
> On Wed, Apr 22, 2009 at 12:53:51PM -0600, Tim Gardner wrote:
>> 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
--
Tim Gardner [email protected] http://www.tpi.com
OR 503-601-0234 x102 MT 406-443-5357
On Wed, Apr 22, 2009 at 02:07:14PM -0600, Tim Gardner wrote:
> 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.
--
Matthew Garrett | [email protected]
On Wed, Apr 22, 2009 at 09:11:15PM +0100, Matthew Garrett wrote:
> On Wed, Apr 22, 2009 at 02:07:14PM -0600, Tim Gardner wrote:
>
> > 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.
--
mattia
:wq!
On Wed, Apr 22, 2009 at 12:53:51PM -0600, Tim Gardner wrote:
> Mattia,
>
> Does this patch makes sense for just the Vaio? I've had positive feedback from
> the tester in the bug report.
>
> rtg
> -----------------
> From 083972abb1d5e84845927b3925f077953c207b82 Mon 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.
>
> Bug: https://bugs.launchpad.net/bugs/364678
>
> 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)?
> Signed-off-by: Tim Gardner <[email protected]>
> ---
> drivers/misc/sony-laptop.c | 24 +++++++++++++++++++++++-
> 1 files changed, 23 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/misc/sony-laptop.c b/drivers/misc/sony-laptop.c
> index 7e73acc..eab7011 100644
> --- a/drivers/misc/sony-laptop.c
> +++ b/drivers/misc/sony-laptop.c
> @@ -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,
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.
--
mattia
:wq!
Mattia Dongili wrote:
> On Wed, Apr 22, 2009 at 12:53:51PM -0600, Tim Gardner wrote:
>> Mattia,
>>
>> Does this patch makes sense for just the Vaio? I've had positive feedback from
>> the tester in the bug report.
>>
>> rtg
>> -----------------
>> From 083972abb1d5e84845927b3925f077953c207b82 Mon 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.
>>
>> Bug: https://bugs.launchpad.net/bugs/364678
>>
>> 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 ?
>> Signed-off-by: Tim Gardner <[email protected]>
>> ---
>> drivers/misc/sony-laptop.c | 24 +++++++++++++++++++++++-
>> 1 files changed, 23 insertions(+), 1 deletions(-)
>>
>> diff --git a/drivers/misc/sony-laptop.c b/drivers/misc/sony-laptop.c
>> index 7e73acc..eab7011 100644
>> --- a/drivers/misc/sony-laptop.c
>> +++ b/drivers/misc/sony-laptop.c
>> @@ -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,
>
> 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.
Agreed.
rtg
--
Tim Gardner [email protected] http://www.tpi.com
OR 503-601-0234 x102 MT 406-443-5357