the following patch was just integrated into master:
commit 5c0b7abe786d7b4370ed1dd7ef323a3091d9620c
Author: Gabe Black <gabeblack(a)google.com>
Date: Fri Feb 22 16:38:53 2013 -0800
libpayload: Generalize and redistribute timekeeping code
The timekeeping code in libpayload was dependent on rdtsc, and when it was
split up by arch, that code was duplicated even though it was mostly the same.
This change factors out actually reading the count from the timer and the
speed of the timer and puts the definitions of ndelay, udelay, mdelay and
delay into generic code. Then, in x86, the timer_hz and timer_get_raw_value
functions which used to be in depthcharge were moved over to libpayload's
arch/x86/timer.c. In ARM where there isn't a single, canonical timer, those
functions are omitted with the intention that they'll be implemented by a
specific timer driver chosen elsewhere.
Change-Id: I9c919bed712ace941f417c1d58679d667b2d8269
Signed-off-by: Gabe Black <gabeblack(a)google.com>
Reviewed-on: http://review.coreboot.org/2717
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich(a)gmail.com>
Build-Tested: build bot (Jenkins) at Thu Mar 14 01:12:54 2013, giving +1
Reviewed-By: Ronald G. Minnich <rminnich(a)gmail.com> at Thu Mar 14 04:53:33 2013, giving +2
See http://review.coreboot.org/2717 for details.
-gerrit
the following patch was just integrated into master:
commit 6a0b3611c5b057907aabe2a1c05302440da9bc19
Author: Gabe Black <gabeblack(a)google.com>
Date: Wed Jan 16 03:18:45 2013 -0800
libpayload: Put dump_td/dump_ed in ohci.c behind #ifdef USB_DEBUG
This function is static and not used in that file. To avoid the compiler
complaining about that fact, put the two functions and the call to dump_ed
(currently #if 0) behind #ifdef USB_DEBUG
Change-Id: Ic373313b5fff81f09800f286b32238350ab699c6
Signed-off-by: Gabe Black <gabeblack(a)google.com>
Signed-off-by: Stefan Reinauer <reinauer(a)google.com>
Reviewed-on: http://review.coreboot.org/2716
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich(a)gmail.com>
Build-Tested: build bot (Jenkins) at Thu Mar 14 01:02:58 2013, giving +1
Reviewed-By: Ronald G. Minnich <rminnich(a)gmail.com> at Thu Mar 14 04:52:38 2013, giving +2
See http://review.coreboot.org/2716 for details.
-gerrit
the following patch was just integrated into master:
commit 6d04f0f89e4bf8ea4bea35dd850dad7469ca5ab3
Author: Aaron Durbin <adurbin(a)chromium.org>
Date: Wed Oct 31 22:57:16 2012 -0500
haswell: always use MMIO PCI config accesses
Add a bootblock.c file for the northbridge and setup the
PCIEXBAR as the first thing using IO PCI config acceses.
After that all PCI config accesses can use MMIO.
Change-Id: I51d229c626c45705dda1757c2f14265cbc0e6183
Signed-off-by: Aaron Durbin <adurbin(a)chromium.org>
Reviewed-on: http://review.coreboot.org/2617
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich(a)gmail.com>
Build-Tested: build bot (Jenkins) at Mon Mar 11 23:44:11 2013, giving +1
Reviewed-By: Ronald G. Minnich <rminnich(a)gmail.com> at Thu Mar 14 01:45:50 2013, giving +2
See http://review.coreboot.org/2617 for details.
-gerrit
the following patch was just integrated into master:
commit 76c3700f02f79b49fec30d6ef18d336f122cbf50
Author: Aaron Durbin <adurbin(a)chromium.org>
Date: Tue Oct 30 09:03:43 2012 -0500
haswell: Add initial support for Haswell platforms
The Haswell parts use a PCH code named Lynx Point (Series 8). Therefore,
the southbridge support is included as well. The basis for this code is
the Sandybridge code. Management Engine, IRQ routing, and ACPI still requires
more attention, but this is a good starting point.
This code partially gets up through the romstage just before training
memory on a Haswell reference board.
Change-Id: If572d6c21ca051b486b82a924ca0ffe05c4d0ad4
Signed-off-by: Aaron Durbin <adurbin(a)chromium.org>
Reviewed-on: http://review.coreboot.org/2616
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich(a)gmail.com>
Build-Tested: build bot (Jenkins) at Mon Mar 11 23:53:50 2013, giving +1
Reviewed-By: Ronald G. Minnich <rminnich(a)gmail.com> at Thu Mar 14 01:44:40 2013, giving +2
See http://review.coreboot.org/2616 for details.
-gerrit
the following patch was just integrated into master:
commit cc86e63e835ab0bceb62215460a13266a791cdd3
Author: Gabe Black <gabeblack(a)google.com>
Date: Sat Mar 2 03:32:19 2013 -0800
libpayload: Don't declare the loop counter within the for loop
'for' loop initial declarations are only allowed in C99 mode
I didn't realize we don't enable 14 year old features when building
libpayload, and I must have accidentally not rebuilt everything when making my
final tweaks to my earlier change.
Change-Id: I6caeeffad177b6d61fa30175f767e85084c061f4
Signed-off-by: Gabe Black <gabeblack(a)google.com>
Reviewed-on: http://review.coreboot.org/2718
Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net>
Tested-by: build bot (Jenkins)
Build-Tested: build bot (Jenkins) at Thu Mar 14 01:22:57 2013, giving +1
See http://review.coreboot.org/2718 for details.
-gerrit
Stefan Reinauer (stefan.reinauer(a)coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2718
-gerrit
commit 0cb105daa485dcaef74c28734113e1d29c8d16a9
Author: Gabe Black <gabeblack(a)google.com>
Date: Sat Mar 2 03:32:19 2013 -0800
libpayload: Don't declare the loop counter within the for loop
'for' loop initial declarations are only allowed in C99 mode
I didn't realize we don't enable 14 year old features when building
libpayload, and I must have accidentally not rebuilt everything when making my
final tweaks to my earlier change.
Change-Id: I6caeeffad177b6d61fa30175f767e85084c061f4
Signed-off-by: Gabe Black <gabeblack(a)google.com>
---
payloads/libpayload/arch/armv7/exception.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/payloads/libpayload/arch/armv7/exception.c b/payloads/libpayload/arch/armv7/exception.c
index 8462b11..5e4e23d 100644
--- a/payloads/libpayload/arch/armv7/exception.c
+++ b/payloads/libpayload/arch/armv7/exception.c
@@ -45,11 +45,12 @@ void exception_fiq(uint32_t *);
static void print_regs(uint32_t *regs)
{
+ int i;
/* Don't print the link register and stack pointer since we don't have their
* actual value. They are hidden by the 'shadow' registers provided
* by the trap hardware.
*/
- for (int i = 0; i < 16; i++) {
+ for (i = 0; i < 16; i++) {
if (i == 15)
printf("PC");
else if (i == 14)