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

Reply via email to