michaelh/zephyr
1
0
Fork 0
Code Releases Activity

tests: spi: add return value check

Browse source 
Add function return value check. This was caught by
Coverity.

Coverity-CID: 151950

Change-Id: Iee550e15d124f05f0b0514fdad22d06c617beac2
Signed-off-by: Baohong Liu <baohong.liu@intel.com>
This commit is contained in:
Baohong Liu 2016-11-17 14:42:47 -08:00
commit d5ccac16be
1 changed files with 22 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

26
tests/drivers/spi_test/src/spi.c
View file

@ -72,21 +72,39 @@ void main(void)
spi = device_get_binding(SPI_DRV_NAME); spi = device_get_binding(SPI_DRV_NAME);
if (!spi) {
printk("SPI device not found\n");
return;
}
printk("Running...\n"); printk("Running...\n");
spi_configure(spi, &spi_conf); if (spi_configure(spi, &spi_conf) != 0) {
spi_slave_select(spi, SPI_SLAVE); printk("SPI config failed\n");
return;
}
if (spi_slave_select(spi, SPI_SLAVE) != 0) {
printk("SPI slave select failed\n");
return;
}
_spi_show(&spi_conf); _spi_show(&spi_conf);
printk("Writing...\n"); printk("Writing...\n");
spi_write(spi, (uint8_t *) wbuf, 6); if (spi_write(spi, (uint8_t *) wbuf, 6) != 0) {
printk("SPI write failed\n");
return;
}
printk("SPI sent: %s\n", wbuf); printk("SPI sent: %s\n", wbuf);
print_buf_hex(rbuf, 6); print_buf_hex(rbuf, 6);
strcpy((char *)wbuf, "So what then?"); strcpy((char *)wbuf, "So what then?");
spi_transceive(spi, wbuf, 14, rbuf, 16); if (spi_transceive(spi, wbuf, 14, rbuf, 16) != 0) {
printk("SPI transceive failed\n");
return;
}
printk("SPI transceived: %s\n", rbuf); printk("SPI transceived: %s\n", rbuf);
print_buf_hex(rbuf, 6); print_buf_hex(rbuf, 6);