Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <20190419084547.26220-1-crag0715@gmail.com> <20190419193747.GA29188@kroah.com>
In-Reply-To: <20190419193747.GA29188@kroah.com>
From:   Crag Wang <crag0715@gmail.com>
Date:   Mon, 22 Apr 2019 09:55:51 +0800
Message-ID: <CAP-8N0i-47x0ZvuSed7aWm_KcjSF1wn9kMYTKHL6zyHO8pqzAQ@mail.gmail.com>
Subject: Re: [PATCH 1/1] r8152: sync sa_family with the media type of network device
To:     Greg KH <gregkh@linuxfoundation.org>
Cc:     linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org,
        hayeswang@realtek.com,
        Mario Limonciello <mario_limonciello@dell.com>,
        "Crag.Wang" <crag.wang@dell.com>,
        "David S. Miller" <davem@davemloft.net>,
        Simon Glass <sjg@chromium.org>,
        Sean Paul <seanpaul@chromium.org>,
        Grant Grundler <grundler@chromium.org>,
        David Chen <david.chen7@dell.com>,
        Florian Fainelli <f.fainelli@gmail.com>,
        "Gustavo A. R. Silva" <gustavo@embeddedor.com>,
        zhong jiang <zhongjiang@huawei.com>,
        Charles Hyde <charles.hyde@dell.corp-partner.google.com>,
        Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Greg KH <gregkh@linuxfoundation.org> =E6=96=BC 2019=E5=B9=B44=E6=9C=8820=E6=
=97=A5 =E9=80=B1=E5=85=AD =E4=B8=8A=E5=8D=883:37=E5=AF=AB=E9=81=93=EF=BC=9A
>
> On Fri, Apr 19, 2019 at 04:45:46PM +0800, crag0715@gmail.com wrote:
> > From: "Crag.Wang" <crag.wang@dell.com>
> >
> > Without this patch the socket address family sporadically gets wrong
> > value ends up the dev_set_mac_address() fails to set the desired MAC
> > address.
> >
> > Change-Id: Ib48a303730bee81667a2859be32e48cc8a66f277
>
> No need for this line here, I don't think you ran checkpatch.pl on your
> change before sending it out :(

I added param to skip the verification of gerrit change-id, will
remove this line in v2 and upload.

Crag