Skip to content

Remove calls from net drivers to legacy miiphy_register() entry-point

Karl Beldan u-boot 2 versions – New

Get series with b4
b4 am 20160820090845.GA4865@gobelin

Versions newest first · linked by author and title

  1. v2 Latest posting 2/20 of 5 patches 0 repliesNew
  2. v2 posting 1/20 of 5 patches 0 repliesSuperseded

Patches in v2 posting 2/2 No applies recorded yet

None of the 5 patches the cover letter announces are in this archive.

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.

Read the v2 thread

Cover letter [U-Boot] [PATCH v2 0/5] Remove calls from net drivers to legacy miiphy_register() entry-point
Quoted 13 lines · Joe Hershberger

Hi Karl,

Quoted 10 lines · Karl Beldan
Quoted 6 lines · Joe Hershberger

This is an effort to migrate closer to the phylib approach so that a new DM approach can be developed from a single starting point. The more of this legacy code that can be removed, the easier it will be to move forward.

Hi Joe,

Quoted 8 lines
Quoted 4 lines

It was nice to see spatch put to work. However davinci_emac crashes U-Boot now. It is an invasive patch so it might take time to check everything is in order, so unless you already have a follow-up series I can send in a trivial patch just now to fix that for davinci_emac.

That's a bummer. Is it a systemic problem that you expect to affect more drivers or is it something that is likely to only affect davinci?

It is likely to only affect davinci, just a mix up of the internal MDIO accessors return code (not in the changes made by spatch proper).

It would be better in the future in the davinci code to just use the same convention as the mdiodev accessors but I chose a non-invasive quick to check follow-up patch.

Rgds,
Karl