[flashrom-gerrit] [S] Change in flashrom[master]: cli_classic.c: Be consistent with pointer types

[Anastasia Klimchuk (Code Review)]

Anastasia Klimchuk has submitted this change. ( https://review.coreboot.org/c/flashrom/+/69451 )

(

1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.
 )Change subject: cli_classic.c: Be consistent with pointer types
......................................................................

cli_classic.c: Be consistent with pointer types

With `i586-pc-msdosdjgpp-gcc (GCC) 12.2.0`, `uint32_t` is defined as
`long unsigned int`, which is not the same as `unsigned int`. As the
`flashrom_layout_get_region_range()` function is part of libflashrom
API, adjust `cli_classic.c` instead to avoid type mismatches.

Change-Id: Ie8f5bc0d9296f7c6b8f8a351b53052f5fe86b09d
Signed-off-by: Angel Pons 
Reviewed-on: https://review.coreboot.org/c/flashrom/+/69451
Reviewed-by: Arthur Heymans 
Reviewed-by: Evan Benn 
Reviewed-by: Nikolai Artemiev 
Tested-by: build bot (Jenkins) 
---
M cli_classic.c
1 file changed, 22 insertions(+), 2 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Arthur Heymans: Looks good to me, approved
  Nikolai Artemiev: Looks good to me, but someone else must approve
  Evan Benn: Looks good to me, but someone else must approve

diff --git a/cli_classic.c b/cli_classic.c
index bd98a07..fe50cf4 100644
--- a/cli_classic.c
+++ b/cli_classic.c
@@ -127,7 +127,7 @@
    return true;
 }

-static int parse_wp_range(uint32_t *start, uint32_t *len)
+static int parse_wp_range(unsigned int *start, unsigned int *len)
 {
    char *endptr = NULL, *token = NULL;

@@ -571,7 +571,7 @@
    int namelen, opt, i, j;
    int startchip = -1, chipcount = 0, option_index = 0;
    int operation_specified = 0;
-	uint32_t wp_start = 0, wp_len = 0;
+	unsigned int wp_start = 0, wp_len = 0;
    bool force = false, ifd = false, fmap = false;
 #if CONFIG_PRINT_WIKI == 1
    bool list_supported_wiki = false;