numbers.h: downgrade MIN() and MAX() to unsafe versions

For compatibility with Zephyr, MIN and MAX macros are downgraded to the
basic, compile-time, standard compliant and unsafe implementation.

This means the code will now be the same whether it's compiled with Zephyr
versus not.

Fixes zephyrproject-rtos/zephyr#33567 which
means it's now possible to use MIN() or MAX() in either Zephyr's
BUILD_ASSERT() or in SOF's equivalent STATIC_ASSERT() or in C11's
_Static_assert()

This implementation is unsafe because it can evaluate its arguments
twice which is obviously not desired when the argument has side-effects:

https://wiki.sei.cmu.edu/confluence/display/c/PRE31-C.+Avoid+side+effects+in+arguments+to+unsafe+macros

Signed-off-by: Marc Herbert <marc.herbert@intel.com>

Author: marc-hb

src/include/sof/math/numbers.h

@@ -16,23 +16,17 @@
 #ifdef MIN
 #undef MIN
 #endif
-
-#define MIN(a, b) ({		\
-	typeof(a) __a = (a);	\
-	typeof(b) __b = (b);	\
-	__a > __b ? __b : __a;	\
-})
+/* Unsafe and portable macros for consistency with Zephyr.
+ * See SEI CERT-C PRE31-C
+ */
+#define MIN(a, b) ((a) < (b) ? (a) : (b))
 
 /* Zephyr defines this - remove local copy once Zephyr integration complete */
 #ifdef MAX
 #undef MAX
 #endif
+#define MAX(a, b) ((a) < (b) ? (b) : (a))
 
-#define MAX(a, b) ({		\
-	typeof(a) __a = (a);	\
-	typeof(b) __b = (b);	\
-	__a < __b ? __b : __a;	\
-})
 #define ABS(a) ({		\
 	typeof(a) __a = (a);	\
 	__a < 0 ? -__a : __a;	\