jtagspi: fix build on MacOS
authorAntonio Borneo <borneo.antonio@gmail.com>
Sat, 13 Nov 2021 13:38:21 +0000 (14:38 +0100)
committerAntonio Borneo <borneo.antonio@gmail.com>
Sat, 13 Nov 2021 22:36:03 +0000 (22:36 +0000)
Commit be57b0ab847e ("Update jtagspi driver for 1-, 2- and 4-byte
addresses") introduces two incorrect format string for uint32_t
data types.
This cause build failure on MacOS:

src/flash/nor/jtagspi.c:474:35: error: format specifies type 'unsigned char'
but the argument has type 'uint32_t' (aka 'unsigned int') [-Werror,-Wformat]
        LOG_DEBUG("status=0x%02" PRIx8, *status);
        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~

src/flash/nor/jtagspi.c:513:65: error: format specifies type 'unsigned char'
but the argument has type 'uint32_t' (aka 'unsigned int') [-Werror,-Wformat]
        LOG_ERROR("Cannot enable write to flash. Status=0x%02" PRIx8, status);
        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~

Fix the format string.

Change-Id: I209053317c8b26c35c6f11be0553ccccc698c551
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Fixes: be57b0ab847e ("Update jtagspi driver for 1-, 2- and 4-byte addresses")
Reviewed-on: https://review.openocd.org/c/openocd/+/6701
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
Tested-by: jenkins
src/flash/nor/jtagspi.c

index 9d377ce704a373b2ae6ecfd6a2d571366eee02c8..a5672c63e99186d803592c783c1897d028edf6a6 100644 (file)
@@ -471,7 +471,7 @@ static int jtagspi_read_status(struct flash_bank *bank, uint32_t *status)
        int err = jtagspi_cmd(bank, SPIFLASH_READ_STATUS, NULL, 0, &buf, -1);
        if (err == ERROR_OK) {
                *status = buf;
-               LOG_DEBUG("status=0x%02" PRIx8, *status);
+               LOG_DEBUG("status=0x%02" PRIx32, *status);
        }
        return err;
 }
@@ -510,7 +510,7 @@ static int jtagspi_write_enable(struct flash_bank *bank)
                return retval;
 
        if ((status & SPIFLASH_WE_BIT) == 0) {
-               LOG_ERROR("Cannot enable write to flash. Status=0x%02" PRIx8, status);
+               LOG_ERROR("Cannot enable write to flash. Status=0x%02" PRIx32, status);
                return ERROR_FAIL;
        }
        return ERROR_OK;