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

Sverdlin, Alexander
2025-04-01T07:06:24Z

Thanks for review, Tony!

On Tue, 2025-04-01 at 06:57 +0300, Tony Lindgren wrote:
> > > 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.

I'll re-spin with more clear description!

--
Alexander Sverdlin
Siemens AG