On 3/6/23 10:10 AM, Niklas Schnelle wrote: > As the name suggests zpci_bus_scan_device() is used to scan a specific > device and thus pci_bus_add_device() for that device is sufficient. > Furthermore move this call inside the rescan/remove locking. > > Signed-off-by: Niklas Schnelle <schnelle@xxxxxxxxxxxxx> I think a suggested by tag from Bjorn is appropriate here Otherwise: Reviewed-by: Matthew Rosato <mjrosato@xxxxxxxxxxxxx> > --- > arch/s390/pci/pci_bus.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/arch/s390/pci/pci_bus.c b/arch/s390/pci/pci_bus.c > index a99926af2b69..ae46c280b35f 100644 > --- a/arch/s390/pci/pci_bus.c > +++ b/arch/s390/pci/pci_bus.c > @@ -85,9 +85,8 @@ int zpci_bus_scan_device(struct zpci_dev *zdev) > if (!pdev) > return -ENODEV; > > - pci_bus_add_device(pdev); > pci_lock_rescan_remove(); > - pci_bus_add_devices(zdev->zbus->bus); > + pci_bus_add_device(pdev); > pci_unlock_rescan_remove(); > > return 0;
