[PATCH 2/2] ti: Do not remove ft_board_setup for CONFIG_NO_NET

Maarten Brock Maarten.Brock at sttls.nl
Fri Sep 5 12:06:10 CEST 2025


From: Tom Rini
> On Thu, Sep 04, 2025 at 01:19:26PM +0000, Maarten Brock wrote:
> > From: Tom Rini
> >
> > > On Mon, Sep 01, 2025 at 04:06:10PM +0000, Maarten Brock wrote:
> > >
> > > > Even when CONFIG_NO_NET is selected there is still need for a
> > > > definition of ft_board_setup(). Thus move the checks for
> > > > CONFIG_DM_ETH and CONFIG_DRIVER_TI_CPSW inside the function.
> > > >
> > > > Signed-off-by: Maarten Brock <maarten.brock at sttls.nl>
> > >
> > > Shouldn't OF_BOARD_SETUP be disabled in that case?
> > >
> > > --
> > > Tom
> >
> > Had there also been non-NET-related fixups in this ft_board_setup(),
> > would that also have been your answer?
> > I don't see why disabling NET drivers would have to impact anything else.
> 
> Yes it would have been my answer. And because not all dependencies are
> easily expressible in Kconfig.

If this is your opinion then I propose to remove the checks for DM_ETH and
DRIVER_TI_CPSW alltogether. They obviously serve no purpose as one is
required to deselect OF_BOARD_SETUP when NO_NET is selected.

-#if defined(CONFIG_OF_BOARD_SETUP) && defined(CONFIG_OF_CONTROL) && \
-	defined(CONFIG_DM_ETH) && defined(CONFIG_DRIVER_TI_CPSW)
+#if defined(CONFIG_OF_BOARD_SETUP) && defined(CONFIG_OF_CONTROL)

I must hope that there will never be the need for other fixups for e.g.
the RTC or the user is left with either choosing both RTC and NET or neither.

Kind regards,
Maarten


More information about the U-Boot mailing list