[PATCH v3 3/9] ram: k3-ddrss: Add comment about ecc_reserved_space

Neha Malcom Francis n-francis at ti.com
Tue Aug 12 14:43:18 CEST 2025


The reserved space needed for storing the parity remains the same no
matter the size of the region that is being protected. Add this as a
comment for better code understanding.

Reviewed-by: Udit Kumar <u-kumar1 at ti.com>
Signed-off-by: Neha Malcom Francis <n-francis at ti.com>
---
Changes since v2:
- slight modification in commit message

 drivers/ram/k3-ddrss/k3-ddrss.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/drivers/ram/k3-ddrss/k3-ddrss.c b/drivers/ram/k3-ddrss/k3-ddrss.c
index 1c122c68784..04fab6518cf 100644
--- a/drivers/ram/k3-ddrss/k3-ddrss.c
+++ b/drivers/ram/k3-ddrss/k3-ddrss.c
@@ -737,6 +737,12 @@ static void k3_ddrss_lpddr4_ecc_calc_reserved_mem(struct k3_ddrss_desc *ddrss)
 {
 	fdtdec_setup_mem_size_base_lowest();
 
+	/*
+	 * For every 512-byte data block, 64 bytes are used to store inline ECC
+	 * information into a reserved region. It remains 1/9th of the total DDR
+	 * size irrespective of the size of the region under protection.
+	 */
+
 	ddrss->ecc_reserved_space = ddrss->ddr_ram_size;
 	do_div(ddrss->ecc_reserved_space, 9);
 
-- 
2.34.1



More information about the U-Boot mailing list