Hi Heinrich, On 25/02/2026 07:14, Heinrich Schuchardt wrote: > For enabling the clock driver we use symbol CONFIG_CLK. > Select this symbol for the HiSilicon Fast Ethernet Controller driver. > > Signed-off-by: Heinrich Schuchardt <[email protected]> > --- > @Jerome > > I could not find any usage of the drivers. > Should they be removed?
You mean no defconfig enables them? It could be that some project downstream depends on them though. But do we care if we don't build them upstream? Tom, what do you think? Thanks, -- Jerome > --- > drivers/net/Kconfig | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig > index d3ef050d1a1..17ceb7be57a 100644 > --- a/drivers/net/Kconfig > +++ b/drivers/net/Kconfig > @@ -979,7 +979,7 @@ source "drivers/net/mtk_eth/Kconfig" > > config HIFEMAC_ETH > bool "HiSilicon Fast Ethernet Controller" > - select DM_CLK > + select CLK > select DM_RESET > select PHYLIB > help > @@ -989,7 +989,7 @@ config HIFEMAC_ETH > config HIFEMAC_MDIO > bool "HiSilicon Fast Ethernet Controller MDIO interface" > depends on DM_MDIO > - select DM_CLK > + select CLK > help > This driver supports the internal MDIO interface of HIFEMAC > Ethernet controller.
