[Openocd-development] New patch to review for openocd: 2f7714a buspirate: ignore return value and fix warning

[Gerrit]

This is an automated email from Gerrit.

?yvind Harboe (oyvindhar...@gmail.com) just uploaded a new patch set to Gerrit, 
which you can find at http://openocd.zylin.com/43

-- gerrit
commit 2f7714a90cdb829a7a5d4f7596b189728ba0ac8d
Author: Øyvind Harboe <oyvind.har...@zylin.com>
Date:   Sat Oct 22 00:40:51 2011 +0200

    buspirate: ignore return value and fix warning
    
    Perhaps we could do one better and propagate the error?
    
    Change-Id: Idc45f516c26f09de4ee01fe05e8d3475f4b80db3
    Signed-off-by: Øyvind Harboe <oyvind.har...@zylin.com>

diff --git a/src/jtag/drivers/buspirate.c b/src/jtag/drivers/buspirate.c
index 62ab008..3a368eb 100644
--- a/src/jtag/drivers/buspirate.c
+++ b/src/jtag/drivers/buspirate.c
@@ -757,13 +757,13 @@ static void buspirate_jtag_enable(int fd)
 
 static void buspirate_jtag_reset(int fd)
 {
-       int ret;
        char tmp[5];
 
        tmp[0] = 0x00; /* exit OCD1 mode */
        buspirate_serial_write(fd, tmp, 1);
        usleep(10000);
-       ret = buspirate_serial_read(fd, tmp, 5);
+       /* We ignore the return value here purposly, nothing we can do */
+       buspirate_serial_read(fd, tmp, 5);
        if (strncmp(tmp, "BBIO1", 5) == 0) {
                tmp[0] = 0x0F; /*  reset BP */
                buspirate_serial_write(fd, tmp, 1);

-- 

_______________________________________________
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development