* [PATCH] [ZynqMP] Writing correct value to ANALOG_BUS.
@ 2021-06-07 17:53 Adrian Fiergolski
2021-06-08 7:17 ` Michal Simek
0 siblings, 1 reply; 2+ messages in thread
From: Adrian Fiergolski @ 2021-06-07 17:53 UTC (permalink / raw)
To: u-boot; +Cc: michal.simek, Adrian Fiergolski
The default register configuration after powerup for PSSYSMON_ANALOG_BUS
register is incorrect. Hence, fix this in SPL by writing correct fixed
value. It follows UG1085 chapter 'PS SYSMON Analog_Bus' and reflects commit
sw_apps:zynq ("056ca65d44549ce27f716d423e8dfdefeee7440c")
in Xilinx:embeddedsw[1].
[1] https://github.com/Xilinx/embeddedsw
Signed-off-by: Adrian Fiergolski <adrian.fiergolski@fastree3d.com>
%% original patch: 0001-zynqmp-Writing-correct-value-to-ANALOG_BUS.patch
---
arch/arm/mach-zynqmp/include/mach/hardware.h | 5 +++++
board/xilinx/zynqmp/zynqmp.c | 11 +++++++++++
2 files changed, 16 insertions(+)
diff --git a/arch/arm/mach-zynqmp/include/mach/hardware.h b/arch/arm/mach-zynqmp/include/mach/hardware.h
index a0acfa2ff1..4774ed4089 100644
--- a/arch/arm/mach-zynqmp/include/mach/hardware.h
+++ b/arch/arm/mach-zynqmp/include/mach/hardware.h
@@ -15,6 +15,11 @@
#define ZYNQMP_CRL_APB_BOOT_PIN_CTRL_OUT_EN_SHIFT 0
#define ZYNQMP_CRL_APB_BOOT_PIN_CTRL_OUT_VAL_SHIFT 8
+#define ZYNQMP_AMS_PS_SYSMON_BASEADDR 0XFFA50800
+#define ZYNQMP_AMS_PS_SYSMON_ANALOG_BUS ((ZYNQMP_AMS_PS_SYSMON_BASEADDR) \
+ + 0x00000114)
+#define ZYNQMP_PS_SYSMON_ANALOG_BUS_VAL 0x00003210
+
#define PS_MODE0 BIT(0)
#define PS_MODE1 BIT(1)
#define PS_MODE2 BIT(2)
diff --git a/board/xilinx/zynqmp/zynqmp.c b/board/xilinx/zynqmp/zynqmp.c
index 8acd871ec2..9e7475dfaa 100644
--- a/board/xilinx/zynqmp/zynqmp.c
+++ b/board/xilinx/zynqmp/zynqmp.c
@@ -272,6 +272,17 @@ int board_early_init_f(void)
#if defined(CONFIG_ZYNQMP_PSU_INIT_ENABLED)
ret = psu_init();
+
+ /*
+ * PS_SYSMON_ANALOG_BUS register determines mapping between SysMon
+ * supply sense channel to SysMon supply registers inside the IP.
+ * This register must be programmed to complete SysMon IP
+ * configuration. The default register configuration after
+ * power-up is incorrect. Hence, fix this by writing the
+ * correct value - 0x3210.
+ */
+ writel(ZYNQMP_PS_SYSMON_ANALOG_BUS_VAL,
+ ZYNQMP_AMS_PS_SYSMON_ANALOG_BUS);
#endif
return ret;
--
2.31.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] [ZynqMP] Writing correct value to ANALOG_BUS.
2021-06-07 17:53 [PATCH] [ZynqMP] Writing correct value to ANALOG_BUS Adrian Fiergolski
@ 2021-06-08 7:17 ` Michal Simek
0 siblings, 0 replies; 2+ messages in thread
From: Michal Simek @ 2021-06-08 7:17 UTC (permalink / raw)
To: Adrian Fiergolski, u-boot; +Cc: michal.simek
On 6/7/21 7:53 PM, Adrian Fiergolski wrote:
> The default register configuration after powerup for PSSYSMON_ANALOG_BUS
> register is incorrect. Hence, fix this in SPL by writing correct fixed
> value. It follows UG1085 chapter 'PS SYSMON Analog_Bus' and reflects commit
> sw_apps:zynq ("056ca65d44549ce27f716d423e8dfdefeee7440c")
> in Xilinx:embeddedsw[1].
>
> [1] https://github.com/Xilinx/embeddedsw
>
> Signed-off-by: Adrian Fiergolski <adrian.fiergolski@fastree3d.com>
>
> %% original patch: 0001-zynqmp-Writing-correct-value-to-ANALOG_BUS.patch
Remove this line it shouldn't be the part of the commit message.
> ---
> arch/arm/mach-zynqmp/include/mach/hardware.h | 5 +++++
> board/xilinx/zynqmp/zynqmp.c | 11 +++++++++++
> 2 files changed, 16 insertions(+)
>
> diff --git a/arch/arm/mach-zynqmp/include/mach/hardware.h b/arch/arm/mach-zynqmp/include/mach/hardware.h
> index a0acfa2ff1..4774ed4089 100644
> --- a/arch/arm/mach-zynqmp/include/mach/hardware.h
> +++ b/arch/arm/mach-zynqmp/include/mach/hardware.h
> @@ -15,6 +15,11 @@
> #define ZYNQMP_CRL_APB_BOOT_PIN_CTRL_OUT_EN_SHIFT 0
> #define ZYNQMP_CRL_APB_BOOT_PIN_CTRL_OUT_VAL_SHIFT 8
>
> +#define ZYNQMP_AMS_PS_SYSMON_BASEADDR 0XFFA50800
> +#define ZYNQMP_AMS_PS_SYSMON_ANALOG_BUS ((ZYNQMP_AMS_PS_SYSMON_BASEADDR) \
> + + 0x00000114)
> +#define ZYNQMP_PS_SYSMON_ANALOG_BUS_VAL 0x00003210
> +
> #define PS_MODE0 BIT(0)
> #define PS_MODE1 BIT(1)
> #define PS_MODE2 BIT(2)
> diff --git a/board/xilinx/zynqmp/zynqmp.c b/board/xilinx/zynqmp/zynqmp.c
> index 8acd871ec2..9e7475dfaa 100644
> --- a/board/xilinx/zynqmp/zynqmp.c
> +++ b/board/xilinx/zynqmp/zynqmp.c
> @@ -272,6 +272,17 @@ int board_early_init_f(void)
>
> #if defined(CONFIG_ZYNQMP_PSU_INIT_ENABLED)
> ret = psu_init();
> +
> + /*
> + * PS_SYSMON_ANALOG_BUS register determines mapping between SysMon
> + * supply sense channel to SysMon supply registers inside the IP.
> + * This register must be programmed to complete SysMon IP
> + * configuration. The default register configuration after
> + * power-up is incorrect. Hence, fix this by writing the
> + * correct value - 0x3210.
> + */
> + writel(ZYNQMP_PS_SYSMON_ANALOG_BUS_VAL,
> + ZYNQMP_AMS_PS_SYSMON_ANALOG_BUS);
> #endif
>
> return ret;
>
And please rebase this on the top of v2021.07-rc4 tag.
Thanks,
Michal
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-06-08 7:17 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-07 17:53 [PATCH] [ZynqMP] Writing correct value to ANALOG_BUS Adrian Fiergolski
2021-06-08 7:17 ` Michal Simek
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.