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
Ben.
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(-)