Angel Pons submitted this change.

View Change

mb/asrock/e350m1: Do not redefine AGESA_VERSION_STRING

This is the only AGESA f14 board which has a different version string.
As it is most likely a copy-paste error, drop the redefinition of this
macro from buildOpts.c and use the value defined in AGESA f14 headers.

Change-Id: I384bd96db51457e68a320b99ecdbb2ada0dfbdd5
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/41621
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Mike Banon <mikebdp2@gmail.com>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
---
M src/mainboard/asrock/e350m1/buildOpts.c
1 file changed, 0 insertions(+), 6 deletions(-)

diff --git a/src/mainboard/asrock/e350m1/buildOpts.c b/src/mainboard/asrock/e350m1/buildOpts.c
index 7a2cc3b..1c4d62c 100644
--- a/src/mainboard/asrock/e350m1/buildOpts.c
+++ b/src/mainboard/asrock/e350m1/buildOpts.c
@@ -38,11 +38,5 @@
 #include "cpuLateInit.h"
 #include "GnbInterface.h"
 
-/* FIXME: This is most likely wrong */
-#undef AGESA_VERSION_STRING
-                  // This is the release version number of the AGESA component
-                  // This string MUST be exactly 12 characters long
-#define AGESA_VERSION_STRING  {'V', '0', '.', '0', '.', '0', '.', '1', ' ', ' ', ' ', ' '}
-
 /* Instantiate all solution relevant data */
 #include <PlatformInstall.h>

To view, visit change 41621. To unsubscribe, or for help writing mail filters, visit settings.