[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