Re: [PATCH v4 20/30] PCI/sparc: Use pci_scan_root_bridge() for simplicity

From: Yijing Wang
Date: Tue Mar 03 2015 - 03:50:08 EST


>> - pci_of_scan_bus(pbm, node, bus);
>> - pci_bus_add_devices(bus);
>> - pci_bus_register_of_sysfs(bus);
>> + pci_bus_add_devices(host->bus);
>> + pci_bus_register_of_sysfs(host->bus);
>>
>> - pci_claim_bus_resources(bus);
>> + pci_claim_bus_resources(host->bus);
>
> Isn't this a case where pci_claim_bus_resources() should be done before
> pci_bus_add_devices()? You didn't introduce this problem, but since you're
> fixing similar issues, seems like we should fix this one, too. In a patch
> of its own, of course.

OK, will fix in a another patch.

>
>>
>> - return bus;
>> + return host->bus;
>> }
>>
>> void pcibios_fixup_bus(struct pci_bus *pbus)
>> --
>> 1.7.1
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>> the body of a message to majordomo@xxxxxxxxxxxxxxx
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>> Please read the FAQ at http://www.tux.org/lkml/
>
> .
>


--
Thanks!
Yijing

--
To unsubscribe from this list: send the line "unsubscribe linux-alpha" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html