Skip to content

Commit 7ba7a70

Browse files
committed
HID: apple: add support for internal keyboards
Apple MacBook keyboards started using HID over SPI in 2015. With the addition of the SPI HID transport they can be supported by this driver. Support all product ids over with the Apple SPI vendor id for now. Individual product ids will have to be added for a correct Fn/function key mapping. Enable by default on the Apple Arm platform. Signed-off-by: Janne Grunau <j@jannau.net>
1 parent 952c50e commit 7ba7a70

2 files changed

Lines changed: 7 additions & 1 deletion

File tree

drivers/hid/Kconfig

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ config HID_APPLE
129129
tristate "Apple {i,Power,Mac}Books"
130130
depends on LEDS_CLASS
131131
depends on NEW_LEDS
132-
default !EXPERT
132+
default !EXPERT || SPI_HID_APPLE
133133
help
134134
Support for some Apple devices which less or more break
135135
HID specification.

drivers/hid/hid-apple.c

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -903,6 +903,10 @@ static int apple_probe(struct hid_device *hdev,
903903
struct apple_sc *asc;
904904
int ret;
905905

906+
if (id->bus == BUS_SPI && id->vendor == SPI_VENDOR_ID_APPLE &&
907+
hdev->type != HID_TYPE_SPI_KEYBOARD)
908+
return -ENODEV;
909+
906910
asc = devm_kzalloc(&hdev->dev, sizeof(*asc), GFP_KERNEL);
907911
if (asc == NULL) {
908912
hid_err(hdev, "can't alloc apple descriptor\n");
@@ -1158,6 +1162,8 @@ static const struct hid_device_id apple_devices[] = {
11581162
.driver_data = APPLE_HAS_FN | APPLE_ISO_TILDE_QUIRK | APPLE_RDESC_BATTERY },
11591163
{ HID_BLUETOOTH_DEVICE(BT_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_MAGIC_KEYBOARD_NUMPAD_2021),
11601164
.driver_data = APPLE_HAS_FN | APPLE_ISO_TILDE_QUIRK },
1165+
{ HID_SPI_DEVICE(SPI_VENDOR_ID_APPLE, HID_ANY_ID),
1166+
.driver_data = APPLE_HAS_FN | APPLE_ISO_TILDE_QUIRK },
11611167
{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_TOUCHBAR_BACKLIGHT),
11621168
.driver_data = APPLE_MAGIC_BACKLIGHT },
11631169

0 commit comments

Comments
 (0)