[coreboot-gerrit] Change in coreboot[master]: src/include: Remove min/max() from <stdlib.h>

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/37378 )

Change subject: src/include: Remove min/max() from <stdlib.h>
......................................................................

src/include: Remove min/max() from <stdlib.h>

Change-Id: I9ded44422a267e244343502dd5d6ab355e5a788d
Signed-off-by: Elyes HAOUAS 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/37378
Reviewed-by: Patrick Georgi 
Reviewed-by: Kyösti Mälkki 
Tested-by: build bot (Jenkins) 
---
M src/include/stdlib.h
M src/vendorcode/cavium/bdk/libdram/dram-util.h
2 files changed, 0 insertions(+), 6 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Patrick Georgi: Looks good to me, approved
  Kyösti Mälkki: Looks good to me, but someone else must approve

diff --git a/src/include/stdlib.h b/src/include/stdlib.h
index 34ef93a..a8297f8 100644
--- a/src/include/stdlib.h
+++ b/src/include/stdlib.h
@@ -3,9 +3,6 @@

#include <stddef.h>

-#define min(a, b) MIN((a), (b))
-#define max(a, b) MAX((a), (b))
-
 void *memalign(size_t boundary, size_t size);
 void *malloc(size_t size);
 /* We never free memory */
diff --git a/src/vendorcode/cavium/bdk/libdram/dram-util.h b/src/vendorcode/cavium/bdk/libdram/dram-util.h
index c9a96ba..f8ab6c1 100644
--- a/src/vendorcode/cavium/bdk/libdram/dram-util.h
+++ b/src/vendorcode/cavium/bdk/libdram/dram-util.h
@@ -42,8 +42,6 @@
  * are not meant for users's of the libdram API.
  */

-#if 0
-/* FIXME(dhendrix): min/max are defined in stdlib.h */
 /**
  * Standard min(a,b) macro
  */
@@ -58,7 +56,6 @@
 #define max(X, Y)				\
    ({ typeof (X) __x = (X); typeof(Y) __y = (Y);	\
    	(__x > __y) ? __x : __y; })
-#endif

/**
  * Absolute value of an integer