platform: cznic: turris-omnia-mcu: fix Kconfig dependencies
authorArnd Bergmann <arnd@arndb.de>
Mon, 15 Jul 2024 06:02:30 +0000 (08:02 +0200)
committerArnd Bergmann <arnd@arndb.de>
Mon, 15 Jul 2024 06:08:58 +0000 (08:08 +0200)
The newly added driver causes a Kconfig warning:

WARNING: unmet direct dependencies detected for RTC_CLASS
  Depends on [n]: !S390 [=y]
  Selected by [m]:
  - TURRIS_OMNIA_MCU [=m] && CZNIC_PLATFORMS [=y] && (MACH_ARMADA_38X || COMPILE_TEST [=y]) && I2C [=m] && OF [=y] && WATCHDOG [=y]

The problem here is that it selects entire subsystems, which normal
device drivers should not do. Changes all of these to 'depends on'
instead.

Fixes: dfa556e45ae9e ("platform: cznic: turris-omnia-mcu: Add support for MCU connected GPIOs")
Fixes: 90e700fd12b61 ("platform: cznic: turris-omnia-mcu: Add support for poweroff and wakeup")
Fixes: ab89fb5fb92c7 ("platform: cznic: turris-omnia-mcu: Add support for MCU watchdog")
Fixes: 41bb142a40289 ("platform: cznic: turris-omnia-mcu: Add support for MCU provided TRNG")
Reported-by: Nathan Chancellor <nathan@kernel.org>
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
drivers/platform/cznic/Kconfig

index 2a5235cf6844c3de7dbf528732ed9bac3dacb56a..cb0d4d686d8a5123e2f0511a03af343e5854bd74 100644 (file)
@@ -18,11 +18,11 @@ config TURRIS_OMNIA_MCU
        depends on I2C
        depends on OF
        depends on WATCHDOG
-       select GPIOLIB
+       depends on GPIOLIB
+       depends on HW_RANDOM
+       depends on RTC_CLASS
+       depends on WATCHDOG_CORE
        select GPIOLIB_IRQCHIP
-       select HW_RANDOM
-       select RTC_CLASS
-       select WATCHDOG_CORE
        help
          Say Y here to add support for the features implemented by the
          microcontroller on the CZ.NIC's Turris Omnia SOHO router.