sysmo-bts
/
barebox
9
0
Fork 0
Code Releases Activity

i.MX6: esdctl: Fix a bug in memory probing code 

Old version of imx6_mmdc_add_mem did not use 64-bit arithmetic and
thus was prone to overflow on systems with 4GB of memory. It also did
not take into account the fact that i.MX6 does not support more than
3.8GB of memory and would report incorrect memory size. This commit
fixes both issues.

Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
This commit is contained in:
Andrey Smirnov 2014-11-13 15:24:57 -08:00 committed by Sascha Hauer
parent a56dcaa9d4
commit 074960339e
1 changed files with 21 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
arch/arm/mach-imx/esdctl.c
View File

@ -280,11 +280,30 @@ static void imx_esdctl_v4_add_mem(void *esdctlbase, struct imx_esdctl_data *data
data->base1, imx_v4_sdram_size(esdctlbase, 1));
}
/*
* On i.MX6 the adress space reserved for SDRAM is 0x10000000 to 0xFFFFFFFF
* which makes the maximum supported RAM size 0xF0000000.
*/
#define IMX6_MAX_SDRAM_SIZE 0xF0000000
static void imx6_mmdc_add_mem(void *mmdcbase, struct imx_esdctl_data *data)
{
/*
* It is possible to have a configuration in which both chip
* selects of the memory controller have 2GB of memory. To
* account for this case we need to use 64-bit arithmetic and
* also make sure we do not report more than
* IMX6_MAX_SDRAM_SIZE bytes of memory available.
*/
u64 size_cs0 = imx6_mmdc_sdram_size(mmdcbase, 0);
u64 size_cs1 = imx6_mmdc_sdram_size(mmdcbase, 1);
u64 total = size_cs0 + size_cs1;
resource_size_t size = min(total, (u64)IMX6_MAX_SDRAM_SIZE);
arm_add_mem_device("ram0", data->base0,
imx6_mmdc_sdram_size(mmdcbase, 0) +
imx6_mmdc_sdram_size(mmdcbase, 1));
size);
}
static int imx_esdctl_probe(struct device_d *dev)