* [PATCH v8 24/30] PCI/tegra: Remove redundant tegra_pcie_scan_bus() [not found] <1427168064-8657-1-git-send-email-wangyijing@huawei.com> @ 2015-03-24 3:34 ` Yijing Wang 2015-03-24 12:31 ` Thierry Reding 0 siblings, 1 reply; 3+ messages in thread From: Yijing Wang @ 2015-03-24 3:34 UTC (permalink / raw) To: Bjorn Helgaas Cc: Jiang Liu, linux-pci, Yinghai Lu, linux-kernel, Marc Zyngier, linux-arm-kernel, Russell King, x86, Thomas Gleixner, Benjamin Herrenschmidt, Rusty Russell, Tony Luck, linux-ia64, David S. Miller, Guan Xuetao, linux-alpha, linux-m68k, Liviu Dudau, Arnd Bergmann, Geert Uytterhoeven, Yijing Wang, Thierry Reding, linux-tegra Now pci_scan_root_bus() is almost similar to pci_create_root_bus() + pci_scan_child_bus(). So we could use common pci_scan_root_bus() in pci_common_init_dev() to scan pci busses. tegra_pcie_scan_bus() is redundant, remove it. Signed-off-by: Yijing Wang <wangyijing@huawei.com> CC: Thierry Reding <thierry.reding@gmail.com> CC: linux-tegra@vger.kernel.org --- drivers/pci/host/pci-tegra.c | 16 ---------------- 1 files changed, 0 insertions(+), 16 deletions(-) diff --git a/drivers/pci/host/pci-tegra.c b/drivers/pci/host/pci-tegra.c index 94e9362..10c0571 100644 --- a/drivers/pci/host/pci-tegra.c +++ b/drivers/pci/host/pci-tegra.c @@ -630,21 +630,6 @@ static int tegra_pcie_map_irq(const struct pci_dev *pdev, u8 slot, u8 pin) return irq; } -static struct pci_bus *tegra_pcie_scan_bus(int nr, struct pci_sys_data *sys) -{ - struct tegra_pcie *pcie = sys_to_pcie(sys); - struct pci_bus *bus; - - bus = pci_create_root_bus(pcie->dev, -1, sys->busnr, &tegra_pcie_ops, - sys, &sys->resources); - if (!bus) - return NULL; - - pci_scan_child_bus(bus); - - return bus; -} - static irqreturn_t tegra_pcie_isr(int irq, void *arg) { const char *err_msg[] = { @@ -1831,7 +1816,6 @@ static int tegra_pcie_enable(struct tegra_pcie *pcie) hw.private_data = (void **)&pcie; hw.setup = tegra_pcie_setup; hw.map_irq = tegra_pcie_map_irq; - hw.scan = tegra_pcie_scan_bus; hw.ops = &tegra_pcie_ops; pci_common_init_dev(pcie->dev, &hw); -- 1.7.1 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v8 24/30] PCI/tegra: Remove redundant tegra_pcie_scan_bus() 2015-03-24 3:34 ` [PATCH v8 24/30] PCI/tegra: Remove redundant tegra_pcie_scan_bus() Yijing Wang @ 2015-03-24 12:31 ` Thierry Reding 2015-03-25 1:22 ` Yijing Wang 0 siblings, 1 reply; 3+ messages in thread From: Thierry Reding @ 2015-03-24 12:31 UTC (permalink / raw) To: Yijing Wang Cc: Bjorn Helgaas, Jiang Liu, linux-pci, Yinghai Lu, linux-kernel, Marc Zyngier, linux-arm-kernel, Russell King, x86, Thomas Gleixner, Benjamin Herrenschmidt, Rusty Russell, Tony Luck, linux-ia64, David S. Miller, Guan Xuetao, linux-alpha, linux-m68k, Liviu Dudau, Arnd Bergmann, Geert Uytterhoeven, linux-tegra [-- Attachment #1: Type: text/plain, Size: 765 bytes --] On Tue, Mar 24, 2015 at 11:34:18AM +0800, Yijing Wang wrote: > Now pci_scan_root_bus() is almost similar to > pci_create_root_bus() + pci_scan_child_bus(). > So we could use common pci_scan_root_bus() in > pci_common_init_dev() to scan pci busses. > tegra_pcie_scan_bus() is redundant, remove it. > > Signed-off-by: Yijing Wang <wangyijing@huawei.com> > CC: Thierry Reding <thierry.reding@gmail.com> > CC: linux-tegra@vger.kernel.org > --- > drivers/pci/host/pci-tegra.c | 16 ---------------- > 1 files changed, 0 insertions(+), 16 deletions(-) Jetson TK1 still boots over NFS root via the onboard PCIe network card with this series applied, so: Tested-by: Thierry Reding <treding@nvidia.com> Acked-by: Thierry Reding <treding@nvidia.com> [-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v8 24/30] PCI/tegra: Remove redundant tegra_pcie_scan_bus() 2015-03-24 12:31 ` Thierry Reding @ 2015-03-25 1:22 ` Yijing Wang 0 siblings, 0 replies; 3+ messages in thread From: Yijing Wang @ 2015-03-25 1:22 UTC (permalink / raw) To: Thierry Reding Cc: Bjorn Helgaas, Jiang Liu, linux-pci, Yinghai Lu, linux-kernel, Marc Zyngier, linux-arm-kernel, Russell King, x86, Thomas Gleixner, Benjamin Herrenschmidt, Rusty Russell, Tony Luck, linux-ia64, David S. Miller, Guan Xuetao, linux-alpha, linux-m68k, Liviu Dudau, Arnd Bergmann, Geert Uytterhoeven, linux-tegra On 2015/3/24 20:31, Thierry Reding wrote: > On Tue, Mar 24, 2015 at 11:34:18AM +0800, Yijing Wang wrote: >> Now pci_scan_root_bus() is almost similar to >> pci_create_root_bus() + pci_scan_child_bus(). >> So we could use common pci_scan_root_bus() in >> pci_common_init_dev() to scan pci busses. >> tegra_pcie_scan_bus() is redundant, remove it. >> >> Signed-off-by: Yijing Wang <wangyijing@huawei.com> >> CC: Thierry Reding <thierry.reding@gmail.com> >> CC: linux-tegra@vger.kernel.org >> --- >> drivers/pci/host/pci-tegra.c | 16 ---------------- >> 1 files changed, 0 insertions(+), 16 deletions(-) > > Jetson TK1 still boots over NFS root via the onboard PCIe network card > with this series applied, so: > > Tested-by: Thierry Reding <treding@nvidia.com> > Acked-by: Thierry Reding <treding@nvidia.com> Thanks! > -- Thanks! Yijing ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-03-25 1:22 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1427168064-8657-1-git-send-email-wangyijing@huawei.com>
2015-03-24 3:34 ` [PATCH v8 24/30] PCI/tegra: Remove redundant tegra_pcie_scan_bus() Yijing Wang
2015-03-24 12:31 ` Thierry Reding
2015-03-25 1:22 ` Yijing Wang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).