From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Andrew Harvey (agh)" <agh@cisco.com>
Subject: Re: [PATCH v7 2/4] ixgbe: add ops to support ethtool ops
Date: Fri, 26 Jun 2015 06:26:18 +0000
Message-ID: <D1B23DE0.35586%agh@cisco.com>
References: <1432946276-9424-1-git-send-email-liang-min.wang@intel.com>
 <1434579735-15496-1-git-send-email-liang-min.wang@intel.com>
 <1434579735-15496-3-git-send-email-liang-min.wang@intel.com>
 <20150625094546.77e95f26@uryu.home.lan>
Mime-Version: 1.0
Content-Type: text/plain; charset="Windows-1252"
Content-Transfer-Encoding: quoted-printable
Cc: "dev@dpdk.org" <dev@dpdk.org>
To: Stephen Hemminger <stephen@networkplumber.org>, Liang-Min Larry Wang
 <liang-min.wang@intel.com>
Return-path: <dev-bounces@dpdk.org>
Received: from rcdn-iport-3.cisco.com (rcdn-iport-3.cisco.com [173.37.86.74])
 by dpdk.org (Postfix) with ESMTP id 11ACDC7C8
 for <dev@dpdk.org>; Fri, 26 Jun 2015 08:26:20 +0200 (CEST)
In-Reply-To: <20150625094546.77e95f26@uryu.home.lan>
Content-Language: en-US
Content-ID: <0B8781AC5D36B24BBEF09E01E9FCFCBA@emea.cisco.com>
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>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

On 6/25/15, 6:45 AM, "dev on behalf of Stephen Hemminger"
<dev-bounces@dpdk.org on behalf of stephen@networkplumber.org> wrote:

>On Wed, 17 Jun 2015 18:22:13 -0400
>Liang-Min Larry Wang <liang-min.wang@intel.com> wrote:
>
>> +
>> +static reg_info ixgbe_regs_general[] =3D {
>> +	{IXGBE_CTRL, 1, 1, "IXGBE_CTRL"},
>> +	{IXGBE_STATUS, 1, 1, "IXGBE_STATUS"},
>> +	{IXGBE_CTRL_EXT, 1, 1, "IXGBE_CTRL_EXT"},
>> +	{IXGBE_ESDP, 1, 1, "IXGBE_ESDP"},
>> +	{IXGBE_EODSDP, 1, 1, "IXGBE_EODSDP"},
>> +	{IXGBE_LEDCTL, 1, 1, "IXGBE_LEDCTL"},
>> +	{IXGBE_FRTIMER, 1, 1, "IXGBE_FRTIMER"},
>> +	{IXGBE_TCPTIMER, 1, 1, "IXGBE_TCPTIMER"},
>> +	{0, 0, 0, ""}
>> +};
>> +
>> +static reg_info ixgbevf_regs_general[] =3D {
>> +	{IXGBE_CTRL, 1, 1, "IXGBE_CTRL"},
>> +	{IXGBE_STATUS, 1, 1, "IXGBE_STATUS"},
>> +	{IXGBE_VFLINKS, 1, 1, "IXGBE_VFLINKS"},
>> +	{IXGBE_FRTIMER, 1, 1, "IXGBE_FRTIMER"},
>> +	{IXGBE_VFMAILBOX, 1, 1, "IXGBE_VFMAILBOX"},
>> +	{IXGBE_VFMBMEM, 16, 4, "IXGBE_VFMBMEM"},
>> +	{IXGBE_VFRXMEMWRAP, 1, 1, "IXGBE_VFRXMEMWRAP"},
>> +	{0, 0, 0, ""}
>> +};
>> +
>
>All these tables should be const
>and API may need to change.
Good point, I have sent a patch to Larry for inclusion in the next version.

Thanks =8B Andy