diff options
| author | Maximilian Luz <luzmaximilian@gmail.com> | 2022-12-02 23:33:26 +0100 |
|---|---|---|
| committer | Hans de Goede <hdegoede@redhat.com> | 2023-02-02 22:48:20 +0100 |
| commit | 78abf1b5205534bb7deda408aa5b9b7e0bf1982e (patch) | |
| tree | 3a4a7491db9959e229d24179aadd5216bea415ac /drivers/hid/surface-hid/surface_hid.c | |
| parent | HID: surface-hid: Use target-ID enum instead of hard-coding values (diff) | |
| download | linux-78abf1b5205534bb7deda408aa5b9b7e0bf1982e.tar.xz linux-78abf1b5205534bb7deda408aa5b9b7e0bf1982e.zip | |
platform/surface: aggregator: Enforce use of target-ID enum in device ID macros
Similar to the target category (TC), the target ID (TID) can be one
value out of a small number of choices, given in enum ssam_ssh_tid.
In the device ID macros, SSAM_SDEV() and SSAM_VDEV() we already use text
expansion to, both, remove some textual clutter for the target category
values and enforce that the value belongs to the known set. Now that we
know the names for the target IDs, use the same trick for them as well.
Also rename the SSAM_ANY_x macros to SSAM_SSH_x_ANY to better fit in.
Signed-off-by: Maximilian Luz <luzmaximilian@gmail.com>
Acked-by: Sebastian Reichel <sebastian.reichel@collabora.com>
Link: https://lore.kernel.org/r/20221202223327.690880-9-luzmaximilian@gmail.com
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Diffstat (limited to 'drivers/hid/surface-hid/surface_hid.c')
| -rw-r--r-- | drivers/hid/surface-hid/surface_hid.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/hid/surface-hid/surface_hid.c b/drivers/hid/surface-hid/surface_hid.c index d4aa8c81903a..aa80d83a83d1 100644 --- a/drivers/hid/surface-hid/surface_hid.c +++ b/drivers/hid/surface-hid/surface_hid.c @@ -230,7 +230,7 @@ static void surface_hid_remove(struct ssam_device *sdev) } static const struct ssam_device_id surface_hid_match[] = { - { SSAM_SDEV(HID, SSAM_ANY_TID, SSAM_ANY_IID, 0x00) }, + { SSAM_SDEV(HID, ANY, SSAM_SSH_IID_ANY, 0x00) }, { }, }; MODULE_DEVICE_TABLE(ssam, surface_hid_match); |