Re: [PATCH 0/2] HID: Add support for Surface Aggregator Module HID transport

From: Maximilian Luz
Date: Sat Mar 27 2021 - 16:51:19 EST


On 3/10/21 11:53 PM, Maximilian Luz wrote:
This series adds support for the Surface System Aggregator Module (SSAM)
HID transport subsystem.

The SSAM is an embedded controller, found on 5th- and later generation
Microsoft Surface devices. On some of these devices (specifically
Surface Laptops 1, 2, and 3, as well as Surface Book 3), built-in input
devices are connected via the SSAM. These devices communicate (mostly)
via normal HID reports, so adding support for them is (mostly) just a
matter of implementing an HID transport driver.

SSAM actually has two different HID interfaces: One (legacy) interface
used on Surface Laptops 1 and 2, and a newer interface for the rest. The
newer interface allows for multiple HID devices to be addressed and is
implemented in the first patch. The older interface only allows a single
HID device to be connected and, furthermore, only allows a single output
report, specifically one for the caps lock LED. This is implemented in
the second patch.

See the commit messages of the respective patches for more details.

Regards,
Max

Note: This patch depends on the

platform/surface: Add Surface Aggregator device registry

series. More specifically patch

platform/surface: Set up Surface Aggregator device registry

The full series has been merged into the for-next branch of the
platform-drivers-x86 tree [1]. The commit in question can be found at
[2].

[1]: https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/log/?h=for-next
[2]: https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/commit/?h=for-next&id=fc622b3d36e6d91330fb21506b9ad1e3206a4dde

Maximilian Luz (2):
HID: Add support for Surface Aggregator Module HID transport
HID: surface-hid: Add support for legacy keyboard interface

MAINTAINERS | 7 +
drivers/hid/Kconfig | 2 +
drivers/hid/Makefile | 2 +
drivers/hid/surface-hid/Kconfig | 42 +++
drivers/hid/surface-hid/Makefile | 7 +
drivers/hid/surface-hid/surface_hid.c | 253 +++++++++++++++++
drivers/hid/surface-hid/surface_hid_core.c | 272 +++++++++++++++++++
drivers/hid/surface-hid/surface_hid_core.h | 77 ++++++
drivers/hid/surface-hid/surface_kbd.c | 300 +++++++++++++++++++++
9 files changed, 962 insertions(+)
create mode 100644 drivers/hid/surface-hid/Kconfig
create mode 100644 drivers/hid/surface-hid/Makefile
create mode 100644 drivers/hid/surface-hid/surface_hid.c
create mode 100644 drivers/hid/surface-hid/surface_hid_core.c
create mode 100644 drivers/hid/surface-hid/surface_hid_core.h
create mode 100644 drivers/hid/surface-hid/surface_kbd.c


Hi, is there any status update on this?

Regards,
Max