generic: ar8216: improve ar8xxx_is_possible check
authorChristian Lamparter <chunkeey@googlemail.com>
Wed, 5 Oct 2016 13:19:34 +0000 (15:19 +0200)
committerJohn Crispin <john@phrozen.org>
Sat, 15 Oct 2016 09:36:50 +0000 (11:36 +0200)
commite9455c561d4af68840421e3d442e4ad5ebfbedc6
tree96e5867729c8c23e6f3f6983c68dc5d22fcfa3e5
parentbcfb535730fde6f1543f7140e818992ea13f4d30
generic: ar8216: improve ar8xxx_is_possible check

The commit "generic: ar8216: add sanity check to ar8216_probe"
(774da6c7a40320a320b28d71291c0e61fcf7bc8a) stated that PHY IDs
should be checked at address 0-4. However, the PHY 4 was
never check by the loop. This patch extends the check to be
similar to the Atheors SDK. It tries all 4 ports and skips
unconnected PHYs if necessary. If it cannot find any familiar
PHYs, it will prevent the phy driver from initializing.

This patch is necessary for the C-60. It doesn't have a
PHY at port 3, so this caused the check in ar8xxx_is_possible
to fail. As a result, the ethernet ports on the C-60 didn't
work.

Signed-off-by: Christian Lamparter <chunkeey@gmail.com>
target/linux/generic/files/drivers/net/phy/ar8216.c