Received-SPF: pass (google.com: domain of linux-kernel+bounces-22571-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
From: Frank Li <Frank.Li@nxp.com>
To: frank.li@nxp.com,
	robh@kernel.org
Cc: alexandre.belloni@bootlin.com,
	conor.culhane@silvaco.com,
	gregkh@linuxfoundation.org,
	imx@lists.linux.dev,
	jirislaby@kernel.org,
	joe@perches.com,
	linux-i3c@lists.infradead.org,
	linux-kernel@vger.kernel.org,
	linux-serial@vger.kernel.org,
	miquel.raynal@bootlin.com,
	zbigniew.lukwinski@linux.intel.com,
	devicetree@vger.kernel.org,
	krzysztof.kozlowski+dt@linaro.org,
	krzysztof.kozlowski@linaro.org
Subject: [PATCH v2 6/7] i3c: add API i3c_dev_gettstatus_format1() to get target device status
Date: Wed, 10 Jan 2024 12:52:20 -0500
Message-Id: <20240110175221.2335480-7-Frank.Li@nxp.com>
In-Reply-To: <20240110175221.2335480-1-Frank.Li@nxp.com>
References: <20240110175221.2335480-1-Frank.Li@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?SXOxOmlXUOj6QMKGLXWkEMiPZwNlC6XPD81RsGvw7ZvpVQ4+FMgwFRKmtOXq?=
 =?us-ascii?Q?WozUj2/4SmkF12p/hUK2+jKTN5PDdcs+SWE09YiZ/ne9GFHr60iPxU1VTSZJ?=
 =?us-ascii?Q?MNm1anCsIjWgI3LdmerlnBtGW8Yo1up7eq8yvlc84/YZ1SXZJ0KHH6vyYULL?=
 =?us-ascii?Q?ebIU2bQCy2WCyz7INL+xCJ/a35Tf5T4Q9X55XJvCp1ngKrtrD2ciJTr9isWu?=
 =?us-ascii?Q?c0JQaK/5OJMdZD8bMldYMWJkHeNeJ24HIJcTgdpBbj8nNDFJWB+gPjslTwfA?=
 =?us-ascii?Q?Qjllyc1BHLKcuqC8mLbCMqj7jZ6poOJ391TuSi9uPeN9f5hsxS1B1mDgHHr9?=
 =?us-ascii?Q?nRjxrpFA5FWHWvsG2j+tZK2Jr1jqbTO8qgVbwffpOX11DXKEdOyxBJcLN37f?=
 =?us-ascii?Q?0g31LimKl1beEVYqt5YDQnHGKyOwuB3LLVMptXcoiAtlD0cQBqqbsMStfeCB?=
 =?us-ascii?Q?NS1DOvRJYB8c++MBocMsATeOMmnDDzpY/7U/QpJG8eolu3OaNrXQMMc6kX8C?=
 =?us-ascii?Q?1wu5N22HUQBiBt67/kfNSM1kke8/Yyo3RFHuQUDH3ELNKUrA7uc3JIZzX4Sm?=
 =?us-ascii?Q?KVSn8TVMsS1WHkTVc0mXfpoDZzcySamxmkLMabYDncb6+LWr1HNbV2iHPTr0?=
 =?us-ascii?Q?4xU2aqULwZCdblL5lZiinFHKEL84ryzop2Xub1I+IrBUYYOpXaCXHgn3i7cm?=
 =?us-ascii?Q?htdBQrc39zYnEtbG5EvuXD3PzPKLKIxPVWT+Z/CCySJE6Vk21+CxVHoNUEMI?=
 =?us-ascii?Q?6oPYT2QdvNWaOMlZTlWMToO6qic8cBMf1cwL1Dt/4yr4qqufcJ7OwpLT30YI?=
 =?us-ascii?Q?sdRbzeBF1mHVtmokg47HFSFtbzV7sSqEZDmkij2jCi5wSCrLyXj30xEraTIH?=
 =?us-ascii?Q?tW/vOYjBQH1dIwKMmtPgkRe4XWK4893lv7eguRLd8ezMC55OMO2NNXxNwXq/?=
 =?us-ascii?Q?ReUhm3FX/wuGEmSbhsdyXNTJ8r64LRsnNyv+ilSlg6R6pKz5Jm35yCjsItVV?=
 =?us-ascii?Q?eYtUUne4qOXcFp/ns0fBYe/c3PkWAPASGiWZ47pfrXyw5TW+I7v6qldroY43?=
 =?us-ascii?Q?IxZNA0hTuKTZK3uPWYCSAf3aPNo5PsM315JgPKysBYok1oIdYKxQHzEofcRF?=
 =?us-ascii?Q?yLpd7r56U/5FJfCHiAyOXHeMRKoFcv9zMiuabvuwyJInXoS82FTgZwKWtLWN?=
 =?us-ascii?Q?SVEf+nsMtRpth350Ce4JcYRPlZwjowaUzsN66X9wtqp/io2G2b3UbeJZXbC3?=
 =?us-ascii?Q?sqUgjXUlO2Vn4rAhXs0bNY9JiOYsiFDeZuGoY+WNzFXiYSExaRkgT9rsEBjR?=
 =?us-ascii?Q?qZI4h5Lk4pIEGNM+xOjqCN2BTj1u/pAKqeju4YmWZppEYvsLU88F916rq/B1?=
 =?us-ascii?Q?x4E5zj+Fs6pHSj9lMlNAPoU4il61+RVpvU72xQv13V+0izy7Zv4VfT1/AESH?=
 =?us-ascii?Q?K0Ozd6xtj/sfCr2+p/IWD3dJ/fnL5UxOzQhGSUukSvL6Ilt423nq4cnedhP3?=
 =?us-ascii?Q?6vFp2UBEwKB0DIW5f5qIJtb5h2kmvjxQxG+IyimBUiRlZSSp9rKMuK9F272Q?=
 =?us-ascii?Q?3SVAmmNew0u+u/wlh6o=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 1e8e635c-ab26-40a6-3c1c-08dc1204feec
X-MS-Exchange-CrossTenant-AuthSource: AS8PR04MB7511.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Jan 2024 17:53:04.7471
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: DlXhbrcutyVNt88c8j/hgDrsrxwd2UHPuf53f5tnytENUt7YU3Hi1NoFn21yu2zFWY2ebH3sE25OReYUl5ws4w==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR04MB9332

I3C standard 5.1.9.3.15 Get Device Status (GETSTATUS):
Get request for one I3C Target Device to return its current status.

Add API to fetch it with format1.

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 drivers/i3c/device.c       | 24 ++++++++++++++++++++++++
 drivers/i3c/internals.h    |  1 +
 drivers/i3c/master.c       | 26 ++++++++++++++++++++++++++
 include/linux/i3c/device.h |  1 +
 4 files changed, 52 insertions(+)

diff --git a/drivers/i3c/device.c b/drivers/i3c/device.c
index 1a6a8703dbc3a..aa26cf50ab9c6 100644
--- a/drivers/i3c/device.c
+++ b/drivers/i3c/device.c
@@ -196,6 +196,30 @@ void i3c_device_free_ibi(struct i3c_device *dev)
 }
 EXPORT_SYMBOL_GPL(i3c_device_free_ibi);
 
+/**
+ * i3c_device_getstatus_format1() - Get device status with format 1.
+ * @dev: device for which you want to get status.
+ * @status: I3C status format 1
+ *
+ * Return: 0 in case of success, a negative error core otherwise.
+ */
+int i3c_device_getstatus_format1(struct i3c_device *dev, u16 *status)
+{
+	int ret = -EINVAL;
+
+	if (!status)
+		return -EINVAL;
+
+	i3c_bus_normaluse_lock(dev->bus);
+	if (dev->desc)
+		ret = i3c_dev_getstatus_format1_locked(dev->desc, status);
+
+	i3c_bus_normaluse_unlock(dev->bus);
+
+	return ret;
+}
+EXPORT_SYMBOL_GPL(i3c_device_getstatus_format1);
+
 /**
  * i3cdev_to_dev() - Returns the device embedded in @i3cdev
  * @i3cdev: I3C device
diff --git a/drivers/i3c/internals.h b/drivers/i3c/internals.h
index 908a807badaf9..976ad26ca79c2 100644
--- a/drivers/i3c/internals.h
+++ b/drivers/i3c/internals.h
@@ -24,4 +24,5 @@ int i3c_dev_enable_ibi_locked(struct i3c_dev_desc *dev);
 int i3c_dev_request_ibi_locked(struct i3c_dev_desc *dev,
 			       const struct i3c_ibi_setup *req);
 void i3c_dev_free_ibi_locked(struct i3c_dev_desc *dev);
+int i3c_dev_getstatus_format1_locked(struct i3c_dev_desc *dev, u16 *status);
 #endif /* I3C_INTERNAL_H */
diff --git a/drivers/i3c/master.c b/drivers/i3c/master.c
index d3b56c9f601e2..81611a3e3585a 100644
--- a/drivers/i3c/master.c
+++ b/drivers/i3c/master.c
@@ -2923,6 +2923,32 @@ void i3c_dev_free_ibi_locked(struct i3c_dev_desc *dev)
 	dev->ibi = NULL;
 }
 
+int i3c_dev_getstatus_format1_locked(struct i3c_dev_desc *dev, u16 *status)
+{
+	struct i3c_master_controller *master = i3c_dev_get_master(dev);
+	struct i3c_ccc_getstatus *format1;
+	struct i3c_ccc_cmd_dest dest;
+	struct i3c_ccc_cmd cmd;
+	int ret;
+
+	format1 = i3c_ccc_cmd_dest_init(&dest, dev->info.dyn_addr, sizeof(*format1));
+	if (!format1)
+		return -ENOMEM;
+
+	i3c_ccc_cmd_init(&cmd, true, I3C_CCC_GETSTATUS, &dest, 1);
+
+	ret = i3c_master_send_ccc_cmd_locked(master, &cmd);
+	if (ret)
+		goto out;
+
+	*status = be16_to_cpu(format1->status);
+
+out:
+	i3c_ccc_cmd_dest_cleanup(&dest);
+
+	return ret;
+}
+
 static int __init i3c_init(void)
 {
 	int res;
diff --git a/include/linux/i3c/device.h b/include/linux/i3c/device.h
index ef6217da8253b..5f511bd400f11 100644
--- a/include/linux/i3c/device.h
+++ b/include/linux/i3c/device.h
@@ -345,4 +345,5 @@ void i3c_device_free_ibi(struct i3c_device *dev);
 int i3c_device_enable_ibi(struct i3c_device *dev);
 int i3c_device_disable_ibi(struct i3c_device *dev);
 
+int i3c_device_getstatus_format1(struct i3c_device *dev, u16 *status);
 #endif /* I3C_DEV_H */
-- 
2.34.1