When CLK_BCM2711_DVP is enabled and RESET_CONTROLLER is disabled, it
results in the following Kbuild warning:

WARNING: unmet direct dependencies detected for RESET_SIMPLE
  Depends on [n]: RESET_CONTROLLER [=n]
  Selected by [y]:
  - CLK_BCM2711_DVP [=y] && (ARCH_BCM2835 [=n] || COMPILE_TEST [=y]) && 
COMMON_CLK [=y]

The reason is that CLK_BCM2711_DVP selects RESET_SIMPLE without depending
on or selecting RESET_CONTROLLER while RESET_SIMPLE is subordinate to
RESET_CONTROLLER.

Honor the kconfig menu hierarchy to remove kconfig dependency warnings.

Fixes: 1bc95972715a ("clk: bcm: Add BCM2711 DVP driver")
Signed-off-by: Necip Fazil Yildiran <fazilyildi...@gmail.com>
---
 drivers/clk/bcm/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clk/bcm/Kconfig b/drivers/clk/bcm/Kconfig
index 784f12c72365..ec738f74a026 100644
--- a/drivers/clk/bcm/Kconfig
+++ b/drivers/clk/bcm/Kconfig
@@ -5,6 +5,7 @@ config CLK_BCM2711_DVP
        depends on ARCH_BCM2835 ||COMPILE_TEST
        depends on COMMON_CLK
        default ARCH_BCM2835
+       select RESET_CONTROLLER
        select RESET_SIMPLE
        help
          Enable common clock framework support for the Broadcom BCM2711
-- 
2.25.1

Reply via email to