[PATCH v3 1/5] firmware: arm_scmi: Remove superfluous handle_to_scmi_info

Luke Parkin posted 5 patches 1 year, 6 months ago
There is a newer version of this series
[PATCH v3 1/5] firmware: arm_scmi: Remove superfluous handle_to_scmi_info
Posted by Luke Parkin 1 year, 6 months ago
Remove duplicate handle_to_scmi_info

Signed-off-by: Luke Parkin <luke.parkin@arm.com>
Reviewed-by: Cristian Marussi <cristian.marussi@arm.com>
---
 drivers/firmware/arm_scmi/driver.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/firmware/arm_scmi/driver.c b/drivers/firmware/arm_scmi/driver.c
index 6b6957f4743f..56a93d20bf23 100644
--- a/drivers/firmware/arm_scmi/driver.c
+++ b/drivers/firmware/arm_scmi/driver.c
@@ -1261,9 +1261,6 @@ static int scmi_wait_for_reply(struct device *dev, const struct scmi_desc *desc,
 					    xfer->rx.buf, xfer->rx.len);
 
 			if (IS_ENABLED(CONFIG_ARM_SCMI_RAW_MODE_SUPPORT)) {
-				struct scmi_info *info =
-					handle_to_scmi_info(cinfo->handle);
-
 				scmi_raw_message_report(info->raw, xfer,
 							SCMI_RAW_REPLY_QUEUE,
 							cinfo->id);
-- 
2.34.1
Re: [PATCH v3 1/5] firmware: arm_scmi: Remove superfluous handle_to_scmi_info
Posted by Cristian Marussi 1 year, 6 months ago
On Mon, Jul 15, 2024 at 02:37:47PM +0100, Luke Parkin wrote:
> Remove duplicate handle_to_scmi_info
> 
> Signed-off-by: Luke Parkin <luke.parkin@arm.com>
> Reviewed-by: Cristian Marussi <cristian.marussi@arm.com>

Your Signed-off-by should always come last after any Reviewd-by, CC or
any other tag...

Thanks,
Cristian