[U-Boot] [PATCH 3/8] RFC: dm: pci: Set up the SDRAM mapping correctly

Stephen Warren swarren at wwwdotorg.org
Wed Oct 21 22:19:23 CEST 2015


On 10/17/2015 11:50 AM, Simon Glass wrote:
> SDRAM doesn't always start at 0. Adjust the region mapping so that it works
> on platforms where SDRAM is somewhere else.
>
> This needs testing on other platforms.

> diff --git a/drivers/pci/pci-uclass.c b/drivers/pci/pci-uclass.c

> @@ -729,8 +729,11 @@ static int decode_regions(struct pci_controller *hose, const void *blob,
>   	addr = gd->ram_size;
>   	if (gd->pci_ram_top && gd->pci_ram_top < addr)
>   		addr = gd->pci_ram_top;
> -	pci_set_region(hose->regions + hose->region_count++, 0, 0, addr,
> -		       PCI_REGION_MEM | PCI_REGION_SYS_MEMORY);
> +#ifdef CONFIG_SYS_SDRAM_BASE
> +	base = CONFIG_SYS_SDRAM_BASE;
> +#endif
> +	pci_set_region(hose->regions + hose->region_count++, base, base,
> +		       addr, PCI_REGION_MEM | PCI_REGION_SYS_MEMORY);

The variable naming here is extremely misleading; "addr" is actually the 
size of the region not its address. I think it'd be a good idea to 
rename the addr variable while you're fixing this bug, and also 
pci_ram_top too since that's really pci_ram_size.

FWIW, on Tegra210 when I set gd->pci_ram_top = 0x80000000 (coupled with 
SDRAM_BASE being 0x80000000) this function seems to do the right thing; 
at least a PCIe NIC works OK.


More information about the U-Boot mailing list