kaspar/RIOT
1
0
Fork 0
mirror of https://github.com/RIOT-OS/RIOT.git synced 2025-12-22 13:03:54 +01:00
Code Releases Activity

cpu/stm32f1: Reduce scope of ret variable in spi_transfer_bytes.

Browse Source 
Reduced scope to inside the while loop as per comments in RIOT-OS/RIOT#1630

Signed-off-by: Joakim Gebart <joakim@gebart.se>
This commit is contained in:
Joakim Gebart 2014-09-03 10:28:33 +02:00
parent a12b879027
commit 21fc1bd7a6
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
cpu/stm32f1/periph/spi.c
View File

@ -123,12 +123,11 @@ int spi_transfer_byte(spi_t dev, char out, char *in)
int spi_transfer_bytes(spi_t dev, char *out, char *in, unsigned int length) int spi_transfer_bytes(spi_t dev, char *out, char *in, unsigned int length)
{ {
int transfered = 0; int transfered = 0;
int ret = 0;
if (out != NULL) { if (out != NULL) {
DEBUG("out*: %p out: %x length: %x\n", out, *out, length); DEBUG("out*: %p out: %x length: %x\n", out, *out, length);
while (length--) { while (length--) {
ret = spi_transfer_byte(dev, *(out)++, 0); int ret = spi_transfer_byte(dev, *(out)++, 0);
if (ret < 0) { if (ret < 0) {
return ret; return ret;
} }
@ -137,7 +136,7 @@ int spi_transfer_bytes(spi_t dev, char *out, char *in, unsigned int length)
} }
if (in != NULL) { if (in != NULL) {
while (length--) { while (length--) {
ret = spi_transfer_byte(dev, 0, in++); int ret = spi_transfer_byte(dev, 0, in++);
if (ret < 0) { if (ret < 0) {
return ret; return ret;
} }