diff mbox series

[3/3] km/common: fix for CPUWD reset reason

Message ID 20200110114743.3660-3-holger.brunck@ch.abb.com
State New
Headers show
Series [1/3] kmp204x: do not make FPGA config error fail board_early_init_r | expand

Commit Message

Holger Brunck Jan. 10, 2020, 11:47 a.m. UTC 
From: Rainer Boschung <rainer.boschung at ch.abb.com>

The CPUWD reset reason is used for kmp204x.
And the qrio cpu reset request is configured to operate in
core reset mode.
But for the evaluation of the qrio's reset reason register
the CPUWD figures as a unit reset source rather than a core
reset source. This patch defines the CPUWD reset as a core
reset source when evaluating the reset reason register.

Signed-off-by: Rainer Boschung <rainer.boschung at ch.abb.com>
CC: Priyanka Jain <priyanka.jain at nxp.com>
---
 board/keymile/common/qrio.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

Comments

Tom Rini Jan. 23, 2020, 10 p.m. UTC | #1 
On Fri, Jan 10, 2020 at 12:47:43PM +0100, Holger Brunck wrote:

> From: Rainer Boschung <rainer.boschung at ch.abb.com>
> 
> The CPUWD reset reason is used for kmp204x.
> And the qrio cpu reset request is configured to operate in
> core reset mode.
> But for the evaluation of the qrio's reset reason register
> the CPUWD figures as a unit reset source rather than a core
> reset source. This patch defines the CPUWD reset as a core
> reset source when evaluating the reset reason register.
> 
> Signed-off-by: Rainer Boschung <rainer.boschung at ch.abb.com>
> CC: Priyanka Jain <priyanka.jain at nxp.com>

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

Patch

diff --git a/board/keymile/common/qrio.c b/board/keymile/common/qrio.c
index 0641ffa8b2..0cb33663aa 100644
--- a/board/keymile/common/qrio.c
+++ b/board/keymile/common/qrio.c
@@ -199,10 +199,9 @@  void qrio_cpuwd_flag(bool flag)
 #define REASON0_KBRST	0x02
 #define REASON0_POWUP	0x01
 #define UNIT_RESET\
-	((REASON1_CPUWD << 8) |\
-	REASON0_POWUP | REASON0_COPRST | REASON0_KBRST |\
-	REASON0_BPRST | REASON0_SWURST | REASON0_WDRST)
-#define CORE_RESET      REASON0_SWCRST
+	(REASON0_POWUP | REASON0_COPRST | REASON0_KBRST |\
+	 REASON0_BPRST | REASON0_SWURST | REASON0_WDRST)
+#define CORE_RESET      ((REASON1_CPUWD << 8) | REASON0_SWCRST)
 
 bool qrio_reason_unitrst(void)
 {