From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1751640AbdJRVbI (ORCPT <rfc822;w@1wt.eu>);
        Wed, 18 Oct 2017 17:31:08 -0400
Received: from mail-qt0-f195.google.com ([209.85.216.195]:48553 "EHLO
        mail-qt0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1751141AbdJRVbF (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Wed, 18 Oct 2017 17:31:05 -0400
X-Google-Smtp-Source: ABhQp+QOjXlc2nIV4CdpFxw9yQ1P58qaRorvW8PdA2tk3UWitdah+kuOnxkEx0qetNOsy1GNd82CUg==
From: Andy Gospodarek <andrew.gospodarek@broadcom.com>
X-Google-Original-From: Andy Gospodarek <gospo@broadcom.com>
Date: Wed, 18 Oct 2017 17:30:58 -0400
To: Scott Branden <scott.branden@broadcom.com>
Cc: Andrew Lunn <andrew@lunn.ch>, "David S. Miller" <davem@davemloft.net>,
        "Allan W. Nielsen" <allan.nielsen@microsemi.com>,
        Raju Lakkaraju <Raju.Lakkaraju@microsemi.com>,
        Florian Fainelli <f.fainelli@gmail.com>,
        BCM Kernel Feedback <bcm-kernel-feedback-list@broadcom.com>,
        linux-kernel@vger.kernel.org, Steve Lin <steven.lin1@broadcom.com>
Subject: Re: [PATCH 0/1] net: ethtool: add SmartNIC reset support
Message-ID: <20171018213058.GD2847@C02RW35GFVH8.dhcp.broadcom.net>
References: <1508342496-5047-1-git-send-email-scott.branden@broadcom.com>
 <20171018162823.GB15371@lunn.ch>
 <b9f6864a-e746-bb0d-ec8f-b85924276266@broadcom.com>
 <20171018191639.GI15371@lunn.ch>
 <e8f5b581-2613-0dda-d9cc-e826a14ee394@broadcom.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <e8f5b581-2613-0dda-d9cc-e826a14ee394@broadcom.com>
User-Agent: Mutt/1.8.0 (2017-02-23)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Wed, Oct 18, 2017 at 12:31:28PM -0700, Scott Branden wrote:
> Hi Andrew,
> 
> 
> On 17-10-18 12:16 PM, Andrew Lunn wrote:
> > > Yes, there is also a management processor.
> > O.K.
> > 
> > Maybe it would be nice to add some more text to the commit message to
> > make this clear. Define what an application processor is, and how it
> > differs from a management processor. But othersize:
> OK -will add more description to differentiate management processor vs.
> application processor(s).
> > 
> > Reviewed-by: Andrew Lunn <andrew@lunn.ch>
> > 
> > I assume you have another kernel patch to actually make use of this?
> > It is normal to post the user of a new API in the same series as the
> > API.
> I actually wanted to get agreement that the bit define could be added to
> ethtool before implementing it in driver.
> If this direction approved we'll implement in driver and submit with this
> patch series.

I just noticed that you did not also post this to netdev@vger.kernel.org.  I
suspect you are more likely to get review and acceptance if that list is cc'd.

I'm not positive that Linus will take networking patches off this list.

> > 
> >      Andrew
> Thanks,
>  Scott