Use 'phy-handle' in FDT to find PHY address

[aryeeteygerald_rogers.com]

Summary

Based on @manu's patch.
Since 'phy_id' is no longer dts as per in r337703, update driver to match device tree.

Test Plan

Tested on a beaglebone black

[andrew]

sys/arm/ti/cpsw/if_cpsw.c

762 ↗

(On Diff #55780)

I don't think this is needed.

[aryeeteygerald_rogers.com]

sys/arm/ti/cpsw/if_cpsw.c

762 ↗

(On Diff #55780)

I don't think this is needed.

Yea, I think fdt_get_phyaddr should already handle it.

779 ↗

(On Diff #55780)

I think I should re-add this check in case a slave isn't found somehow ...

[aryeeteygerald_rogers.com]

• Remove extra phy-handlei check and add 'no children' check

[manu]

sys/arm/ti/cpsw/if_cpsw.c

763 ↗

(On Diff #55783)

This would return ENXIO if the first slave node isn't the one used, I don't know if any boards are like that but if they are they worked before and should after.

[manu]

sys/arm/ti/cpsw/if_cpsw.c

763 ↗

(On Diff #55783)

You might also want to get phy_id if phy-handle don't exists so people with old DTB could still boot.

[manu]

sys/arm/ti/cpsw/if_cpsw.c

763 ↗

(On Diff #55783)

Mhm no the old code only check the first slave@ node that matches the full node name based on the port and this is still the case.
A correct way would be to rely on the 'slaves' property which contain the number of slaves used and always parse every slave until this number is reached instead of this but that might be out of the scope of this review.
So just add a fallback to read phy_id if phy-handle doesn't exists and we're good to go for this one.

[aryeeteygerald_rogers.com]

• Support old DTBs which still use phy_id

[manu]

LGTM.