Am 21.11.2017 um 02:34 schrieb Andrew Lunn: > Hi Heiner Hi Andrew, > > Do you have access to the data sheet? > Not to more recent ones. I only have two older data sheets for early members of the rtl8169 family.
> I had a quick look through the driver. It would be nice to refactor it > to follow the usual Linux conventions: > > Turn the MDIO read/write functions into an MDIO bus driver. > I thought the same when looking at this driver. It's a nightmare to maintain a driver with almost 9.000 lines of code and numerous "switch mac_id" clauses. Not sure whether still members of this chip family are being developed that we may have to add in future. My first thought was to factor out support for the original 8169 family (mac id <= 6) into a separate driver as first step as it differs significantly from later members of the chip family (e.g. TBI support in addition to MII). > Move the PHY code into drivers/net/phy/realtek.c, and in the process, > replace all the magic numbers with #defines. > Yes, this definitely would be desirable. However I found that the available datasheets for the external PHY's usually only document the registers on page 0. All the magic settings on other pages often are just copied from vendor drivers. Or are there other, more comprehensive versions of the datasheets being available under NDA? > Do you have any interest in doing this? > Would be a nice challenge. I'm willing to look into this if I can get hold of the official datasheets. > Andrew > Heiner
