From patchwork Mon Jul 1 10:45:25 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Aditya Garg X-Patchwork-Id: 13717844 X-Patchwork-Delegate: jikos@jikos.cz Received: from IND01-MAX-obe.outbound.protection.outlook.com (mail-maxind01olkn2059.outbound.protection.outlook.com [40.92.102.59]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8F26657C9A; Mon, 1 Jul 2024 10:45:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.92.102.59 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1719830732; cv=fail; b=Fct3SNgs9Tci/MPsSo93poHPD84POCS+pWajHyMR1uCtI1XGct8+y07tmDs3piem8x1s75Mb9XBynz5ovVASAW8meVc/wZsGIrDlK8DStI6o/395oEQGQF7tHn/hNTBoP3XAuwU1qsBeDtPv+pwHFDV7/72AYyt3n6Uehh7KZCw= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1719830732; c=relaxed/simple; bh=Fw5W6Mg3CDWVxNIbWH4Q9MQvSK98cK8h35loj/05puc=; h=From:To:CC:Subject:Date:Message-ID:Content-Type:MIME-Version; b=Ery1pxpQgECqPGbRGCGKpliZiyk5irpMZ8RZ7jptyhd5RwVCTOsOx++q1o8GmvnfT/mUFA979ogDMC+nRfgfakMwgf6EwdFXsTAzv5ssgM0VKFgTk0RMkv0nGlRiQlk8PGeZ+PHiE+WGr/NLeA7FeL6BnSPlRN1XDmB1+uxsRvM= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=live.com; spf=pass smtp.mailfrom=live.com; dkim=pass (2048-bit key) header.d=live.com header.i=@live.com header.b=pk8k7Vdv; arc=fail smtp.client-ip=40.92.102.59 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=live.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=live.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=live.com header.i=@live.com header.b="pk8k7Vdv" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=eSkv11KCd9gXv7mRIROS1hSytdPGAkgzvOSnaDAxCjJs4dVPSshe1u3oisIsu9XYr79ONzkr4k1k0gZao9nJSeNzyNXygwaDJ/VoG+8Cj2L/gmKoDfuBekf5LGznU+WGRDG6LnBdcyo691lg1x+l3v7DGh9FtqCh/36OK6GPQ3HIAloiTEO3xd5cnjxs8skT9foiSethPe3w8JEJk4KvnU51mUWUIMLv9LP1n6YZM8F3eXpS2W6DMgSf1OMAYutiQkMW9LtrOidY3ZyXEZSddBnR/PqfAsa4Ar9lBiARogP2BJdIMGXu2MoUNoLk6OHOs09II8kB8+Z6hIXSX6porg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=EK+8Q8ctycK/JqpR1S/x6bk1jvDpwYa0GayZEaWoDdU=; b=JRK2cuiyWVfCKpKYCCPGNRAIDe6zSOHf8TnxPVRosvzti3aVyTBP6S1IA1JeyLHwWfbXxSJZ4A93nd9wKPTqaX4+mRYfWJ1myNrAIxF2SZv8/ytBuzExOVQuKeQOMKv3Nbf68yndWmSqE/Ew5TyeGmseEH/37W1PXmsYTvBOmqL4FW8sbE9ku4sqN7Ih1JNzwFL9Z2L94CjzA+sSew/0eoKSlQo+64hLhGv8j5s7uyWnWexkIpYPHfsn/ZU3EpWTH9Uy677zVJDi2T5nmLYBmgu6ZAbqjlqS9M6CHuMT5326mqosdByqOdDMi9X/MmrXWV4fb4G9mpR0iBMn5INJpQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=live.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=EK+8Q8ctycK/JqpR1S/x6bk1jvDpwYa0GayZEaWoDdU=; b=pk8k7VdvJEWIgG98krxa59nY7B4V7OfKPC6FsQgeIWbZisH2FNCxAKgHEmO556Ra05CI/+l0yc4tqpCKMxPXFtZXXAqd0w9hbliUUW0eIrcJrm8JJfXFNfwS6tNthgeP8ZyohloaS4eMcQxszV8JrFn/ItCjMoV22Prw4Sbd+rv3QSniGf0x8anQxsrz+8/qKSrxWc6Ub0dh7NGp9IgOXHNJ6zpFhyrLjzPSLOUNDuNXaerhm6mv/DlZT0jbenLAkSf1CLvFJLN6YxcggFgw+1Yd/Kn3bBv44lyv1bJsXS2JxjmNfwqs5ezf/haHtv+hONAIdNWNUF2pMTdPs0icAQ== Received: from MA0P287MB0217.INDP287.PROD.OUTLOOK.COM (2603:1096:a01:b3::9) by PN3P287MB0180.INDP287.PROD.OUTLOOK.COM (2603:1096:c01:d5::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7719.28; Mon, 1 Jul 2024 10:45:25 +0000 Received: from MA0P287MB0217.INDP287.PROD.OUTLOOK.COM ([fe80::98d2:3610:b33c:435a]) by MA0P287MB0217.INDP287.PROD.OUTLOOK.COM ([fe80::98d2:3610:b33c:435a%7]) with mapi id 15.20.7719.029; Mon, 1 Jul 2024 10:45:25 +0000 From: Aditya Garg To: Jiri Kosina , "bentiss@kernel.org" CC: "linux-input@vger.kernel.org" , Linux Kernel Mailing List , Orlando Chamberlain Subject: [PATCH] HID: apple: Add support for magic keyboard backlight on T2 Macs Thread-Topic: [PATCH] HID: apple: Add support for magic keyboard backlight on T2 Macs Thread-Index: AQHay6PHPmmDJHTE8kuvuT8p1BrWxw== Date: Mon, 1 Jul 2024 10:45:25 +0000 Message-ID: Accept-Language: en-IN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-messagesentrepresentingtype: 1 x-tmn: [SPF4WkZQgmTDLAv4vo3dG6GrwOB/Hd1ubWjFzirFwTGb3Tnu3P4/eVDayxWMaYc9uBiBZArh5Vo=] x-ms-publictraffictype: Email x-ms-traffictypediagnostic: MA0P287MB0217:EE_|PN3P287MB0180:EE_ x-ms-office365-filtering-correlation-id: c6b02bf5-1150-40ae-9389-08dc99baea2a x-microsoft-antispam: BCL:0;ARA:14566002|8060799006|461199028|3412199025|440099028|102099032; x-microsoft-antispam-message-info: v7YAaHTKoy1+ytK+ZAQ3r6KHq0ftRI5yReCCSWpJt/T/KGkX0MoESJhuEw1coQnacDsWCrc3NZkdJXofOtAOQtpbHX5jxfEMnlylVSZKIVPaZg/riw2t3hx7AQD/dVVlzdsfGAAgbt8Ps8gJSdzcdTZNKVg/Uur8Cz8otTRQ10Z5OT+nqEuwmEp+52O8kL7rzt7+aIpwidcen5mMB+IxV0vCKE3O6E49d4yhC1CRvmRS2SXoy8M+1P1Wq5Hv/qm8vxzyDpS8TQqBcAEPvlTEs2nw8zquMpPfuqQboZo+PS1kP88t+2VYNsOA3l+A6FC4ZaB0V8uYnOsIV2Q1RZUwQmBFM6/DdupkzPRWlmqP/qX4doIDis0EWGeFINHZq2gCojbIMA3R7xkLmaRRhh71HoMsNs1D4eWSpwzuK9lYJDyLBDPh8C/82ikKw+365+IPqz3hwFO1Aqu7OwSNccjFe2Rn9YeXt4q7l55ueifT4l23ceEFlvL3hylF0VG0a+rL7RdsuBArVtmMIbjLPklbQHxg2umaCb6Bg2KAbXL8x+pVsYhseIrIQKUEw2dptBfWjkhVhKKhseQS6p8bDbK04TcVnw363GqXZSehwCa3aygTBrrWYnYSm5yOg2M2EkPE x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: 65SFwUfL5p4Th+dfpLKaDBT8D9i5H6ftpdYXHIUKZ5DIx0qZueENq7DLksSTC6gUm1Fo7dslXcdrhyl0Qu1Mt+PzarUI9m87DA14ouygxLmperI5+15wQYDoMInc7JQvgEAR6v//X7jGikP1OW0kq+GZFJsiyFfVaG40CyC9hLXgNZ5Gqs+vb6pK/20wMtGfeyBkmNvYcZt6BcNgFlRWkzecw8qt6Vh2pnKMYeuJ0VEMax8xqLseFF/3sqVzkI6yjjUXkug3FTjH4HcxIuXva10QfQAz4CSNNISvGZRg8B8h258LLTE2I9CBHHNpyHtnZV/2TzgrnDwQS17Jtftx7gi746TDCHzuzLovyePURTuAWaU08Er7zTVhk3aPVM9vCjzpW6T80cTKxx0yQwO5vs9jMuRHticD/wILJtClMHkT3nWhz0BPjTztqZElkd7a9BqXxqf07RMNqam9YRj15J/Ro0T/80MOP2OHficS2lni8MgwWj86isz9OB3zg+Ns7IgW4r3r0qaX3mOPWUEEm9yM9UeF5Ygvmq2poFBX7NhT9VdGxlzZEZwkSA7WnfYZf5hNlgoFlK0C7zIMS4pcClltdlHv1C6e9EH0OjkNTiGVvp6Pf8fOa7h/aUUJE1g4jfvy2Ufyi2jPcTmn3KgwisJe/YmwyrwUm/G4sTKMNyKkUqXV9wYy1bsO6yMlgDEh47d6S75wAx4TZBMTHrPIZevREyPeLaqt5xKhoGSPdX8UYkO9+/XoMTU5p4LARBLeUaQCVKJZrLlnc/zVvCx937zdrcBJhDrFOsDrn9jyAkyzwX2WpeG/hCL0+IZPBwf78d9yWHmgzUOhWcyU+y57LJMTcAN6g/FOTQ/HFLIIfPWqpUUTsKdN5YvbVkGcdeWEgXUKQZmLd9hFJlBDQ6HW8hVkevpChtEH6kB3ZsCHPAYquMOC2xNulp1hCkEEXJuFBdRflM5kHXtOxRCjTNhfXPLW5V7prY7NCZELS72fyVsSUFgYq95XOr0oPaAKjc54NUWpCGy4W+niRSKfYRNSvyi0oeTofWm2ywpZ9W1rmeSxt06UM2sGFpZw8iSN/WyHUh/YgCgJZczjmzOIJNYOJd65yuvghMWxb+GEZOeleZJ/YXHPxs1JyaZlxWwPNZM6fPTxQjqxAupxo5nZ+CP4LEnjj+xed5A4k8bh9rg0xabdc7VZ9vMquAvD99bz9fZBtVDQI4WVFhXv8iHNO8QRdFScj1gYtcNPDa88HqJdgLTnYg/ekVI+SBqyVOPh98dJ/i23o61MN5Ho0FogWXZL2B+XyMJ7yCsdQUB0aKAk1txsNmGnhMs6/XtQ45jXFclN Content-ID: <3EDDD37BEE915F4D9CD9F6613872E129@INDP287.PROD.OUTLOOK.COM> Precedence: bulk X-Mailing-List: linux-input@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-OriginatorOrg: sct-15-20-4755-11-msonline-outlook-bafef.templateTenant X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: MA0P287MB0217.INDP287.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-CrossTenant-Network-Message-Id: c6b02bf5-1150-40ae-9389-08dc99baea2a X-MS-Exchange-CrossTenant-rms-persistedconsumerorg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-CrossTenant-originalarrivaltime: 01 Jul 2024 10:45:25.1248 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-Transport-CrossTenantHeadersStamped: PN3P287MB0180 From: Orlando Chamberlain Unlike T2 Macs with Butterfly keyboard, who have their keyboard backlight on the USB device the T2 Macs with Magic keyboard have their backlight on the Touchbar backlight device (05ac:8102). Support for Butterfly keyboards has already been added in 9018eacbe623 ("HID: apple: Add support for keyboard backlight on certain T2 Macs."). This patch adds support for the Magic keyboards. Co-developed-by: Aditya Garg Signed-off-by: Aditya Garg Signed-off-by: Orlando Chamberlain --- drivers/hid/hid-apple.c | 87 ++++++++++++++++++++++++++++++++++++++++- 1 file changed, 86 insertions(+), 1 deletion(-) diff --git a/drivers/hid/hid-apple.c b/drivers/hid/hid-apple.c index bd022e004..db279948c 100644 --- a/drivers/hid/hid-apple.c +++ b/drivers/hid/hid-apple.c @@ -8,6 +8,8 @@ * Copyright (c) 2006-2007 Jiri Kosina * Copyright (c) 2008 Jiri Slaby * Copyright (c) 2019 Paul Pawlowski + * Copyright (c) 2023 Orlando Chamberlain + * Copyright (c) 2024 Aditya Garg */ /* @@ -23,6 +25,7 @@ #include #include #include +#include #include "hid-ids.h" @@ -37,13 +40,18 @@ #define APPLE_NUMLOCK_EMULATION BIT(8) #define APPLE_RDESC_BATTERY BIT(9) #define APPLE_BACKLIGHT_CTL BIT(10) -#define APPLE_IS_NON_APPLE BIT(11) +#define APPLE_MAGIC_BACKLIGHT BIT(11) +#define APPLE_IS_NON_APPLE BIT(12) #define APPLE_FLAG_FKEY 0x01 #define HID_COUNTRY_INTERNATIONAL_ISO 13 #define APPLE_BATTERY_TIMEOUT_MS 60000 +#define HID_USAGE_MAGIC_BL 0xff00000f +#define APPLE_MAGIC_REPORT_ID_POWER 3 +#define APPLE_MAGIC_REPORT_ID_BRIGHTNESS 1 + static unsigned int fnmode = 3; module_param(fnmode, uint, 0644); MODULE_PARM_DESC(fnmode, "Mode of fn key on Apple keyboards (0 = disabled, " @@ -81,6 +89,12 @@ struct apple_sc_backlight { struct hid_device *hdev; }; +struct apple_magic_backlight { + struct led_classdev cdev; + struct hid_report *brightness; + struct hid_report *power; +}; + struct apple_sc { struct hid_device *hdev; unsigned long quirks; @@ -822,6 +836,72 @@ static int apple_backlight_init(struct hid_device *hdev) return ret; } +static void apple_magic_backlight_report_set(struct hid_report *rep, s32 value, u8 rate) +{ + rep->field[0]->value[0] = value; + rep->field[1]->value[0] = 0x5e; /* Mimic Windows */ + rep->field[1]->value[0] |= rate << 8; + + hid_hw_request(rep->device, rep, HID_REQ_SET_REPORT); +} + +static void apple_magic_backlight_set(struct apple_magic_backlight *backlight, + int brightness, char rate) +{ + apple_magic_backlight_report_set(backlight->power, brightness ? 1 : 0, rate); + if (brightness) + apple_magic_backlight_report_set(backlight->brightness, brightness, rate); +} + +static int apple_magic_backlight_led_set(struct led_classdev *led_cdev, + enum led_brightness brightness) +{ + struct apple_magic_backlight *backlight = container_of(led_cdev, + struct apple_magic_backlight, cdev); + + apple_magic_backlight_set(backlight, brightness, 1); + return 0; +} + +static int apple_magic_backlight_init(struct hid_device *hdev) +{ + struct apple_magic_backlight *backlight; + int rc; + + /* + * Ensure this usb endpoint is for the keyboard backlight, not touchbar + * backlight. + */ + if (hdev->collection[0].usage != HID_USAGE_MAGIC_BL) + return -ENODEV; + + backlight = devm_kzalloc(&hdev->dev, sizeof(*backlight), GFP_KERNEL); + if (!backlight) + return -ENOMEM; + + backlight->brightness = hid_register_report(hdev, HID_FEATURE_REPORT, + APPLE_MAGIC_REPORT_ID_BRIGHTNESS, 0); + backlight->power = hid_register_report(hdev, HID_FEATURE_REPORT, + APPLE_MAGIC_REPORT_ID_POWER, 0); + + if (!backlight->brightness || !backlight->power) { + rc = -ENODEV; + goto hw_stop; + } + + backlight->cdev.name = ":white:" LED_FUNCTION_KBD_BACKLIGHT; + backlight->cdev.max_brightness = backlight->brightness->field[0]->logical_maximum; + backlight->cdev.brightness_set_blocking = apple_magic_backlight_led_set; + + apple_magic_backlight_set(backlight, 0, 0); + + return devm_led_classdev_register(&hdev->dev, &backlight->cdev); + +hw_stop: + hid_hw_stop(hdev); + return rc; +} + static int apple_probe(struct hid_device *hdev, const struct hid_device_id *id) { @@ -860,6 +940,9 @@ static int apple_probe(struct hid_device *hdev, if (quirks & APPLE_BACKLIGHT_CTL) apple_backlight_init(hdev); + if (quirks & APPLE_MAGIC_BACKLIGHT) + apple_magic_backlight_init(hdev); + return 0; } @@ -1073,6 +1156,8 @@ static const struct hid_device_id apple_devices[] = { .driver_data = APPLE_HAS_FN | APPLE_ISO_TILDE_QUIRK | APPLE_RDESC_BATTERY }, { HID_BLUETOOTH_DEVICE(BT_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_MAGIC_KEYBOARD_NUMPAD_2021), .driver_data = APPLE_HAS_FN | APPLE_ISO_TILDE_QUIRK }, + { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_TOUCHBAR_BACKLIGHT), + .driver_data = APPLE_MAGIC_BACKLIGHT }, { } };