On 05.07.21 10:22, Cedric Hombourger wrote: > Changes in v3: > - place wdat.c first and note its precedence as done for ipc4x7e_wdt.c >
Thoroughness trumps speed: Please read and address ALL my remarks. Jan > Changes in v2: > - addressed stylistic review comments from Jan > - (TYPE *)var instead of (TYPE *) var > - do not indent "case" in switch statements > - remove over-defensive if statements > - let the compiler decide when to inline > > - move REBOOT action up > - no longer print a warning for unsupported actions (let callers do > that) > - removed changes to main.c > - moved wdat.c before itco.c so it takes precedence at runtime > - call readl() (as expected) for width == 3 > > New revision of this patch tested on SIMATIC IPC127E. > > > Cedric Hombourger (1): > drivers/watchdog: add support for WDAT (ACPI) watchdogs > > Makefile.am | 2 + > drivers/watchdog/wdat.c | 425 ++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 427 insertions(+) > create mode 100644 drivers/watchdog/wdat.c > -- Siemens AG, T RDA IOT Corporate Competence Center Embedded Linux -- You received this message because you are subscribed to the Google Groups "EFI Boot Guard" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/efibootguard-dev/3179223f-b3fd-7052-b2be-44a0674ce18c%40siemens.com.
