On Aug 4, 2009, at 1:09 AM, Wolfgang Denk wrote: > Dear Kumar Gala, > > In message <1249352037-25832-2-git-send-email-ga...@kernel.crashing.org > > you wrote: >> Every platform that calls fsl_pci_init calls >> fsl_pci_setup_inbound_windows >> before it calls fsl_pci_init. There isn't any reason to just call it >> from fsl_pci_init and simplify things a bit. > ... >> diff --git a/board/atum8548/atum8548.c b/board/atum8548/atum8548.c >> index 7a02cdc..85c0adc 100644 >> --- a/board/atum8548/atum8548.c >> +++ b/board/atum8548/atum8548.c >> @@ -216,9 +216,6 @@ pci_init_board(void) >> } >> printf ("\n"); >> >> - /* inbound */ >> - r += fsl_pci_setup_inbound_windows(r); >> - >> /* outbound memory */ >> pci_set_region(r++, >> CONFIG_SYS_PCIE1_MEM_BASE, > > Now pci_set_region() will see a different value of "r". Is this OK?
It is, the order of regions is not important. - k _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot