This is an automated email from Gerrit.

Tarek BOCHKATI (tarek.bouchk...@gmail.com) just uploaded a new patch set to 
Gerrit, which you can find at http://openocd.zylin.com/5940

-- gerrit

commit 0fb5c803c2fcd01e3fddbea2dc39699bba77cf4e
Author: Tarek BOCHKATI <tarek.bouchk...@st.com>
Date:   Fri Nov 20 20:25:38 2020 +0100

    flash/nor/sfdp: fix build issue with clang on mac OS
    
    Change-Id: I3b3aa4236125523ad65fd615ada0f5647d26f526
    Signed-off-by: Tarek BOCHKATI <tarek.bouchk...@st.com>

diff --git a/src/flash/nor/sfdp.c b/src/flash/nor/sfdp.c
index 2fbc37d..fb98618 100644
--- a/src/flash/nor/sfdp.c
+++ b/src/flash/nor/sfdp.c
@@ -91,7 +91,7 @@ int spi_sfdp(struct flash_bank *bank, struct flash_device 
*dev,
        }
        if (((header.revision >> 24) & 0xFF) != SFDP_ACCESS_PROT) {
                LOG_ERROR("access protocol 0x%02" PRIx8 " not implemented",
-                       (header.revision >> 24) & 0xFF);
+                       (uint8_t) (header.revision >> 24));
                return ERROR_FLASH_BANK_NOT_PROBED;
        }
 
diff --git a/src/flash/nor/stmqspi.c b/src/flash/nor/stmqspi.c
index 11529f4..85b7cb1 100644
--- a/src/flash/nor/stmqspi.c
+++ b/src/flash/nor/stmqspi.c
@@ -456,7 +456,7 @@ static int qspi_write_enable(struct flash_bank *bank)
                != BIT(SPI_FSEL_FLASH))
                if ((status & (SPIFLASH_WE_BIT | SPIFLASH_BSY_BIT)) != 
SPIFLASH_WE_BIT) {
                        LOG_ERROR("Cannot write enable flash1. Status=0x%02" 
PRIx8,
-                               status & 0xFF);
+                               (uint8_t) status);
                        return ERROR_FLASH_OPERATION_FAILED;
                }
 
@@ -465,7 +465,7 @@ static int qspi_write_enable(struct flash_bank *bank)
        if ((stmqspi_info->saved_cr & (BIT(SPI_DUAL_FLASH) | 
BIT(SPI_FSEL_FLASH))) != 0)
                if ((status & (SPIFLASH_WE_BIT | SPIFLASH_BSY_BIT)) != 
SPIFLASH_WE_BIT) {
                        LOG_ERROR("Cannot write enable flash2. Status=0x%02" 
PRIx8,
-                               status & 0xFF);
+                               (uint8_t) status);
                        return ERROR_FLASH_OPERATION_FAILED;
                }
 
@@ -549,7 +549,7 @@ COMMAND_HANDLER(stmqspi_handle_mass_erase_command)
                != BIT(SPI_FSEL_FLASH)) && ((status & SPIFLASH_BSY_BIT) == 0) &&
                ((status & SPIFLASH_WE_BIT) != 0)) {
                LOG_ERROR("Mass erase command not accepted by flash1. 
Status=0x%02" PRIx8,
-                       status & 0xFF);
+                       (uint8_t) status);
                retval = ERROR_FLASH_OPERATION_FAILED;
                goto err;
        }
@@ -560,7 +560,7 @@ COMMAND_HANDLER(stmqspi_handle_mass_erase_command)
                ((status & SPIFLASH_BSY_BIT) == 0) &&
                ((status & SPIFLASH_WE_BIT) != 0)) {
                LOG_ERROR("Mass erase command not accepted by flash2. 
Status=0x%02" PRIx8,
-                       status & 0xFF);
+                       (uint8_t) status);
                retval = ERROR_FLASH_OPERATION_FAILED;
                goto err;
        }
@@ -960,7 +960,7 @@ static int qspi_erase_sector(struct flash_bank *bank, 
unsigned int sector)
                != BIT(SPI_FSEL_FLASH)) && ((status & SPIFLASH_BSY_BIT) == 0) &&
                ((status & SPIFLASH_WE_BIT) != 0)) {
                LOG_ERROR("Sector erase command not accepted by flash1. 
Status=0x%02" PRIx8,
-                       status & 0xFF);
+                       (uint8_t) status);
                retval = ERROR_FLASH_OPERATION_FAILED;
                goto err;
        }
@@ -972,7 +972,7 @@ static int qspi_erase_sector(struct flash_bank *bank, 
unsigned int sector)
                ((status & SPIFLASH_BSY_BIT) == 0) &&
                ((status & SPIFLASH_WE_BIT) != 0)) {
                LOG_ERROR("Sector erase command not accepted by flash2. 
Status=0x%02" PRIx8,
-                       status & 0xFF);
+                       (uint8_t) status);
                retval = ERROR_FLASH_OPERATION_FAILED;
                goto err;
        }
@@ -1214,7 +1214,7 @@ static int stmqspi_blank_check(struct flash_bank *bank)
                        /* we need le_32_to_h, but that's the same as 
h_to_le_32 */
                        result = h_to_le_32(erase_check_info.result);
                        bank->sectors[sector + index].is_erased = ((result & 
0xFF) == 0xFF);
-                       LOG_DEBUG("Flash sector %u checked: 0x%04" PRIx16, 
sector + index, result & 0xFFFF);
+                       LOG_DEBUG("Flash sector %u checked: 0x%04" PRIx16, 
sector + index, (uint16_t) result);
                }
                keep_alive();
                sector += count;

-- 


_______________________________________________
OpenOCD-devel mailing list
OpenOCD-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to