public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot-Users] [PATCH] Add support CONFIG_PCIAUTO_SKIP_HOST_BRIDGE in pci_auto.c
@ 2008-03-17 22:08 Tsi-Chung Liew
  2008-03-19  5:17 ` Stefan Roese
  2008-03-19  7:58 ` Jean-Christophe PLAGNIOL-VILLARD
  0 siblings, 2 replies; 6+ messages in thread
From: Tsi-Chung Liew @ 2008-03-17 22:08 UTC (permalink / raw)
  To: u-boot

Provide common configuration in pciauto_config_device() usable for
all architectures

Signed-off-by: TsiChung Liew <Tsi-Chung.Liew@freescale.com>
---
 drivers/pci/pci_auto.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/pci/pci_auto.c b/drivers/pci/pci_auto.c
index acfda83..1085a6c 100644
--- a/drivers/pci/pci_auto.c
+++ b/drivers/pci/pci_auto.c
@@ -383,7 +383,7 @@ int pciauto_config_device(struct pci_controller *hose, pci_dev_t dev)
 		hose->current_busno++;
 		break;
 
-#ifdef CONFIG_MPC5200
+#if defined(CONFIG_MPC5200) || defined(CONFIG_PCIAUTO_SKIP_HOST_BRIDGE)
 	case PCI_CLASS_BRIDGE_OTHER:
 		DEBUGF("PCI Autoconfig: Skipping bridge device %d\n",
 		       PCI_DEV(dev));
-- 
1.5.4.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [U-Boot-Users] [PATCH] Add support CONFIG_PCIAUTO_SKIP_HOST_BRIDGE in pci_auto.c
  2008-03-17 22:08 [U-Boot-Users] [PATCH] Add support CONFIG_PCIAUTO_SKIP_HOST_BRIDGE in pci_auto.c Tsi-Chung Liew
@ 2008-03-19  5:17 ` Stefan Roese
  2008-03-19  7:58 ` Jean-Christophe PLAGNIOL-VILLARD
  1 sibling, 0 replies; 6+ messages in thread
From: Stefan Roese @ 2008-03-19  5:17 UTC (permalink / raw)
  To: u-boot

On Monday 17 March 2008, Tsi-Chung Liew wrote:
> Provide common configuration in pciauto_config_device() usable for
> all architectures
>
> Signed-off-by: TsiChung Liew <Tsi-Chung.Liew@freescale.com>

Acked-by: Stefan Roese <sr@denx.de>

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
=====================================================================

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [U-Boot-Users] [PATCH] Add support CONFIG_PCIAUTO_SKIP_HOST_BRIDGE in pci_auto.c
  2008-03-17 22:08 [U-Boot-Users] [PATCH] Add support CONFIG_PCIAUTO_SKIP_HOST_BRIDGE in pci_auto.c Tsi-Chung Liew
  2008-03-19  5:17 ` Stefan Roese
@ 2008-03-19  7:58 ` Jean-Christophe PLAGNIOL-VILLARD
  2008-03-19  8:34   ` Stefan Roese
  1 sibling, 1 reply; 6+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2008-03-19  7:58 UTC (permalink / raw)
  To: u-boot

On 17:08 Mon 17 Mar     , Tsi-Chung Liew wrote:
> Provide common configuration in pciauto_config_device() usable for
> all architectures
> 
> Signed-off-by: TsiChung Liew <Tsi-Chung.Liew@freescale.com>
> ---
>  drivers/pci/pci_auto.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/pci/pci_auto.c b/drivers/pci/pci_auto.c
> index acfda83..1085a6c 100644
> --- a/drivers/pci/pci_auto.c
> +++ b/drivers/pci/pci_auto.c
> @@ -383,7 +383,7 @@ int pciauto_config_device(struct pci_controller *hose, pci_dev_t dev)
>  		hose->current_busno++;
>  		break;
>  
> -#ifdef CONFIG_MPC5200
> +#if defined(CONFIG_MPC5200) || defined(CONFIG_PCIAUTO_SKIP_HOST_BRIDGE)

I think that you can remove the CONFIG_MPC5200 and define it in the
configs of the boards

Best Regards
J.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [U-Boot-Users] [PATCH] Add support CONFIG_PCIAUTO_SKIP_HOST_BRIDGE in pci_auto.c
  2008-03-19  7:58 ` Jean-Christophe PLAGNIOL-VILLARD
@ 2008-03-19  8:34   ` Stefan Roese
  2008-03-19 18:45     ` Liew Tsi Chung
  0 siblings, 1 reply; 6+ messages in thread
From: Stefan Roese @ 2008-03-19  8:34 UTC (permalink / raw)
  To: u-boot

On Wednesday 19 March 2008, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > diff --git a/drivers/pci/pci_auto.c b/drivers/pci/pci_auto.c
> > index acfda83..1085a6c 100644
> > --- a/drivers/pci/pci_auto.c
> > +++ b/drivers/pci/pci_auto.c
> > @@ -383,7 +383,7 @@ int pciauto_config_device(struct pci_controller
> > *hose, pci_dev_t dev) hose->current_busno++;
> >  		break;
> >
> > -#ifdef CONFIG_MPC5200
> > +#if defined(CONFIG_MPC5200) || defined(CONFIG_PCIAUTO_SKIP_HOST_BRIDGE)
>
> I think that you can remove the CONFIG_MPC5200 and define it in the
> configs of the boards

That would be the optimal version of course.

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
=====================================================================

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [U-Boot-Users] [PATCH] Add support CONFIG_PCIAUTO_SKIP_HOST_BRIDGE in pci_auto.c
  2008-03-19  8:34   ` Stefan Roese
@ 2008-03-19 18:45     ` Liew Tsi Chung
  0 siblings, 0 replies; 6+ messages in thread
From: Liew Tsi Chung @ 2008-03-19 18:45 UTC (permalink / raw)
  To: u-boot

I will change it at the later patch with three CF EVBs.

Regards,
TsiChung 

-----Original Message-----
From: Stefan Roese [mailto:sr at denx.de] 
Sent: Wednesday, March 19, 2008 3:34 AM
To: u-boot-users at lists.sourceforge.net
Cc: Jean-Christophe PLAGNIOL-VILLARD; Liew Tsi Chung; Wolfgang Denx;
Rigby John
Subject: Re: [U-Boot-Users] [PATCH] Add support
CONFIG_PCIAUTO_SKIP_HOST_BRIDGE in pci_auto.c

On Wednesday 19 March 2008, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > diff --git a/drivers/pci/pci_auto.c b/drivers/pci/pci_auto.c index 
> > acfda83..1085a6c 100644
> > --- a/drivers/pci/pci_auto.c
> > +++ b/drivers/pci/pci_auto.c
> > @@ -383,7 +383,7 @@ int pciauto_config_device(struct pci_controller 
> > *hose, pci_dev_t dev) hose->current_busno++;
> >  		break;
> >
> > -#ifdef CONFIG_MPC5200
> > +#if defined(CONFIG_MPC5200) || 
> > +defined(CONFIG_PCIAUTO_SKIP_HOST_BRIDGE)
>
> I think that you can remove the CONFIG_MPC5200 and define it in the 
> configs of the boards

That would be the optimal version of course.

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
=====================================================================

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [U-Boot-Users] [PATCH] Add support CONFIG_PCIAUTO_SKIP_HOST_BRIDGE in pci_auto.c
@ 2008-03-20  0:55 Tsi-Chung Liew
  0 siblings, 0 replies; 6+ messages in thread
From: Tsi-Chung Liew @ 2008-03-20  0:55 UTC (permalink / raw)
  To: u-boot

Provide common configuration in pciauto_config_device() usable for
all architectures

Signed-off-by: TsiChung Liew <Tsi-Chung.Liew@freescale.com>
Acked-by: Stefan Roese <sr@denx.de>
---
 drivers/pci/pci_auto.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/pci/pci_auto.c b/drivers/pci/pci_auto.c
index acfda83..1085a6c 100644
--- a/drivers/pci/pci_auto.c
+++ b/drivers/pci/pci_auto.c
@@ -383,7 +383,7 @@ int pciauto_config_device(struct pci_controller *hose, pci_dev_t dev)
 		hose->current_busno++;
 		break;
 
-#ifdef CONFIG_MPC5200
+#if defined(CONFIG_MPC5200) || defined(CONFIG_PCIAUTO_SKIP_HOST_BRIDGE)
 	case PCI_CLASS_BRIDGE_OTHER:
 		DEBUGF("PCI Autoconfig: Skipping bridge device %d\n",
 		       PCI_DEV(dev));
-- 
1.5.4.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2008-03-20  0:55 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-03-17 22:08 [U-Boot-Users] [PATCH] Add support CONFIG_PCIAUTO_SKIP_HOST_BRIDGE in pci_auto.c Tsi-Chung Liew
2008-03-19  5:17 ` Stefan Roese
2008-03-19  7:58 ` Jean-Christophe PLAGNIOL-VILLARD
2008-03-19  8:34   ` Stefan Roese
2008-03-19 18:45     ` Liew Tsi Chung
  -- strict thread matches above, loose matches on Subject: below --
2008-03-20  0:55 Tsi-Chung Liew

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox