]> git.proxmox.com Git - mirror_ubuntu-hirsute-kernel.git/commit
net: dsa: bcm_sf2: do not use indirect reads and writes for 7445E0
authorFlorian Fainelli <f.fainelli@gmail.com>
Wed, 15 Jul 2015 23:09:32 +0000 (16:09 -0700)
committerDavid S. Miller <davem@davemloft.net>
Mon, 20 Jul 2015 23:47:30 +0000 (16:47 -0700)
commitb8c6cd1d316f3b01ae578d8e29179f6396c0eaa2
tree72718fbde0231c635ee9c9b9119244e2cccac6dd
parent7d5cd2ce5292b45e555de776cb9e72975a07460d
net: dsa: bcm_sf2: do not use indirect reads and writes for 7445E0

7445E0 contains an ECO which disconnected the internal SF2 pseudo-PHY which was
known to conflict with the external pseudo-PHY of BCM53125 switches. This
motivated the need to utilize the internal SF2 MDIO controller via indirect
register reads/writes to control external Broadcom switches due to this address
conflict (both responded at address 30d).

For 7445E0, the internal pseudo-PHY of the SF2 switch got disconnected, and as
a consequence this prevents the internal SF2 MDIO bus controller from reading
data (reads back everything as 0) since the MDI line is tied low.

Fix this by making the indirect register reads and writes conditional to
7445D0, on 7445E0 we can utilize the SWITCH_MDIO controller (backed by
mdio-unimac and not the DSA created slave MII bus).

We utilize of_machine_is_compatible() here since this is the only way for use
to differentiate between these two chips in a way that does not violate layers
or becomes (too) vendor-specific.

Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/dsa/bcm_sf2.c