[PATCH v5 7/9] ram: k3-ddrss: Remove 'ti,ecc-enable' support

Wadim Egorov w.egorov at phytec.de
Fri Dec 20 07:49:59 CET 2024


Am 13.12.24 um 17:31 schrieb Santhosh Kumar K:
> The functionality of enabling Inline ECC is now controlled by
> CONFIG_K3_INLINE_ECC. So, remove the support for 'ti,ecc-enable'
> property to avoid redundancy and to ensure the Inline ECC feature is
> mananged through build-time config.
> 
> Signed-off-by: Santhosh Kumar K <s-k6 at ti.com>

Reviewed-by: Wadim Egorov <w.egorov at phytec.de>

> ---
>   drivers/ram/k3-ddrss/k3-ddrss.c | 5 +----
>   1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/drivers/ram/k3-ddrss/k3-ddrss.c b/drivers/ram/k3-ddrss/k3-ddrss.c
> index e4f9891bee20..05ea61bbfe9a 100644
> --- a/drivers/ram/k3-ddrss/k3-ddrss.c
> +++ b/drivers/ram/k3-ddrss/k3-ddrss.c
> @@ -151,7 +151,6 @@ struct k3_ddrss_desc {
>   	lpddr4_privatedata pd;
>   	struct k3_ddrss_ecc_region ecc_regions[K3_DDRSS_MAX_ECC_REGIONS];
>   	u64 ecc_reserved_space;
> -	bool ti_ecc_enabled;
>   	u64 ddr_bank_base[CONFIG_NR_DRAM_BANKS];
>   	u64 ddr_bank_size[CONFIG_NR_DRAM_BANKS];
>   	u64 ddr_ram_size;
> @@ -408,8 +407,6 @@ static int k3_ddrss_ofdata_to_priv(struct udevice *dev)
>   	if (ret)
>   		dev_err(dev, "ddr fhs cnt not populated %d\n", ret);
>   
> -	ddrss->ti_ecc_enabled = dev_read_bool(dev, "ti,ecc-enable");
> -
>   	return ret;
>   }
>   
> @@ -792,7 +789,7 @@ static int k3_ddrss_probe(struct udevice *dev)
>   
>   	k3_ddrss_ddr_bank_base_size_calc(ddrss);
>   
> -	if (ddrss->ti_ecc_enabled) {
> +	if (IS_ENABLED(CONFIG_K3_INLINE_ECC)) {
>   		if (!ddrss->ddrss_ss_cfg) {
>   			printf("%s: ss_cfg is required if ecc is enabled but not provided.",
>   			       __func__);



More information about the U-Boot mailing list