Re: [PATCH v4 5/8] drivers: ram: Kconfig: Add CONFIG_K3_INLINE_ECC

On October 21, 2024 thus sayeth Santhosh Kumar K:

From: Neha Malcom Francis n-francis@ti.com

Add CONFIG_K3_INLINE_ECC so that ECC functions can be compiled into R5 SPL only when the config has been enabled.

Signed-off-by: Neha Malcom Francis n-francis@ti.com

drivers/ram/Kconfig | 10 ++++++++++ 1 file changed, 10 insertions(+)

diff --git a/drivers/ram/Kconfig b/drivers/ram/Kconfig index f7e357f24da7..533d0c629a64 100644 --- a/drivers/ram/Kconfig +++ b/drivers/ram/Kconfig @@ -116,6 +116,16 @@ config IMXRT_SDRAM to support external memories like sdram, psram & nand. This driver is for the sdram memory interface with the SEMC.

+config K3_INLINE_ECC

• bool "Enable TI Inline ECC support"
• depends on K3_DDRSS
• help

•  Enable Inline ECC support on K3 platforms. 1/9th of the SDRAM space
  

•  is used for ECC storage and the rest 8/9th is available for system
  

•  use. Enabling ECC increases boot time as the ECC protected regions
  

•  need to be primed with a predefined value prior to enabling ECC
  

•  check.
  

I agree size will always be a complaint at such an early stage of boot but we're already adding the ECC configuration in the memory{} node.

We should do one or the other but not both. It would be hard to debug problems if someone added the protected{} node but not enable this config