From 3703027b42a51ee0a28f1a75f255f7d03fbca71b Mon Sep 17 00:00:00 2001
From: Tomasz Bursztyka <tobu@bang-olufsen.dk>
Date: Thu, 12 Jun 2025 12:10:41 +0200
Subject: [PATCH] i2c: Add target related commands to the shell module

For testing/debugging purposes, it will be possible then to register
or unregister an i2c target.

Signed-off-by: Tomasz Bursztyka <tobu@bang-olufsen.dk>
Co-authored-by: Hans Binderup <habi@bang-olufsen.dk>
---
 drivers/i2c/i2c_shell.c | 68 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 68 insertions(+)

diff --git a/drivers/i2c/i2c_shell.c b/drivers/i2c/i2c_shell.c
index 41f0518780c..3a2cf31fb39 100644
--- a/drivers/i2c/i2c_shell.c
+++ b/drivers/i2c/i2c_shell.c
@@ -319,6 +319,58 @@ static int cmd_i2c_speed(const struct shell *sh, size_t argc, char **argv)
 	return 0;
 }
 
+/* i2c target register <device> */
+static int cmd_i2c_target_register(const struct shell *sh,
+				   size_t argc, char **argv)
+{
+	char *s_dev_name = argv[ARGV_DEV];
+	const struct device *dev;
+	int ret;
+
+	dev = shell_device_get_binding(s_dev_name);
+	if (!dev) {
+		shell_error(sh, "I2C: Device driver %s not found.", s_dev_name);
+		return -ENODEV;
+	}
+
+	ret = i2c_target_driver_register(dev);
+	if (ret < 0) {
+		shell_error(sh, "I2C: Failed to register %s with err=%d",
+			    s_dev_name, ret);
+		return ret;
+	}
+
+	shell_print(sh, "I2C: Successfully registered %s", s_dev_name);
+
+	return 0;
+}
+
+/* i2c target unregister <device> */
+static int cmd_i2c_target_unregister(const struct shell *sh,
+				     size_t argc, char **argv)
+{
+	char *s_dev_name = argv[ARGV_DEV];
+	const struct device *dev;
+	int ret;
+
+	dev = shell_device_get_binding(s_dev_name);
+	if (!dev) {
+		shell_error(sh, "I2C: Device driver %s not found.", s_dev_name);
+		return -ENODEV;
+	}
+
+	ret = i2c_target_driver_unregister(dev);
+	if (ret < 0) {
+		shell_error(sh, "I2C: Failed to unregister %s with err=%d",
+			    s_dev_name, ret);
+		return ret;
+	}
+
+	shell_print(sh, "I2C: Successfully unregistered %s", s_dev_name);
+
+	return 0;
+}
+
 static bool device_is_i2c(const struct device *dev)
 {
 #ifdef CONFIG_I3C
@@ -340,6 +392,19 @@ static void device_name_get(size_t idx, struct shell_static_entry *entry)
 
 SHELL_DYNAMIC_CMD_CREATE(dsub_device_name, device_name_get);
 
+SHELL_STATIC_SUBCMD_SET_CREATE(
+	sub_i2c_target,
+	SHELL_CMD_ARG(register, &dsub_device_name,
+		      "Register an i2c-target on its respective bus.\n"
+		      "Usage: target register <device>",
+		      cmd_i2c_target_register, 2, 0),
+	SHELL_CMD_ARG(unregister, &dsub_device_name,
+		      "Unegister an i2c-target from its respective bus.\n"
+		      "Usage: target unregister <device>",
+		      cmd_i2c_target_unregister, 2, 0),
+	SHELL_SUBCMD_SET_END     /* Array terminated. */
+);
+
 SHELL_STATIC_SUBCMD_SET_CREATE(sub_i2c_cmds,
 	SHELL_CMD_ARG(scan, &dsub_device_name,
 		      "Scan I2C devices\n"
@@ -374,6 +439,9 @@ SHELL_STATIC_SUBCMD_SET_CREATE(sub_i2c_cmds,
 		      "Configure I2C bus speed\n"
 		      "Usage: speed <device> <speed>",
 		      cmd_i2c_speed, 3, 0),
+	SHELL_CMD_ARG(target, &sub_i2c_target,
+		      "Subcommands operating on i2c targets.",
+		      NULL, 3, 0),
 	SHELL_SUBCMD_SET_END     /* Array terminated. */
 );