[PATCH] HID: hid-input: battery quirk for Apple keyboard

Subsystems: hid core layer, the rest

STALE5034d

4 messages, 4 authors, 2012-11-26 · open the first message on its own page

[PATCH] HID: hid-input: battery quirk for Apple keyboard

From: Ortwin Glück <hidden>
Date: 2012-11-25 14:05:58

Support battery capacity on another Apple wireless
keyboard.

NB: most likely all other APPLE_ALU_WIRELESS_*
keyboards should be added as well.

Signed-off-by: Ortwin Glück <redacted>
---
diff --git a/drivers/hid/hid-input.c b/drivers/hid/hid-input.c
index 5301006..b2ae097 100644
--- a/drivers/hid/hid-input.c
+++ b/drivers/hid/hid-input.c
@@ -299,6 +299,9 @@ static enum power_supply_property hidinput_battery_props[] = {

  static const struct hid_device_id hid_battery_quirks[] = {
         { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_APPLE,
+                              USB_DEVICE_ID_APPLE_ALU_WIRELESS_2009_ISO),
+         HID_BATTERY_QUIRK_PERCENT | HID_BATTERY_QUIRK_FEATURE },
+       { 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,

Re: [PATCH] HID: hid-input: battery quirk for Apple keyboard

From: devendra.aaru <hidden>
Date: 2012-11-25 14:13:35

i think you should simply do a  cc: stable@vger.kernel.org just right
above your signedoff-by, which is the way that stable patches gets
into i think.

Re: [PATCH] HID: hid-input: battery quirk for Apple keyboard

From: Ben Hutchings <hidden>
Date: 2012-11-26 00:45:46

On Sun, 2012-11-25 at 09:13 -0500, devendra.aaru wrote:
i think you should simply do a  cc: stable@vger.kernel.org just right
above your signedoff-by, which is the way that stable patches gets
into i think.
It's OK to cc stable@ when submitting a patch, but that line must be
included in the commit message to ensure it actually gets considered for
stable updates.

Ben.

-- 
Ben Hutchings
Never attribute to conspiracy what can adequately be explained by stupidity.

Re: [PATCH] HID: hid-input: battery quirk for Apple keyboard

From: Jiri Kosina <hidden>
Date: 2012-11-26 13:27:19

On Sun, 25 Nov 2012, Ortwin Glück wrote:
Support battery capacity on another Apple wireless
keyboard.
Your patch has been whitespace damaged. I have fixed it by hand and 
applied, but please fix your process for any future patch submissions. 
Thanks.
NB: most likely all other APPLE_ALU_WIRELESS_* keyboards should be added 
as well.
I think it's likely true, but I prefer to have it actually tested before 
doing such changes blindly.

Thanks,

-- 
Jiri Kosina
SUSE Labs
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help