buspirate: ignore return value and fix warning
authorØyvind Harboe <oyvind.harboe@zylin.com>
Fri, 21 Oct 2011 22:40:51 +0000 (00:40 +0200)
committerØyvind Harboe <oyvindharboe@gmail.com>
Sat, 22 Oct 2011 09:01:00 +0000 (09:01 +0000)
Perhaps we could do one better and propagate the error?

Change-Id: Idc45f516c26f09de4ee01fe05e8d3475f4b80db3
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
Reviewed-on: http://openocd.zylin.com/43
Tested-by: jenkins
Reviewed-by: Øyvind Harboe <oyvindharboe@gmail.com>
src/jtag/drivers/buspirate.c

index 62ab008af2cd029ada82160e3cb17063bbca280b..3a368ebbfadc3e115bb9d425d16ebacb237ad364 100644 (file)
@@ -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);