[U-Boot] [PATCH] ppc4xx: Make PCIE support selectable

[U-Boot] [PATCH] ppc4xx: Make PCIE support selectable

[Dirk Eibach]

On some platforms PCIE support is not required, but would be included
because the cpu supports it. To reduce fooprint it is now configurable
via CONFIG_PCI_DISABLE_PCIE.

Signed-off-by: Dirk Eibach <eibach@gdsys.de>
---
 cpu/ppc4xx/4xx_pci.c  |    5 +++--
 cpu/ppc4xx/4xx_pcie.c |    2 +-
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/cpu/ppc4xx/4xx_pci.c b/cpu/ppc4xx/4xx_pci.c
index e8871fc..2e75886 100644
--- a/cpu/ppc4xx/4xx_pci.c
+++ b/cpu/ppc4xx/4xx_pci.c
@@ -588,8 +588,9 @@ void pci_init_board(void)
 	int busno;
 
 	busno = pci_440_init (&ppc440_hose);
-#if defined(CONFIG_440SPE) || \
-    defined(CONFIG_460EX) || defined(CONFIG_460GT)
+#if (defined(CONFIG_440SPE) || \
+    defined(CONFIG_460EX) || defined(CONFIG_460GT)) && \
+    !defined(CONFIG_PCI_DISABLE_PCIE)
 	pcie_setup_hoses(busno + 1);
 #endif
 }
diff --git a/cpu/ppc4xx/4xx_pcie.c b/cpu/ppc4xx/4xx_pcie.c
index fd40d8a..58d96bb 100644
--- a/cpu/ppc4xx/4xx_pcie.c
+++ b/cpu/ppc4xx/4xx_pcie.c
@@ -33,7 +33,7 @@
 
 #if (defined(CONFIG_440SPE) || defined(CONFIG_405EX) ||	\
     defined(CONFIG_460EX) || defined(CONFIG_460GT)) && \
-    defined(CONFIG_PCI)
+    defined(CONFIG_PCI) && !defined(CONFIG_PCI_DISABLE_PCIE)
 
 #include <asm/4xx_pcie.h>
 
-- 
1.5.6.5

[U-Boot] [PATCH] ppc4xx: Make PCIE support selectable

[Stefan Roese]

On Tuesday 03 February 2009, Dirk Eibach wrote:
> On some platforms PCIE support is not required, but would be included
> because the cpu supports it. To reduce fooprint it is now configurable
> via CONFIG_PCI_DISABLE_PCIE.

Applied to ppc4xx/master. Thanks.

BTW: Please CC me on 4xx related patches in the future. This makes it easier 
for me to spot those patches.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================

[U-Boot] [PATCH] ppc4xx: Make PCIE support selectable

[Wolfgang Denk]

Dear Dirk Eibach,

In message <1233670521-24501-1-git-send-email-eibach@gdsys.de> you wrote:
> On some platforms PCIE support is not required, but would be included
> because the cpu supports it. To reduce fooprint it is now configurable
> via CONFIG_PCI_DISABLE_PCIE.
> 
> Signed-off-by: Dirk Eibach <eibach@gdsys.de>
> ---
>  cpu/ppc4xx/4xx_pci.c  |    5 +++--
>  cpu/ppc4xx/4xx_pcie.c |    2 +-
>  2 files changed, 4 insertions(+), 3 deletions(-)

Sorry, I'm too late with my review comment:

Please document the new CONFIG_PCI_DISABLE_PCIE in the README.


As Stefan has already pushed this into mainline, please send it as
additional patch.

Thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Be careful what you wish for. You never know who will be listening.
                                      - Terry Pratchett, _Soul Music_