From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <dev-bounces@dpdk.org>
Received: from dpdk.org (dpdk.org [92.243.14.124])
	by dpdk.space (Postfix) with ESMTP id 31407A068F
	for <public@inbox.dpdk.org>; Mon,  1 Apr 2019 08:32:36 +0200 (CEST)
Received: from [92.243.14.124] (localhost [127.0.0.1])
	by dpdk.org (Postfix) with ESMTP id E9CF634F3;
	Mon,  1 Apr 2019 08:32:34 +0200 (CEST)
Received: from mga01.intel.com (mga01.intel.com [192.55.52.88])
 by dpdk.org (Postfix) with ESMTP id BE96A343C;
 Mon,  1 Apr 2019 08:32:33 +0200 (CEST)
X-Amp-Result: SKIPPED(no attachment in message)
X-Amp-File-Uploaded: False
Received: from orsmga007.jf.intel.com ([10.7.209.58])
 by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384;
 31 Mar 2019 23:32:32 -0700
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.60,295,1549958400"; d="scan'208";a="127537225"
Received: from kmsmsx156.gar.corp.intel.com ([172.21.138.133])
 by orsmga007.jf.intel.com with ESMTP; 31 Mar 2019 23:32:31 -0700
Received: from pgsmsx103.gar.corp.intel.com ([169.254.2.210]) by
 KMSMSX156.gar.corp.intel.com ([169.254.1.198]) with mapi id 14.03.0415.000;
 Mon, 1 Apr 2019 14:32:30 +0800
From: "Zhao1, Wei" <wei.zhao1@intel.com>
To: "Stillwell Jr, Paul M" <paul.m.stillwell.jr@intel.com>, "dev@dpdk.org"
 <dev@dpdk.org>
CC: "Lu, Wenzhuo" <wenzhuo.lu@intel.com>, "stable@dpdk.org" <stable@dpdk.org>, 
 "Zhang, Qi Z" <qi.z.zhang@intel.com>
Thread-Topic: [dpdk-dev] [PATCH v2] net/ixgbe: enable 10Mb/s link setup for
 x553
Thread-Index: AQHU5guKgX5Bz2WlrUuqK++iCMX7KKYibWGAgARvw5A=
Date: Mon, 1 Apr 2019 06:32:29 +0000
Message-ID:
 <A2573D2ACFCADC41BB3BE09C6DE313CA07ECD7E3@PGSMSX103.gar.corp.intel.com>
References: <1553844685-52291-1-git-send-email-wei.zhao1@intel.com>
 <1553847345-54013-1-git-send-email-wei.zhao1@intel.com>
 <F8A4ECA1C1D86B4081DD6BF503D1F436C746BA94@fmsmsx120.amr.corp.intel.com>
In-Reply-To: <F8A4ECA1C1D86B4081DD6BF503D1F436C746BA94@fmsmsx120.amr.corp.intel.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
dlp-product: dlpe-windows
dlp-version: 11.0.400.15
dlp-reaction: no-action
x-originating-ip: [172.30.20.206]
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Subject: Re: [dpdk-dev] [PATCH v2] net/ixgbe: enable 10Mb/s link setup for
 x553
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>
Message-ID: <20190401063229.7mw3IL34XuqCEuKVlcmJWE8FkyNkAWPd5IjHjT1Upos@z>



> -----Original Message-----
> From: Stillwell Jr, Paul M
> Sent: Saturday, March 30, 2019 2:47 AM
> To: Zhao1, Wei <wei.zhao1@intel.com>; dev@dpdk.org
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; stable@dpdk.org; Zhang, Qi Z
> <qi.z.zhang@intel.com>; Zhao1, Wei <wei.zhao1@intel.com>
> Subject: RE: [dpdk-dev] [PATCH v2] net/ixgbe: enable 10Mb/s link setup fo=
r
> x553
>=20
>=20
> > -----Original Message-----
> > From: dev <dev-bounces@dpdk.org> On Behalf Of Wei Zhao
> > Sent: Friday, March 29, 2019 1:16 AM
> > To: dev@dpdk.org
> > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; stable@dpdk.org; Zhang, Qi Z
> > <qi.z.zhang@intel.com>; Zhao1, Wei <wei.zhao1@intel.com>
> > Subject: [dpdk-dev] [PATCH v2] net/ixgbe: enable 10Mb/s link setup for
> > x553
> >
> > There is need to eanble 10Mb/s link foixgbe NIC of x553.
>=20
> Couple of typos, how about:
>=20
> Enable 10Mb/s link for ixgbe x553.


Good,  update in v2

>=20
> > This new device has own device id of 0x15E4 and 0x15E5, so ixgbe PMD
> > driver need to special check when setup link for these two types of dev=
ice.
> >
> > Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
> >
> > ---
> >
> > v2:
> > delete test code and change permit link speed
> > ---
> >  drivers/net/ixgbe/ixgbe_ethdev.c | 12 +++++++++++-
> >  1 file changed, 11 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c
> > b/drivers/net/ixgbe/ixgbe_ethdev.c
> > index 97e1021..0aef577 100644
> > --- a/drivers/net/ixgbe/ixgbe_ethdev.c
> > +++ b/drivers/net/ixgbe/ixgbe_ethdev.c
> > @@ -2701,6 +2701,10 @@ static int eth_ixgbevf_pci_remove(struct
> > rte_pci_device *pci_dev)
> >  		allowed_speeds =3D ETH_LINK_SPEED_100M |
> ETH_LINK_SPEED_1G |
> >  			ETH_LINK_SPEED_2_5G |  ETH_LINK_SPEED_5G |
> >  			ETH_LINK_SPEED_10G;
> > +		if (hw->device_id =3D=3D IXGBE_DEV_ID_X550EM_A_1G_T ||
> > +				hw->device_id =3D=3D
> > IXGBE_DEV_ID_X550EM_A_1G_T_L)
> > +			allowed_speeds =3D ETH_LINK_SPEED_10M |
> > +				ETH_LINK_SPEED_100M |
> > ETH_LINK_SPEED_1G;
> >  		break;
> >  	default:
> >  		allowed_speeds =3D ETH_LINK_SPEED_100M |
> ETH_LINK_SPEED_1G | @@
> > -2742,6 +2746,8 @@ static int eth_ixgbevf_pci_remove(struct
> > rte_pci_device *pci_dev)
> >  			speed |=3D IXGBE_LINK_SPEED_1GB_FULL;
> >  		if (*link_speeds & ETH_LINK_SPEED_100M)
> >  			speed |=3D IXGBE_LINK_SPEED_100_FULL;
> > +		if (*link_speeds & ETH_LINK_SPEED_10M)
> > +			speed |=3D IXGBE_LINK_SPEED_10_FULL;
> >  	}
> >
> >  	err =3D ixgbe_setup_link(hw, speed, link_up); @@ -4061,8 +4067,12
> @@
> > static int ixgbevf_dev_xstats_get_names(__rte_unused struct
> > rte_eth_dev *dev,
> >  	switch (link_speed) {
> >  	default:
> >  	case IXGBE_LINK_SPEED_UNKNOWN:
> > +		if (hw->device_id =3D=3D IXGBE_DEV_ID_X550EM_A_1G_T ||
> > +			hw->device_id =3D=3D
> > IXGBE_DEV_ID_X550EM_A_1G_T_L)
> > +			link.link_speed =3D ETH_SPEED_NUM_10M;
> > +		else
> > +			link.link_speed =3D ETH_SPEED_NUM_100M;
> >  		link.link_duplex =3D ETH_LINK_FULL_DUPLEX;
> > -		link.link_speed =3D ETH_SPEED_NUM_100M;
> >  		break;
> >
> >  	case IXGBE_LINK_SPEED_100_FULL:
> > --
> > 1.8.3.1