Update SCMI reset controller driver to get its assigned SCMI channel
during initialization. This change allows SCMI reset domain protocol
to use a dedicated channel when defined in the DT. The reference is
saved in SCMI reset controller driver private data.

Signed-off-by: Etienne Carriere <etienne.carri...@linaro.org>
---
Changes since v1:
- Define a private struct to hold channel reference rather than using
  device private data reference as opaque channel reference.

---
 drivers/reset/reset-scmi.c | 25 ++++++++++++++++++++++---
 1 file changed, 22 insertions(+), 3 deletions(-)

diff --git a/drivers/reset/reset-scmi.c b/drivers/reset/reset-scmi.c
index 30b26ec9d31..122556162ec 100644
--- a/drivers/reset/reset-scmi.c
+++ b/drivers/reset/reset-scmi.c
@@ -1,6 +1,6 @@
 // SPDX-License-Identifier: GPL-2.0+
 /*
- * Copyright (C) 2019-2020 Linaro Limited
+ * Copyright (C) 2019-2022 Linaro Limited
  */
 
 #define LOG_CATEGORY UCLASS_RESET
@@ -13,8 +13,17 @@
 #include <scmi_protocols.h>
 #include <asm/types.h>
 
+/**
+ * struct scmi_reset_priv - Private data for SCMI reset controller
+ * @channel: Reference to the SCMI channel to use
+ */
+struct scmi_reset_priv {
+       struct scmi_channel *channel;
+};
+
 static int scmi_reset_set_level(struct reset_ctl *rst, bool 
assert_not_deassert)
 {
+       struct scmi_reset_priv *priv = dev_get_priv(rst->dev);
        struct scmi_rd_reset_in in = {
                .domain_id = rst->id,
                .flags = assert_not_deassert ? SCMI_RD_RESET_FLAG_ASSERT : 0,
@@ -26,7 +35,7 @@ static int scmi_reset_set_level(struct reset_ctl *rst, bool 
assert_not_deassert)
                                          in, out);
        int ret;
 
-       ret = devm_scmi_process_msg(rst->dev, NULL, &msg);
+       ret = devm_scmi_process_msg(rst->dev, priv->channel, &msg);
        if (ret)
                return ret;
 
@@ -45,6 +54,7 @@ static int scmi_reset_deassert(struct reset_ctl *rst)
 
 static int scmi_reset_request(struct reset_ctl *rst)
 {
+       struct scmi_reset_priv *priv = dev_get_priv(rst->dev);
        struct scmi_rd_attr_in in = {
                .domain_id = rst->id,
        };
@@ -58,7 +68,7 @@ static int scmi_reset_request(struct reset_ctl *rst)
         * We don't really care about the attribute, just check
         * the reset domain exists.
         */
-       ret = devm_scmi_process_msg(rst->dev, NULL, &msg);
+       ret = devm_scmi_process_msg(rst->dev, priv->channel, &msg);
        if (ret)
                return ret;
 
@@ -71,8 +81,17 @@ static const struct reset_ops scmi_reset_domain_ops = {
        .rst_deassert   = scmi_reset_deassert,
 };
 
+static int scmi_reset_probe(struct udevice *dev)
+{
+       struct scmi_reset_priv *priv = dev_get_priv(dev);
+
+       return devm_scmi_of_get_channel(dev, &priv->channel);
+}
+
 U_BOOT_DRIVER(scmi_reset_domain) = {
        .name = "scmi_reset_domain",
        .id = UCLASS_RESET,
        .ops = &scmi_reset_domain_ops,
+       .probe = scmi_reset_probe,
+       .priv_auto = sizeof(struct scmi_reset_priv *),
 };
-- 
2.25.1

Reply via email to