DM_REGULATOR_SCMI selects SCMI_AGENT, but that Kconfig symbol does not
exist. As a result, enabling the regulator option does not ensure that
the SCMI agent uclass and transport code are built.

Make the option depend on SCMI_FIRMWARE, matching the dependency used by
the SCMI clock, reset and power-domain consumers. This also leaves
transport selection to SCMI_FIRMWARE and its existing dependencies.

Commit 80bdc86db5a5 ("scmi: Rework SCMI_FIRMWARE implementation") made
the same select-to-dependency correction for RESET_SCMI after randconfig
exposed the incomplete SCMI configuration.

Fixes: 1f213ee4dbf2 ("firmware: scmi: voltage regulator")
Signed-off-by: Pengpeng Hou <[email protected]>
---
 drivers/power/regulator/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/power/regulator/Kconfig b/drivers/power/regulator/Kconfig
index 00a25acfdf9..924d38e17ff 100644
--- a/drivers/power/regulator/Kconfig
+++ b/drivers/power/regulator/Kconfig
@@ -524,7 +524,7 @@ config DM_REGULATOR_TPS65941
 config DM_REGULATOR_SCMI
        bool "Enable driver for SCMI voltage domain regulators"
        depends on DM_REGULATOR
-       select SCMI_AGENT
+       depends on SCMI_FIRMWARE
        help
          Enable this option if you want to support regulators exposed through
          the SCMI voltage domain protocol by a SCMI server.
-- 
2.50.1 (Apple Git-155)

Reply via email to