On Fri, Jan 3, 2025 at 3:38 PM Bjorn Helgaas <helgaas@xxxxxxxxxx> wrote: > > On Fri, Jan 03, 2025 at 03:31:28PM -0600, Bjorn Helgaas wrote: > > From: Bjorn Helgaas <bhelgaas@xxxxxxxxxx> > > > > of_pci_parse_bus_range() looks for a DT "bus-range" property. If none > > exists, devm_of_pci_get_host_bridge_resources() defaults to the [bus 00-ff] > > range supplied by its caller, pci_parse_request_of_pci_ranges(). > > > > devm_of_pci_get_host_bridge_resources() is static and has no other callers, > > so there's no reason to complicate its interface by passing the default bus > > range. > > > > Drop the busno and bus_max parameters and use 0x0 and 0xff directly in > > devm_of_pci_get_host_bridge_resources(). > > Since we default this if the DT lacks "bus-range", is there any point > in repeating "bus-range = <0x00 0xff>;" in all the host bridge > descriptions? No. And I tell people to drop it if that's what they have. Rob
