diff mbox

Add Apple wireless keyboard to the quirk list

Message ID CACo8zOd77ZJNNzmwNWTJGM4H9U3F5Je90ADcO+PiK=jqESQixQ@mail.gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Daniel Nicoletti July 4, 2012, 1:40 p.m. UTC 
2012/6/5 Jiri Kosina <jkosina@suse.cz>
> > I've tested and with this, it works just fine, thanks.
>
> Thanks for the patch. It's however missing your Signed-off-by: line, could
> you please resend the patch including it, so that I could apply it?
Ok sorry for the delay (too much stuff),
I'm used to have commit access on the stuff I contribute so
I didn't know this git format-patch thing, which someone pointed out to me.
Should I use git send-email too?

Thanks.

From cc0225ce44f037f7ce9ecb876e65b5e8079d5665 Mon Sep 17 00:00:00 2001
From: Daniel Nicoletti <dantti12@gmail.com>
Date: Wed, 4 Jul 2012 10:20:31 -0300
Subject: [PATCH 1/1] hid-input: Add Apple's Wireless keyboard
 (USB_DEVICE_ID_APPLE_ALU_WIRELESS_ANSI), to the quirk
 list since it report wrong feature type and wrong
 percentage range.


Signed-off-by: Daniel Nicoletti <dantti12@gmail.com>
---
 drivers/hid/hid-input.c |    3 +++
 1 file changed, 3 insertions(+)

Comments

Jiri Kosina July 4, 2012, 1:48 p.m. UTC | #1 
On Wed, 4 Jul 2012, Daniel Nicoletti wrote:

> Ok sorry for the delay (too much stuff),
> I'm used to have commit access on the stuff I contribute so
> I didn't know this git format-patch thing, which someone pointed out to me.
> Should I use git send-email too?
> 
> Thanks.
> 
> >From cc0225ce44f037f7ce9ecb876e65b5e8079d5665 Mon Sep 17 00:00:00 2001
> From: Daniel Nicoletti <dantti12@gmail.com>
> Date: Wed, 4 Jul 2012 10:20:31 -0300
> Subject: [PATCH 1/1] hid-input: Add Apple's Wireless keyboard
>  (USB_DEVICE_ID_APPLE_ALU_WIRELESS_ANSI), to the quirk
>  list since it report wrong feature type and wrong
>  percentage range.
> 
> 
> Signed-off-by: Daniel Nicoletti <dantti12@gmail.com>
> ---
>  drivers/hid/hid-input.c |    3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/hid/hid-input.c b/drivers/hid/hid-input.c
> index 132b001..33d88d1 100644
> --- a/drivers/hid/hid-input.c
> +++ b/drivers/hid/hid-input.c
> @@ -301,6 +301,9 @@ static const struct hid_device_id hid_battery_quirks[] = {
>         { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_APPLE,
>                                USB_DEVICE_ID_APPLE_ALU_WIRELESS_2011_ANSI),
>           HID_BATTERY_QUIRK_PERCENT | HID_BATTERY_QUIRK_FEATURE },
> +       { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_APPLE,
> +                              USB_DEVICE_ID_APPLE_ALU_WIRELESS_ANSI),
> +         HID_BATTERY_QUIRK_PERCENT | HID_BATTERY_QUIRK_FEATURE },

Daniel,

the patch has been whitespace damaged. I have fixed it now and applied, 
but please fix your workflow (likely e-mail client) for further 
submissions in this respect.
Daniel Nicoletti July 4, 2012, 1:51 p.m. UTC | #2 
2012/7/4 Jiri Kosina <jkosina@suse.cz>:
> Daniel,
>
> the patch has been whitespace damaged. I have fixed it now and applied,
> but please fix your workflow (likely e-mail client) for further
> submissions in this respect.
Ok I've just used gmail web interface with plain text,
so I guess it's better to use git send email next time right?

Best,
Daniel.
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/hid/hid-input.c b/drivers/hid/hid-input.c
index 132b001..33d88d1 100644
--- a/drivers/hid/hid-input.c
+++ b/drivers/hid/hid-input.c
@@ -301,6 +301,9 @@  static const struct hid_device_id hid_battery_quirks[] = {
        { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_APPLE,
                               USB_DEVICE_ID_APPLE_ALU_WIRELESS_2011_ANSI),
          HID_BATTERY_QUIRK_PERCENT | HID_BATTERY_QUIRK_FEATURE },
+       { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_APPLE,
+                              USB_DEVICE_ID_APPLE_ALU_WIRELESS_ANSI),
+         HID_BATTERY_QUIRK_PERCENT | HID_BATTERY_QUIRK_FEATURE },
        {}
 };