From bf85c62a8df4e9c198ff670414338946d88ed0de Mon Sep 17 00:00:00 2001 From: Felix Singer Date: Fri, 19 Aug 2022 02:44:28 +0200 Subject: [PATCH] flashrom.c: Retype appropriate variables with bool Use the bool type instead of an integer for appropriate variables, since this represents their purpose much better. Signed-off-by: Felix Singer Change-Id: I6629f391284c8f1266e4ba66c9976f3df43955d4 Reviewed-on: https://review.coreboot.org/c/flashrom/+/66883 Reviewed-by: Anastasia Klimchuk Tested-by: build bot (Jenkins) Reviewed-by: Angel Pons --- flashrom.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/flashrom.c b/flashrom.c index 0e2fa42ea..a1e4d2301 100644 --- a/flashrom.c +++ b/flashrom.c @@ -19,6 +19,7 @@ * GNU General Public License for more details. */ +#include #include #include #include @@ -61,7 +62,7 @@ static struct shutdown_func_data { /* Initialize to 0 to make sure nobody registers a shutdown function before * programmer init. */ -static int may_register_shutdown = 0; +static bool may_register_shutdown = false; /* Did we change something or was every erase/write skipped (if any)? */ static bool all_skipped = true; @@ -145,7 +146,7 @@ int programmer_init(const struct programmer_entry *prog, const char *param) /* Default to top aligned flash at 4 GB. */ flashbase = 0; /* Registering shutdown functions is now allowed. */ - may_register_shutdown = 1; + may_register_shutdown = true; /* Default to allowing writes. Broken programmers set this to 0. */ programmer_may_write = 1; @@ -181,7 +182,7 @@ int programmer_shutdown(void) int ret = 0; /* Registering shutdown functions is no longer allowed. */ - may_register_shutdown = 0; + may_register_shutdown = false; while (shutdown_fn_count > 0) { int i = --shutdown_fn_count; ret |= shutdown_fn[i].func(shutdown_fn[i].data); @@ -517,7 +518,7 @@ static unsigned int get_next_write(const uint8_t *have, const uint8_t *want, uns unsigned int *first_start, enum write_granularity gran) { - int need_write = 0; + bool need_write = false; unsigned int rel_start = 0, first_len = 0; unsigned int i, limit, stride; @@ -562,7 +563,7 @@ static unsigned int get_next_write(const uint8_t *have, const uint8_t *want, uns if (memcmp(have + i * stride, want + i * stride, limit)) { if (!need_write) { /* First location where have and want differ. */ - need_write = 1; + need_write = true; rel_start = i * stride; } } else {