[S] Change in flashrom[master]: cli_classic.c: Make count_max_decode_exceedings() pure

[Edward O'Callaghan (Code Review)]

Edward O'Callaghan has submitted this change. ( https://review.coreboot.org/c/flashrom/+/68479 )

(

5 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: Make count_max_decode_exceedings() pure
......................................................................

cli_classic.c: Make count_max_decode_exceedings() pure

Pass by argument the max_rom_decode structure such that the
function is pure and defined upon its parameters.

Note, unfortunately a itermediate step of a '_' suffix is
required for the 'max_rom_decode' parameter as to not alias
the global symbol within the function body.

Change-Id: Ia01f77993deab68e251850008e885828e55b9462
Signed-off-by: Edward O'Callaghan 
Reviewed-on: https://review.coreboot.org/c/flashrom/+/68479
Reviewed-by: Angel Pons 
Reviewed-by: Felix Singer 
Tested-by: build bot (Jenkins) 
---
M cli_classic.c
1 file changed, 32 insertions(+), 10 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Felix Singer: Looks good to me, approved
  Angel Pons: Looks good to me, approved

diff --git a/cli_classic.c b/cli_classic.c
index fe50cf4..5cc14a0 100644
--- a/cli_classic.c
+++ b/cli_classic.c
@@ -520,43 +520,44 @@

/* Returns the number of buses commonly supported by the current programmer and flash chip where the latter
  * can not be completely accessed due to size/address limits of the programmer. */
-static unsigned int count_max_decode_exceedings(const struct flashctx *flash)
+static unsigned int count_max_decode_exceedings(const struct flashctx *flash,
+		const struct decode_sizes *max_rom_decode_)
 {
    unsigned int limitexceeded = 0;
    uint32_t size = flash->chip->total_size * 1024;
    enum chipbustype buses = flash->mst->buses_supported & flash->chip->bustype;

-	if ((buses & BUS_PARALLEL) && (max_rom_decode.parallel < size)) {
+	if ((buses & BUS_PARALLEL) && (max_rom_decode_->parallel < size)) {
    	limitexceeded++;
    	msg_pdbg("Chip size %u kB is bigger than supported "
    		 "size %u kB of chipset/board/programmer "
    		 "for %s interface, "
    		 "probe/read/erase/write may fail. ", size / 1024,
-			 max_rom_decode.parallel / 1024, "Parallel");
+			 max_rom_decode_->parallel / 1024, "Parallel");
    }
-	if ((buses & BUS_LPC) && (max_rom_decode.lpc < size)) {
+	if ((buses & BUS_LPC) && (max_rom_decode_->lpc < size)) {
    	limitexceeded++;
    	msg_pdbg("Chip size %u kB is bigger than supported "
    		 "size %u kB of chipset/board/programmer "
    		 "for %s interface, "
    		 "probe/read/erase/write may fail. ", size / 1024,
-			 max_rom_decode.lpc / 1024, "LPC");
+			 max_rom_decode_->lpc / 1024, "LPC");
    }
-	if ((buses & BUS_FWH) && (max_rom_decode.fwh < size)) {
+	if ((buses & BUS_FWH) && (max_rom_decode_->fwh < size)) {
    	limitexceeded++;
    	msg_pdbg("Chip size %u kB is bigger than supported "
    		 "size %u kB of chipset/board/programmer "
    		 "for %s interface, "
    		 "probe/read/erase/write may fail. ", size / 1024,
-			 max_rom_decode.fwh / 1024, "FWH");
+			 max_rom_decode_->fwh / 1024, "FWH");
    }
-	if ((buses & BUS_SPI) && (max_rom_decode.spi < size)) {
+	if ((buses & BUS_SPI) && (max_rom_decode_->spi < size)) {
    	limitexceeded++;
    	msg_pdbg("Chip size %u kB is bigger than supported "
    		 "size %u kB of chipset/board/programmer "
    		 "for %s interface, "
    		 "probe/read/erase/write may fail. ", size / 1024,
-			 max_rom_decode.spi / 1024, "SPI");
+			 max_rom_decode_->spi / 1024, "SPI");
    }
    return limitexceeded;
 }
@@ -1060,7 +1061,7 @@

print_chip_support_status(fill_flash->chip);

-	unsigned int limitexceeded = count_max_decode_exceedings(fill_flash);
+	unsigned int limitexceeded = count_max_decode_exceedings(fill_flash, &max_rom_decode);
    if (limitexceeded > 0 && !force) {
    	enum chipbustype commonbuses = fill_flash->mst->buses_supported & fill_flash->chip->bustype;