diff mbox series

board: phytec: k3: k3_ddrss_patch: Add ddr phy reg count

Message ID 20240702092240.34349-1-d.haller@phytec.de
State Accepted
Commit 9945a30e2ed1e1a62585355865755e0e1b744d0a
Delegated to: Tom Rini
Headers show
Series board: phytec: k3: k3_ddrss_patch: Add ddr phy reg count | expand

Commit Message

Dominik Haller July 2, 2024, 9:22 a.m. UTC
Add and use the correct number of ddr phy registers to update the
corresponding settings.

Fixes: cbf5c99ef317 ("board: phytec: common: Introduce a method to inject DDR timings deltas")
Signed-off-by: Dominik Haller <d.haller@phytec.de>
---
 board/phytec/common/k3/k3_ddrss_patch.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Wadim Egorov July 2, 2024, 1:12 p.m. UTC | #1
Am 02.07.24 um 11:22 schrieb Dominik Haller:
> Add and use the correct number of ddr phy registers to update the
> corresponding settings.
> 
> Fixes: cbf5c99ef317 ("board: phytec: common: Introduce a method to inject DDR timings deltas")
> Signed-off-by: Dominik Haller <d.haller@phytec.de>
Reviewed-by: Wadim Egorov <w.egorov@phytec.de>

> ---
>   board/phytec/common/k3/k3_ddrss_patch.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/board/phytec/common/k3/k3_ddrss_patch.c b/board/phytec/common/k3/k3_ddrss_patch.c
> index 39f7be8dc922..5afe5a20c7f3 100644
> --- a/board/phytec/common/k3/k3_ddrss_patch.c
> +++ b/board/phytec/common/k3/k3_ddrss_patch.c
> @@ -12,6 +12,7 @@
>   #ifdef CONFIG_K3_AM64_DDRSS
>   #define LPDDR4_INTR_CTL_REG_COUNT (423U)
>   #define LPDDR4_INTR_PHY_INDEP_REG_COUNT (345U)
> +#define LPDDR4_INTR_PHY_REG_COUNT (1406U)
>   #endif
>   
>   static int fdt_setprop_inplace_idx_u32(void *fdt, int nodeoffset,
> @@ -54,7 +55,7 @@ int fdt_apply_ddrss_timings_patch(void *fdt, struct ddrss *ddrss)
>   					return ret;
>   			}
>   
> -	for (i = 0; i < LPDDR4_INTR_PHY_INDEP_REG_COUNT; i++)
> +	for (i = 0; i < LPDDR4_INTR_PHY_REG_COUNT; i++)
>   		for (j = 0; j < ddrss->phy_regs_num; j++)
>   			if (i == ddrss->phy_regs[j].off) {
>   				ret = fdt_setprop_inplace_idx_u32(fdt,
Tom Rini July 11, 2024, 9:27 p.m. UTC | #2
On Tue, 02 Jul 2024 11:22:40 +0200, Dominik Haller wrote:

> Add and use the correct number of ddr phy registers to update the
> corresponding settings.
> 
> 

Applied to u-boot/master, thanks!
diff mbox series

Patch

diff --git a/board/phytec/common/k3/k3_ddrss_patch.c b/board/phytec/common/k3/k3_ddrss_patch.c
index 39f7be8dc922..5afe5a20c7f3 100644
--- a/board/phytec/common/k3/k3_ddrss_patch.c
+++ b/board/phytec/common/k3/k3_ddrss_patch.c
@@ -12,6 +12,7 @@ 
 #ifdef CONFIG_K3_AM64_DDRSS
 #define LPDDR4_INTR_CTL_REG_COUNT (423U)
 #define LPDDR4_INTR_PHY_INDEP_REG_COUNT (345U)
+#define LPDDR4_INTR_PHY_REG_COUNT (1406U)
 #endif
 
 static int fdt_setprop_inplace_idx_u32(void *fdt, int nodeoffset,
@@ -54,7 +55,7 @@  int fdt_apply_ddrss_timings_patch(void *fdt, struct ddrss *ddrss)
 					return ret;
 			}
 
-	for (i = 0; i < LPDDR4_INTR_PHY_INDEP_REG_COUNT; i++)
+	for (i = 0; i < LPDDR4_INTR_PHY_REG_COUNT; i++)
 		for (j = 0; j < ddrss->phy_regs_num; j++)
 			if (i == ddrss->phy_regs[j].off) {
 				ret = fdt_setprop_inplace_idx_u32(fdt,