On Fri, Feb 15, 2019 at 09:57:49PM +0100, Heiner Kallweit wrote: > Similar to the existing helpers for the Clause 22 registers add helpers > to deal with converting Clause 45 advertisement registers to / from > link mode bitmaps. > > Note that these helpers are defined in linux/mdio.h, not like the > Clause 22 helpers in linux/mii.h. Reason is that the Clause 45 register > constants are defined in uapi/linux/mdio.h. And uapi/linux/mdio.h > includes linux/mii.h before defining the C45 register constants.
Hi Heiner You add three helpers, but the followup patch only uses one of them. Maybe you should wait until you have real uses of the other two? Or just add the one helper. Andrew