[coreboot-gerrit] New patch to review for coreboot: 0e11ce1 AMD: Use same sourcecode for reset in romstage as ramstage

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Mon Jun 17 08:49:17 CEST 2013


Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3487

-gerrit

commit 0e11ce1312788ef6f202bfff115d94290689429a
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date:   Mon Jun 17 01:09:07 2013 +0300

    AMD: Use same sourcecode for reset in romstage as ramstage
    
    Confusingly, romstage compiled in different copy of soft_reset()
    than ramstage. Use source in reset.c for both.
    
    Change-Id: I2e4b6d1b89c859c7cf5d9e9c8f7748b43d369775
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
 src/mainboard/advansus/a785e-i/Makefile.inc | 3 +--
 src/mainboard/advansus/a785e-i/romstage.c   | 7 -------
 src/mainboard/asus/m5a88-v/Makefile.inc     | 1 +
 src/mainboard/asus/m5a88-v/romstage.c       | 7 -------
 src/mainboard/avalue/eax-785e/Makefile.inc  | 3 +--
 src/mainboard/avalue/eax-785e/romstage.c    | 8 --------
 6 files changed, 3 insertions(+), 26 deletions(-)

diff --git a/src/mainboard/advansus/a785e-i/Makefile.inc b/src/mainboard/advansus/a785e-i/Makefile.inc
index d69a9bf..d7290af 100644
--- a/src/mainboard/advansus/a785e-i/Makefile.inc
+++ b/src/mainboard/advansus/a785e-i/Makefile.inc
@@ -1,5 +1,4 @@
-#romstage-y += reset.c #FIXME romstage have include test_rest.c
-
+romstage-y += reset.c
 ramstage-y += reset.c
 
 #SB800 CIMx share AGESA V5 lib code
diff --git a/src/mainboard/advansus/a785e-i/romstage.c b/src/mainboard/advansus/a785e-i/romstage.c
index ae283a4..1a4a276 100644
--- a/src/mainboard/advansus/a785e-i/romstage.c
+++ b/src/mainboard/advansus/a785e-i/romstage.c
@@ -72,14 +72,7 @@ static int spd_read_byte(u32 device, u32 address)
 #include "cpu/amd/model_10xxx/init_cpus.c"
 #include "northbridge/amd/amdfam10/early_ht.c"
 #include "spd.h"
-
 #include <reset.h>
-void soft_reset(void)
-{
-	set_bios_reset();
-	/* link reset */
-	outb(0x06, 0x0cf9);
-}
 
 void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
 {
diff --git a/src/mainboard/asus/m5a88-v/Makefile.inc b/src/mainboard/asus/m5a88-v/Makefile.inc
index 35b2043..d7290af 100644
--- a/src/mainboard/asus/m5a88-v/Makefile.inc
+++ b/src/mainboard/asus/m5a88-v/Makefile.inc
@@ -1,3 +1,4 @@
+romstage-y += reset.c
 ramstage-y += reset.c
 
 #SB800 CIMx share AGESA V5 lib code
diff --git a/src/mainboard/asus/m5a88-v/romstage.c b/src/mainboard/asus/m5a88-v/romstage.c
index dbae2d3..ddb0e6f 100644
--- a/src/mainboard/asus/m5a88-v/romstage.c
+++ b/src/mainboard/asus/m5a88-v/romstage.c
@@ -72,14 +72,7 @@ static int spd_read_byte(u32 device, u32 address)
 #include "cpu/amd/model_10xxx/init_cpus.c"
 #include "northbridge/amd/amdfam10/early_ht.c"
 #include "spd.h"
-
 #include <reset.h>
-void soft_reset(void)
-{
-	set_bios_reset();
-	/* link reset */
-	outb(0x06, 0x0cf9);
-}
 
 #define SERIAL_DEV PNP_DEV(0x4e, IT8721F_SP1)
 void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
diff --git a/src/mainboard/avalue/eax-785e/Makefile.inc b/src/mainboard/avalue/eax-785e/Makefile.inc
index d69a9bf..d7290af 100644
--- a/src/mainboard/avalue/eax-785e/Makefile.inc
+++ b/src/mainboard/avalue/eax-785e/Makefile.inc
@@ -1,5 +1,4 @@
-#romstage-y += reset.c #FIXME romstage have include test_rest.c
-
+romstage-y += reset.c
 ramstage-y += reset.c
 
 #SB800 CIMx share AGESA V5 lib code
diff --git a/src/mainboard/avalue/eax-785e/romstage.c b/src/mainboard/avalue/eax-785e/romstage.c
index d4704e8..5af67f7 100644
--- a/src/mainboard/avalue/eax-785e/romstage.c
+++ b/src/mainboard/avalue/eax-785e/romstage.c
@@ -72,15 +72,7 @@ static int spd_read_byte(u32 device, u32 address)
 #include "cpu/amd/model_10xxx/init_cpus.c"
 #include "northbridge/amd/amdfam10/early_ht.c"
 #include "spd.h"
-
 #include <reset.h>
-void soft_reset(void)
-{
-	set_bios_reset();
-	/* link reset */
-	outb(0x06, 0x0cf9);
-}
-
 
 void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
 {



More information about the coreboot-gerrit mailing list