net: dsa: mt7530: skip locking if MDIO bus isn't present
authorDaniel Golle <daniel@makrotopia.org>
Mon, 3 Apr 2023 01:19:28 +0000 (02:19 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 3 Apr 2023 09:13:00 +0000 (10:13 +0100)
commit54d4147a121cec5004a673a58572da346e4458f8
treebed2b26b76b3cd7db066641a4c7ec518bb3f72e1
parentcb675afcddbbeb2bfa6596e3bc236bc026cd425f
net: dsa: mt7530: skip locking if MDIO bus isn't present

As MT7530 and MT7531 internally use 32-bit wide registers, each access
to any register of the switch requires several operations on the MDIO
bus. Hence if there is congruent access, e.g. due to PCS or PHY
polling, this can mess up and interfere with another ongoing register
access sequence.

However, the MDIO bus mutex is only relevant for MDIO-connected
switches. Prepare switches which have there registers directly mapped
into the SoCs register space via MMIO which do not require such
locking. There we can simply use regmap's default locking mechanism.

Hence guard mutex operations to only be performed in case of MDIO
connected switches.

Signed-off-by: Daniel Golle <daniel@makrotopia.org>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/dsa/mt7530.c