Re: [PATCH v1 2/2] net: emac: fix and unify emac_mdio functions

2017-06-05 Thread Christian Lamparter
On Monday, June 5, 2017 11:43:33 PM CEST Andrew Lunn wrote: > On Mon, Jun 05, 2017 at 10:49:40PM +0200, Christian Lamparter wrote: > > emac_mdio_read_link() was not copying the requested phy settings > > back into the emac driver's own phy api. This has caused a link > > speed mismatch issue for th

Re: [PATCH v1 2/2] net: emac: fix and unify emac_mdio functions

2017-06-05 Thread Andrew Lunn
On Mon, Jun 05, 2017 at 10:49:40PM +0200, Christian Lamparter wrote: > emac_mdio_read_link() was not copying the requested phy settings > back into the emac driver's own phy api. This has caused a link > speed mismatch issue for the AR8035 as the emac driver kept > trying to connect with 10/100MBps

[PATCH v1 2/2] net: emac: fix and unify emac_mdio functions

2017-06-05 Thread Christian Lamparter
emac_mdio_read_link() was not copying the requested phy settings back into the emac driver's own phy api. This has caused a link speed mismatch issue for the AR8035 as the emac driver kept trying to connect with 10/100MBps on a 1GBit/s link. This patch also unifies shared code between emac_setup_a