Re: [PATCH] Revert "bus: ti-sysc: Probe for l4_wkup and l4_cfg interconnect devices first"

Tony Lindgren
2025-04-01T03:58:02Z

* Andreas Kemnade <andreas@kemnade.info> [250331 09:00]:
> Am Thu, 13 Mar 2025 10:47:06 +0100
> schrieb "A. Sverdlin" <alexander.sverdlin@siemens.com>:
>
> > From: Alexander Sverdlin <alexander.sverdlin@siemens.com>
> >
> > This reverts commit 4700a00755fb5a4bb5109128297d6fd2d1272ee6.
> >
> > It brakes target-module@2b300050 ("ti,sysc-omap2") probe on AM62x in a case
> > when minimally-configured system tries to network-boot:
> >
> > [ 6.888776] probe of 2b300050.target-module returned 517 after 258 usecs
> > [ 17.129637] probe of 2b300050.target-module returned 517 after 708 usecs
> > [ 17.137397] platform 2b300050.target-module: deferred probe pending: (reason unknown)
> > [ 26.878471] Waiting up to 100 more seconds for network.
> >
> > Arbitrary 10 deferrals is really not a solution to any problem.
> > Stable mmc enumeration can be achiever by filling /aliases node properly
> > (4700a00755fb commit's rationale).

Would be nice to update the description to make it clear it breaks in all
cases if there are not enough devices.

> > After revert:
> >
> > [ 9.006816] IP-Config: Complete:
> > [ 9.010058] device=lan0, ...
> >
> > Signed-off-by: Alexander Sverdlin <alexander.sverdlin@siemens.com>
>
> No regressions seen on GTA04, Pandaboard and BT200.
> So
>
> Tested-by: Andreas Kemnade <andreas@kemnade.info> # GTA04, Panda, BT200

Good to hear, best that you or Kevin pick this up:

Reviewed-by: Tony Lindgren <tony@atomide.com>