From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <thomas.monjalon@6wind.com>
Received: from mail-wm0-f45.google.com (mail-wm0-f45.google.com [74.125.82.45])
 by dpdk.org (Postfix) with ESMTP id BFB021E33
 for <dev@dpdk.org>; Mon,  7 Nov 2016 20:48:50 +0100 (CET)
Received: by mail-wm0-f45.google.com with SMTP id a197so204138934wmd.0
 for <dev@dpdk.org>; Mon, 07 Nov 2016 11:48:50 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=6wind-com.20150623.gappssmtp.com; s=20150623;
 h=from:to:cc:subject:date:message-id:user-agent:in-reply-to
 :references:mime-version:content-transfer-encoding;
 bh=AsYPrn7hxpJb4pUVjX8DxeYdhB9R8qmQzAvPN31lMJ4=;
 b=hjiryFMTroyFFqlJ8j7i74ywJw1rr+z13nDhcO8UmpldKJReEsSVbKj+h/q9IFwxP6
 Q4FDWWKtex9qwfcw+jGmjM4t1p4aUU3a7iyAAsBvcg6PJFks1+dS+ts824QmNP8+fJ9w
 JUjaipwQRLBbNg/PbWy5i4qQ3KTmE6Atvpz8/d9yocyxDbRiuuQM8+eOFY5pVKg3Xa72
 nOUtfGl6k0pxvaL1XBUpyLrSSV9Qk/sSqOx/YQFJmRomwcFH1E2HIsIgxOgRBceIgMUz
 q8fh4UTb0uMdoUxd581mZUkSBdIez3F4cf7wrD5IRhLCxls6/SHyMXNnE4Y5rnMLnRAF
 hd6Q==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=1e100.net; s=20130820;
 h=x-gm-message-state:from:to:cc:subject:date:message-id:user-agent
 :in-reply-to:references:mime-version:content-transfer-encoding;
 bh=AsYPrn7hxpJb4pUVjX8DxeYdhB9R8qmQzAvPN31lMJ4=;
 b=hW74n092MkH0aMW10M08gBHTqKlDmUyBpQEXVHQ5waveisbbAePk+qJV978yDbA/8M
 d+hDl7N1S53dIupjuRx47RAOcDoeoi18x/GQOdI9YYkDhN9IYjB9TagrjVgvstmXp2TF
 Tu+SczRTncW/PEF0JejocI9C3T9oeNAJ5N2xdf4Ie8P7lhTNVkfIJW9LoybPpIBhFtm/
 9sr7cA1G1FXUuIvOv+AHN8ya9494G+pF4zEYTE/YhbCG0SEdkCWJjRSZIl+hNxyAuge4
 jjarvjmxg4JwKrxaB2naBlTZjH7Oj3wdXI0ejWuTnUTvJFnF+pg5GPWKUkJT6BuB5Vuq
 qP9w==
X-Gm-Message-State: ABUngvfkPMNGDDnNZyO09jgOf0vZOOapJpfjk/W4uEqHPrhxrGoAl0T0X20IA9Jq7FMDMRFU
X-Received: by 10.194.101.97 with SMTP id ff1mr8635344wjb.67.1478548130269;
 Mon, 07 Nov 2016 11:48:50 -0800 (PST)
Received: from xps13.localnet (184.203.134.77.rev.sfr.net. [77.134.203.184])
 by smtp.gmail.com with ESMTPSA id s204sm15379648wmd.1.2016.11.07.11.48.49
 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);
 Mon, 07 Nov 2016 11:48:49 -0800 (PST)
From: Thomas Monjalon <thomas.monjalon@6wind.com>
To: Rasesh Mody <Rasesh.Mody@cavium.com>,
 Harish Patil <harish.patil@qlogic.com>
Cc: bruce.richardson@intel.com, dev@dpdk.org, Dept-EngDPDKDev@qlogic.com
Date: Mon, 07 Nov 2016 20:48:48 +0100
Message-ID: <1656706.VhxROYD8UU@xps13>
User-Agent: KMail/4.14.10 (Linux/4.5.4-1-ARCH; KDE/4.14.11; x86_64; ; )
In-Reply-To: <1477938901-24999-2-git-send-email-Rasesh.Mody@cavium.com>
References: <1477721677-25668-1-git-send-email-Rasesh.Mody@cavium.com>
 <1477938901-24999-1-git-send-email-Rasesh.Mody@cavium.com>
 <1477938901-24999-2-git-send-email-Rasesh.Mody@cavium.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
Subject: Re: [dpdk-dev] [PATCH v3] net/qede: fix advertising link speed
	capability
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: patches and discussions about DPDK <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Mon, 07 Nov 2016 19:48:50 -0000

2016-10-31 11:35, Rasesh Mody:
> From: Harish Patil <harish.patil@qlogic.com>
> 
> Fix to advertise device's link speed capability based on NVM
> port configuration instead of returning driver supported speeds.
> 
> Fixes: 95e67b479506 ("net/qede: add 100G link speed capability")
> 
> Signed-off-by: Harish Patil <harish.patil@qlogic.com>
[...]
> +	/* Fill up the native advertised speed */
> +	switch (params.speed.advertised_speeds) {
> +	case NVM_CFG1_PORT_DRV_SPEED_CAPABILITY_MASK_10G:
> +		adv_speed = 10000;
> +	break;
> +	case NVM_CFG1_PORT_DRV_SPEED_CAPABILITY_MASK_25G:
> +		adv_speed = 25000;
> +	break;
> +	case NVM_CFG1_PORT_DRV_SPEED_CAPABILITY_MASK_40G:
> +		adv_speed = 40000;
> +	break;
> +	case NVM_CFG1_PORT_DRV_SPEED_CAPABILITY_MASK_50G:
> +		adv_speed = 50000;
> +	break;
> +	case NVM_CFG1_PORT_DRV_SPEED_CAPABILITY_MASK_BB_100G:
> +		adv_speed = 100000;
> +	break;
> +	default:
> +		DP_NOTICE(hwfn, false, "Unknown speed\n");
> +		adv_speed = 0;
> +	}
> +	if_link->adv_speed = adv_speed;

The qede devices support only one speed?
I guess it is wrong but it is a step in right direction so it
will be enough for 16.11.

Applied