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@8000000port@1: phy_connect() failed am65_cpsw_nuss_port ethernet@8000000ethernet@8000000port@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@8000000port@1: phy_connect() failed am65_cpsw_nuss_port ethernet@8000000ethernet@800000: phy_connect() failed No ethernet found.
Seems that the topic can be closed at least for now. Thanks for looking into that!