Skip to content

Commit 1eece1d

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 8173acb commit 1eece1d

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
@@ -829,6 +829,10 @@ static int apple_probe(struct hid_device *hdev,
829829
struct apple_sc *asc;
830830
int ret;
831831

832+
if (id->bus == BUS_SPI && id->vendor == SPI_VENDOR_ID_APPLE &&
833+
hdev->type != HID_TYPE_SPI_KEYBOARD)
834+
return -ENODEV;
835+
832836
asc = devm_kzalloc(&hdev->dev, sizeof(*asc), GFP_KERNEL);
833837
if (asc == NULL) {
834838
hid_err(hdev, "can't alloc apple descriptor\n");
@@ -1073,6 +1077,8 @@ static const struct hid_device_id apple_devices[] = {
10731077
.driver_data = APPLE_HAS_FN | APPLE_ISO_TILDE_QUIRK | APPLE_RDESC_BATTERY },
10741078
{ HID_BLUETOOTH_DEVICE(BT_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_MAGIC_KEYBOARD_NUMPAD_2021),
10751079
.driver_data = APPLE_HAS_FN | APPLE_ISO_TILDE_QUIRK },
1080+
{ HID_SPI_DEVICE(SPI_VENDOR_ID_APPLE, HID_ANY_ID),
1081+
.driver_data = APPLE_HAS_FN | APPLE_ISO_TILDE_QUIRK },
10761082

10771083
{ }
10781084
};

0 commit comments

Comments
 (0)