arm_adi_v5: fix return value of mem_ap_read/write for size 0
Unhandled marginal case produced a warning in Clang static analyzer. Change-Id: I3e2fc4182fa4f863acfb972b1e7a512fce5bf33a Signed-off-by: Tomas Vanek <vanekt@fbl.cz> Suggested-by: Matthias Welwarsky <matthias@welwarsky.de> Reviewed-on: http://openocd.zylin.com/4357 Tested-by: jenkins Reviewed-by: Matthias Welwarsky <matthias@welwarsky.de>
This commit is contained in:
parent
1ac0f5d493
commit
dd890d4cad
|
@ -325,7 +325,7 @@ static int mem_ap_write(struct adiv5_ap *ap, const uint8_t *buffer, uint32_t siz
|
||||||
const uint32_t csw_addrincr = addrinc ? CSW_ADDRINC_SINGLE : CSW_ADDRINC_OFF;
|
const uint32_t csw_addrincr = addrinc ? CSW_ADDRINC_SINGLE : CSW_ADDRINC_OFF;
|
||||||
uint32_t csw_size;
|
uint32_t csw_size;
|
||||||
uint32_t addr_xor;
|
uint32_t addr_xor;
|
||||||
int retval;
|
int retval = ERROR_OK;
|
||||||
|
|
||||||
/* TI BE-32 Quirks mode:
|
/* TI BE-32 Quirks mode:
|
||||||
* Writes on big-endian TMS570 behave very strangely. Observed behavior:
|
* Writes on big-endian TMS570 behave very strangely. Observed behavior:
|
||||||
|
@ -455,7 +455,7 @@ static int mem_ap_read(struct adiv5_ap *ap, uint8_t *buffer, uint32_t size, uint
|
||||||
const uint32_t csw_addrincr = addrinc ? CSW_ADDRINC_SINGLE : CSW_ADDRINC_OFF;
|
const uint32_t csw_addrincr = addrinc ? CSW_ADDRINC_SINGLE : CSW_ADDRINC_OFF;
|
||||||
uint32_t csw_size;
|
uint32_t csw_size;
|
||||||
uint32_t address = adr;
|
uint32_t address = adr;
|
||||||
int retval;
|
int retval = ERROR_OK;
|
||||||
|
|
||||||
/* TI BE-32 Quirks mode:
|
/* TI BE-32 Quirks mode:
|
||||||
* Reads on big-endian TMS570 behave strangely differently than writes.
|
* Reads on big-endian TMS570 behave strangely differently than writes.
|
||||||
|
|
Loading…
Reference in New Issue