input: edt-ft5x06: Include I2C details in names for the devices
authorDave Stevenson <dave.stevenson@raspberrypi.com>
Wed, 8 Nov 2023 15:50:39 +0000 (15:50 +0000)
committerDom Cobley <popcornmix@gmail.com>
Mon, 19 Feb 2024 11:35:16 +0000 (11:35 +0000)
libinput uses the input device name alone. If you have two
identical input devices, then there is no way to differentiate
between them, and in the case of touchscreens that means no
way to associate them with the appropriate display device.

Add the I2C bus and address to the start of the input device
name so that the name is always unique within the system.

Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com>
drivers/input/touchscreen/edt-ft5x06.c

index 9c7fdbb..1b82821 100644 (file)
@@ -69,6 +69,7 @@
 #define TOUCH_EVENT_RESERVED           0x03
 
 #define EDT_NAME_LEN                   23
+#define EDT_NAME_PREFIX_LEN            8
 #define EDT_SWITCH_MODE_RETRIES                10
 #define EDT_SWITCH_MODE_DELAY          5 /* msec */
 #define EDT_RAW_DATA_RETRIES           100
@@ -145,7 +146,7 @@ struct edt_ft5x06_ts_data {
        int tdata_offset;
        unsigned int known_ids;
 
-       char name[EDT_NAME_LEN];
+       char name[EDT_NAME_PREFIX_LEN + EDT_NAME_LEN];
        char fw_version[EDT_NAME_LEN];
        int init_td_status;
 
@@ -937,6 +938,9 @@ static int edt_ft5x06_ts_identify(struct i2c_client *client,
        char *model_name = tsdata->name;
        char *fw_version = tsdata->fw_version;
 
+       snprintf(model_name, EDT_NAME_PREFIX_LEN, "%s ", dev_name(&client->dev));
+       model_name += strlen(model_name);
+
        /* see what we find if we assume it is a M06 *
         * if we get less than EDT_NAME_LEN, we don't want
         * to have garbage in there