[U-Boot] [PATCH] pci: Divided pci code of the powerpc

Stefan Roese sr at denx.de
Tue Sep 23 10:03:06 CEST 2008


On Monday 22 September 2008, Andrew Dyer wrote:
> >> I did a simple check.
> >>
> >> Only powerpc and i386 and sh seem to use PCI.
> >> If powerpc does not have a problem, would you commit it?
> >
> > Do I have your "go" (or at least no veto) to apply this patch?
>
> (forgot to reply all on this, already sent to the O.P.)
>
> I think this is going to break some boards/archs.  From the top
> of the u-boot tree, if you do
>
> find . -type f -exec grep pci_register_hose {} /dev/null \;
>
> it shows all the source files that are registering a PCI bus.  A quick
> look shows there are MIPS, ARM, x86 in that list at least, that aren't
> addressed by this change.

That's correct. This patch moves the original defines to the PPC specific 
header and introduces new defines in the SH specific header. All other ARCH's 
will not compile anymore when they try to include PCI support.

So, NAK from me. An acceptable patch would be to move the original defines at 
least to all ARCH's already using PCI.

BTW: I fail to see what's really PPC specific about the pci code in question. 
Nobuhiro, could you please elaborate what's the problem here?

Thanks.

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


More information about the U-Boot mailing list