From: Peng Fan <[email protected]>

Add conditional compilation for SCMI protocol support in scmi_get_protocol()
and scmi_add_protocol() based on corresponding Kconfig options. This ensures
that only the enabled protocols are compiled and accessed, and reducing binary
size.

Signed-off-by: Peng Fan <[email protected]>
---
 drivers/firmware/scmi/scmi_agent-uclass.c | 20 ++++++++++++++++++++
 include/scmi_agent-uclass.h               | 10 ++++++++++
 2 files changed, 30 insertions(+)

diff --git a/drivers/firmware/scmi/scmi_agent-uclass.c 
b/drivers/firmware/scmi/scmi_agent-uclass.c
index 
69a277e8786b46994ccb83a04c155175858791ad..f48c2fdb27b16e0693eca6316d2ef6b27bc1eedf
 100644
--- a/drivers/firmware/scmi/scmi_agent-uclass.c
+++ b/drivers/firmware/scmi/scmi_agent-uclass.c
@@ -86,21 +86,31 @@ struct udevice *scmi_get_protocol(struct udevice *dev,
        case SCMI_PROTOCOL_ID_BASE:
                proto = priv->base_dev;
                break;
+#if IS_ENABLED(CONFIG_SCMI_POWER_DOMAIN)
        case SCMI_PROTOCOL_ID_POWER_DOMAIN:
                proto = priv->pwdom_dev;
                break;
+#endif
+#if IS_ENABLED(CONFIG_CLK_SCMI)
        case SCMI_PROTOCOL_ID_CLOCK:
                proto = priv->clock_dev;
                break;
+#endif
+#if IS_ENABLED(CONFIG_RESET_SCMI)
        case SCMI_PROTOCOL_ID_RESET_DOMAIN:
                proto = priv->resetdom_dev;
                break;
+#endif
+#if IS_ENABLED(CONFIG_DM_REGULATOR_SCMI)
        case SCMI_PROTOCOL_ID_VOLTAGE_DOMAIN:
                proto = priv->voltagedom_dev;
                break;
+#endif
+#if IS_ENABLED(CONFIG_PINCTRL_IMX_SCMI)
        case SCMI_PROTOCOL_ID_PINCTRL:
                proto = priv->pinctrl_dev;
                break;
+#endif
        default:
                dev_err(dev, "Protocol not supported\n");
                proto = NULL;
@@ -139,21 +149,31 @@ static int scmi_add_protocol(struct udevice *dev,
        case SCMI_PROTOCOL_ID_BASE:
                priv->base_dev = proto;
                break;
+#if IS_ENABLED(CONFIG_SCMI_POWER_DOMAIN)
        case SCMI_PROTOCOL_ID_POWER_DOMAIN:
                priv->pwdom_dev = proto;
                break;
+#endif
+#if IS_ENABLED(CONFIG_CLK_SCMI)
        case SCMI_PROTOCOL_ID_CLOCK:
                priv->clock_dev = proto;
                break;
+#endif
+#if IS_ENABLED(CONFIG_RESET_SCMI)
        case SCMI_PROTOCOL_ID_RESET_DOMAIN:
                priv->resetdom_dev = proto;
                break;
+#endif
+#if IS_ENABLED(CONFIG_DM_REGULATOR_SCMI)
        case SCMI_PROTOCOL_ID_VOLTAGE_DOMAIN:
                priv->voltagedom_dev = proto;
                break;
+#endif
+#if IS_ENABLED(CONFIG_PINCTRL_IMX_SCMI)
        case SCMI_PROTOCOL_ID_PINCTRL:
                priv->pinctrl_dev = proto;
                break;
+#endif
        default:
                dev_err(dev, "Protocol not supported\n");
                return -EPROTO;
diff --git a/include/scmi_agent-uclass.h b/include/scmi_agent-uclass.h
index 
d6586eb3ff9d48073cafbb4c036874dbe4e0ee6a..ccedd42f3670c95376412f686c67d462e9740c06
 100644
--- a/include/scmi_agent-uclass.h
+++ b/include/scmi_agent-uclass.h
@@ -40,11 +40,21 @@ struct scmi_agent_priv {
        u8 *agent_name;
        u32 agent_id;
        struct udevice *base_dev;
+#if IS_ENABLED(CONFIG_SCMI_POWER_DOMAIN)
        struct udevice *pwdom_dev;
+#endif
+#if IS_ENABLED(CONFIG_CLK_SCMI)
        struct udevice *clock_dev;
+#endif
+#if IS_ENABLED(CONFIG_RESET_SCMI)
        struct udevice *resetdom_dev;
+#endif
+#if IS_ENABLED(CONFIG_DM_REGULATOR_SCMI)
        struct udevice *voltagedom_dev;
+#endif
+#if IS_ENABLED(CONFIG_PINCTRL_IMX_SCMI)
        struct udevice *pinctrl_dev;
+#endif
 };
 
 static inline u32 scmi_version(struct udevice *dev)

-- 
2.35.3

Reply via email to