Re: PHY issue with SJA1105Q/DP84849I Design

From: Jagan Teki
Date: Tue Dec 26 2023 - 02:41:30 EST


On Fri, Dec 22, 2023 at 8:21 PM Vladimir Oltean <olteanv@xxxxxxxxx> wrote:
>
> On Wed, Dec 20, 2023 at 12:49:30PM +0530, Jagan Teki wrote:
> > [ 1255.021656] Modules linked in: mdio_netlink(O) bnep tag_sja1105 sja1105 pcs_xpcs brcmfmac brcmutil imx_sdma coda_vpu v4l2_jpeg imx_vdoa evbug
> > [ 1255.034578] CPU: 0 PID: 374 Comm: ip Tainted: G O 6.1.58 #1
>
> Can you please reproduce the issue with no out-of-tree modules used
> (here mdio_netlink)?

root@imx6solo:~# ip link set dev eth0 down
root@imx6solo:~# ip link set dev ethphy0 down
root@imx6solo:~# ip link set dev ethphy1 down
root@imx6solo:~# ip link set dev ethphy0 address 00:11:22:33:44:55
root@imx6solo:~# ip link set dev ethphy1 address 00:11:22:33:44:66
root@imx6solo:~# ip addr add dev ethphy0 192.168.1.1/16
root@imx6solo:~# ip addr add dev ethphy1 192.168.1.5/16
root@imx6solo:~# ip link set dev eth0 up
[ 617.126753] dp83848_config_init: ret 0
[ 617.130831] TI DP83849I 10/100 Mbps PHY 2188000.ethernet-1:00:
attached PHY driver (mii_bus:phy_addr=2188000.ethernet-1:00, irq=POLL)
root@imx6solo:~# [ 619.207519] fec 2188000.ethernet eth0: Link is Up
- 100Mbps/Full - flow control rx/tx
[ 619.215698] IPv6: ADDRCONF(NETDEV_CHANGE): eth0: link becomes ready

root@imx6solo:~# ip link set dev ethphy0 up
[ 634.114608] sja1105 spi1.0 ethphy0: configuring for phy/mii link mode
[ 634.126617] ------------[ cut here ]------------
[ 634.131463] WARNING: CPU: 0 PID: 330 at drivers/net/phy/phy.c:1157
phy_start+0x58/0xb4
[ 634.139627] called from state RUNNING
[ 634.143310] Modules linked in: bnep tag_sja1105 sja1105 pcs_xpcs
brcmfmac brcmutil imx_sdma coda_vpu v4l2_jpeg imx_vdoa evbug
[ 634.154815] CPU: 0 PID: 330 Comm: ip Not tainted 6.1.58 #1
[ 634.160321] Hardware name: Freescale i.MX6 Quad/DualLite (Device Tree)
[ 634.166866] unwind_backtrace from show_stack+0x10/0x14
[ 634.172125] show_stack from dump_stack_lvl+0x58/0x70
[ 634.177205] dump_stack_lvl from __warn+0x70/0x1a4
[ 634.182023] __warn from warn_slowpath_fmt+0x1bc/0x270
[ 634.187181] warn_slowpath_fmt from phy_start+0x58/0xb4
[ 634.192427] phy_start from dsa_port_enable_rt+0x54/0x9c
[ 634.197765] dsa_port_enable_rt from dsa_slave_open+0xb8/0x17c
[ 634.203624] dsa_slave_open from __dev_open+0xec/0x1ac
[ 634.208788] __dev_open from __dev_change_flags+0x18c/0x224
[ 634.214377] __dev_change_flags from dev_change_flags+0x14/0x44
[ 634.220313] dev_change_flags from do_setlink+0x254/0xe4c
[ 634.225737] do_setlink from rtnl_newlink+0x598/0x80c
[ 634.230811] rtnl_newlink from rtnetlink_rcv_msg+0x168/0x574
[ 634.236485] rtnetlink_rcv_msg from netlink_rcv_skb+0xb8/0x11c
[ 634.242337] netlink_rcv_skb from netlink_unicast+0x1a8/0x2dc
[ 634.248108] netlink_unicast from netlink_sendmsg+0x1d8/0x450
[ 634.253884] netlink_sendmsg from ____sys_sendmsg+0x188/0x258
[ 634.259656] ____sys_sendmsg from ___sys_sendmsg+0x6c/0xa4
[ 634.265164] ___sys_sendmsg from sys_sendmsg+0x50/0x98
[ 634.270324] sys_sendmsg from ret_fast_syscall+0x0/0x1c
[ 634.275569] Exception stack(0xf1071fa8 to 0xf1071ff0)
[ 634.280637] 1fa0: 00000000 00000001 00000003
bef4d710 00000000 00000000
[ 634.288828] 1fc0: 00000000 00000001 b6f415a0 00000128 626ad475
00000000 00531cc0 00531cc0
[ 634.297016] 1fe0: 00000128 bef4d6b0 b6e3f6d3 b6db1ae6
[ 634.302150] irq event stamp: 3327
[ 634.305485] hardirqs last enabled at (3335): [<c0195dbc>]
__up_console_sem+0x50/0x60
[ 634.313404] hardirqs last disabled at (3344): [<c0195da8>]
__up_console_sem+0x3c/0x60
[ 634.321318] softirqs last enabled at (3230): [<c01017d0>]
__do_softirq+0x2c0/0x57c
[ 634.329055] softirqs last disabled at (3187): [<c012f204>]
__irq_exit_rcu+0x138/0x17c
[ 634.336971] ---[ end trace 0000000000000000 ]---
root@imx6solo:~# ip link set dev ethphy1 up
[ 644.613564] sja1105 spi1.0 ethphy1: configuring for phy/mii link mode
root@imx6solo:~# [ 646.727549] sja1105 spi1.0 ethphy1: Link is Up -
100Mbps/Full - flow control off
[ 646.735033] IPv6: ADDRCONF(NETDEV_CHANGE): ethphy1: link becomes ready

Thanks,
Jagan.