| Message ID | 20090330165009.18855.43672.stgit@bob.kio (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Headers | show |
Reviewed-by: Kenji Kaneshige <kaneshige.kenji@jp.fujitsu.com> Thanks, Kenji Kaneshige Alex Chiang wrote: > There is no reason to prevent removal of root bus devices. A subsequent > rescan will find them just fine. > > Signed-off-by: Alex Chiang <achiang@hp.com> > --- > > drivers/pci/pci-sysfs.c | 4 ---- > 1 files changed, 0 insertions(+), 4 deletions(-) > > diff --git a/drivers/pci/pci-sysfs.c b/drivers/pci/pci-sysfs.c > index e9a8706..7b2cb27 100644 > --- a/drivers/pci/pci-sysfs.c > +++ b/drivers/pci/pci-sysfs.c > @@ -277,14 +277,10 @@ remove_store(struct device *dev, struct device_attribute *dummy, > { > int ret = 0; > unsigned long val; > - struct pci_dev *pdev = to_pci_dev(dev); > > if (strict_strtoul(buf, 0, &val) < 0) > return -EINVAL; > > - if (pci_is_root_bus(pdev->bus)) > - return -EBUSY; > - > /* An attribute cannot be unregistered by one of its own methods, > * so we have to use this roundabout approach. > */ > > -- > To unsubscribe from this list: send the line "unsubscribe linux-pci" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, 30 Mar 2009 10:50:09 -0600 Alex Chiang <achiang@hp.com> wrote: > There is no reason to prevent removal of root bus devices. A > subsequent rescan will find them just fine. > > Signed-off-by: Alex Chiang <achiang@hp.com> Applied this series, thanks.
diff --git a/drivers/pci/pci-sysfs.c b/drivers/pci/pci-sysfs.c index e9a8706..7b2cb27 100644 --- a/drivers/pci/pci-sysfs.c +++ b/drivers/pci/pci-sysfs.c @@ -277,14 +277,10 @@ remove_store(struct device *dev, struct device_attribute *dummy, { int ret = 0; unsigned long val; - struct pci_dev *pdev = to_pci_dev(dev); if (strict_strtoul(buf, 0, &val) < 0) return -EINVAL; - if (pci_is_root_bus(pdev->bus)) - return -EBUSY; - /* An attribute cannot be unregistered by one of its own methods, * so we have to use this roundabout approach. */
There is no reason to prevent removal of root bus devices. A subsequent rescan will find them just fine. Signed-off-by: Alex Chiang <achiang@hp.com> --- drivers/pci/pci-sysfs.c | 4 ---- 1 files changed, 0 insertions(+), 4 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html