From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from v094114.home.net.pl ([79.96.170.134]:63004 "HELO v094114.home.net.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1756697Ab3K0A4r (ORCPT ); Tue, 26 Nov 2013 19:56:47 -0500 From: "Rafael J. Wysocki" To: Yinghai Lu Cc: Bjorn Helgaas , "Rafael J. Wysocki" , Gu Zheng , Guo Chao , linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 01/10] PCI: Use device_release_driver in pci_stop_root_bus Date: Wed, 27 Nov 2013 02:09:35 +0100 Message-ID: <1669330.9A2ve7gAuo@vostro.rjw.lan> In-Reply-To: <1385429290-25397-2-git-send-email-yinghai@kernel.org> References: <1385429290-25397-1-git-send-email-yinghai@kernel.org> <1385429290-25397-2-git-send-email-yinghai@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Sender: linux-pci-owner@vger.kernel.org List-ID: On Monday, November 25, 2013 05:28:01 PM Yinghai Lu wrote: > To be consistent with change in > | PCI: Move device_del() from pci_stop_dev() to pci_destroy_dev() > > Use device_release_driver for root bus/hostbridge. > > Also use device_unregister() in pci_remove_root_bus() instead of > device_del/put_device, that will be corresponding device_register() > for pci_create_root_bus for hostbridge. > > Signed-off-by: Yinghai Lu Acked-by: Rafael J. Wysocki > --- > drivers/pci/remove.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/pci/remove.c b/drivers/pci/remove.c > index cc9337a..692f4c3 100644 > --- a/drivers/pci/remove.c > +++ b/drivers/pci/remove.c > @@ -128,7 +128,7 @@ void pci_stop_root_bus(struct pci_bus *bus) > pci_stop_bus_device(child); > > /* stop the host bridge */ > - device_del(&host_bridge->dev); > + device_release_driver(&host_bridge->dev); > } > > void pci_remove_root_bus(struct pci_bus *bus) > @@ -147,5 +147,5 @@ void pci_remove_root_bus(struct pci_bus *bus) > host_bridge->bus = NULL; > > /* remove the host bridge */ > - put_device(&host_bridge->dev); > + device_unregister(&host_bridge->dev); > } > -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center.