From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH 1/1] net-next/hinic:add shutdown callback Date: Fri, 28 Dec 2018 21:49:05 -0800 (PST) Message-ID: <20181228.214905.2208487974538707696.davem@davemloft.net> References: <20181228002040.1256-1-xuechaojing@huawei.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: linux-kernel@vger.kernel.org, netdev@vger.kernel.org, wulike1@huawei.com, chiqijun@huawei.com, fy.wang@huawei.com, tony.qu@huawei.com, luoshaokai@huawei.com To: xuechaojing@huawei.com Return-path: In-Reply-To: <20181228002040.1256-1-xuechaojing@huawei.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org From: Xue Chaojing Date: Fri, 28 Dec 2018 00:20:40 +0000 > @@ -1106,6 +1106,13 @@ static void hinic_remove(struct pci_dev *pdev) > dev_info(&pdev->dev, "HiNIC driver - removed\n"); > } > > +static void hinic_shutdown(struct pci_dev *pdev) > +{ > + pci_disable_device(pdev); > + > + dev_info(&pdev->dev, "HiNIC driver - shutdown\n"); > +} I do not think these messages are appropriate. I didn't catch them before, but I'm not going to let new ones get added.