On Thu, Mar 28, 2024 at 07:29:10AM +0100, A. Sverdlin wrote: > From: Alexander Sverdlin <alexander.sverd...@siemens.com> > > am65_cpsw_ofdata_parse_phy() tries to handle the case when PHY is not > specified in DT gracefully: > > am65_cpsw_nuss_port ethernet@8000000port@1: can't parse phy-handle port 1 (-2) > > am65_cpsw_mdio_init() in turn is prepared for this, checks > if priv->has_phy == 0 and bails out (leaving cpsw_common->bus == NULL). > > am65_cpsw_phy_init() however is not prepared for this and calls > phy_connect(cpsw_common->bus, ...) unconditionally, which leads to: > > "Synchronous Abort" handler, esr 0x8600000d, far 0x0 > elr: ffffffff808ab000 lr : 000000008083bde4 (reloc) > > where lr points to the instruction right after bus->read() in get_phy_id(). > > Fixes: 9d0dca1199d1 ("net: ethernet: ti: Introduce am654 gigabit eth switch > subsystem driver") > Signed-off-by: Alexander Sverdlin <alexander.sverd...@siemens.com>
Reviewed-by: Siddharth Vadapalli <s-vadapa...@ti.com> Regards, Siddharth.