net: ethernet: renesas: rcar_gen4_ptp: Depend on PTP_1588_CLOCK
authorNiklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Wed, 29 Nov 2023 11:11:42 +0000 (12:11 +0100)
committerJakub Kicinski <kuba@kernel.org>
Sat, 2 Dec 2023 04:17:55 +0000 (20:17 -0800)
When breaking out the Gen4 gPTP support to its own module the dependency
on the PTP_1588_CLOCK framework was left as optional and only stated for
the driver using the module. This leads to issues when doing
COMPILE_TEST of RENESAS_GEN4_PTP separately and PTP_1588_CLOCK is built
as a module and the other as a built-in. Add an explicit depend on
PTP_1588_CLOCK.

While at it remove the optional support for PTP_1588_CLOCK from
RENESAS_ETHER_SWITCH as the driver unconditionally calls the Gen4 gPTP
module and thus also requires the PTP_1588_CLOCK framework.

Reported-by: Arnd Bergmann <arnd@arndb.de>
Fixes: 8c1c66235e03 ("net: ethernet: renesas: rcar_gen4_ptp: Break out to module")
Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Reviewed-by: Arnd Bergmann <arnd@arndb.de>
Link: https://lore.kernel.org/r/20231129111142.3322667-1-niklas.soderlund+renesas@ragnatech.se
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/renesas/Kconfig

index 733cbb6eb3ed95f3c1d38f4a18211eebbe2524a2..d6136fe5c20645989a6affbf9483fcd54522b9f1 100644 (file)
@@ -40,7 +40,7 @@ config RAVB
 config RENESAS_ETHER_SWITCH
        tristate "Renesas Ethernet Switch support"
        depends on ARCH_RENESAS || COMPILE_TEST
-       depends on PTP_1588_CLOCK_OPTIONAL
+       depends on PTP_1588_CLOCK
        select CRC32
        select MII
        select PHYLINK
@@ -50,6 +50,7 @@ config RENESAS_ETHER_SWITCH
 
 config RENESAS_GEN4_PTP
        tristate "Renesas R-Car Gen4 gPTP support" if COMPILE_TEST
+       depends on PTP_1588_CLOCK
        select CRC32
        select MII
        select PHYLIB