# HG changeset patch # User Paul Boddie # Date 1440420520 -7200 # Node ID d26f14ad413357030df79c643e09251530edfec2 # Parent c44b58a7fc6977f0f5e1431f959b8017db4441ab Removed apparently superfluous failure handling. diff -r c44b58a7fc69 -r d26f14ad4133 upload.py --- a/upload.py Mon Aug 24 13:00:39 2015 +0200 +++ b/upload.py Mon Aug 24 14:48:40 2015 +0200 @@ -86,21 +86,13 @@ if not verify_only: writeToPort("P%05x%02x\n" % (addr, value)) resp = readFromPort() - if resp == "P": - i += 1 - continue - else: - closePort() - openPort() - - for attempt in "first", "second": + success = resp == "P" + else: writeToPort("R%05x\n" % addr) resp = readFromPort() - if resp == "%02X" % value or resp == "%X" % value: - break - closePort() - openPort() - else: + success = resp == "%02X" % value or resp == "%X" % value + + if not success: print >>sys.stderr, "%s of location %05x failed: %s" % (verify_only and "Verify" or "Program", addr, resp) return False