Page MenuHomeFreeBSD

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

Authored by aryeeteygerald_rogers.com on Apr 3 2019, 4:07 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 15, 5:40 AM
Unknown Object (File)
Sun, Nov 10, 6:18 PM
Unknown Object (File)
Sun, Nov 10, 6:02 PM
Unknown Object (File)
Tue, Nov 5, 3:48 AM
Unknown Object (File)
Fri, Nov 1, 12:53 PM
Unknown Object (File)
Tue, Oct 29, 9:35 PM
Unknown Object (File)
Oct 20 2024, 10:34 PM
Unknown Object (File)
Oct 20 2024, 10:34 PM
Subscribers

Details

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

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 23501
Build 22507: arc lint + arc unit

Event Timeline

sys/arm/ti/cpsw/if_cpsw.c
762–770

I don't think this is needed.

sys/arm/ti/cpsw/if_cpsw.c
762–770

I don't think this is needed.

Yea, I think fdt_get_phyaddr should already handle it.

785

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

  • Remove extra phy-handlei check and add 'no children' check
sys/arm/ti/cpsw/if_cpsw.c
772

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.

sys/arm/ti/cpsw/if_cpsw.c
772

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

sys/arm/ti/cpsw/if_cpsw.c
772

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.

  • Support old DTBs which still use phy_id
This revision is now accepted and ready to land.Apr 3 2019, 8:31 PM
This revision was automatically updated to reflect the committed changes.