diff options
author | Rolf Meeser <rolfm_9dq@yahoo.de> | 2010-11-27 23:09:59 +0100 |
---|---|---|
committer | Øyvind Harboe <oyvind.harboe@zylin.com> | 2010-12-01 08:47:51 +0100 |
commit | 1bd87f5dbab730a5fe048bb18a952e4ffe01d7ec (patch) | |
tree | e43cd33b0f2220f2f08654a2c859ebc1de5cc916 /src/flash/nor | |
parent | 6356604382698463dd0bc0e8d71cb6d5f480ecaa (diff) | |
download | openocd_libswd-1bd87f5dbab730a5fe048bb18a952e4ffe01d7ec.tar.gz openocd_libswd-1bd87f5dbab730a5fe048bb18a952e4ffe01d7ec.tar.bz2 openocd_libswd-1bd87f5dbab730a5fe048bb18a952e4ffe01d7ec.tar.xz openocd_libswd-1bd87f5dbab730a5fe048bb18a952e4ffe01d7ec.zip |
LPC2900: Avoid false positive reprobing if first probing fails
Diffstat (limited to 'src/flash/nor')
-rw-r--r-- | src/flash/nor/lpc2900.c | 18 |
1 files changed, 11 insertions, 7 deletions
diff --git a/src/flash/nor/lpc2900.c b/src/flash/nor/lpc2900.c index e17823ec..99d6d1dc 100644 --- a/src/flash/nor/lpc2900.c +++ b/src/flash/nor/lpc2900.c @@ -132,6 +132,11 @@ struct lpc2900_flash_bank { /** + * This flag is set when the device has been successfully probed. + */ + bool is_probed; + + /** * Holds the value read from CHIPID register. * The driver will not load if the chipid doesn't match the expected * value of 0x209CE02B of the LPC2900 family. A probe will only be done @@ -255,7 +260,7 @@ static uint32_t lpc2900_is_ready( struct flash_bank *bank ) { struct lpc2900_flash_bank *lpc2900_info = bank->driver_priv; - if( lpc2900_info->chipid != EXPECTED_CHIPID ) + if( !lpc2900_info->is_probed ) { return ERROR_FLASH_BANK_NOT_PROBED; } @@ -512,8 +517,6 @@ static uint32_t lpc2900_calc_tr( uint32_t clock_var, uint32_t time_var ) /* ((time[µs]/1e6) * f[Hz]) + 511 * FPTR.TR = ------------------------------- * 512 - * - * The result is the */ uint32_t tr_val = (uint32_t)((((time_var / 1e6) * clock_var) + 511.0) / 512.0); @@ -1050,6 +1053,7 @@ FLASH_BANK_COMMAND_HANDLER(lpc2900_flash_bank_command) /* Chip ID will be obtained by probing the device later */ lpc2900_info->chipid = 0; + lpc2900_info->is_probed = false; return ERROR_OK; } @@ -1554,10 +1558,8 @@ static int lpc2900_probe(struct flash_bank *bank) return ERROR_TARGET_NOT_HALTED; } - /* We want to do this only once. Check if we already have a valid CHIPID, - * because then we will have already successfully probed the device. - */ - if (lpc2900_info->chipid == EXPECTED_CHIPID) + /* We want to do this only once. */ + if (lpc2900_info->is_probed) { return ERROR_OK; } @@ -1723,6 +1725,8 @@ static int lpc2900_probe(struct flash_bank *bank) offset += bank->sectors[i].size; } + lpc2900_info->is_probed = true; + /* Read sector security status */ if ( lpc2900_read_security_status(bank) != ERROR_OK ) { |