From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ferruh Yigit <ferruh.yigit@intel.com>
Subject: Re: [PATCH 05/28] bnxt: add support for PF/VF
 communications
Date: Thu, 11 May 2017 18:00:06 +0100
Message-ID: <6a55004d-4787-c6b0-27db-b9da9fb466b5@intel.com>
References: <20170328034903.41482-1-ajit.khaparde@broadcom.com>
 <20170328034903.41482-6-ajit.khaparde@broadcom.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Cc: dev@dpdk.org
To: Ajit Khaparde <ajit.khaparde@broadcom.com>,
 Stephen Hurd <stephen.hurd@broadcom.com>
Return-path: <dev-bounces@dpdk.org>
Received: from mga04.intel.com (mga04.intel.com [192.55.52.120])
 by dpdk.org (Postfix) with ESMTP id B593647CE
 for <dev@dpdk.org>; Thu, 11 May 2017 19:00:09 +0200 (CEST)
In-Reply-To: <20170328034903.41482-6-ajit.khaparde@broadcom.com>
Content-Language: en-US
List-Id: DPDK patches and discussions <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 3/28/2017 4:48 AM, Ajit Khaparde wrote:
> Set up interrupts and default completion ring during device init to
> allow VF communications.
> 
> Request most VF HWRM commands be forwarded to the PF driver.  In the
> future, this should be configurable.
> 
> Fix reporting of active VFs.
> 
> Add a log message when a completion is ignored on the default completion
> ring since this will indicate support that needs to be added.
> 
> Do not initialize the default completion ring in bnxt_alloc_hwrm_rings(),
> but move it out to bnxt_dev_init().
> 
> Create a structure bnxt_child_vf_info for PF to cache VF specific info.
> 
> Signed-off-by: Stephen Hurd <stephen.hurd@broadcom.com>
> Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>

Hi Ajit, Stephen,

This patch doesn't cleanly apply on top of latest next-net, can you
please rebase the patchset?

Also as reminder, there was previously reported  32bit and shared
library builds.

And there are some checkpatch warnings in patch 25 and 28.

Thanks,
ferruh