From 11a6ac2f4fc41b1231efec63fdfe4746fb994710 Mon Sep 17 00:00:00 2001 From: Yuri D'Elia Date: Tue, 22 Jun 2021 17:08:32 +0200 Subject: [PATCH] dump parsing: refuse to continue on incomplete D23 dumps We should try harder to handle incomplete D21/D23 dumps in the future, but until D21 handled too, bail in D23 as well. --- tools/lib/dump.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/tools/lib/dump.py b/tools/lib/dump.py index 1c541a2a..8d6b1059 100644 --- a/tools/lib/dump.py +++ b/tools/lib/dump.py @@ -137,9 +137,10 @@ def decode_dump(path): # check if the dump is complete if len(ranges) != 1 or ranges[0][0] != 0 or \ ranges[0][1] != avr.SRAM_START + avr.SRAM_SIZE: - print('warning: incomplete D23 dump', file=sys.stderr) - else: - regs = True + print('error: incomplete D23 dump', file=sys.stderr) + return None + + regs = True if reason is None: print('warning: no error line in D23', file=sys.stderr)