On Fri, Dec 08, 2017 at 02:07:54PM +0530, Manikanta Maddireddy wrote: > > > On 06-Dec-17 1:34 AM, Bjorn Helgaas wrote: > > On Fri, Dec 01, 2017 at 02:50:02PM +0530, Manikanta Maddireddy wrote: > >> PCI subsystem pass pci_bus pointer to pci_ops callback functions, Tegra > > > > s/PCI subsystem pass/The PCI subsystem passes a/ > > s/, Tegra/. The Tegra/ > > > >> host driver use pci_find_host_bridge() to get pci_host_bridge from pci_bus. > > > > s/use/uses/ > > > >> Export pci_find_host_bridge() to allow Tegra PCIe driver to be compiled as > >> loadable kernel module. > > > > s/allow Tegra/allow the Tegra/ > > s/loadable/a loadable/ > > > > But why do you need pci_find_host_bridge() at all? Most drivers use > > the pci_bus.sysdata pointer for this purpose, e.g., see db271747a04d > > ("PCI: mediatek: Use bus->sysdata to get host private data") > > I will send a patch to use bus->sysdata instead of pci_find_host_bridge(). That would be excellent! Thank you! Bjorn -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html