[PATCH] phylib: add driver for aquantia phy

2015-07-23 Thread shh.xie
From: Shaohui Xie This patch added driver to support Aquantia PHYs AQ1202, AQ2104, AQR105, AQR405, which accessed through clause 45. Signed-off-by: Shaohui Xie --- drivers/net/phy/Kconfig| 5 ++ drivers/net/phy/Makefile | 1 + drivers/net/phy/aquantia.c | 157

Re: [PATCH] phylib: add driver for aquantia phy

2015-07-23 Thread Florian Fainelli
Le 07/23/15 20:46, shh@gmail.com a écrit : > From: Shaohui Xie > > This patch added driver to support Aquantia PHYs AQ1202, AQ2104, AQR105, > AQR405, which accessed through clause 45. Could you prefix your patches with "net: phy: " in the future to be consistent with what is typically used?

RE: [PATCH] phylib: add driver for aquantia phy

2015-07-27 Thread Shaohui Xie
> -Original Message- > From: Florian Fainelli [mailto:f.faine...@gmail.com] > Sent: Friday, July 24, 2015 12:39 PM > To: shh@gmail.com; netdev@vger.kernel.org; da...@davemloft.net > Cc: Xie Shaohui-B21989 > Subject: Re: [PATCH] phylib: add driver for aquantia phy >

Re: [PATCH] phylib: add driver for aquantia phy

2015-07-27 Thread Florian Fainelli
gt; Subject: Re: [PATCH] phylib: add driver for aquantia phy >> >> Le 07/23/15 20:46, shh@gmail.com a écrit : >>> From: Shaohui Xie >>> >>> This patch added driver to support Aquantia PHYs AQ1202, AQ2104, >>> AQR105, AQR405, which accessed through

RE: [PATCH] phylib: add driver for aquantia phy

2015-07-27 Thread Shaohui Xie
> > for this Aquantia PHY, SUPPORTED_1baseT_Full is a valid define, > should I set it as below: > > .features = PHY_GBIT_FEATURES | SUPPORTED_1baseT_Full, > > PHY_GBIT_FEATURES means 10/100/1000 half and full-duplex are supported, > which are not supported as you indicated above, I would