diff options
author | Antonio Borneo <borneo.antonio@gmail.com> | 2021-09-22 19:14:29 +0200 |
---|---|---|
committer | Antonio Borneo <borneo.antonio@gmail.com> | 2021-10-25 16:10:04 +0000 |
commit | 327040ad745300c6c2117b81c584ff868567ef83 (patch) | |
tree | 9e76d49d9473ee12fa0c27b68506a8ed92f74500 /src/target | |
parent | 01de751025cfaebb7c1090f1d1ff00071ce39c71 (diff) | |
download | riscv-openocd-327040ad745300c6c2117b81c584ff868567ef83.zip riscv-openocd-327040ad745300c6c2117b81c584ff868567ef83.tar.gz riscv-openocd-327040ad745300c6c2117b81c584ff868567ef83.tar.bz2 |
target/lakemont: fix dead assignment
Clang scan-build complains for a dead assignment:
Value stored to 'tapstatus' is never read
Remove the assignment and add a comment to point for a potential
removal of the line.
Change-Id: Iad2fdc7e6faf650e24cc086ee74c745acb0d1c73
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/6591
Tested-by: jenkins
Diffstat (limited to 'src/target')
-rw-r--r-- | src/target/lakemont.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/target/lakemont.c b/src/target/lakemont.c index 576956e..e46ee5c 100644 --- a/src/target/lakemont.c +++ b/src/target/lakemont.c @@ -1070,7 +1070,8 @@ int lakemont_step(struct target *t, int current, LOG_DEBUG("EFLAGS [TF] [RF] bits set=0x%08" PRIx32 ", PMCR=0x%08" PRIx32 ", EIP=0x%08" PRIx32, eflags, pmcr, eip); - tapstatus = get_tapstatus(t); + /* Returned value unused. Can this line be removed? */ + get_tapstatus(t); t->debug_reason = DBG_REASON_SINGLESTEP; t->state = TARGET_DEBUG_RUNNING; |