nuttx: unify MIN, MAX and ABS macro definition across the code

Signed-off-by: Petro Karashchenko <petro.karashchenko@gmail.com>

arch/arm/src/imxrt/imxrt_mpuinit.c

@@ -41,12 +41,12 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#ifndef MAX
-#  define MAX(a,b) a > b ? a : b
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
-#ifndef MIN
-#  define MIN(a,b) a < b ? a : b
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 #ifndef CONFIG_ARMV7M_DCACHE

arch/arm/src/kinetis/kinetis_mpuinit.c

@@ -40,12 +40,12 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#ifndef MAX
-#  define MAX(a,b) a > b ? a : b
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
-#ifndef MIN
-#  define MIN(a,b) a < b ? a : b
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

arch/arm/src/kinetis/kinetis_usbdev.c

@@ -345,11 +345,11 @@ const struct trace_msg_t g_usb_trace_strings_deverror[] =
 /* Ever-present MIN and MAX macros */
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) (a > b ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /* Byte ordering in host-based values */

arch/arm/src/lpc17xx_40xx/lpc17_40_mpuinit.c

@@ -37,12 +37,12 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#ifndef MAX
-#  define MAX(a,b) a > b ? a : b
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
-#ifndef MIN
-#  define MIN(a,b) a < b ? a : b
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

arch/arm/src/lpc43xx/lpc43_mpuinit.c

@@ -37,12 +37,12 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#ifndef MAX
-#  define MAX(a,b) a > b ? a : b
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
-#ifndef MIN
-#  define MIN(a,b) a < b ? a : b
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

arch/arm/src/lpc54xx/lpc54_mpuinit.c

@@ -37,12 +37,12 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#ifndef MAX
-#  define MAX(a,b) a > b ? a : b
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
-#ifndef MIN
-#  define MIN(a,b) a < b ? a : b
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

arch/arm/src/max326xx/common/max326_mpuinit.c

@@ -37,12 +37,12 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#ifndef MAX
-#  define MAX(a,b) a > b ? a : b
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
-#ifndef MIN
-#  define MIN(a,b) a < b ? a : b
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

arch/arm/src/rtl8720c/amebaz_coex.c

@@ -30,6 +30,10 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
+#endif
+
 #define HCI_VENDOR_MAILBOX_CMD 0xfc8f
 
 /* static net_buf_simple *Xiaomi_buf; */
@@ -110,9 +114,6 @@ void bt_coex_handle_specific_evt(uint8_t *p, uint8_t len)
   rltk_coex_mailbox_to_wifi(p, len);
 }
 
-#ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
-#endif
 static const char *bt_hex_real(const void *buf, size_t len)
 {
   static const char hex[] = "0123456789abcdef";

arch/arm/src/sam34/sam_mpuinit.c

@@ -37,12 +37,12 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#ifndef MAX
-#  define MAX(a,b) a > b ? a : b
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
-#ifndef MIN
-#  define MIN(a,b) a < b ? a : b
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

arch/arm/src/sam34/sam_udp.c

@@ -178,11 +178,11 @@
 /* Ever-present MIN and MAX macros */
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) (a > b ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /* Byte ordering in host-based values */

arch/arm/src/sama5/sam_can.c

@@ -80,11 +80,11 @@
 /* Common definitions *******************************************************/
 
 #ifndef MIN
-#  define MIN(a,b) ((a < b) ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) ((a > b) ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /* Mailboxes ****************************************************************/

arch/arm/src/sama5/sam_hsmci.c

@@ -340,8 +340,13 @@
 
 /* Some semi-standard definitions */
 
-#define MAX(a,b) (((a) > (b)) ? (a) : (b))
-#define MIN(a,b) (((a) < (b)) ? (a) : (b))
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
+#endif
+
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
+#endif
 
 /****************************************************************************
  * Private Types

arch/arm/src/sama5/sam_udphs.c

@@ -210,11 +210,11 @@
 /* Ever-present MIN and MAX macros */
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) (a > b ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /* Byte ordering in host-based values */

arch/arm/src/samd2l2/sam_usb.c

@@ -214,11 +214,11 @@
 /* Ever-present MIN and MAX macros */
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) (a > b ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /* Byte ordering in host-based values */

arch/arm/src/samd5e5/sam_mpuinit.c

@@ -37,12 +37,12 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#ifndef MAX
-#  define MAX(a,b) a > b ? a : b
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
-#ifndef MIN
-#  define MIN(a,b) a < b ? a : b
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

arch/arm/src/samd5e5/sam_usb.c

@@ -236,11 +236,11 @@
 /* Ever-present MIN and MAX macros */
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) (a > b ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /* Byte ordering in host-based values */

arch/arm/src/samv7/sam_hsmci.c

@@ -273,8 +273,13 @@
 
 /* Some semi-standard definitions */
 
-#define MAX(a,b) (((a) > (b)) ? (a) : (b))
-#define MIN(a,b) (((a) < (b)) ? (a) : (b))
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
+#endif
+
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
+#endif
 
 /****************************************************************************
  * Private Types

arch/arm/src/samv7/sam_mcan.c

@@ -62,11 +62,11 @@
 /* Common definitions *******************************************************/
 
 #ifndef MIN
-#  define MIN(a,b) ((a < b) ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) ((a > b) ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /* Clock source *************************************************************/
@@ -131,7 +131,7 @@
 
 #  define MCAN0_TSEG1  (CONFIG_SAMV7_MCAN0_PROPSEG + CONFIG_SAMV7_MCAN0_PHASESEG1)
 #  define MCAN0_TSEG2  CONFIG_SAMV7_MCAN0_PHASESEG2
-#  define MCAN0_BRP    ((uint32_t)(((float) SAMV7_MCANCLK_FREQUENCY / \
+#  define MCAN0_BRP    ((uint32_t)(((float)SAMV7_MCANCLK_FREQUENCY / \
                        ((float)(MCAN0_TSEG1 + MCAN0_TSEG2 + 3) * \
                         (float)CONFIG_SAMV7_MCAN0_BITRATE)) - 1))
 #  define MCAN0_SJW    (CONFIG_SAMV7_MCAN0_FSJW - 1)
@@ -148,7 +148,7 @@
 
 #  define MCAN0_DTSEG1 (CONFIG_SAMV7_MCAN0_FPROPSEG + CONFIG_SAMV7_MCAN0_FPHASESEG1)
 #  define MCAN0_DTSEG2 (CONFIG_SAMV7_MCAN0_FPHASESEG2)
-#  define MCAN0_DBRP   ((uint32_t)(((float) SAMV7_MCANCLK_FREQUENCY / \
+#  define MCAN0_DBRP   ((uint32_t)(((float)SAMV7_MCANCLK_FREQUENCY / \
                        ((float)(MCAN0_DTSEG1 + MCAN0_DTSEG2 + 3) * \
                         (float)CONFIG_SAMV7_MCAN0_FBITRATE)) - 1))
 #  define MCAN0_DSJW   (CONFIG_SAMV7_MCAN0_FFSJW - 1)
@@ -416,7 +416,7 @@
 
 #  define MCAN1_TSEG1  (CONFIG_SAMV7_MCAN1_PROPSEG + CONFIG_SAMV7_MCAN1_PHASESEG1)
 #  define MCAN1_TSEG2  CONFIG_SAMV7_MCAN1_PHASESEG2
-#  define MCAN1_BRP    ((uint32_t)(((float) SAMV7_MCANCLK_FREQUENCY / \
+#  define MCAN1_BRP    ((uint32_t)(((float)SAMV7_MCANCLK_FREQUENCY / \
                        ((float)(MCAN1_TSEG1 + MCAN1_TSEG2 + 3) * \
                         (float)CONFIG_SAMV7_MCAN1_BITRATE)) - 1))
 #  define MCAN1_SJW    (CONFIG_SAMV7_MCAN1_FSJW - 1)
@@ -433,7 +433,7 @@
 
 #  define MCAN1_DTSEG1 (CONFIG_SAMV7_MCAN1_FPROPSEG + CONFIG_SAMV7_MCAN1_FPHASESEG1)
 #  define MCAN1_DTSEG2 (CONFIG_SAMV7_MCAN1_FPHASESEG2)
-#  define MCAN1_DBRP   ((uint32_t)(((float) SAMV7_MCANCLK_FREQUENCY / \
+#  define MCAN1_DBRP   ((uint32_t)(((float)SAMV7_MCANCLK_FREQUENCY / \
                        ((float)(MCAN1_DTSEG1 + MCAN1_DTSEG2 + 3) * \
                         (float)CONFIG_SAMV7_MCAN1_FBITRATE)) - 1))
 #  define MCAN1_DSJW   (CONFIG_SAMV7_MCAN1_FFSJW - 1)

arch/arm/src/samv7/sam_mpuinit.c

@@ -41,12 +41,12 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#ifndef MAX
-#  define MAX(a,b) a > b ? a : b
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
-#ifndef MIN
-#  define MIN(a,b) a < b ? a : b
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

arch/arm/src/samv7/sam_usbdevhs.c

@@ -274,11 +274,11 @@
 /* Ever-present MIN and MAX macros */
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) (a > b ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /* Byte ordering in host-based values */

arch/arm/src/stm32/stm32_mpuinit.c

@@ -37,12 +37,12 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#ifndef MAX
-#  define MAX(a,b) a > b ? a : b
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
-#ifndef MIN
-#  define MIN(a,b) a < b ? a : b
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

arch/arm/src/stm32/stm32_usbdev.c

@@ -235,11 +235,11 @@
 /* Ever-present MIN and MAX macros */
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) (a > b ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /* Byte ordering in host-based values */

arch/arm/src/stm32/stm32_usbfs.c

@@ -216,11 +216,11 @@
 /* Ever-present MIN and MAX macros */
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) (a > b ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /* Byte ordering in host-based values */

arch/arm/src/stm32f0l0g0/stm32_usbdev.c

@@ -220,11 +220,11 @@
 /* Ever-present MIN and MAX macros */
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) (a > b ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /* Byte ordering in host-based values */

arch/arm/src/stm32f7/stm32_mpuinit.c

@@ -37,12 +37,12 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#ifndef MAX
-#  define MAX(a,b) a > b ? a : b
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
-#ifndef MIN
-#  define MIN(a,b) a < b ? a : b
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

arch/arm/src/stm32h7/stm32_mpuinit.c

@@ -37,12 +37,12 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#ifndef MAX
-#  define MAX(a,b) a > b ? a : b
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
-#ifndef MIN
-#  define MIN(a,b) a < b ? a : b
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

arch/arm/src/stm32l4/stm32l4_mpuinit.c

@@ -37,12 +37,12 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#ifndef MAX
-#  define MAX(a,b) a > b ? a : b
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
-#ifndef MIN
-#  define MIN(a,b) a < b ? a : b
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

arch/arm/src/stm32l4/stm32l4_usbdev.c

@@ -220,11 +220,11 @@
 /* Ever-present MIN and MAX macros */
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) (a > b ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /* Byte ordering in host-based values */

arch/arm/src/stm32l5/stm32l5_mpuinit.c

@@ -37,12 +37,12 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#ifndef MAX
-#  define MAX(a,b) a > b ? a : b
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
-#ifndef MIN
-#  define MIN(a,b) a < b ? a : b
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

arch/arm/src/stm32u5/stm32_mpuinit.c

@@ -37,12 +37,12 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#ifndef MAX
-#  define MAX(a,b) a > b ? a : b
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
-#ifndef MIN
-#  define MIN(a,b) a < b ? a : b
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

arch/arm/src/stm32wl5/stm32wl5_mpuinit.c

@@ -37,12 +37,12 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#ifndef MAX
-#  define MAX(a,b) a > b ? a : b
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
-#ifndef MIN
-#  define MIN(a,b) a < b ? a : b
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

arch/arm/src/tiva/common/tiva_mpuinit.c

@@ -37,14 +37,13 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#ifndef MAX
-#  define MAX(a,b) a > b ? a : b
-#endif
-
 #ifndef MIN
-#  define MIN(a,b) a < b ? a : b
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
+#endif
 /****************************************************************************
  * Private Data
  ****************************************************************************/

arch/arm/src/xmc4/xmc4_mpuinit.c

@@ -37,12 +37,12 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#ifndef MAX
-#  define MAX(a,b) a > b ? a : b
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
-#ifndef MIN
-#  define MIN(a,b) a < b ? a : b
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

arch/mips/src/pic32mx/pic32mx_ethernet.c

@@ -271,13 +271,12 @@
 /* Ever-present MIN and MAX macros */
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) (a > b ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
-
 /****************************************************************************
  * Private Types
  ****************************************************************************/

arch/mips/src/pic32mx/pic32mx_usbdev.c

@@ -243,11 +243,11 @@
 /* Ever-present MIN and MAX macros */
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) (a > b ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /* Byte ordering in host-based values */

arch/risc-v/src/esp32c3/esp32c3_pm.c

@@ -75,7 +75,7 @@
 #define RTC_MODULE_SLEEP_PREPARE_CYCLES (6)
 
 #ifndef MAX
-#define MAX(a,b) a > b ? a : b
+#  define MAX(a,b) ((a) > (b) ? (a) : (b))
 #endif
 
 /* Time from VDD_SDIO power up to first flash read in ROM code */
@@ -1193,4 +1193,4 @@ uint32_t IRAM_ATTR esp32c3_pm_lockstatus(void)
   return pm_wakelock;
 }
 
-#endif /* CONFIG_PM */
+#endif /* CONFIG_PM */

arch/xtensa/src/esp32/esp32_dma.c

@@ -35,7 +35,7 @@
  ****************************************************************************/
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef ALIGN_UP

arch/xtensa/src/esp32/esp32_emac.c

@@ -170,7 +170,7 @@
 
 #ifdef MIN
 #  undef MIN
-#  define MIN(a,b) (((a)<(b))?(a):(b))
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 /* Check if current TX description is busy */

arch/xtensa/src/esp32/esp32_pm.c

@@ -66,7 +66,7 @@
 #define LIGHT_SLEEP_MIN_TIME_US    200
 
 #ifndef MAX
-#define MAX(a,b) a > b ? a : b
+#  define MAX(a,b) ((a) > (b) ? (a) : (b))
 #endif
 
 /* Time from VDD_SDIO power up to first flash read in ROM code */
@@ -1170,4 +1170,4 @@ uint32_t IRAM_ATTR esp32_pm_lockstatus(void)
   return pm_wakelock;
 }
 
-#endif /* CONFIG_PM */
+#endif /* CONFIG_PM */

binfmt/elf.c

@@ -61,7 +61,7 @@
 #endif
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

binfmt/nxflat.c

@@ -58,7 +58,7 @@
 #endif
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

boards/arm/sam34/sam4l-xplained/src/sam_slcd.c

@@ -71,11 +71,11 @@
 /* The ever-present MIN/MAX macros ******************************************/
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) (a > b ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /* LCD **********************************************************************/

boards/arm/stm32/stm3210e-eval/src/stm32_pmbuttons.c

@@ -99,10 +99,11 @@
 /* Miscellaneous Definitions ************************************************/
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
+
 #ifndef MAX
-#  define MAX(a,b) (a > b ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 #define MIN_BUTTON MIN(CONFIG_PM_BUTTONS_MIN, CONFIG_PM_IRQBUTTONS_MIN)

boards/arm/stm32/stm32ldiscovery/src/stm32_lcd.c

@@ -74,11 +74,11 @@
 /* The ever-present MIN/MAX macros ******************************************/
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) (a > b ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /* LCD **********************************************************************/

boards/mips/pic32mx/sure-pic32mx/src/pic32mx_lcd1602.c

@@ -100,11 +100,11 @@
 /* The ever-present MIN/MAX macros ******************************************/
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) (a > b ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /* LCD **********************************************************************/

drivers/lcd/st7735.c

@@ -45,7 +45,9 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#define MAX(a,b)    ((a)>(b)?(a):(b))
+#ifndef MAX
+#  define MAX(a,b) ((a) > (b) ? (a) : (b))
+#endif
 
 /* Verify that all configuration requirements have been met */
 

drivers/usbdev/cdcacm.h

@@ -170,11 +170,11 @@
 /* MIN/MAX macros */
 
 #ifndef MIN
-#  define MIN(a,b) ((a)<(b)?(a):(b))
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) ((a)>(b)?(a):(b))
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /* Trace values *************************************************************/

drivers/usbdev/cdcecm.h

@@ -58,7 +58,7 @@
 #define CDCECM_REMOTEWAKEUP      (0)
 
 #ifndef MIN
-#  define MIN(a,b) ((a)<(b)?(a):(b))
+#  define MIN(a,b) ((a) < (b) ? (a) : (b))
 #endif
 
 #endif /* __DRIVERS_USBDEV_CDCECM_H */

graphics/nxbe/nxbe_setsize.c

@@ -46,11 +46,11 @@
  ****************************************************************************/
 
 #ifndef MIN
-#  define MIN(a,b) ((a < b) ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b) ((a > b) ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

libs/libc/fixedmath/lib_b16atan2.c

@@ -37,16 +37,16 @@
 #define B16_HALFPI 0x00019220 /* 1.570796327 */
 #define B16_PI     0x00032440 /* 3.141592654 */
 
-#ifndef MAX
-#  define MAX(a,b) (a > b ? a : b)
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
-#ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#ifndef MAX
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 #ifndef ABS
-#  define ABS(a)   (a < 0 ? -a : a)
+#  define ABS(a)   ((a) < 0 ? -(a) : (a))
 #endif
 
 /****************************************************************************

libs/libc/stdio/lib_libdtoa.c

@@ -59,11 +59,11 @@
  ****************************************************************************/
 
 #ifndef MIN
-#  define MIN(a,b)          (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 #ifndef MAX
-#  define MAX(a,b)          (a > b ? a : b)
+#  define MAX(a,b) (((a) > (b)) ? (a) : (b))
 #endif
 
 /* Use the maximim precision with %g format if no precision is specified.

libs/libdsp/lib_misc_b16.c

@@ -32,7 +32,7 @@
 #define FAST_ATAN2_SMALLNUM       (1)
 
 #ifndef ABS
-#  define ABS(a)   (a < 0 ? -a : a)
+#  define ABS(a)   ((a) < 0 ? -(a) : (a))
 #endif
 
 /****************************************************************************

sched/module/mod_insmod.c

@@ -43,7 +43,7 @@
  ****************************************************************************/
 
 #ifndef MIN
-#  define MIN(a,b) (a < b ? a : b)
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
 #endif
 
 /****************************************************************************

tools/rp2040/make_flash_fs.c

@@ -38,7 +38,9 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
-#define MIN(a,b) (((a) < (b)) ? (a) : (b))
+#ifndef MIN
+#  define MIN(a,b) (((a) < (b)) ? (a) : (b))
+#endif
 
 #define MAX_NAME_LEN     16
 #define MAX_SECTOR_DATA  (1024 - 10)