From 8323e9a7f56c4aa377a0a50717b05e12886606c2 Mon Sep 17 00:00:00 2001 From: "Sadik.Ozer" Date: Mon, 28 Mar 2022 15:58:27 +0300 Subject: [PATCH] Fix GCC_ARM warnings Signed-off-by: Sadik.Ozer --- .../Libraries/PeriphDrivers/Source/RTC/rtc_reva.c | 2 +- .../Libraries/PeriphDrivers/Source/SPI/spi_me15.c | 12 ++---------- .../Libraries/PeriphDrivers/Source/SPI/spi_reva.c | 5 ++++- .../Libraries/PeriphDrivers/Source/UART/uart_me15.c | 6 +++--- .../Libraries/PeriphDrivers/Source/UART/uart_revb.c | 2 +- 5 files changed, 11 insertions(+), 16 deletions(-) diff --git a/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/RTC/rtc_reva.c b/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/RTC/rtc_reva.c index 20000413d8..8967e93cb4 100644 --- a/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/RTC/rtc_reva.c +++ b/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/RTC/rtc_reva.c @@ -325,7 +325,7 @@ int MXC_RTC_RevA_GetTime(uint32_t* sec, uint32_t* subsec) // Read the seconds count. temp_sec = MXC_RTC_RevA_GetSecond(); - if (temp_sec == E_BUSY) { + if ((int)temp_sec == E_BUSY) { return E_BUSY; } diff --git a/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/SPI/spi_me15.c b/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/SPI/spi_me15.c index 1e77c0da1c..c2e0a1e93a 100644 --- a/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/SPI/spi_me15.c +++ b/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/SPI/spi_me15.c @@ -51,11 +51,7 @@ int MXC_SPI_Init(mxc_spi_regs_t* spi, int masterMode, int quadModeUsed, int numSlaves, unsigned ssPolarity, unsigned int hz, unsigned int drv_ssel) { - int spi_num; - - spi_num = MXC_SPI_GET_IDX(spi); - MXC_ASSERT(spi_num >= 0); - + if (numSlaves > MXC_SPI_SS_INSTANCES) { return E_BAD_PARAM; } @@ -91,11 +87,7 @@ int MXC_SPI_Init(mxc_spi_regs_t* spi, int masterMode, int quadModeUsed, int numS } int MXC_SPI_Shutdown(mxc_spi_regs_t* spi) -{ - int spi_num; - spi_num = MXC_SPI_GET_IDX(spi); - MXC_ASSERT(spi_num >= 0); - +{ MXC_SPI_RevA_Shutdown ((mxc_spi_reva_regs_t*) spi); if (spi == MXC_SPI0) { diff --git a/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/SPI/spi_reva.c b/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/SPI/spi_reva.c index 715b6752c1..174817d21f 100644 --- a/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/SPI/spi_reva.c +++ b/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/SPI/spi_reva.c @@ -803,7 +803,10 @@ uint32_t MXC_SPI_RevA_TransHandler (mxc_spi_reva_regs_t *spi, mxc_spi_reva_req_t uint32_t tx_length = 0, rx_length = 0; uint8_t bits; spi_num = MXC_SPI_GET_IDX ((mxc_spi_regs_t*) spi); - + if (spi_num == -1) { + return E_BAD_PARAM; + } + bits = MXC_SPI_GetDataSize ((mxc_spi_regs_t*) req->spi); //MXC_F_SPI_REVA_CTRL2_NUMBITS data bits diff --git a/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/UART/uart_me15.c b/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/UART/uart_me15.c index 361507566c..691ed1f5c2 100644 --- a/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/UART/uart_me15.c +++ b/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/UART/uart_me15.c @@ -167,8 +167,8 @@ int MXC_UART_ReadyForSleep(mxc_uart_regs_t* uart) int MXC_UART_SetFrequency(mxc_uart_regs_t* uart, unsigned int baud, mxc_uart_clock_t clock) { int freq; - int mod = 0; - int clkdiv = 0; + unsigned int mod = 0; + unsigned int clkdiv = 0; int div = 8; if (MXC_UART_GET_IDX (uart) < 0) { @@ -230,7 +230,7 @@ int MXC_UART_SetFrequency(mxc_uart_regs_t* uart, unsigned int baud, mxc_uart_clo freq = MXC_UART_GetFrequency (uart); } else { - int clkDiv = 0, mod = 0; + unsigned int clkDiv = 0, mod = 0; if (MXC_UART_GET_IDX ((mxc_uart_regs_t*) uart) < 0) { return E_BAD_PARAM; } diff --git a/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/UART/uart_revb.c b/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/UART/uart_revb.c index 5c9acefad8..e60f69338e 100644 --- a/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/UART/uart_revb.c +++ b/targets/TARGET_Maxim/TARGET_MAX32670/Libraries/PeriphDrivers/Source/UART/uart_revb.c @@ -115,7 +115,7 @@ int MXC_UART_RevB_ReadyForSleep (mxc_uart_revb_regs_t* uart) int MXC_UART_RevB_SetFrequency (mxc_uart_revb_regs_t* uart, unsigned int baud, mxc_uart_revb_clock_t clock) { - int clkDiv = 0, mod = 0; + unsigned int clkDiv = 0, mod = 0; if (MXC_UART_GET_IDX ((mxc_uart_regs_t*) uart) < 0) { return E_BAD_PARAM; }