From: Cristian Marussi <cristian.marussi@arm.com>
Date: Mon, 20 Apr 2020 15:23:13 +0000 (+0100)
Subject: firmware: arm_scmi: Clear channel on reception of unexpected responses
X-Git-Tag: v5.8-rc1~138^2~26^2~4
X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=b37f5cc8d243479d7572445010fb6c9a4dff6dc4;p=linux.git

firmware: arm_scmi: Clear channel on reception of unexpected responses

When an unexpected response message is received we currently warn the user
and bail-out, ensure to also free the channel by invoking the transport
independent operation .clear_channel()

Link: https://lore.kernel.org/r/20200420152315.21008-3-cristian.marussi@arm.com
Signed-off-by: Cristian Marussi <cristian.marussi@arm.com>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
---

diff --git a/drivers/firmware/arm_scmi/driver.c b/drivers/firmware/arm_scmi/driver.c
index 90c7a0bb62ef..31c6a89a6edd 100644
--- a/drivers/firmware/arm_scmi/driver.c
+++ b/drivers/firmware/arm_scmi/driver.c
@@ -242,6 +242,7 @@ static void scmi_handle_response(struct scmi_chan_info *cinfo,
 	/* Are we even expecting this? */
 	if (!test_bit(xfer_id, minfo->xfer_alloc_table)) {
 		dev_err(dev, "message for %d is not expected!\n", xfer_id);
+		info->desc->ops->clear_channel(cinfo);
 		return;
 	}