Hi Jonas,
On 3/29/24 20:01, Jonas Karlman wrote:
Imply support for GbE PHY status parsing and configuration when support
for onboard ethernet is enabled.
Signed-off-by: Jonas Karlman <jo...@kwiboo.se>
---
arch/arm/mach-rockchip/Kconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/mach-rockchip/Kconfig b/arch/arm/mach-rockchip/Kconfig
index d518913f8a37..d85b59a92da2 100644
--- a/arch/arm/mach-rockchip/Kconfig
+++ b/arch/arm/mach-rockchip/Kconfig
@@ -316,6 +316,7 @@ config ROCKCHIP_RK3568
imply MISC_INIT_R
imply MMC_HS200_SUPPORT if MMC_SDHCI_ROCKCHIP
imply OF_LIBFDT_OVERLAY
+ imply PHY_GIGE if DWC_ETH_QOS_ROCKCHIP
Is this really something we expect most devices to use?
We have two products based on RK3588, none use it. If I'm not mistaken,
Rock5B doesn't as well as Orange Pi 5 Plus, RK3588 EVB, Rock5A, (likely
not the Edgeble as well since they have 2.5Gbps connectors), NanoPC T6,
IndieDroid Nova, Cool Pi 4B, Cool Pi CM5 EVB, NanoPi R6S, Rockchip
Toybrick TB-RK3588X.
So, I'm not sure it's worth it making it the default? (Even though we
could remove it from the defconfig manually). Wouldn't this make more
sense in your generic defconfigs?
Cheers,
Quentin