From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ferruh Yigit <ferruh.yigit@intel.com>
Subject: Re: [PATCH 02/29] ixgbe/base: add sgmii link for X550
Date: Mon, 13 Jun 2016 17:27:37 +0100
Message-ID: <575EDEF9.6070803@intel.com>
References: <1462514861-29419-1-git-send-email-beilei.xing@intel.com>
 <1462514861-29419-3-git-send-email-beilei.xing@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: quoted-printable
Cc: dev@dpdk.org
To: Beilei Xing <beilei.xing@intel.com>, helin.zhang@intel.com
Return-path: <dev-bounces@dpdk.org>
Received: from mga14.intel.com (mga14.intel.com [192.55.52.115])
 by dpdk.org (Postfix) with ESMTP id 81CCD5958
 for <dev@dpdk.org>; Mon, 13 Jun 2016 18:27:40 +0200 (CEST)
In-Reply-To: <1462514861-29419-3-git-send-email-beilei.xing@intel.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 5/6/2016 7:07 AM, Beilei Xing wrote:
> It adds sgmii link for X550.
>=20
> Signed-off-by: Beilei Xing <beilei.xing@intel.com>
> ---
>  drivers/net/ixgbe/base/ixgbe_type.h |   9 +++
>  drivers/net/ixgbe/base/ixgbe_x550.c | 131 ++++++++++++++++++++++++++++=
+++++---
>  2 files changed, 129 insertions(+), 11 deletions(-)
>=20
...

>  	}
> @@ -512,8 +546,8 @@ s32 ixgbe_init_ops_X550EM(struct ixgbe_hw *hw)
>  		link->addr =3D IXGBE_CS4227;
>  	}
>  	if (hw->mac.type =3D=3D ixgbe_mac_X550EM_a) {
> -		mac->ops.read_iosf_sb_reg =3D ixgbe_read_iosf_sb_reg_x550a;
> -		mac->ops.write_iosf_sb_reg =3D ixgbe_write_iosf_sb_reg_x550a;
> +		mac->ops.read_iosf_sb_reg =3D ixgbe_read_iosf_sb_reg_x550;
> +		mac->ops.write_iosf_sb_reg =3D ixgbe_write_iosf_sb_reg_x550;
Is it possible that these lines related to the patch 7:
[PATCH 07/29] ixgbe/base: change access method

It looks like first changes ops from ..x550a to ...x550 here and in
patch 7, using ops function instead fixed one.

Also if there is more details about ..x550a to ...x550 change it can be
good to add patch 7 comment.

>  		mac->ops.acquire_swfw_sync =3D ixgbe_acquire_swfw_sync_X550a;
>  		mac->ops.release_swfw_sync =3D ixgbe_release_swfw_sync_X550a;
>  	}
...