This is an automated email from Gerrit. "Benjamin Vernoux <bvern...@gmail.com>" just uploaded a new patch set to Gerrit, which you can find at https://review.openocd.org/c/openocd/+/7803
-- gerrit commit 5024aa0615e3d2726b7e119ac66c9031a3f16dc2 Author: Benjamin Vernoux <bvern...@gmail.com> Date: Thu Aug 31 11:25:57 2023 +0200 Minor cosmetic fixes Change-Id: I5b11cc6c3faa7a7801ecf253027ef0d7cab87525 Signed-off-by: Benjamin Vernoux <bvern...@gmail.com> diff --git a/src/jtag/drivers/buspirate.c b/src/jtag/drivers/buspirate.c index 27d900f5f5..306e0e05f4 100644 --- a/src/jtag/drivers/buspirate.c +++ b/src/jtag/drivers/buspirate.c @@ -111,7 +111,7 @@ static bool swd_mode; static int queued_retval; static char swd_features; #ifdef _WIN32 -static fd_handle_t buspirate_fd = NULL; +static fd_handle_t buspirate_fd; #else static fd_handle_t buspirate_fd = -1; #endif @@ -305,7 +305,7 @@ static int buspirate_init(void) buspirate_fd = buspirate_serial_open(buspirate_port); #ifdef _WIN32 if (buspirate_fd == NULL) { -#else +#else if (buspirate_fd == -1) { #endif LOG_ERROR("Could not open serial port"); @@ -381,7 +381,7 @@ COMMAND_HANDLER(buspirate_handle_adc_command) #ifdef _WIN32 if (buspirate_fd == NULL) return ERROR_OK; -#else +#else if (buspirate_fd == -1) return ERROR_OK; #endif @@ -1243,7 +1243,7 @@ static int buspirate_serial_setspeed(fd_handle_t fd, char speed, cc_t timeout) dcbSerialParams.ErrorChar = 0; // error replacement character dcbSerialParams.EofChar = 0; // end of input character dcbSerialParams.EvtChar = 0; // received event character - + if (!SetCommState(fd, &dcbSerialParams)) return -1; --