[PATCH v1 1/1] arch: arm: zynqmp: mp.c: tcminit halt both cores in split mode

The "zynqmp tcminit split" command should halt both cores and not just RPU1 when configuring the TCM memory for split mode.
Signed-off-by: Neal Frager neal.frager@amd.com --- arch/arm/mach-zynqmp/mp.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/arch/arm/mach-zynqmp/mp.c b/arch/arm/mach-zynqmp/mp.c index 4f1ed44afb..2407eaf00b 100644 --- a/arch/arm/mach-zynqmp/mp.c +++ b/arch/arm/mach-zynqmp/mp.c @@ -244,8 +244,10 @@ void initialize_tcm(bool mode) release_r5_reset(ZYNQMP_CORE_RPU0, LOCK); } else { set_r5_tcm_mode(SPLIT); + set_r5_halt_mode(ZYNQMP_CORE_RPU0, HALT, SPLIT); set_r5_halt_mode(ZYNQMP_CORE_RPU1, HALT, SPLIT); enable_clock_r5(); + release_r5_reset(ZYNQMP_CORE_RPU0, SPLIT); release_r5_reset(ZYNQMP_CORE_RPU1, SPLIT); } }

On 3/23/23 09:25, Neal Frager wrote:
The "zynqmp tcminit split" command should halt both cores and not just RPU1 when configuring the TCM memory for split mode.
Signed-off-by: Neal Frager neal.frager@amd.com
arch/arm/mach-zynqmp/mp.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/arch/arm/mach-zynqmp/mp.c b/arch/arm/mach-zynqmp/mp.c index 4f1ed44afb..2407eaf00b 100644 --- a/arch/arm/mach-zynqmp/mp.c +++ b/arch/arm/mach-zynqmp/mp.c @@ -244,8 +244,10 @@ void initialize_tcm(bool mode) release_r5_reset(ZYNQMP_CORE_RPU0, LOCK); } else { set_r5_tcm_mode(SPLIT);
set_r5_halt_mode(ZYNQMP_CORE_RPU1, HALT, SPLIT); enable_clock_r5();set_r5_halt_mode(ZYNQMP_CORE_RPU0, HALT, SPLIT);
release_r5_reset(ZYNQMP_CORE_RPU1, SPLIT); } }release_r5_reset(ZYNQMP_CORE_RPU0, SPLIT);
Applied. M
participants (2)
-
Michal Simek
-
Neal Frager