linux-acpi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/9] sony-laptop for 2.6.30
@ 2009-03-01  4:40 Mattia Dongili
  2009-03-01  4:40 ` [PATCH 1/9] [sony-laptop] merge Type4 into Type3 Mattia Dongili
                   ` (9 more replies)
  0 siblings, 10 replies; 12+ messages in thread
From: Mattia Dongili @ 2009-03-01  4:40 UTC (permalink / raw)
  To: Len Brown; +Cc: linux-acpi

Hello Len,

This is a list of long overdue patches I had sitting in my local tree.
Please could you include them in you batch for 2.6.30?

PS: apologies to those who receive these emails twice (due to me trying
git-send-email ;))

Thanks
---
 drivers/platform/x86/sony-laptop.c |  126 +++++++++++++++++++++++++-----------
 include/linux/sonypi.h             |    3 +
 2 files changed, 91 insertions(+), 38 deletions(-)
-- 
mattia


^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 1/9] [sony-laptop] merge Type4 into Type3
  2009-03-01  4:40 [PATCH 0/9] sony-laptop for 2.6.30 Mattia Dongili
@ 2009-03-01  4:40 ` Mattia Dongili
  2009-03-01  4:40 ` [PATCH 2/9] [sony-laptop] VGN-A317M hotkey support Mattia Dongili
                   ` (8 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Mattia Dongili @ 2009-03-01  4:40 UTC (permalink / raw)
  To: Len Brown; +Cc: linux-acpi, Mattia Dongili

Creating Type4 was a mistake in the first place. Some users report that
also Type3 vaios require the same extra hotkey handling which the Type4
for was menat to guard from.
Merging down Type4 into Type3 will just remove a useless distinction.

Signed-off-by: Mattia Dongili <malattia@linux.it>
---
 drivers/platform/x86/sony-laptop.c |   28 ++++++----------------------
 1 files changed, 6 insertions(+), 22 deletions(-)

diff --git a/drivers/platform/x86/sony-laptop.c b/drivers/platform/x86/sony-laptop.c
index 537959d..5e43d0a 100644
--- a/drivers/platform/x86/sony-laptop.c
+++ b/drivers/platform/x86/sony-laptop.c
@@ -1195,7 +1195,6 @@ static struct acpi_driver sony_nc_driver = {
 #define SONYPI_TYPE1_OFFSET	0x04
 #define SONYPI_TYPE2_OFFSET	0x12
 #define SONYPI_TYPE3_OFFSET	0x12
-#define SONYPI_TYPE4_OFFSET	0x12
 
 struct sony_pic_ioport {
 	struct acpi_resource_io	io1;
@@ -1438,14 +1437,6 @@ static struct sonypi_eventtypes type3_events[] = {
 	{ 0x31, SONYPI_MEMORYSTICK_MASK, sonypi_memorystickev },
 	{ 0x41, SONYPI_BATTERY_MASK, sonypi_batteryev },
 	{ 0x31, SONYPI_PKEY_MASK, sonypi_pkeyev },
-	{ 0 },
-};
-static struct sonypi_eventtypes type4_events[] = {
-	{ 0, 0xffffffff, sonypi_releaseev },
-	{ 0x21, SONYPI_FNKEY_MASK, sonypi_fnkeyev },
-	{ 0x31, SONYPI_WIRELESS_MASK, sonypi_wlessev },
-	{ 0x31, SONYPI_MEMORYSTICK_MASK, sonypi_memorystickev },
-	{ 0x41, SONYPI_BATTERY_MASK, sonypi_batteryev },
 	{ 0x05, SONYPI_PKEY_MASK, sonypi_pkeyev },
 	{ 0x05, SONYPI_ZOOM_MASK, sonypi_zoomev },
 	{ 0x05, SONYPI_CAPTURE_MASK, sonypi_captureev },
@@ -1511,11 +1502,11 @@ static u8 sony_pic_call3(u8 dev, u8 fn, u8 v)
 /*
  * minidrivers for SPIC models
  */
-static int type4_handle_irq(const u8 data_mask, const u8 ev)
+static int type3_handle_irq(const u8 data_mask, const u8 ev)
 {
 	/*
 	 * 0x31 could mean we have to take some extra action and wait for
-	 * the next irq for some Type4 models, it will generate a new
+	 * the next irq for some Type3 models, it will generate a new
 	 * irq and we can read new data from the device:
 	 *  - 0x5c and 0x5f requires 0xA0
 	 *  - 0x61 requires 0xB3
@@ -1545,16 +1536,10 @@ static struct device_ctrl spic_types[] = {
 	},
 	{
 		.model = SONYPI_DEVICE_TYPE3,
-		.handle_irq = NULL,
+		.handle_irq = type3_handle_irq,
 		.evport_offset = SONYPI_TYPE3_OFFSET,
 		.event_types = type3_events,
 	},
-	{
-		.model = SONYPI_DEVICE_TYPE4,
-		.handle_irq = type4_handle_irq,
-		.evport_offset = SONYPI_TYPE4_OFFSET,
-		.event_types = type4_events,
-	},
 };
 
 static void sony_pic_detect_device_type(struct sony_pic_dev *dev)
@@ -1578,14 +1563,14 @@ static void sony_pic_detect_device_type(struct sony_pic_dev *dev)
 	pcidev = pci_get_device(PCI_VENDOR_ID_INTEL,
 			PCI_DEVICE_ID_INTEL_ICH7_1, NULL);
 	if (pcidev) {
-		dev->control = &spic_types[3];
+		dev->control = &spic_types[2];
 		goto out;
 	}
 
 	pcidev = pci_get_device(PCI_VENDOR_ID_INTEL,
 			PCI_DEVICE_ID_INTEL_ICH8_4, NULL);
 	if (pcidev) {
-		dev->control = &spic_types[3];
+		dev->control = &spic_types[2];
 		goto out;
 	}
 
@@ -1598,8 +1583,7 @@ out:
 
 	printk(KERN_INFO DRV_PFX "detected Type%d model\n",
 			dev->control->model == SONYPI_DEVICE_TYPE1 ? 1 :
-			dev->control->model == SONYPI_DEVICE_TYPE2 ? 2 :
-			dev->control->model == SONYPI_DEVICE_TYPE3 ? 3 : 4);
+			dev->control->model == SONYPI_DEVICE_TYPE2 ? 2 : 3);
 }
 
 /* camera tests and poweron/poweroff */
-- 
1.6.1.3


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 2/9] [sony-laptop] VGN-A317M hotkey support
  2009-03-01  4:40 [PATCH 0/9] sony-laptop for 2.6.30 Mattia Dongili
  2009-03-01  4:40 ` [PATCH 1/9] [sony-laptop] merge Type4 into Type3 Mattia Dongili
@ 2009-03-01  4:40 ` Mattia Dongili
  2009-03-01  4:40 ` [PATCH 3/9] [sony-laptop] Eliminate BKL in ioctls Mattia Dongili
                   ` (7 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Mattia Dongili @ 2009-03-01  4:40 UTC (permalink / raw)
  To: Len Brown; +Cc: linux-acpi, Harald Jenny, Mattia Dongili

From: Harald Jenny <harald@a-little-linux-box.at>

This laptop has 5 SPIC managed buttons above the keyboard:
sound + and - as well as brightness, zoom and S1.
Possibly the entire VGN-A serie behaves the same.

Signed-off-by: Mattia Dongili <malattia@linux.it>
---
 drivers/platform/x86/sony-laptop.c |   24 +++++++++++++++++++++++-
 include/linux/sonypi.h             |    3 +++
 2 files changed, 26 insertions(+), 1 deletions(-)

diff --git a/drivers/platform/x86/sony-laptop.c b/drivers/platform/x86/sony-laptop.c
index 5e43d0a..c710840 100644
--- a/drivers/platform/x86/sony-laptop.c
+++ b/drivers/platform/x86/sony-laptop.c
@@ -211,6 +211,9 @@ static int sony_laptop_input_index[] = {
 	48,	/* 61 SONYPI_EVENT_WIRELESS_OFF */
 	49,	/* 62 SONYPI_EVENT_ZOOM_IN_PRESSED */
 	50,	/* 63 SONYPI_EVENT_ZOOM_OUT_PRESSED */
+	51,	/* 64 SONYPI_EVENT_VOLUME_INC_PRESSED */
+	52,	/* 65 SONYPI_EVENT_VOLUME_DEC_PRESSED */
+	-1,	/* 66 SONYPI_EVENT_BRIGHTNESS_PRESSED */
 };
 
 static int sony_laptop_input_keycode_map[] = {
@@ -264,7 +267,9 @@ static int sony_laptop_input_keycode_map[] = {
 	KEY_WLAN,	/* 47 SONYPI_EVENT_WIRELESS_ON */
 	KEY_WLAN,	/* 48 SONYPI_EVENT_WIRELESS_OFF */
 	KEY_ZOOMIN,	/* 49 SONYPI_EVENT_ZOOM_IN_PRESSED */
-	KEY_ZOOMOUT	/* 50 SONYPI_EVENT_ZOOM_OUT_PRESSED */
+	KEY_ZOOMOUT,	/* 50 SONYPI_EVENT_ZOOM_OUT_PRESSED */
+	KEY_VOLUMEUP,	/* 51 SONYPI_EVENT_VOLUME_INC_PRESSED */
+	KEY_VOLUMEDOWN,	/* 52 SONYPI_EVENT_VOLUME_DEC_PRESSED */
 };
 
 /* release buttons after a short delay if pressed */
@@ -1327,6 +1332,7 @@ static struct sonypi_event sonypi_pkeyev[] = {
 	{ 0x01, SONYPI_EVENT_PKEY_P1 },
 	{ 0x02, SONYPI_EVENT_PKEY_P2 },
 	{ 0x04, SONYPI_EVENT_PKEY_P3 },
+	{ 0x20, SONYPI_EVENT_PKEY_P1 },
 	{ 0, 0 }
 };
 
@@ -1370,6 +1376,7 @@ static struct sonypi_event sonypi_zoomev[] = {
 	{ 0x39, SONYPI_EVENT_ZOOM_PRESSED },
 	{ 0x10, SONYPI_EVENT_ZOOM_IN_PRESSED },
 	{ 0x20, SONYPI_EVENT_ZOOM_OUT_PRESSED },
+	{ 0x04, SONYPI_EVENT_ZOOM_PRESSED },
 	{ 0, 0 }
 };
 
@@ -1400,6 +1407,19 @@ static struct sonypi_event sonypi_batteryev[] = {
 	{ 0, 0 }
 };
 
+/* The set of possible volume events */
+static struct sonypi_event sonypi_volumeev[] = {
+	{ 0x01, SONYPI_EVENT_VOLUME_INC_PRESSED },
+	{ 0x02, SONYPI_EVENT_VOLUME_DEC_PRESSED },
+	{ 0, 0 }
+};
+
+/* The set of possible brightness events */
+static struct sonypi_event sonypi_brightnessev[] = {
+	{ 0x80, SONYPI_EVENT_BRIGHTNESS_PRESSED },
+	{ 0, 0 }
+};
+
 static struct sonypi_eventtypes type1_events[] = {
 	{ 0, 0xffffffff, sonypi_releaseev },
 	{ 0x70, SONYPI_MEYE_MASK, sonypi_meyeev },
@@ -1440,6 +1460,8 @@ static struct sonypi_eventtypes type3_events[] = {
 	{ 0x05, SONYPI_PKEY_MASK, sonypi_pkeyev },
 	{ 0x05, SONYPI_ZOOM_MASK, sonypi_zoomev },
 	{ 0x05, SONYPI_CAPTURE_MASK, sonypi_captureev },
+	{ 0x05, SONYPI_PKEY_MASK, sonypi_volumeev },
+	{ 0x05, SONYPI_PKEY_MASK, sonypi_brightnessev },
 	{ 0 },
 };
 
diff --git a/include/linux/sonypi.h b/include/linux/sonypi.h
index f41ffd7..ce0c80f 100644
--- a/include/linux/sonypi.h
+++ b/include/linux/sonypi.h
@@ -103,6 +103,9 @@
 #define SONYPI_EVENT_WIRELESS_OFF		61
 #define SONYPI_EVENT_ZOOM_IN_PRESSED		62
 #define SONYPI_EVENT_ZOOM_OUT_PRESSED		63
+#define SONYPI_EVENT_VOLUME_INC_PRESSED		64
+#define SONYPI_EVENT_VOLUME_DEC_PRESSED		65
+#define SONYPI_EVENT_BRIGHTNESS_PRESSED		66
 
 /* get/set brightness */
 #define SONYPI_IOCGBRT		_IOR('v', 0, __u8)
-- 
1.6.1.3


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 3/9] [sony-laptop] Eliminate BKL in ioctls
  2009-03-01  4:40 [PATCH 0/9] sony-laptop for 2.6.30 Mattia Dongili
  2009-03-01  4:40 ` [PATCH 1/9] [sony-laptop] merge Type4 into Type3 Mattia Dongili
  2009-03-01  4:40 ` [PATCH 2/9] [sony-laptop] VGN-A317M hotkey support Mattia Dongili
@ 2009-03-01  4:40 ` Mattia Dongili
  2009-03-01  4:40 ` [PATCH 4/9] [sony-laptop] detect the ICH9 chipset as Type3 Mattia Dongili
                   ` (6 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Mattia Dongili @ 2009-03-01  4:40 UTC (permalink / raw)
  To: Len Brown; +Cc: linux-acpi, Alan Cox, Alan Cox, Mattia Dongili

From: Alan Cox <alan@lxorguk.ukuu.org.uk>

Signed-off-by: Alan Cox <alan@redhat.com>
Signed-off-by: Mattia Dongili <malattia@linux.it>
---
 drivers/platform/x86/sony-laptop.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/platform/x86/sony-laptop.c b/drivers/platform/x86/sony-laptop.c
index c710840..35491d9 100644
--- a/drivers/platform/x86/sony-laptop.c
+++ b/drivers/platform/x86/sony-laptop.c
@@ -1996,8 +1996,8 @@ static int ec_read16(u8 addr, u16 *value)
 	return 0;
 }
 
-static int sonypi_misc_ioctl(struct inode *ip, struct file *fp,
-			     unsigned int cmd, unsigned long arg)
+static long sonypi_misc_ioctl(struct file *fp, unsigned int cmd,
+							unsigned long arg)
 {
 	int ret = 0;
 	void __user *argp = (void __user *)arg;
@@ -2131,7 +2131,7 @@ static const struct file_operations sonypi_misc_fops = {
 	.open		= sonypi_misc_open,
 	.release	= sonypi_misc_release,
 	.fasync		= sonypi_misc_fasync,
-	.ioctl		= sonypi_misc_ioctl,
+	.unlocked_ioctl	= sonypi_misc_ioctl,
 };
 
 static struct miscdevice sonypi_misc_device = {
-- 
1.6.1.3


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 4/9] [sony-laptop] detect the ICH9 chipset as Type3
  2009-03-01  4:40 [PATCH 0/9] sony-laptop for 2.6.30 Mattia Dongili
                   ` (2 preceding siblings ...)
  2009-03-01  4:40 ` [PATCH 3/9] [sony-laptop] Eliminate BKL in ioctls Mattia Dongili
@ 2009-03-01  4:40 ` Mattia Dongili
  2009-03-01  4:40 ` [PATCH 5/9] [sony-laptop] call the extra enable function for SNC on Vaio Z Mattia Dongili
                   ` (5 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Mattia Dongili @ 2009-03-01  4:40 UTC (permalink / raw)
  To: Len Brown; +Cc: linux-acpi, ISHIKAWA Mutsumi, Mattia Dongili

From: ISHIKAWA Mutsumi <ishikawa@hanzubon.jp>

Signed-off-by: Mattia Dongili <malattia@linux.it>
---
 drivers/platform/x86/sony-laptop.c |    7 +++++++
 1 files changed, 7 insertions(+), 0 deletions(-)

diff --git a/drivers/platform/x86/sony-laptop.c b/drivers/platform/x86/sony-laptop.c
index 35491d9..3322368 100644
--- a/drivers/platform/x86/sony-laptop.c
+++ b/drivers/platform/x86/sony-laptop.c
@@ -1596,6 +1596,13 @@ static void sony_pic_detect_device_type(struct sony_pic_dev *dev)
 		goto out;
 	}
 
+	pcidev = pci_get_device(PCI_VENDOR_ID_INTEL,
+			PCI_DEVICE_ID_INTEL_ICH9_1, NULL);
+	if (pcidev) {
+		dev->control = &spic_types[2];
+		goto out;
+	}
+
 	/* default */
 	dev->control = &spic_types[1];
 
-- 
1.6.1.3


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 5/9] [sony-laptop] call the extra enable function for SNC on Vaio Z
  2009-03-01  4:40 [PATCH 0/9] sony-laptop for 2.6.30 Mattia Dongili
                   ` (3 preceding siblings ...)
  2009-03-01  4:40 ` [PATCH 4/9] [sony-laptop] detect the ICH9 chipset as Type3 Mattia Dongili
@ 2009-03-01  4:40 ` Mattia Dongili
  2009-03-01  4:40 ` [PATCH 6/9] [sony-laptop] notify the hardware of a state change in wwanpower Mattia Dongili
                   ` (4 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Mattia Dongili @ 2009-03-01  4:40 UTC (permalink / raw)
  To: Len Brown; +Cc: linux-acpi, ISHIKAWA Mutsumi, Mattia Dongili

From: ISHIKAWA Mutsumi <ishikawa@hanzubon.jp>

Signed-off-by: Mattia Dongili <malattia@linux.it>
---
 drivers/platform/x86/sony-laptop.c |    9 +++++++++
 1 files changed, 9 insertions(+), 0 deletions(-)

diff --git a/drivers/platform/x86/sony-laptop.c b/drivers/platform/x86/sony-laptop.c
index 3322368..f3bd13a 100644
--- a/drivers/platform/x86/sony-laptop.c
+++ b/drivers/platform/x86/sony-laptop.c
@@ -894,6 +894,15 @@ static const struct dmi_system_id sony_nc_ids[] = {
 				DMI_MATCH(DMI_PRODUCT_NAME, "VGN-N"),
 			},
 		},
+		{
+			.ident = "Sony Vaio Z Series",
+			.callback = sony_nc_C_enable,
+			.driver_data = sony_C_events,
+			.matches = {
+				DMI_MATCH(DMI_SYS_VENDOR, "Sony Corporation"),
+				DMI_MATCH(DMI_PRODUCT_NAME, "VGN-Z"),
+			},
+		},
 		{ }
 };
 
-- 
1.6.1.3


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 6/9] [sony-laptop] notify the hardware of a state change in wwanpower
  2009-03-01  4:40 [PATCH 0/9] sony-laptop for 2.6.30 Mattia Dongili
                   ` (4 preceding siblings ...)
  2009-03-01  4:40 ` [PATCH 5/9] [sony-laptop] call the extra enable function for SNC on Vaio Z Mattia Dongili
@ 2009-03-01  4:40 ` Mattia Dongili
  2009-03-01  4:40 ` [PATCH 7/9] [sony-laptop] Add FW specific hotkey events and DMI Mattia Dongili
                   ` (3 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Mattia Dongili @ 2009-03-01  4:40 UTC (permalink / raw)
  To: Len Brown; +Cc: linux-acpi, Sergey Yanovich, Mattia Dongili

From: Sergey Yanovich <ynvich@gmail.com>

From: Sergey Yanovich <ynvich@gmail.com>
Signed-off-by: Mattia Dongili <malattia@linux.it>
---
 drivers/platform/x86/sony-laptop.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/drivers/platform/x86/sony-laptop.c b/drivers/platform/x86/sony-laptop.c
index f3bd13a..8568d1e 100644
--- a/drivers/platform/x86/sony-laptop.c
+++ b/drivers/platform/x86/sony-laptop.c
@@ -1785,6 +1785,7 @@ static void sony_pic_set_wwanpower(u8 state)
 		return;
 	}
 	sony_pic_call2(0xB0, state);
+	sony_pic_call1(0x82);
 	spic_dev.wwan_power = state;
 	mutex_unlock(&spic_dev.lock);
 }
-- 
1.6.1.3


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 7/9] [sony-laptop] Add FW specific hotkey events and DMI
  2009-03-01  4:40 [PATCH 0/9] sony-laptop for 2.6.30 Mattia Dongili
                   ` (5 preceding siblings ...)
  2009-03-01  4:40 ` [PATCH 6/9] [sony-laptop] notify the hardware of a state change in wwanpower Mattia Dongili
@ 2009-03-01  4:40 ` Mattia Dongili
  2009-03-10  1:53   ` Mattia Dongili
  2009-03-01  4:40 ` [PATCH 8/9] [sony-laptop] Rename the SNC callback and events Mattia Dongili
                   ` (2 subsequent siblings)
  9 siblings, 1 reply; 12+ messages in thread
From: Mattia Dongili @ 2009-03-01  4:40 UTC (permalink / raw)
  To: Len Brown; +Cc: linux-acpi, Anton Veretenenko, Mattia Dongili

From: Anton Veretenenko <anton@veretenenko.ru>

From: Anton Veretenenko <anton@veretenenko.ru>
Signed-off-by: Mattia Dongili <malattia@linux.it>
---
 drivers/platform/x86/sony-laptop.c |   18 ++++++++++++++++++
 1 files changed, 18 insertions(+), 0 deletions(-)

diff --git a/drivers/platform/x86/sony-laptop.c b/drivers/platform/x86/sony-laptop.c
index 8568d1e..576c9c2 100644
--- a/drivers/platform/x86/sony-laptop.c
+++ b/drivers/platform/x86/sony-laptop.c
@@ -843,16 +843,25 @@ static int sony_nc_C_enable(const struct dmi_system_id *id)
 static struct sony_nc_event sony_C_events[] = {
 	{ 0x81, SONYPI_EVENT_FNKEY_F1 },
 	{ 0x01, SONYPI_EVENT_FNKEY_RELEASED },
+	{ 0x82, SONYPI_EVENT_FNKEY_F2 },
+	{ 0x02, SONYPI_EVENT_FNKEY_RELEASED },
 	{ 0x85, SONYPI_EVENT_FNKEY_F5 },
 	{ 0x05, SONYPI_EVENT_FNKEY_RELEASED },
+	{ 0x85, SONYPI_EVENT_FNKEY_F5 },
 	{ 0x86, SONYPI_EVENT_FNKEY_F6 },
 	{ 0x06, SONYPI_EVENT_FNKEY_RELEASED },
 	{ 0x87, SONYPI_EVENT_FNKEY_F7 },
 	{ 0x07, SONYPI_EVENT_FNKEY_RELEASED },
+	{ 0x89, SONYPI_EVENT_FNKEY_F9 },
+	{ 0x09, SONYPI_EVENT_FNKEY_RELEASED },
 	{ 0x8A, SONYPI_EVENT_FNKEY_F10 },
 	{ 0x0A, SONYPI_EVENT_FNKEY_RELEASED },
 	{ 0x8C, SONYPI_EVENT_FNKEY_F12 },
 	{ 0x0C, SONYPI_EVENT_FNKEY_RELEASED },
+	{ 0x90, SONYPI_EVENT_PKEY_P1 },
+	{ 0x10, SONYPI_EVENT_FNKEY_RELEASED },
+	{ 0xA1, SONYPI_EVENT_PKEY_P2 },
+	{ 0x21, SONYPI_EVENT_FNKEY_RELEASED },
 	{ 0, 0 },
 };
 
@@ -868,6 +877,15 @@ static const struct dmi_system_id sony_nc_ids[] = {
 			},
 		},
 		{
+			.ident = "Sony Vaio FW Series",
+			.callback = sony_nc_C_enable,
+			.driver_data = sony_C_events,
+			.matches = {
+				DMI_MATCH(DMI_SYS_VENDOR, "Sony Corporation"),
+				DMI_MATCH(DMI_PRODUCT_NAME, "VGN-FW"),
+			},
+		},
+		{
 			.ident = "Sony Vaio FZ Series",
 			.callback = sony_nc_C_enable,
 			.driver_data = sony_C_events,
-- 
1.6.1.3


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 8/9] [sony-laptop] Rename the SNC callback and events
  2009-03-01  4:40 [PATCH 0/9] sony-laptop for 2.6.30 Mattia Dongili
                   ` (6 preceding siblings ...)
  2009-03-01  4:40 ` [PATCH 7/9] [sony-laptop] Add FW specific hotkey events and DMI Mattia Dongili
@ 2009-03-01  4:40 ` Mattia Dongili
  2009-03-01  4:40 ` [PATCH 9/9] [sony-laptop] add Sony Vaio VGN-SR19 support Mattia Dongili
  2009-03-16  5:10 ` [PATCH 0/9] sony-laptop for 2.6.30 Len Brown
  9 siblings, 0 replies; 12+ messages in thread
From: Mattia Dongili @ 2009-03-01  4:40 UTC (permalink / raw)
  To: Len Brown; +Cc: linux-acpi, Mattia Dongili

This two objects are not specific to the C series.

Signed-off-by: Mattia Dongili <malattia@linux.it>
---
 drivers/platform/x86/sony-laptop.c |   32 ++++++++++++++++----------------
 1 files changed, 16 insertions(+), 16 deletions(-)

diff --git a/drivers/platform/x86/sony-laptop.c b/drivers/platform/x86/sony-laptop.c
index 576c9c2..021c9d2 100644
--- a/drivers/platform/x86/sony-laptop.c
+++ b/drivers/platform/x86/sony-laptop.c
@@ -816,10 +816,10 @@ struct sony_nc_event {
 
 static struct sony_nc_event *sony_nc_events;
 
-/* Vaio C* --maybe also FE*, N* and AR* ?-- special init sequence
- * for Fn keys
+/* Some Vaio models require additional steps to enable
+ * Fn keys
  */
-static int sony_nc_C_enable(const struct dmi_system_id *id)
+static int sony_SNC_enable_cb(const struct dmi_system_id *id)
 {
 	int result = 0;
 
@@ -840,7 +840,7 @@ static int sony_nc_C_enable(const struct dmi_system_id *id)
 	return 0;
 }
 
-static struct sony_nc_event sony_C_events[] = {
+static struct sony_nc_event sony_SNC_events[] = {
 	{ 0x81, SONYPI_EVENT_FNKEY_F1 },
 	{ 0x01, SONYPI_EVENT_FNKEY_RELEASED },
 	{ 0x82, SONYPI_EVENT_FNKEY_F2 },
@@ -869,8 +869,8 @@ static struct sony_nc_event sony_C_events[] = {
 static const struct dmi_system_id sony_nc_ids[] = {
 		{
 			.ident = "Sony Vaio FE Series",
-			.callback = sony_nc_C_enable,
-			.driver_data = sony_C_events,
+			.callback = sony_SNC_enable_cb,
+			.driver_data = sony_SNC_events,
 			.matches = {
 				DMI_MATCH(DMI_SYS_VENDOR, "Sony Corporation"),
 				DMI_MATCH(DMI_PRODUCT_NAME, "VGN-FE"),
@@ -878,8 +878,8 @@ static const struct dmi_system_id sony_nc_ids[] = {
 		},
 		{
 			.ident = "Sony Vaio FW Series",
-			.callback = sony_nc_C_enable,
-			.driver_data = sony_C_events,
+			.callback = sony_SNC_enable_cb,
+			.driver_data = sony_SNC_events,
 			.matches = {
 				DMI_MATCH(DMI_SYS_VENDOR, "Sony Corporation"),
 				DMI_MATCH(DMI_PRODUCT_NAME, "VGN-FW"),
@@ -887,8 +887,8 @@ static const struct dmi_system_id sony_nc_ids[] = {
 		},
 		{
 			.ident = "Sony Vaio FZ Series",
-			.callback = sony_nc_C_enable,
-			.driver_data = sony_C_events,
+			.callback = sony_SNC_enable_cb,
+			.driver_data = sony_SNC_events,
 			.matches = {
 				DMI_MATCH(DMI_SYS_VENDOR, "Sony Corporation"),
 				DMI_MATCH(DMI_PRODUCT_NAME, "VGN-FZ"),
@@ -896,8 +896,8 @@ static const struct dmi_system_id sony_nc_ids[] = {
 		},
 		{
 			.ident = "Sony Vaio C Series",
-			.callback = sony_nc_C_enable,
-			.driver_data = sony_C_events,
+			.callback = sony_SNC_enable_cb,
+			.driver_data = sony_SNC_events,
 			.matches = {
 				DMI_MATCH(DMI_SYS_VENDOR, "Sony Corporation"),
 				DMI_MATCH(DMI_PRODUCT_NAME, "VGN-C"),
@@ -905,8 +905,8 @@ static const struct dmi_system_id sony_nc_ids[] = {
 		},
 		{
 			.ident = "Sony Vaio N Series",
-			.callback = sony_nc_C_enable,
-			.driver_data = sony_C_events,
+			.callback = sony_SNC_enable_cb,
+			.driver_data = sony_SNC_events,
 			.matches = {
 				DMI_MATCH(DMI_SYS_VENDOR, "Sony Corporation"),
 				DMI_MATCH(DMI_PRODUCT_NAME, "VGN-N"),
@@ -914,8 +914,8 @@ static const struct dmi_system_id sony_nc_ids[] = {
 		},
 		{
 			.ident = "Sony Vaio Z Series",
-			.callback = sony_nc_C_enable,
-			.driver_data = sony_C_events,
+			.callback = sony_SNC_enable_cb,
+			.driver_data = sony_SNC_events,
 			.matches = {
 				DMI_MATCH(DMI_SYS_VENDOR, "Sony Corporation"),
 				DMI_MATCH(DMI_PRODUCT_NAME, "VGN-Z"),
-- 
1.6.1.3


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 9/9] [sony-laptop] add Sony Vaio VGN-SR19 support
  2009-03-01  4:40 [PATCH 0/9] sony-laptop for 2.6.30 Mattia Dongili
                   ` (7 preceding siblings ...)
  2009-03-01  4:40 ` [PATCH 8/9] [sony-laptop] Rename the SNC callback and events Mattia Dongili
@ 2009-03-01  4:40 ` Mattia Dongili
  2009-03-16  5:10 ` [PATCH 0/9] sony-laptop for 2.6.30 Len Brown
  9 siblings, 0 replies; 12+ messages in thread
From: Mattia Dongili @ 2009-03-01  4:40 UTC (permalink / raw)
  To: Len Brown; +Cc: linux-acpi, Wolfram Strepp, Mattia Dongili

From: Wolfram Strepp <wstrepp@gmx.de>

I have a Sony Vaio VGN-SR19XN, running with Debian Lenny,
and i had a problem with some function keys: Backlight Up/Down
(Fn-F5/F6), Hibernate (Fn-F12), and some others, were not working.
I have described the problem in detail here (at the bottom):
https://bugs.launchpad.net/ubuntu/+source/acpi-support/+bug/251665

Signed-off-by: Wolfram Strepp <wstrepp@gmx.de>
Signed-off-by: Mattia Dongili <malattia@linux.it>
---
 drivers/platform/x86/sony-laptop.c |    9 +++++++++
 1 files changed, 9 insertions(+), 0 deletions(-)

diff --git a/drivers/platform/x86/sony-laptop.c b/drivers/platform/x86/sony-laptop.c
index 021c9d2..f0db14f 100644
--- a/drivers/platform/x86/sony-laptop.c
+++ b/drivers/platform/x86/sony-laptop.c
@@ -921,6 +921,15 @@ static const struct dmi_system_id sony_nc_ids[] = {
 				DMI_MATCH(DMI_PRODUCT_NAME, "VGN-Z"),
 			},
 		},
+		{
+			.ident = "Sony Vaio SR Series",
+			.callback = sony_SNC_enable_cb,
+			.driver_data = sony_SNC_events,
+			.matches = {
+				DMI_MATCH(DMI_SYS_VENDOR, "Sony Corporation"),
+				DMI_MATCH(DMI_PRODUCT_NAME, "VGN-SR"),
+			},
+		},
 		{ }
 };
 
-- 
1.6.1.3


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [PATCH 7/9] [sony-laptop] Add FW specific hotkey events and DMI
  2009-03-01  4:40 ` [PATCH 7/9] [sony-laptop] Add FW specific hotkey events and DMI Mattia Dongili
@ 2009-03-10  1:53   ` Mattia Dongili
  0 siblings, 0 replies; 12+ messages in thread
From: Mattia Dongili @ 2009-03-10  1:53 UTC (permalink / raw)
  To: Len Brown; +Cc: linux-acpi, Anton Veretenenko

On Sun, Mar 01, 2009 at 01:40:43PM +0900, Mattia Dongili wrote:
> From: Anton Veretenenko <anton@veretenenko.ru>
> 
> From: Anton Veretenenko <anton@veretenenko.ru>
> Signed-off-by: Mattia Dongili <malattia@linux.it>
> ---
>  drivers/platform/x86/sony-laptop.c |   18 ++++++++++++++++++
>  1 files changed, 18 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/platform/x86/sony-laptop.c b/drivers/platform/x86/sony-laptop.c
> index 8568d1e..576c9c2 100644
> --- a/drivers/platform/x86/sony-laptop.c
> +++ b/drivers/platform/x86/sony-laptop.c
> @@ -843,16 +843,25 @@ static int sony_nc_C_enable(const struct dmi_system_id *id)
>  static struct sony_nc_event sony_C_events[] = {
>  	{ 0x81, SONYPI_EVENT_FNKEY_F1 },
>  	{ 0x01, SONYPI_EVENT_FNKEY_RELEASED },
> +	{ 0x82, SONYPI_EVENT_FNKEY_F2 },
> +	{ 0x02, SONYPI_EVENT_FNKEY_RELEASED },
>  	{ 0x85, SONYPI_EVENT_FNKEY_F5 },
>  	{ 0x05, SONYPI_EVENT_FNKEY_RELEASED },
> +	{ 0x85, SONYPI_EVENT_FNKEY_F5 },

Gah, duplicate Fn+F5. I'll resend the whole serie with this fix and some
more additions

Thanks
-- 
mattia
:wq!

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 0/9] sony-laptop for 2.6.30
  2009-03-01  4:40 [PATCH 0/9] sony-laptop for 2.6.30 Mattia Dongili
                   ` (8 preceding siblings ...)
  2009-03-01  4:40 ` [PATCH 9/9] [sony-laptop] add Sony Vaio VGN-SR19 support Mattia Dongili
@ 2009-03-16  5:10 ` Len Brown
  9 siblings, 0 replies; 12+ messages in thread
From: Len Brown @ 2009-03-16  5:10 UTC (permalink / raw)
  To: Mattia Dongili; +Cc: linux-acpi

I snapped these up when i was traveling -- I think
I didn't get a chance to reply.

a couple of nits for next time...

Subject: [PATCH a/b] [sony-laptop] bla bla bla
should be:
Subject: [PATCH a/b] sony-laptop: bla bla bla

because everything between []'s get deleted by git-am

Also, some of the From: and Signed-off-by's were not quite right.
If you're not the original author, then the From: for the author
should be the first line in the body.  It doesn't again go down
by the Signed-off-by:.  Also, the orignal author should have
the 1st signed-off-by line, as they're generally intended to
show the path that the patch took to the kernel...

thanks,
Len Brown, Intel Open Source Technology Center

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2009-03-16  5:11 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-03-01  4:40 [PATCH 0/9] sony-laptop for 2.6.30 Mattia Dongili
2009-03-01  4:40 ` [PATCH 1/9] [sony-laptop] merge Type4 into Type3 Mattia Dongili
2009-03-01  4:40 ` [PATCH 2/9] [sony-laptop] VGN-A317M hotkey support Mattia Dongili
2009-03-01  4:40 ` [PATCH 3/9] [sony-laptop] Eliminate BKL in ioctls Mattia Dongili
2009-03-01  4:40 ` [PATCH 4/9] [sony-laptop] detect the ICH9 chipset as Type3 Mattia Dongili
2009-03-01  4:40 ` [PATCH 5/9] [sony-laptop] call the extra enable function for SNC on Vaio Z Mattia Dongili
2009-03-01  4:40 ` [PATCH 6/9] [sony-laptop] notify the hardware of a state change in wwanpower Mattia Dongili
2009-03-01  4:40 ` [PATCH 7/9] [sony-laptop] Add FW specific hotkey events and DMI Mattia Dongili
2009-03-10  1:53   ` Mattia Dongili
2009-03-01  4:40 ` [PATCH 8/9] [sony-laptop] Rename the SNC callback and events Mattia Dongili
2009-03-01  4:40 ` [PATCH 9/9] [sony-laptop] add Sony Vaio VGN-SR19 support Mattia Dongili
2009-03-16  5:10 ` [PATCH 0/9] sony-laptop for 2.6.30 Len Brown

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).