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-kum...@ti.com>
Signed-off-by: Neha Malcom Francis <n-fran...@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

Reply via email to