the following patch was just integrated into master:
commit 148598054c34c601273b69a67d7db2d2b88da48c
Author: Kostr <aladyshev(a)nicevt.ru>
Date: Mon Oct 8 22:04:53 2012 +0400
Fix ExecuteFinalHltInstruction function in f15h family code
Current ExecuteFinalHltInstruction function doesn't work well.
(at least in configuration
Supermicro board with Orochi AMD Opteron processors (model OS6234WKTCGGU))
System reboots when trying to halt core 2,4,6,8 or 10
(OS6234WKTCGGU is 12 core processor)
Based on this information, i think that code doesn't really work with
f15 compute unit (CU) system.
Replacing ExecuteFinalHltInstruction function with
analogous function from f15tn family code fix this problem.
Both functions written from the same cahalt.asm file, but f15tn version
seems more completed
Change-Id: I3942abcdf21f1b86a44c01cc477714e44a40b9cf
Signed-off-by: Kostr <aladyshev(a)nicevt.ru>
Build-Tested: build bot (Jenkins) at Mon Oct 8 21:20:32 2012, giving +1
Reviewed-By: Marc Jones <marcj303(a)gmail.com> at Tue Oct 30 05:49:13 2012, giving +2
See http://review.coreboot.org/1569 for details.
-gerrit
the following patch was just integrated into master:
commit 5c74ba9265db51902b01e58b1b471a2ee24069d5
Author: Zheng Bao <fishbaozi(a)gmail.com>
Date: Wed Oct 24 11:20:44 2012 +0800
AMD SB800: PCIE slots on Persimmon
Enable the PCIE bridge which is connected to the PCIE slot.
Change-Id: I1b3fb59990e06d7bc7cf19639f2b93dbb7bf9b3e
Signed-off-by: Zheng Bao <zheng.bao(a)amd.com>
Signed-off-by: zbao <fishbaozi(a)gmail.com>
Build-Tested: build bot (Jenkins) at Wed Oct 24 04:46:12 2012, giving +1
Reviewed-By: Marc Jones <marcj303(a)gmail.com> at Tue Oct 30 05:16:36 2012, giving +2
See http://review.coreboot.org/1098 for details.
-gerrit
the following patch was just integrated into master:
commit cfd7a424b8e18f8301dbcbd921eccfd8c57b0d16
Author: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Date: Tue Aug 28 07:49:49 2012 +0300
Drop get_smbios_data from chip_operations
We only want to add data once per device. Using the one in
chip_operations is not very usable anyway, as different
devices under the same chip directory would need to output
entirely different sets of data.
Change-Id: I96690c4c699667343ebef44a7f3de1f974cf6d6d
Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Build-Tested: build bot (Jenkins) at Fri Oct 26 23:28:26 2012, giving +1
See http://review.coreboot.org/1492 for details.
-gerrit
Siyuan Wang (wangsiyuanbuaa(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1653
-gerrit
commit c19b79a8ee4348d6c6897a60adec124f8a41a7f2
Author: Siyuan Wang <wangsiyuanbuaa(a)gmail.com>
Date: Mon Oct 29 18:34:42 2012 +0800
amd CPU: add funtion disable_cache_as_ram_and_enable_cache()
Now, the mainboard use two functions disable_cache_as_ram()
and enable_cache(). someone think enable_cache() should move to
disable_cache_as_ram(). but there are 2 problems:
1) disable_cache_as_ram is written by assembler language and
enable_cache. if we put them in one function, we have to use
inline assembler. also, enable_cache does not waste time because
it is inline function. so we don't need to rewrite this function
by inline.
2) if we put these two function together, the name disable_cache_as_ram
would not suitable.
3) some boards have use these two functions. if we changed, these
boards may run into problem
I think src/cpu/amd/car/ is the right place to add this function.
let the coreboot community to decide which one is better,
my new method or using two functions.
Change-Id: I99b381e183b80b7b202428ad796e623e9de016fe
Signed-off-by: Siyuan Wang <SiYuan.Wang(a)amd.com>
Signed-off-by: Siyuan Wang <wangsiyuanbuaa(a)gmail.com>
---
.../car/disable_cache_as_ram_and_enable_cache.h | 29 ++++++++++++++++++++++
1 file changed, 29 insertions(+)
diff --git a/src/cpu/amd/car/disable_cache_as_ram_and_enable_cache.h b/src/cpu/amd/car/disable_cache_as_ram_and_enable_cache.h
new file mode 100644
index 0000000..0921a53
--- /dev/null
+++ b/src/cpu/amd/car/disable_cache_as_ram_and_enable_cache.h
@@ -0,0 +1,29 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * original idea yhlu 6.2005 (assembler code)
+ *
+ * Copyright (C) 2010 Rudolf Marek <r.marek(a)assembler.cz>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
+ *
+ * be warned, this file will be used other cores and core 0 / node 0
+ */
+
+extern void disable_cache_as_ram(void); /* cache_as_ram.inc */
+static inline __attribute__((always_inline)) void disable_cache_as_ram_and_enable_cache(void)
+{
+ disable_cache_as_ram();
+ enable_cache();
+}
the following patch was just integrated into master:
commit 91f016fb77ca94e50ecb6723035bf29b1547aaf8
Author: Nico Huber <nico.huber(a)secunet.com>
Date: Mon Sep 24 10:48:43 2012 +0200
Fix reading of number of interrupts for IO-APICs
The number read from the io-apic register represents the index of the
highest interrupt redirection entry, i.e. the number of interrupts
minus one.
Change-Id: I54c992e4ff400de24bb9fef5d82251078f92c588
Signed-off-by: Nico Huber <nico.huber(a)secunet.com>
See http://review.coreboot.org/1624 for details.
-gerrit
the following patch was just integrated into master:
commit dbe054c45f6fe714fe5d448a09983d90dfddf3b0
Author: Peter Stuge <peter(a)stuge.se>
Date: Sat Oct 27 14:20:10 2012 +0200
Hide all _ROM_RUN Kconfig options if the payload is SeaBIOS
The options are shown regardless of payload if CONFIG_EXPERT is set.
Change-Id: I12c81ce41a0e300e852481424eadc83f281863bf
Signed-off-by: Peter Stuge <peter(a)stuge.se>
Reviewed-By: Patrick Georgi <patrick(a)georgi-clan.de> at Mon Oct 29 09:35:49 2012, giving +2
See http://review.coreboot.org/1638 for details.
-gerrit
the following patch was just integrated into master:
commit 9ef50b72a0ca9140b2c0d02fd2e903fb79ebb981
Author: Peter Stuge <peter(a)stuge.se>
Date: Sat Oct 27 14:17:04 2012 +0200
Run option ROMs in coreboot by default only if the payload is not SeaBIOS
Change-Id: I29fb86ff3a3187b720ce5ef246c4eeee696ab5cd
Signed-off-by: Peter Stuge <peter(a)stuge.se>
Reviewed-By: Patrick Georgi <patrick(a)georgi-clan.de> at Mon Oct 29 09:35:33 2012, giving +2
See http://review.coreboot.org/1637 for details.
-gerrit