[U-Boot] [PATCH 2/2 v2] zlib: Add further watchdog reset calls

Patch 253cb831 [zlib: add watchdog reset call] added already a few watchdog reset calls to the new zlib U-Boot port. But on some boards this is not enough. Additional calls are needed on boards with short watchdog timeouts.
This was detected and tested on the lwmon5 board with a very short watchdog timeout. Without this patch, the board resets during Linux kernel decompression. With it, the decompression succeeds.
Signed-off-by: Stefan Roese sr@denx.de Cc: Detlev Zundel dzu@denx.de --- v2: - Changed to use WATCHDOG_RESET macro instead of function pointer
lib/zlib.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/lib/zlib.c b/lib/zlib.c index 2b31dba..e19484a 100644 --- a/lib/zlib.c +++ b/lib/zlib.c @@ -1599,6 +1599,7 @@ int flush; strm->adler = state->check = adler32(0L, Z_NULL, 0); state->mode = TYPE; case TYPE: + WATCHDOG_RESET(); if (flush == Z_BLOCK) goto inf_leave; case TYPEDO: if (state->last) {

Hi Stefan,
Patch 253cb831 [zlib: add watchdog reset call] added already a few watchdog reset calls to the new zlib U-Boot port. But on some boards this is not enough. Additional calls are needed on boards with short watchdog timeouts.
This was detected and tested on the lwmon5 board with a very short watchdog timeout. Without this patch, the board resets during Linux kernel decompression. With it, the decompression succeeds.
Signed-off-by: Stefan Roese sr@denx.de Cc: Detlev Zundel dzu@denx.de
Acked-by: Detlev Zundel dzu@denx.de
Cheers Detlev

Dear Stefan Roese,
In message 1283952931-4406-1-git-send-email-sr@denx.de you wrote:
Patch 253cb831 [zlib: add watchdog reset call] added already a few watchdog reset calls to the new zlib U-Boot port. But on some boards this is not enough. Additional calls are needed on boards with short watchdog timeouts.
This was detected and tested on the lwmon5 board with a very short watchdog timeout. Without this patch, the board resets during Linux kernel decompression. With it, the decompression succeeds.
Signed-off-by: Stefan Roese sr@denx.de Cc: Detlev Zundel dzu@denx.de
v2:
- Changed to use WATCHDOG_RESET macro instead of function pointer
lib/zlib.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-)
Applied, thanks.
Best regards,
Wolfgang Denk
participants (3)
-
Detlev Zundel
-
Stefan Roese
-
Wolfgang Denk