From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thierry Reding Subject: Re: [PATCH v8 24/30] PCI/tegra: Remove redundant tegra_pcie_scan_bus() Date: Tue, 24 Mar 2015 13:31:40 +0100 Message-ID: <20150324123139.GA11625@ulmo.nvidia.com> References: <1427168064-8657-1-git-send-email-wangyijing@huawei.com> <1427168064-8657-25-git-send-email-wangyijing@huawei.com> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="bp/iNruPH9dso1Pn" Return-path: Content-Disposition: inline In-Reply-To: <1427168064-8657-25-git-send-email-wangyijing@huawei.com> Sender: linux-alpha-owner@vger.kernel.org To: Yijing Wang Cc: Bjorn Helgaas , Jiang Liu , linux-pci@vger.kernel.org, Yinghai Lu , linux-kernel@vger.kernel.org, Marc Zyngier , linux-arm-kernel@lists.infradead.org, Russell King , x86@kernel.org, Thomas Gleixner , Benjamin Herrenschmidt , Rusty Russell , Tony Luck , linux-ia64@vger.kernel.org, "David S. Miller" , Guan Xuetao , linux-alpha@vger.kernel.org, linux-m68k@lists.linux-m68k.org, Liviu Dudau , Arnd Bergmann , Geert Uytterhoeven , linux-tegra@vger.kernel.org List-Id: linux-tegra@vger.kernel.org --bp/iNruPH9dso1Pn Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable 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. >=20 > Signed-off-by: Yijing Wang > CC: Thierry Reding > 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 Acked-by: Thierry Reding --bp/iNruPH9dso1Pn Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAABCAAGBQJVEVknAAoJEN0jrNd/PrOhgSAQAKCC9PfTMZLcgurKsdBmqctU 7WkCuoKLDv1sbEfF1mzfk8VI/9kROf6bIyKKgvXV2V50p8OHBbJBJmjR5U48/37v iDGVlSrSII4+G3Mv/HWx/Dvh1whXYp/TH+SNUPNY1BdSIRmvjmLQ3QYBuC6P3N6W kjd5NW5KkgZ0NPORWfXzEabkXIGEjIk8fsB6ILXxBVJMdUxiLA4LyOQAmmuzoviW TAsboT7liAJPbiM2mjnYqo6ZdDR7Y8hotXBf8KtIxlzns12VQZl40GdtvZ7zWi4R tH3zyYqJuhyg6VjvDMZzKZsmU4hctOQ2sgeYOFn52t8nPIZqJMoLT3nAabgUw+7x NNI+oZGD8Wo4XGDXQyiY1ihjObLCD2uDfpKfdwmCRlYFef66rEWW0gImACCCprEc 9jqQgqZAN9AXX+1OkNyX+Ioiy2v7yicVi/JMHawdEEdszSuG3RuZIQQWku+T2UDy WmpI2MdZRUEAJiW1V5rOBibae2uvGEHedkZmOqkjG2NLFqt633FlUg416ucPD+eJ VKWNXwsJyqBtGpifsUaFZfSNFsvSZ4gceI+/KD0ZpmhhAXkh+AVoNTa1iZLjq3X9 q8J1YflS6mHPbE+D37LRDLg47XoHRIf0Ly/9GukoXbowfnP3/Fy1532Zy+1eXyBb Lk3IVyxC8BFsZ0j1FZKG =0jIA -----END PGP SIGNATURE----- --bp/iNruPH9dso1Pn-- From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752376AbbCXMbt (ORCPT ); Tue, 24 Mar 2015 08:31:49 -0400 Received: from mail-pd0-f172.google.com ([209.85.192.172]:33098 "EHLO mail-pd0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752165AbbCXMbp (ORCPT ); Tue, 24 Mar 2015 08:31:45 -0400 Date: Tue, 24 Mar 2015 13:31:40 +0100 From: Thierry Reding To: Yijing Wang Cc: Bjorn Helgaas , Jiang Liu , linux-pci@vger.kernel.org, Yinghai Lu , linux-kernel@vger.kernel.org, Marc Zyngier , linux-arm-kernel@lists.infradead.org, Russell King , x86@kernel.org, Thomas Gleixner , Benjamin Herrenschmidt , Rusty Russell , Tony Luck , linux-ia64@vger.kernel.org, "David S. Miller" , Guan Xuetao , linux-alpha@vger.kernel.org, linux-m68k@vger.kernel.org, Liviu Dudau , Arnd Bergmann , Geert Uytterhoeven , linux-tegra@vger.kernel.org Subject: Re: [PATCH v8 24/30] PCI/tegra: Remove redundant tegra_pcie_scan_bus() Message-ID: <20150324123139.GA11625@ulmo.nvidia.com> References: <1427168064-8657-1-git-send-email-wangyijing@huawei.com> <1427168064-8657-25-git-send-email-wangyijing@huawei.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="bp/iNruPH9dso1Pn" Content-Disposition: inline In-Reply-To: <1427168064-8657-25-git-send-email-wangyijing@huawei.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --bp/iNruPH9dso1Pn Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable 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. >=20 > Signed-off-by: Yijing Wang > CC: Thierry Reding > 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 Acked-by: Thierry Reding --bp/iNruPH9dso1Pn Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAABCAAGBQJVEVknAAoJEN0jrNd/PrOhgSAQAKCC9PfTMZLcgurKsdBmqctU 7WkCuoKLDv1sbEfF1mzfk8VI/9kROf6bIyKKgvXV2V50p8OHBbJBJmjR5U48/37v iDGVlSrSII4+G3Mv/HWx/Dvh1whXYp/TH+SNUPNY1BdSIRmvjmLQ3QYBuC6P3N6W kjd5NW5KkgZ0NPORWfXzEabkXIGEjIk8fsB6ILXxBVJMdUxiLA4LyOQAmmuzoviW TAsboT7liAJPbiM2mjnYqo6ZdDR7Y8hotXBf8KtIxlzns12VQZl40GdtvZ7zWi4R tH3zyYqJuhyg6VjvDMZzKZsmU4hctOQ2sgeYOFn52t8nPIZqJMoLT3nAabgUw+7x NNI+oZGD8Wo4XGDXQyiY1ihjObLCD2uDfpKfdwmCRlYFef66rEWW0gImACCCprEc 9jqQgqZAN9AXX+1OkNyX+Ioiy2v7yicVi/JMHawdEEdszSuG3RuZIQQWku+T2UDy WmpI2MdZRUEAJiW1V5rOBibae2uvGEHedkZmOqkjG2NLFqt633FlUg416ucPD+eJ VKWNXwsJyqBtGpifsUaFZfSNFsvSZ4gceI+/KD0ZpmhhAXkh+AVoNTa1iZLjq3X9 q8J1YflS6mHPbE+D37LRDLg47XoHRIf0Ly/9GukoXbowfnP3/Fy1532Zy+1eXyBb Lk3IVyxC8BFsZ0j1FZKG =0jIA -----END PGP SIGNATURE----- --bp/iNruPH9dso1Pn-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: thierry.reding@gmail.com (Thierry Reding) Date: Tue, 24 Mar 2015 13:31:40 +0100 Subject: [PATCH v8 24/30] PCI/tegra: Remove redundant tegra_pcie_scan_bus() In-Reply-To: <1427168064-8657-25-git-send-email-wangyijing@huawei.com> References: <1427168064-8657-1-git-send-email-wangyijing@huawei.com> <1427168064-8657-25-git-send-email-wangyijing@huawei.com> Message-ID: <20150324123139.GA11625@ulmo.nvidia.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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 > CC: Thierry Reding > CC: linux-tegra at 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 Acked-by: Thierry Reding -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 819 bytes Desc: not available URL: From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thierry Reding Date: Tue, 24 Mar 2015 12:31:40 +0000 Subject: Re: [PATCH v8 24/30] PCI/tegra: Remove redundant tegra_pcie_scan_bus() Message-Id: <20150324123139.GA11625@ulmo.nvidia.com> MIME-Version: 1 Content-Type: multipart/mixed; boundary="bp/iNruPH9dso1Pn" List-Id: References: <1427168064-8657-1-git-send-email-wangyijing@huawei.com> <1427168064-8657-25-git-send-email-wangyijing@huawei.com> In-Reply-To: <1427168064-8657-25-git-send-email-wangyijing@huawei.com> To: Yijing Wang Cc: Bjorn Helgaas , Jiang Liu , linux-pci@vger.kernel.org, Yinghai Lu , linux-kernel@vger.kernel.org, Marc Zyngier , linux-arm-kernel@lists.infradead.org, Russell King , x86@kernel.org, Thomas Gleixner , Benjamin Herrenschmidt , Rusty Russell , Tony Luck , linux-ia64@vger.kernel.org, "David S. Miller" , Guan Xuetao , linux-alpha@vger.kernel.org, linux-m68k@lists.linux-m68k.org, Liviu Dudau , Arnd Bergmann , Geert Uytterhoeven , linux-tegra@vger.kernel.org --bp/iNruPH9dso1Pn Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable 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. >=20 > Signed-off-by: Yijing Wang > CC: Thierry Reding > 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 Acked-by: Thierry Reding --bp/iNruPH9dso1Pn Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAABCAAGBQJVEVknAAoJEN0jrNd/PrOhgSAQAKCC9PfTMZLcgurKsdBmqctU 7WkCuoKLDv1sbEfF1mzfk8VI/9kROf6bIyKKgvXV2V50p8OHBbJBJmjR5U48/37v iDGVlSrSII4+G3Mv/HWx/Dvh1whXYp/TH+SNUPNY1BdSIRmvjmLQ3QYBuC6P3N6W kjd5NW5KkgZ0NPORWfXzEabkXIGEjIk8fsB6ILXxBVJMdUxiLA4LyOQAmmuzoviW TAsboT7liAJPbiM2mjnYqo6ZdDR7Y8hotXBf8KtIxlzns12VQZl40GdtvZ7zWi4R tH3zyYqJuhyg6VjvDMZzKZsmU4hctOQ2sgeYOFn52t8nPIZqJMoLT3nAabgUw+7x NNI+oZGD8Wo4XGDXQyiY1ihjObLCD2uDfpKfdwmCRlYFef66rEWW0gImACCCprEc 9jqQgqZAN9AXX+1OkNyX+Ioiy2v7yicVi/JMHawdEEdszSuG3RuZIQQWku+T2UDy WmpI2MdZRUEAJiW1V5rOBibae2uvGEHedkZmOqkjG2NLFqt633FlUg416ucPD+eJ VKWNXwsJyqBtGpifsUaFZfSNFsvSZ4gceI+/KD0ZpmhhAXkh+AVoNTa1iZLjq3X9 q8J1YflS6mHPbE+D37LRDLg47XoHRIf0Ly/9GukoXbowfnP3/Fy1532Zy+1eXyBb Lk3IVyxC8BFsZ0j1FZKG =0jIA -----END PGP SIGNATURE----- --bp/iNruPH9dso1Pn--