From fa97cefbb3fad90573459e57845b658c9d3351a2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C5=81ukasz=20Dobrowolski?= <lukasz@dobrowolski.io>
Date: Sat, 12 Nov 2016 11:36:59 +0100
Subject: vendorcode/amd/agesa: Remove flawed warning
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The compilation would fail if CFG_MEMORY_LRDIMM_CAPABLE == FALSE
and BLDOPT_REMOVE_LRDIMMS_SUPPORT == TRUE.

Change-Id: I1be37e368bc4ed07e59d0f0bb967bed11143a65b
Signed-off-by: Łukasz Dobrowolski <lukasz@dobrowolski.io>
Reviewed-on: https://review.coreboot.org/17354
Tested-by: build bot (Jenkins)
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
---
 src/vendorcode/amd/agesa/f12/Include/PlatformInstall.h   | 6 ------
 src/vendorcode/amd/agesa/f14/Include/PlatformInstall.h   | 6 ------
 src/vendorcode/amd/agesa/f15/Include/PlatformInstall.h   | 6 ------
 src/vendorcode/amd/agesa/f15tn/Include/PlatformInstall.h | 6 ------
 src/vendorcode/amd/agesa/f16kb/Include/PlatformInstall.h | 6 ------
 5 files changed, 30 deletions(-)

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

diff --git a/src/vendorcode/amd/agesa/f12/Include/PlatformInstall.h b/src/vendorcode/amd/agesa/f12/Include/PlatformInstall.h
index 171adb52af..1fe116a6c7 100644
--- a/src/vendorcode/amd/agesa/f12/Include/PlatformInstall.h
+++ b/src/vendorcode/amd/agesa/f12/Include/PlatformInstall.h
@@ -2056,12 +2056,6 @@ CONST UINT32 ROMDATA AmdPlatformTypeCgf = CFG_AMD_PLATFORM_TYPE;
     #endif
   #endif
 #endif
-// Check LRDIMM CODE and LRDIMM CFG item
-#if CFG_MEMORY_LRDIMM_CAPABLE == FALSE
-  #if  BLDOPT_REMOVE_LRDIMMS_SUPPORT == TRUE
-    #error   Warning: LRDIMM capability is false, but LRIDMM support code included
-  #endif
-#endif
 //  Turn off multi-socket based features if only one node...
 #if OPTION_MULTISOCKET == FALSE
   #undef  OPTION_PARALLEL_TRAINING
diff --git a/src/vendorcode/amd/agesa/f14/Include/PlatformInstall.h b/src/vendorcode/amd/agesa/f14/Include/PlatformInstall.h
index 8092ad2a47..cfcca8278d 100644
--- a/src/vendorcode/amd/agesa/f14/Include/PlatformInstall.h
+++ b/src/vendorcode/amd/agesa/f14/Include/PlatformInstall.h
@@ -1967,12 +1967,6 @@ CONST UINT32 ROMDATA AmdPlatformTypeCgf = CFG_AMD_PLATFORM_TYPE;
     #endif
   #endif
 #endif
-// Check LRDIMM CODE and LRDIMM CFG item
-#if CFG_MEMORY_LRDIMM_CAPABLE == FALSE
-  #if  BLDOPT_REMOVE_LRDIMMS_SUPPORT == TRUE
-    #error   Warning: LRDIMM capability is false, but LRIDMM support code included
-  #endif
-#endif
 //  Turn off multi-socket based features if only one node...
 #if OPTION_MULTISOCKET == FALSE
   #undef  OPTION_PARALLEL_TRAINING
diff --git a/src/vendorcode/amd/agesa/f15/Include/PlatformInstall.h b/src/vendorcode/amd/agesa/f15/Include/PlatformInstall.h
index 7d96fefb76..54f180a94a 100644
--- a/src/vendorcode/amd/agesa/f15/Include/PlatformInstall.h
+++ b/src/vendorcode/amd/agesa/f15/Include/PlatformInstall.h
@@ -2067,12 +2067,6 @@ CONST UINT32 ROMDATA AmdPlatformTypeCgf = CFG_AMD_PLATFORM_TYPE;
     #endif
   #endif
 #endif
-// Check LRDIMM CODE and LRDIMM CFG item
-#if CFG_MEMORY_LRDIMM_CAPABLE == FALSE
-  #if  BLDOPT_REMOVE_LRDIMMS_SUPPORT == TRUE
-    #error   Warning: LRDIMM capability is false, but LRIDMM support code included
-  #endif
-#endif
 //  Turn off multi-socket based features if only one node...
 #if OPTION_MULTISOCKET == FALSE
   #undef  OPTION_PARALLEL_TRAINING
diff --git a/src/vendorcode/amd/agesa/f15tn/Include/PlatformInstall.h b/src/vendorcode/amd/agesa/f15tn/Include/PlatformInstall.h
index d91cec8f76..334c84ee3c 100644
--- a/src/vendorcode/amd/agesa/f15tn/Include/PlatformInstall.h
+++ b/src/vendorcode/amd/agesa/f15tn/Include/PlatformInstall.h
@@ -2465,12 +2465,6 @@ CONST UINT32 ROMDATA AmdPlatformTypeCgf = CFG_AMD_PLATFORM_TYPE;
     #endif
   #endif
 #endif
-// Check LRDIMM CODE and LRDIMM CFG item
-#if CFG_MEMORY_LRDIMM_CAPABLE == FALSE
-  #if  BLDOPT_REMOVE_LRDIMMS_SUPPORT == TRUE
-    #error   Warning: LRDIMM capability is false, but LRIDMM support code included
-  #endif
-#endif
 //  Turn off multi-socket based features if only one node...
 #if OPTION_MULTISOCKET == FALSE
   #undef  OPTION_PARALLEL_TRAINING
diff --git a/src/vendorcode/amd/agesa/f16kb/Include/PlatformInstall.h b/src/vendorcode/amd/agesa/f16kb/Include/PlatformInstall.h
index a313bbb3bb..7ff268d67e 100644
--- a/src/vendorcode/amd/agesa/f16kb/Include/PlatformInstall.h
+++ b/src/vendorcode/amd/agesa/f16kb/Include/PlatformInstall.h
@@ -1521,12 +1521,6 @@ CONST UINT32 ROMDATA AmdPlatformTypeCgf = CFG_AMD_PLATFORM_TYPE;
     #endif
   #endif
 #endif
-// Check LRDIMM CODE and LRDIMM CFG item
-#if CFG_MEMORY_LRDIMM_CAPABLE == FALSE
-  #if  BLDOPT_REMOVE_LRDIMMS_SUPPORT == TRUE
-    #error   Warning: LRDIMM capability is false, but LRIDMM support code included
-  #endif
-#endif
 //  Turn off multi-socket based features if only one node...
 #if OPTION_MULTISOCKET == FALSE
   #undef  OPTION_PARALLEL_TRAINING
-- 
cgit v1.2.3