Re: [PATCH] ARM: shmobile: r8a7743: Fix Watchdog timer clock

Chris Paterson

Hello Ben,

From: Ben Hutchings [mailto:ben.hutchings@...]
Sent: 24 January 2018 22:02

On Tue, 2018-01-09 at 11:48 +0000, Chris Paterson wrote:
R8A7743_CLK_RWDT should be 2 as specified in the hardware manual.
This macro doesn't seem to be used in any in-tree device trees.
You're right; it isn't at the moment.

At some point in the future we'll be adding support for RWDT to the iwg20m, so I thought I'd get this fix submitted now before I forgot.

Should it be?
If we weren't planning to use it - good question!

I guess usually a fix like this would come down the v4.4 LTS route? But as this file has been added directly to CIP this won't happen.

Presumably similar bug fix patches will have to be directly applied to CIP Kernels in the future anyway (once LTS support has ended), even if there is no known use case. There may be users of the CIP Kernel that we aren't aware of?

I've applied this anyway.
Thank you!

Kind regards, Chris


Fixes: 9683f2eba952bc0d ("ARM: shmobile: r8a7743: Add clock index
macros for DT sources")
Reported-by: Thong Ho <thong.ho.px@...>
Signed-off-by: Chris Paterson <chris.paterson2@...>
 include/dt-bindings/clock/r8a7743-clock.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/dt-bindings/clock/r8a7743-clock.h
index a5c7e387e66dfd55..53f54dfce59245d8 100644
--- a/include/dt-bindings/clock/r8a7743-clock.h
+++ b/include/dt-bindings/clock/r8a7743-clock.h
@@ -71,7 +71,7 @@
 #define R8A7743_CLK_USBDMAC1 31
 /* MSTP4 */
-#define R8A7743_CLK_RWDT 4
+#define R8A7743_CLK_RWDT 2
 #define R8A7743_CLK_USB_DDM 6
 #define R8A7743_CLK_IRQC 7
 #define R8A7743_CLK_INTC_SYS 8
Ben Hutchings
Software Developer, Codethink Ltd.

Join { to automatically receive all group messages.