[U-Boot] [PATCH v2] x86: baytrail: pci region 3 is not always mapped to end of ram

Andrew Bradford andrew at bradfordembedded.com
Wed Jun 3 18:23:09 CEST 2015


On 06/03 12:18, andrew at bradfordembedded.com wrote:
> From: Andrew Bradford <andrew.bradford at kodakalaris.com>
> 
> Baytrail physically maps the first 2 GB of SDRAM from 0x0 to 0x7FFFFFFF
> and additional SDRAM is mapped from 0x100000000 and up.  There is a
> physical memory hole from 0x80000000 to 0xFFFFFFFF for other uses.
> Because of this, PCI region 3 should only try to use up to the amount of
> SDRAM or 0x80000000, which ever is less.
> 
> Signed-off-by: Andrew Bradford <andrew.bradford at kodakalaris.com>
> ---
> v2: limit maximum size to lesser of SDRAM or 0x80000000
> ---
>  arch/x86/cpu/baytrail/pci.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/x86/cpu/baytrail/pci.c b/arch/x86/cpu/baytrail/pci.c
> index 6c291f9..53475e8 100644
> --- a/arch/x86/cpu/baytrail/pci.c
> +++ b/arch/x86/cpu/baytrail/pci.c
> @@ -39,7 +39,7 @@ void board_pci_setup_hose(struct pci_controller *hose)
>  	pci_set_region(hose->regions + 3,
>  		       0,
>  		       0,
> -		       gd->ram_size,
> +		       gd->ram_size < 0x80000000 ? gd->ram_size : 0x80000000;,
>  		       PCI_REGION_MEM | PCI_REGION_SYS_MEMORY);
>  
>  	hose->region_count = 4;

I'm very sorry, please disregard this!
It does not actually build.

I will fix and send a v3.

Thanks,
Andrew


More information about the U-Boot mailing list