interesting
/
riscv-openocd
mirror of https://github.com/riscv/riscv-openocd.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

at91smad: Fixes to 'samd_protect' 

Some fixes to 'samd_protect' including:
     - Fix a bug in which the value of 'set' parameter passed into the
     function was ignored so it was impossible to remove flash
     protection once it was set.
     - Check the protection status of the sector via 'is_protected'
     field of the  corresponding 'flash_sector' structure to see if
     any actual HW manipulations needs to be done.
     - Change the way the errors during protection activation are
     handled. Now even in the case of error in the middle of
     protecting a number of sectors the subroutine would still update
     the state of the sector protection in sectors array so as to
     avoid cases where openocd thinks that the sector is not protected
     while it actually is.

Change-Id: I4cc6445a98ec13bdd94c89f0711c17840738a215
Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com>
Reviewed-on: http://openocd.zylin.com/2027
Tested-by: jenkins
Reviewed-by: Andrey Yurovsky <yurovsky@gmail.com>
Reviewed-by: Paul Fertser <fercerpav@gmail.com>
This commit is contained in:
Andrey Smirnov 2014-02-10 08:06:04 -08:00 committed by Paul Fertser
parent c92a605e26
commit dba153bbce
1 changed files with 22 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
src/flash/nor/at91samd.c
View File

@ -254,24 +254,33 @@ static int samd_protect(struct flash_bank *bank, int set, int first, int last)
int res; int res;
struct samd_info *chip = (struct samd_info *)bank->driver_priv; struct samd_info *chip = (struct samd_info *)bank->driver_priv;
res = ERROR_OK;
for (int s = first; s <= last; s++) { for (int s = first; s <= last; s++) {
/* Load an address that is within this sector (we use offset 0) */ if (set != bank->sectors[s].is_protected) {
res = target_write_u32(bank->target, SAMD_NVMCTRL + SAMD_NVMCTRL_ADDR, /* Load an address that is within this sector (we use offset 0) */
s * chip->sector_size); res = target_write_u32(bank->target, SAMD_NVMCTRL + SAMD_NVMCTRL_ADDR,
if (res != ERROR_OK) s * chip->sector_size);
return res; if (res != ERROR_OK)
goto exit;
/* Tell the controller to lock that sector */ /* Tell the controller to lock that sector */
res = target_write_u16(bank->target,
SAMD_NVMCTRL + SAMD_NVMCTRL_CTRLA, uint16_t cmd = (set) ?
SAMD_NVM_CMD(SAMD_NVM_CMD_LR)); SAMD_NVM_CMD(SAMD_NVM_CMD_LR) :
if (res != ERROR_OK) SAMD_NVM_CMD(SAMD_NVM_CMD_UR);
return res;
res = target_write_u16(bank->target,
SAMD_NVMCTRL + SAMD_NVMCTRL_CTRLA,
cmd);
if (res != ERROR_OK)
goto exit;
}
} }
exit:
samd_protect_check(bank); samd_protect_check(bank);
return ERROR_OK; return res;
} }
static bool samd_check_error(struct flash_bank *bank) static bool samd_check_error(struct flash_bank *bank)