From 6b70aacdcded47e61b7830f9aa366743afb650cd Mon Sep 17 00:00:00 2001 From: Felix Singer Date: Fri, 19 Aug 2022 01:02:42 +0200 Subject: serprog.c: Retype variable `have_device` with bool Use the bool type instead of an integer for the variable `have_device`, since this represents its purpose much better. Signed-off-by: Felix Singer Change-Id: Ia12f1503ae3f7be20a065f22b4ab84c4140cd23e Reviewed-on: https://review.coreboot.org/c/flashrom/+/66878 Reviewed-by: Anastasia Klimchuk Tested-by: build bot (Jenkins) Reviewed-by: Angel Pons --- serprog.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'serprog.c') diff --git a/serprog.c b/serprog.c index c68449a7..57196a98 100644 --- a/serprog.c +++ b/serprog.c @@ -15,6 +15,7 @@ * GNU General Public License for more details. */ +#include #include #if ! IS_WINDOWS /* stuff (presumably) needed for sockets only */ #include @@ -571,7 +572,7 @@ static int serprog_init(const struct programmer_cfg *cfg) unsigned char rbuf[3]; unsigned char c; char *device; - int have_device = 0; + bool have_device = false; /* the parameter is either of format "dev=/dev/device[:baud]" or "ip=ip:port" */ device = extract_programmer_param_str(cfg, "dev"); @@ -598,7 +599,7 @@ static int serprog_init(const struct programmer_cfg *cfg) free(device); return 1; } - have_device++; + have_device = true; } } @@ -637,7 +638,7 @@ static int serprog_init(const struct programmer_cfg *cfg) free(device); return 1; } - have_device++; + have_device = true; } } if (device && !strlen(device)) { -- cgit v1.2.3