From cd47560f2ae0758012001d6d954653b4526811ad Mon Sep 17 00:00:00 2001
From: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Date: Mon, 7 Jul 2014 21:08:23 +1000
Subject: vendorcode/amd/agesa: Use macros already defined in stdlib.h

We already have these macros define in 'stdlib.h'. Make good use of them
here to avoid redefinition conflicts of the pre-processor depending on
header inclusion ordering. This has the nice side-effect of syncing up
AGESA families in this particular regard.

Change-Id: Icf911629a4a1a82b01062fe16af4c8f812b05717
Signed-off-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Reviewed-on: http://review.coreboot.org/6199
Tested-by: build bot (Jenkins)
Reviewed-by: Bruce Griffith <Bruce.Griffith@se-eng.com>
---
 .../amd/agesa/f16kb/Proc/GNB/Common/Gnb.h          | 10 ++-----
 src/vendorcode/amd/agesa/f16kb/Proc/Mem/mu.h       | 31 +---------------------
 2 files changed, 3 insertions(+), 38 deletions(-)

(limited to 'src/vendorcode/amd/agesa/f16kb/Proc')

diff --git a/src/vendorcode/amd/agesa/f16kb/Proc/GNB/Common/Gnb.h b/src/vendorcode/amd/agesa/f16kb/Proc/GNB/Common/Gnb.h
index 100ae03036..c01fd5ef98 100644
--- a/src/vendorcode/amd/agesa/f16kb/Proc/GNB/Common/Gnb.h
+++ b/src/vendorcode/amd/agesa/f16kb/Proc/GNB/Common/Gnb.h
@@ -45,6 +45,8 @@
 #ifndef _GNB_H_
 #define _GNB_H_
 
+#include <stdlib.h>
+
 #include  "Ids.h"
 
 #pragma pack (push, 1)
@@ -71,14 +73,6 @@
   #endif
 #endif
 
-#ifndef MIN
-  #define MIN(x, y) (((x) > (y))? (y):(x))
-#endif
-
-#ifndef MAX
-  #define MAX(x, y) (((x) > (y))? (x):(y))
-#endif
-
 #define OFF 0
 
 #define PVOID UINT32
diff --git a/src/vendorcode/amd/agesa/f16kb/Proc/Mem/mu.h b/src/vendorcode/amd/agesa/f16kb/Proc/Mem/mu.h
index 1489fe8b5a..f415e524e0 100644
--- a/src/vendorcode/amd/agesa/f16kb/Proc/Mem/mu.h
+++ b/src/vendorcode/amd/agesa/f16kb/Proc/Mem/mu.h
@@ -60,36 +60,7 @@
   #define PSO_ENTRY UINT8
 #endif
 
-//
-// Maximum value macro
-//
-#ifndef MAX
-  #define MAX(X, Y) (((X) < (Y)) ? (Y) : (X))
-#endif
-//
-// Minimum Value macro
-//
-#ifndef MIN
-  #define MIN(X, Y) (((X) < (Y)) ? (X) : (Y))
-#endif
-//
-// Absolute Value Macro
-//
-#ifndef ABS
-  #define ABS(X) (((X) < 0) ? (-(X)) : (X))
-#endif
-//
-// Taking ceiling of (a / b)
-//
-#ifndef CEIL_DIV
-  #define CEIL_DIV(a, b)  (((a) + (b) - 1) / (b))
-#endif
-//
-// Check if value x is a power of 2 or not
-//
-#ifndef IS_POWER_OF_2
-  #define IS_POWER_OF_2(x)  (((x) & ((x) - 1)) == 0)
-#endif
+#include <stdlib.h>
 //
 // Find offset of a member variable in a struct
 //
-- 
cgit v1.2.3