Dear Poonam Aggrwal, In message <1250775054-20418-1-git-send-email-poonam.aggr...@freescale.com> you wrote: > * Added PCIe support for P1 P2 RDB > * Calls the fsl_pci_init_port function to initialize all the PCIe ports > on the board. > > Signed-off-by: Poonam Aggrwal <poonam.aggr...@freescale.com> > Signed-off-by: Kumar Gala <ga...@kernel.crashing.org> > --- > - applies on git.denx.de/u-boot-mpc85xx.git branch->next > board/freescale/p1_p2_rdb/Makefile | 1 + > board/freescale/p1_p2_rdb/pci.c | 112 > ++++++++++++++++++++++++++++++++++++ > include/configs/P1_P2_RDB.h | 6 ++ > 3 files changed, 119 insertions(+), 0 deletions(-) > create mode 100644 board/freescale/p1_p2_rdb/pci.c > > diff --git a/board/freescale/p1_p2_rdb/Makefile > b/board/freescale/p1_p2_rdb/Makefile > index 9107263..c366f7c 100644 > --- a/board/freescale/p1_p2_rdb/Makefile > +++ b/board/freescale/p1_p2_rdb/Makefile > @@ -27,6 +27,7 @@ LIB = $(obj)lib$(BOARD).a > COBJS-y += $(BOARD).o > COBJS-y += law.o > COBJS-y += tlb.o > +COBJS-$(CONFIG_PCI) += pci.o > COBJS-y += ddr.o
Please (make and) keep list sorted. ... > +void ft_pci_board_setup(void *blob) > +{ > +#ifdef CONFIG_PCIE2 > + ft_fsl_pci_setup(blob, "pci1", &pcie2_hose); PCIE2 ... pci1 ... pcie2 ??? > +#endif > +#ifdef CONFIG_PCIE1 > + ft_fsl_pci_setup(blob, "pci2", &pcie1_hose); PCIE1 ... pci2 ... pcie1 ??? This looks broken to me? 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: w...@denx.de As far as the laws of mathematics refer to reality, they are not cer- tain, and as far as they are certain, they do not refer to reality. -- Albert Einstein _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot