From: Jim Paris Date: Wed, 29 Oct 2014 23:00:52 +0000 (-0400) Subject: nrf51: verify that UICR needs erasing before triggering an error about it X-Git-Tag: v0.9.0-rc1~236 X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=56802d794e9b23c290ccb373c44b86999f35ca20;p=openocd nrf51: verify that UICR needs erasing before triggering an error about it If the UICR is already empty, there's no reason to return an error just because it can't be erased again. This happens, for example, when flashing UICR from GDB after a "monitor nrf51 mass_erase". Change-Id: Ia6d28c43189205fb5a7120b1c7312e45eb32edb7 Signed-off-by: Jim Paris Reviewed-on: http://openocd.zylin.com/2363 Tested-by: jenkins Reviewed-by: Spencer Oliver --- diff --git a/src/flash/nor/nrf51.c b/src/flash/nor/nrf51.c index 8b6a57fb..bfa199a2 100644 --- a/src/flash/nor/nrf51.c +++ b/src/flash/nor/nrf51.c @@ -581,6 +581,12 @@ static int nrf51_erase_page(struct flash_bank *bank, } if ((ppfc & 0xFF) == 0xFF) { + /* We can't erase the UICR. Double-check to + see if it's already erased before complaining. */ + default_flash_blank_check(bank); + if (sector->is_erased == 1) + return ERROR_OK; + LOG_ERROR("The chip was not pre-programmed with SoftDevice stack and UICR cannot be erased separately. Please issue mass erase before trying to write to this region"); return ERROR_FAIL; };