Input: st1232 - read firmware version and revision

According to the data sheet, the st1332 contains a firmware, which may
be updated. The version and revision of the firmware may be read from
the registers of the device.

Read the firmware version and revision and report it to the log when
probing the device.

Suggested-by: Khalid Talash <ktalash@topcon.com>
Signed-off-by: Michael Tretter <m.tretter@pengutronix.de>
Link: https://patch.msgid.link/20260123-input-st1232-firmware-version-v1-1-32df7eefdafe@pengutronix.de
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>

diff --git a/drivers/input/touchscreen/st1232.c b/drivers/input/touchscreen/st1232.c
index 9b3901eec0a5ea29152ed1546b8199d787f3ec87..3dfee15fb3ad294f7bb2bf69a289b77a4fab6161 100644 (file)
--- a/drivers/input/touchscreen/st1232.c
+++ b/drivers/input/touchscreen/st1232.c
@@ -15,6 +15,7 @@
 #include <linux/i2c.h>
 #include <linux/input.h>
 #include <linux/input/mt.h>
+#include <linux/input/touch-overlay.h>
 #include <linux/input/touchscreen.h>
 #include <linux/interrupt.h>
 #include <linux/module.h>
@@ -22,11 +23,14 @@
 #include <linux/pm_qos.h>
 #include <linux/slab.h>
 #include <linux/types.h>
-#include <linux/input/touch-overlay.h>
+#include <asm/byteorder.h>
 
 #define ST1232_TS_NAME "st1232-ts"
 #define ST1633_TS_NAME "st1633-ts"
 
+#define REG_FIRMWARE_VERSION   0x00
+#define REG_FIRMWARE_REVISION_3        0x0C
+
 #define REG_STATUS             0x01    /* Device Status | Error Code */
 
 #define STATUS_NORMAL          0x00
@@ -61,6 +65,8 @@ struct st1232_ts_data {
        struct list_head touch_overlay_list;
        int read_buf_len;
        u8 *read_buf;
+       u8 fw_version;
+       u32 fw_revision;
 };
 
 static int st1232_ts_read_data(struct st1232_ts_data *ts, u8 reg,
@@ -110,6 +116,26 @@ static int st1232_ts_wait_ready(struct st1232_ts_data *ts)
        return -ENXIO;
 }
 
+static int st1232_ts_read_fw_version(struct st1232_ts_data *ts,
+                                    u8 *fw_version, u32 *fw_revision)
+{
+       int error;
+
+       /* select firmware version register */
+       error = st1232_ts_read_data(ts, REG_FIRMWARE_VERSION, 1);
+       if (error)
+               return error;
+       *fw_version = ts->read_buf[0];
+
+       /* select firmware revision register */
+       error = st1232_ts_read_data(ts, REG_FIRMWARE_REVISION_3, 4);
+       if (error)
+               return error;
+       *fw_revision = le32_to_cpup((__le32 *)ts->read_buf);
+
+       return 0;
+}
+
 static int st1232_ts_read_resolution(struct st1232_ts_data *ts, u16 *max_x,
                                     u16 *max_y)
 {
@@ -299,6 +325,16 @@ static int st1232_ts_probe(struct i2c_client *client)
        if (error)
                return error;
 
+       /* Read firmware version from the chip */
+       error = st1232_ts_read_fw_version(ts, &ts->fw_version, &ts->fw_revision);
+       if (error) {
+               dev_err(&client->dev,
+                       "Failed to read firmware version: %d\n", error);
+               return error;
+       }
+       dev_dbg(&client->dev, "Detected firmware version %u, rev %08x\n",
+               ts->fw_version, ts->fw_revision);
+
        if (ts->chip_info->have_z)
                input_set_abs_params(input_dev, ABS_MT_TOUCH_MAJOR, 0,
                                     ts->chip_info->max_area, 0, 0);