[PATCH v2] net: ti: am65-cpsw-nuss: handle missing PHY in am65_cpsw_phy_init() gracefully

[Sverdlin, Alexander]

Hello Nishanth!

On Thu, 2024-03-28 at 07:04 -0500, Nishanth Menon wrote:
> On 11:53-20240328, Sverdlin, Alexander wrote:
> [..]
> > > Btw, this no longer applies on next. only applies on master. Depending
> > > on where Tom would like to apply this change, might need a rebase.
> > 
> > Thanks for the hint! I've totally missed this!
> > And I'm even not sure the patch makes sense on "next".
> > I'd need to retest if the problem is still there (and if there would be one,
> > then it's rather subsystem problem, not am65-cpsw any more.
> > 
> > So the patch only makes sense for stable branches, if any.
> 
> On next, I think a variant of the patch will be needed as well. I see
> the same bug on latest next as well (ab8d9ca3044a Merge tag
> 'v2024.04-rc5' into next)
> 
> https://gist.github.com/nmenon/691ccd210ed7e1add4c865ca9a698f2e

While 2024.04-rc5 (next) aborts:

Net:   am65_cpsw_nuss_port ethernet at 8000000port@1: phy_connect() failed
am65_cpsw_nuss_port ethernet at 8000000ethernet@8000000port at 1: phy_connect() failed
"Synchronous Abort" handler, esr 0x86000004, far 0x3030303030303840
...
Code: "Synchronous Abort" handler, esr 0x96000004, far 0x3030303030303830
elr: 00000000808021d8 lr : 00000000808021b8 (reloc)

2024.07-rc1 (master) handles it gracefully:

Net:   am65_cpsw_nuss_port ethernet at 8000000port@1: phy_connect() failed
am65_cpsw_nuss_port ethernet at 8000000ethernet@800000: phy_connect() failed
No ethernet found.

Seems that the topic can be closed at least for now.
Thanks for looking into that!

-- 
Alexander Sverdlin
Siemens AG
www.siemens.com