diff mbox

[3.5.y.z,extended,stable] Patch "HID: apple: Add support for the 2013 Macbook Air" has been added to staging queue

Message ID 1373281151-22914-1-git-send-email-luis.henriques@canonical.com
State New
Headers show

Commit Message

Luis Henriques July 8, 2013, 10:59 a.m. UTC
This is a note to let you know that I have just added a patch titled

    HID: apple: Add support for the 2013 Macbook Air

to the linux-3.5.y-queue branch of the 3.5.y.z extended stable tree 
which can be found at:

 http://kernel.ubuntu.com/git?p=ubuntu/linux.git;a=shortlog;h=refs/heads/linux-3.5.y-queue

If you, or anyone else, feels it should not be added to this tree, please 
reply to this email.

For more information about the 3.5.y.z tree, see
https://wiki.ubuntu.com/Kernel/Dev/ExtendedStable

Thanks.
-Luis

------

From 3fc49d21abb51328088a21aa35cd081db97d5754 Mon Sep 17 00:00:00 2001
From: Dmitry Torokhov <rydberg@euromail.se>
Date: Mon, 1 Jul 2013 11:46:27 -0700
Subject: [PATCH] HID: apple: Add support for the 2013 Macbook Air

commit 9d9a04ee758b4c1fcc7586d065cdde7a7607e156 upstream.

This patch adds keyboard support for MacbookAir6,2 as WELLSPRING8
(0x0291, 0x0292, 0x0293). The touchpad is handled in a separate
bcm5974 patch, as usual.

Reported-and-tested-by: Brad Ford <plymouthffl@gmail.com>
Signed-off-by: Henrik Rydberg <rydberg@euromail.se>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
[ luis: backported to 3.5: adjusted context ]
Signed-off-by: Luis Henriques <luis.henriques@canonical.com>
---
 drivers/hid/hid-apple.c | 6 ++++++
 drivers/hid/hid-core.c  | 6 ++++++
 drivers/hid/hid-ids.h   | 3 +++
 3 files changed, 15 insertions(+)

--
1.8.1.2

Comments

Henrik Rydberg July 8, 2013, 6:44 p.m. UTC | #1
Hi Luis,

> This is a note to let you know that I have just added a patch titled
> 
>     HID: apple: Add support for the 2013 Macbook Air
> 
> to the linux-3.5.y-queue branch of the 3.5.y.z extended stable tree 
> which can be found at:
> 
>  http://kernel.ubuntu.com/git?p=ubuntu/linux.git;a=shortlog;h=refs/heads/linux-3.5.y-queue
> 
> If you, or anyone else, feels it should not be added to this tree, please 
> reply to this email.

This patch by itself will render the touchpad disabled, and since the
accompanying bcm5974 patch does not apply to v3.5, the result will be
notably inferior.

Thanks,
Henrik
Luis Henriques July 8, 2013, 6:56 p.m. UTC | #2
Hi Henrik,

rydberg@euromail.se writes:

> Hi Luis,
>
>> This is a note to let you know that I have just added a patch titled
>> 
>>     HID: apple: Add support for the 2013 Macbook Air
>> 
>> to the linux-3.5.y-queue branch of the 3.5.y.z extended stable tree 
>> which can be found at:
>> 
>>  http://kernel.ubuntu.com/git?p=ubuntu/linux.git;a=shortlog;h=refs/heads/linux-3.5.y-queue
>> 
>> If you, or anyone else, feels it should not be added to this tree, please 
>> reply to this email.
>
> This patch by itself will render the touchpad disabled, and since the
> accompanying bcm5974 patch does not apply to v3.5, the result will be
> notably inferior.

Thanks for your feedback.
So, you're proposing to drop this patch, correct?

Cheers,
Henrik Rydberg July 8, 2013, 7:18 p.m. UTC | #3
On Mon, Jul 08, 2013 at 07:56:16PM +0100, Luis Henriques wrote:
> Hi Henrik,
> 
> rydberg@euromail.se writes:
> 
> > Hi Luis,
> >
> >> This is a note to let you know that I have just added a patch titled
> >> 
> >>     HID: apple: Add support for the 2013 Macbook Air
> >> 
> >> to the linux-3.5.y-queue branch of the 3.5.y.z extended stable tree 
> >> which can be found at:
> >> 
> >>  http://kernel.ubuntu.com/git?p=ubuntu/linux.git;a=shortlog;h=refs/heads/linux-3.5.y-queue
> >> 
> >> If you, or anyone else, feels it should not be added to this tree, please 
> >> reply to this email.
> >
> > This patch by itself will render the touchpad disabled, and since the
> > accompanying bcm5974 patch does not apply to v3.5, the result will be
> > notably inferior.
> 
> Thanks for your feedback.
> So, you're proposing to drop this patch, correct?

Correct.

Thanks,
Henrik
Luis Henriques July 9, 2013, 8:19 a.m. UTC | #4
rydberg@euromail.se writes:

> On Mon, Jul 08, 2013 at 07:56:16PM +0100, Luis Henriques wrote:
>> Hi Henrik,
>> 
>> rydberg@euromail.se writes:
>> 
>> > Hi Luis,
>> >
>> >> This is a note to let you know that I have just added a patch titled
>> >> 
>> >>     HID: apple: Add support for the 2013 Macbook Air
>> >> 
>> >> to the linux-3.5.y-queue branch of the 3.5.y.z extended stable tree 
>> >> which can be found at:
>> >> 
>> >>  http://kernel.ubuntu.com/git?p=ubuntu/linux.git;a=shortlog;h=refs/heads/linux-3.5.y-queue
>> >> 
>> >> If you, or anyone else, feels it should not be added to this tree, please 
>> >> reply to this email.
>> >
>> > This patch by itself will render the touchpad disabled, and since the
>> > accompanying bcm5974 patch does not apply to v3.5, the result will be
>> > notably inferior.
>> 
>> Thanks for your feedback.
>> So, you're proposing to drop this patch, correct?
>
> Correct.
>
> Thanks,
> Henrik

I'm dropping it from the 3.5 kernel queue.

Cheers,
diff mbox

Patch

diff --git a/drivers/hid/hid-apple.c b/drivers/hid/hid-apple.c
index 585344b..2607e0f 100644
--- a/drivers/hid/hid-apple.c
+++ b/drivers/hid/hid-apple.c
@@ -523,6 +523,12 @@  static const struct hid_device_id apple_devices[] = {
 		.driver_data = APPLE_HAS_FN | APPLE_ISO_KEYBOARD },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING7_JIS),
 		.driver_data = APPLE_HAS_FN | APPLE_RDESC_JIS },
+	{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING8_ANSI),
+		.driver_data = APPLE_HAS_FN },
+	{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING8_ISO),
+		.driver_data = APPLE_HAS_FN | APPLE_ISO_KEYBOARD },
+	{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING8_JIS),
+		.driver_data = APPLE_HAS_FN | APPLE_RDESC_JIS },
 	{ HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_ALU_WIRELESS_2009_ANSI),
 		.driver_data = APPLE_NUMLOCK_EMULATION | APPLE_HAS_FN },
 	{ HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_ALU_WIRELESS_2009_ISO),
diff --git a/drivers/hid/hid-core.c b/drivers/hid/hid-core.c
index 2f85e59..aeb5ce4 100644
--- a/drivers/hid/hid-core.c
+++ b/drivers/hid/hid-core.c
@@ -1516,6 +1516,9 @@  static const struct hid_device_id hid_have_special_driver[] = {
 	{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING7_ANSI) },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING7_ISO) },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING7_JIS) },
+	{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING8_ANSI) },
+	{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING8_ISO) },
+	{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING8_JIS) },
 	{ HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_ALU_WIRELESS_2009_ANSI) },
 	{ HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_ALU_WIRELESS_2009_ISO) },
 	{ HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_ALU_WIRELESS_2009_JIS) },
@@ -2111,6 +2114,9 @@  static const struct hid_device_id hid_mouse_ignore_list[] = {
 	{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING7_ANSI) },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING7_ISO) },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING7_JIS) },
+	{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING8_ANSI) },
+	{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING8_ISO) },
+	{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING8_JIS) },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_FOUNTAIN_TP_ONLY) },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_GEYSER1_TP_ONLY) },
 	{ }
diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h
index 7561d73..94a9378 100644
--- a/drivers/hid/hid-ids.h
+++ b/drivers/hid/hid-ids.h
@@ -133,6 +133,9 @@ 
 #define USB_DEVICE_ID_APPLE_ALU_WIRELESS_2009_JIS   0x023b
 #define USB_DEVICE_ID_APPLE_ALU_WIRELESS_2011_ANSI  0x0255
 #define USB_DEVICE_ID_APPLE_ALU_WIRELESS_2011_ISO   0x0256
+#define USB_DEVICE_ID_APPLE_WELLSPRING8_ANSI	0x0291
+#define USB_DEVICE_ID_APPLE_WELLSPRING8_ISO	0x0292
+#define USB_DEVICE_ID_APPLE_WELLSPRING8_JIS	0x0293
 #define USB_DEVICE_ID_APPLE_FOUNTAIN_TP_ONLY	0x030a
 #define USB_DEVICE_ID_APPLE_GEYSER1_TP_ONLY	0x030b
 #define USB_DEVICE_ID_APPLE_ATV_IRCONTROL	0x8241