Skip to content

Commit b63b7f5

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 e51012b commit b63b7f5

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
@@ -910,6 +910,10 @@ static int apple_probe(struct hid_device *hdev,
910910
struct apple_sc *asc;
911911
int ret;
912912

913+
if (id->bus == BUS_SPI && id->vendor == SPI_VENDOR_ID_APPLE &&
914+
hdev->type != HID_TYPE_SPI_KEYBOARD)
915+
return -ENODEV;
916+
913917
asc = devm_kzalloc(&hdev->dev, sizeof(*asc), GFP_KERNEL);
914918
if (asc == NULL) {
915919
hid_err(hdev, "can't alloc apple descriptor\n");
@@ -1169,6 +1173,8 @@ static const struct hid_device_id apple_devices[] = {
11691173
.driver_data = APPLE_HAS_FN | APPLE_ISO_TILDE_QUIRK | APPLE_RDESC_BATTERY },
11701174
{ HID_BLUETOOTH_DEVICE(BT_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_MAGIC_KEYBOARD_NUMPAD_2021),
11711175
.driver_data = APPLE_HAS_FN | APPLE_ISO_TILDE_QUIRK },
1176+
{ HID_SPI_DEVICE(SPI_VENDOR_ID_APPLE, HID_ANY_ID),
1177+
.driver_data = APPLE_HAS_FN | APPLE_ISO_TILDE_QUIRK },
11721178
{ HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_TOUCHBAR_BACKLIGHT),
11731179
.driver_data = APPLE_MAGIC_BACKLIGHT },
11741180

0 commit comments

Comments
 (0)