mmc: renesas_sdhi: add regulator dependency
authorArnd Bergmann <arnd@arndb.de>
Sat, 29 Mar 2025 16:41:26 +0000 (17:41 +0100)
committerUlf Hansson <ulf.hansson@linaro.org>
Mon, 7 Apr 2025 15:57:01 +0000 (17:57 +0200)
The driver started using the regulator subsystem and fails to build without
a dependeny on CONFIG_REGULATOR:

ERROR: modpost: "rdev_get_drvdata" [drivers/mmc/host/renesas_sdhi_core.ko] undefined!
ERROR: modpost: "devm_regulator_register" [drivers/mmc/host/renesas_sdhi_core.ko] undefined!

The 'select RESET_CONTROLLER' needs to either go away or get changed to a dependency
in order to avoid Kconfig dependency loops here. It also turns out the the superh
version needs neither RESET_CONTROLLER nor REGULATOR, and this works because
CONFIG_OF is not set there.

Change both to a 'depends on', but add '|| !OF' for the superh case.

Fixes: fae80a99dc03 ("mmc: renesas_sdhi: Add support for RZ/G3E SoC")
Tested-by: Biju Das <biju.das.jz@bp.renesas.com>
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Link: https://lore.kernel.org/r/20250329164145.3194284-1-arnd@kernel.org
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/host/Kconfig

index 6824131b69b188cae58c8f48076715ca647ca28c..264e11fa58eafb20fda8b04ff1269ce3974ea776 100644 (file)
@@ -691,8 +691,8 @@ config MMC_TMIO_CORE
 config MMC_SDHI
        tristate "Renesas SDHI SD/SDIO controller support"
        depends on SUPERH || ARCH_RENESAS || COMPILE_TEST
+       depends on (RESET_CONTROLLER && REGULATOR) || !OF
        select MMC_TMIO_CORE
-       select RESET_CONTROLLER if ARCH_RENESAS
        help
          This provides support for the SDHI SD/SDIO controller found in
          Renesas SuperH, ARM and ARM64 based SoCs