HID: logitech-hidpp: Remove hard-coded "Sw. Id." for HID++ 2.0 commands
authorBastien Nocera <hadess@hadess.net>
Tue, 30 Aug 2022 13:25:49 +0000 (15:25 +0200)
committerJiri Kosina <jkosina@suse.cz>
Wed, 7 Sep 2022 20:07:49 +0000 (22:07 +0200)
Some HID++ 2.0 commands had correctly set a non-zero software identifier
directly as part of their function identifiers, but it's more correct to
define the function identifier and the software identifier separately
before combined them when the command is sent.

As this is now done in the previous commit, remove the hard-coded 0x1
software identifiers in the function definitions.

Signed-off-by: Bastien Nocera <hadess@hadess.net>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
drivers/hid/hid-logitech-hidpp.c

index e51ccf2..74013d0 100644 (file)
@@ -859,8 +859,8 @@ static int hidpp_unifying_init(struct hidpp_device *hidpp)
 #define HIDPP_PAGE_ROOT                                        0x0000
 #define HIDPP_PAGE_ROOT_IDX                            0x00
 
-#define CMD_ROOT_GET_FEATURE                           0x01
-#define CMD_ROOT_GET_PROTOCOL_VERSION                  0x11
+#define CMD_ROOT_GET_FEATURE                           0x00
+#define CMD_ROOT_GET_PROTOCOL_VERSION                  0x10
 
 static int hidpp_root_get_feature(struct hidpp_device *hidpp, u16 feature,
        u8 *feature_index, u8 *feature_type)
@@ -937,9 +937,9 @@ print_version:
 
 #define HIDPP_PAGE_GET_DEVICE_NAME_TYPE                        0x0005
 
-#define CMD_GET_DEVICE_NAME_TYPE_GET_COUNT             0x01
-#define CMD_GET_DEVICE_NAME_TYPE_GET_DEVICE_NAME       0x11
-#define CMD_GET_DEVICE_NAME_TYPE_GET_TYPE              0x21
+#define CMD_GET_DEVICE_NAME_TYPE_GET_COUNT             0x00
+#define CMD_GET_DEVICE_NAME_TYPE_GET_DEVICE_NAME       0x10
+#define CMD_GET_DEVICE_NAME_TYPE_GET_TYPE              0x20
 
 static int hidpp_devicenametype_get_count(struct hidpp_device *hidpp,
        u8 feature_index, u8 *nameLength)
@@ -1969,8 +1969,8 @@ static int hidpp_touchpad_fw_items_set(struct hidpp_device *hidpp,
 
 #define HIDPP_PAGE_TOUCHPAD_RAW_XY                     0x6100
 
-#define CMD_TOUCHPAD_GET_RAW_INFO                      0x01
-#define CMD_TOUCHPAD_SET_RAW_REPORT_STATE              0x21
+#define CMD_TOUCHPAD_GET_RAW_INFO                      0x00
+#define CMD_TOUCHPAD_SET_RAW_REPORT_STATE              0x20
 
 #define EVENT_TOUCHPAD_RAW_XY                          0x00