* [PATCH] arm64: zynqmp: Do not clear reset reason
@ 2021-02-09 7:52 Michal Simek
2021-02-23 13:41 ` Michal Simek
0 siblings, 1 reply; 2+ messages in thread
From: Michal Simek @ 2021-02-09 7:52 UTC (permalink / raw
To: u-boot
There is no need to clear reset reason register because it is protected by
PMUFW already which is reported when verbose log is enabled as:
pm_core.c at 733 APU> No write permission to 0xFF5E0220
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---
board/xilinx/zynqmp/zynqmp.c | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/board/xilinx/zynqmp/zynqmp.c b/board/xilinx/zynqmp/zynqmp.c
index a216eeb35f15..6babc0ebce3e 100644
--- a/board/xilinx/zynqmp/zynqmp.c
+++ b/board/xilinx/zynqmp/zynqmp.c
@@ -496,11 +496,7 @@ static int reset_reason(void)
env_set("reset_reason", reason);
- ret = zynqmp_mmio_write((ulong)&crlapb_base->reset_reason, ~0, ~0);
- if (ret)
- return -EINVAL;
-
- return ret;
+ return 0;
}
static int set_fdtfile(void)
--
2.30.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [PATCH] arm64: zynqmp: Do not clear reset reason
2021-02-09 7:52 [PATCH] arm64: zynqmp: Do not clear reset reason Michal Simek
@ 2021-02-23 13:41 ` Michal Simek
0 siblings, 0 replies; 2+ messages in thread
From: Michal Simek @ 2021-02-23 13:41 UTC (permalink / raw
To: u-boot
?t 9. 2. 2021 v 8:52 odes?latel Michal Simek <michal.simek@xilinx.com> napsal:
>
> There is no need to clear reset reason register because it is protected by
> PMUFW already which is reported when verbose log is enabled as:
> pm_core.c at 733 APU> No write permission to 0xFF5E0220
>
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> ---
>
> board/xilinx/zynqmp/zynqmp.c | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/board/xilinx/zynqmp/zynqmp.c b/board/xilinx/zynqmp/zynqmp.c
> index a216eeb35f15..6babc0ebce3e 100644
> --- a/board/xilinx/zynqmp/zynqmp.c
> +++ b/board/xilinx/zynqmp/zynqmp.c
> @@ -496,11 +496,7 @@ static int reset_reason(void)
>
> env_set("reset_reason", reason);
>
> - ret = zynqmp_mmio_write((ulong)&crlapb_base->reset_reason, ~0, ~0);
> - if (ret)
> - return -EINVAL;
> -
> - return ret;
> + return 0;
> }
>
> static int set_fdtfile(void)
> --
> 2.30.0
>
Applied.
M
--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP/Versal SoCs
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-02-23 13:41 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-02-09 7:52 [PATCH] arm64: zynqmp: Do not clear reset reason Michal Simek
2021-02-23 13:41 ` 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.