The original checking is wrong, it basically compares memory address
end = r->start + r->range;
with memory size:
if (end > (ddr_ram_size - ecc_res))
so the if() condition would be always true. This causes the ECC config
never take input from the devicetree.
Fixes: f43f71012254 ("ram: k3-ddrss: Add support for a partial inline ECC
region")
Signed-off-by: Bin Liu <[email protected]>
---
drivers/ram/k3-ddrss/k3-ddrss.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/ram/k3-ddrss/k3-ddrss.c b/drivers/ram/k3-ddrss/k3-ddrss.c
index 5144470b9313..c406998b20aa 100644
--- a/drivers/ram/k3-ddrss/k3-ddrss.c
+++ b/drivers/ram/k3-ddrss/k3-ddrss.c
@@ -931,11 +931,10 @@ static int k3_ddrss_probe(struct udevice *dev)
struct k3_ddrss_ecc_region *r = range;
for (int i = 0; (i < K3_DDRSS_MAX_ECC_REG) && (r->range
!= 0); i++, r++) {
- end = r->start + r->range;
ddr_ram_size = ddrss->ddr_ram_size;
ecc_res = ddrss->ecc_reserved_space;
- if (end > (ddr_ram_size - ecc_res))
+ if (r->range > (ddr_ram_size - ecc_res))
ddrss->ecc_regions[i].range =
ddr_ram_size - ecc_res;
else
ddrss->ecc_regions[i].range = r->range;
--
2.34.1