On Mon, 2018-05-28 at 15:47 +0300, Mika Westerberg wrote: > It is not immediately clear what the two functions actually return so > add kernel-doc comment explaining it a bit better. > Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > Suggested-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx> > Signed-off-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx> > Reviewed-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx> > --- > drivers/pci/probe.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/pci/probe.c b/drivers/pci/probe.c > index 8f384e7ca2c2..7dc0b3c49941 100644 > --- a/drivers/pci/probe.c > +++ b/drivers/pci/probe.c > @@ -999,6 +999,8 @@ static unsigned int > pci_scan_child_bus_extend(struct pci_bus *bus, > * already configured by the BIOS and after we are done with all of > * them, we proceed to assigning numbers to the remaining buses in > * order to avoid overlaps between old and new bus numbers. > + * > + * Return: New subordinate number covering all buses behind this > bridge. > */ > static int pci_scan_bridge_extend(struct pci_bus *bus, struct pci_dev > *dev, > int max, unsigned int > available_buses, > @@ -1226,6 +1228,8 @@ static int pci_scan_bridge_extend(struct pci_bus > *bus, struct pci_dev *dev, > * already configured by the BIOS and after we are done with all of > * them, we proceed to assigning numbers to the remaining buses in > * order to avoid overlaps between old and new bus numbers. > + * > + * Return: New subordinate number covering all buses behind this > bridge. > */ > int pci_scan_bridge(struct pci_bus *bus, struct pci_dev *dev, int > max, int pass) > { -- Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Intel Finland Oy -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html