On 20/03/2020 12.22, Wolfgang Denk wrote:
Dear Rasmus,
In message 20200320105248.24518-1-rasmus.villemoes@prevas.dk you wrote:
The code, which is likely copied from arch/powerpc/lib/interrupts.c, lacks a fallback definition of CONFIG_SYS_WATCHDOG_FREQ and refers to a non-existing timestamp variable - obviously priv->timestamp is meant.
Signed-off-by: Rasmus Villemoes rasmus.villemoes@prevas.dk
drivers/timer/mpc83xx_timer.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)
Could you _please_ get used to add some patch histroy below this line "---", too? I. e. some information so we can see easily what has changed between patch version and and 2, and between version 2 and 3?
Sorry about that. When this changed from a single patch to multiple ones I should have prepended a cover letter as well.
FWIW, patch 1/2 is new in v3, while 2/2 has been extended with documentation of both the existing meaning of CONFIG_SYS_WATCHDOG_FREQ as well as the semantics of setting that to 0, while also making it consistent across (the two implementations on) ppc and m68k.
Can I get you to review v3 as is, or should I rebase to master and resend a v4?
Thanks, Rasmus