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
