[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <BLUPR03MB14754AC9A9793F0AD8B1C945E28D0@BLUPR03MB1475.namprd03.prod.outlook.com>
Date: Tue, 28 Jul 2015 02:23:35 +0000
From: Shaohui Xie <Shaohui.Xie@...escale.com>
To: Florian Fainelli <f.fainelli@...il.com>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"davem@...emloft.net" <davem@...emloft.net>
Subject: RE: [PATCH] phylib: add driver for aquantia phy
> > for this Aquantia PHY, SUPPORTED_10000baseT_Full is a valid define,
> should I set it as below:
> > .features = PHY_GBIT_FEATURES | SUPPORTED_10000baseT_Full,
>
> PHY_GBIT_FEATURES means 10/100/1000 half and full-duplex are supported,
> which are not supported as you indicated above, I would go with adding
> only the supported modes here, this is really important since this is the
> contract between the PHY driver and the Ethernet MAC using it through the
> PHY library.
[S.H] OK. I'll revise the features accordingly.
Thanks.
Shaohui
Powered by blists - more mailing lists