the following patch was just integrated into master:
commit aea318f35d13e1859efa82f664df01c8c13657c1
Author: Bruce Griffith <bruce.griffith(a)se-eng.com>
Date: Mon Jul 29 02:34:26 2013 -0600
AMD Kabini: Add "const" modifier to AGESA function parameters
Add CONST modifiers to read-only pass-by-reference function
parameters in AGESA. This allows the use of "const" modifiers
on the declaration of lookup tables that are pass-by-reference.
These will be used to identify tables that are copied onto the
HEAP but don't need to be.
This same change was made for AMD Trinity APUs (Family15tn) [1].
[1] 283ba78 AGESA: Add "const" modifier to function parameters
Change-Id: I2bdd9fc5e027e938de9df0f923b95da934bb48dc
Signed-off-by: Bruce Griffith <Bruce.Griffith(a)se-eng.com>
Reviewed-by: Dave Frodin <dave.frodin(a)se-eng.com>
Tested-by: Bruce Griffith <bruce.griffith(a)se-eng.com>
Reviewed-on: http://review.coreboot.org/3837
Tested-by: build bot (Jenkins)
Reviewed-by: Marc Jones <marc.jones(a)se-eng.com>
See http://review.coreboot.org/3837 for details.
-gerrit
the following patch was just integrated into master:
commit abebe80161a03c65b22088c7151f01b6681559a5
Author: Paul Menzel <paulepanter(a)users.sourceforge.net>
Date: Sun Apr 28 14:44:08 2013 +0200
AMD SATA: Correct "them implement" to "then implement" in comments
This changelist was cherry-picked from merged community code
for Parmer [1] and the paths modified so that the Parmer
modification is applied against Olive Hill.
[1] 0086162 AMD SATA: Correct _them implement_ ... in comments
Change-Id: I9849e9a75dacfde15331c4200d72343a59036f14
Signed-off-by: Bruce Griffith <bruce.griffith(a)se-eng.com>
Reviewed-on: http://review.coreboot.org/3841
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich(a)gmail.com>
See http://review.coreboot.org/3841 for details.
-gerrit
the following patch was just integrated into master:
commit 76db07e8c20b1e419ceeaf14eb23c7675839067a
Author: Bruce Griffith <Bruce.Griffith(a)se-eng.com>
Date: Sun Jul 7 02:06:53 2013 -0600
AMD Kabini: Add map_oprom() function for Vendor/Device IDs
Change-Id: I14285f0677003fbf8b9b112207af202658807894
Reviewed-by: Marc Jones <marc.jones(a)se-eng.com>
Signed-off-by: Bruce Griffith <Bruce.Griffith(a)se-eng.com>
Reviewed-by: Bruce Griffith <bruce.griffith(a)se-eng.com>
Tested-by: Bruce Griffith <bruce.griffith(a)se-eng.com>
Reviewed-on: http://review.coreboot.org/3806
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martin.roth(a)se-eng.com>
See http://review.coreboot.org/3806 for details.
-gerrit
the following patch was just integrated into master:
commit 17933e8bc17f04a876a51198dabcc2363f4ee334
Author: Bruce Griffith <Bruce.Griffith(a)se-eng.com>
Date: Wed Jul 10 01:26:26 2013 -0600
AMD Olive Hill: Enable HDMI audio setting in build options
Change-Id: Ifc180e6fcd594dbedc2512ea5bef283a3ad689d3
Reviewed-by: Marc Jones <marc.jones(a)se-eng.com>
Signed-off-by: Bruce Griffith <Bruce.Griffith(a)se-eng.com>
Reviewed-by: Dave Frodin <dave.frodin(a)se-eng.com>
Tested-by: Bruce Griffith <bruce.griffith(a)se-eng.com>
Reviewed-on: http://review.coreboot.org/3814
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martin.roth(a)se-eng.com>
See http://review.coreboot.org/3814 for details.
-gerrit
the following patch was just integrated into master:
commit 5912125dffcbe8a1b1ab4d1198c5ff2c9bc0771c
Author: Bruce Griffith <Bruce.Griffith(a)se-eng.com>
Date: Wed Jul 10 22:24:47 2013 -0600
AMD Olive Hill: Eliminate unnecessary memory copy
Eliminate an unnecessary copy of the DDI descriptor list and
the PCIe port descriptor list. As descriptor tables, these
tables do not need dynamic updating and should be used from
ROM without runtime copying.
There will be a corresponding patch for AGESA that adds CONST
modifiers to function parameters that are pass-by-reference
"IN" values (read-only pointers).
Change-Id: I7ab78e58041e9247db22d0f97a6f76d45f338db0
Signed-off-by: Bruce Griffith <Bruce.Griffith(a)se-eng.com>
Reviewed-by: Marc Jones <marc.jones(a)se-eng.com>
Tested-by: Bruce Griffith <bruce.griffith(a)se-eng.com>
Reviewed-by: Dave Frodin <dave.frodin(a)se-eng.com>
Reviewed-on: http://review.coreboot.org/3818
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martin.roth(a)se-eng.com>
See http://review.coreboot.org/3818 for details.
-gerrit
the following patch was just integrated into master:
commit 80cf7d5956283b5c79b293d2ffac82f329f6203c
Author: Siyuan Wang <wangsiyuanbuaa(a)gmail.com>
Date: Tue Jul 9 17:42:43 2013 +0800
AMD Olive Hill: Add new AMD mainboard using Kabini processor
Change-Id: I1f252b67c039d28df96e8dfd458a1ca6a7dbc816
Signed-off-by: Siyuan Wang <SiYuan.Wang(a)amd.com>
Signed-off-by: Siyuan Wang <wangsiyuanbuaa(a)gmail.com>
Reviewed-by: Bruce Griffith <bruce.griffith(a)se-eng.com>
Tested-by: Bruce Griffith <bruce.griffith(a)se-eng.com>
Reviewed-on: http://review.coreboot.org/3784
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martin.roth(a)se-eng.com>
See http://review.coreboot.org/3784 for details.
-gerrit
the following patch was just integrated into master:
commit 37a1d6c9552b267c06c471cced5928d2385767e2
Author: Bruce Griffith <bruce.griffith(a)se-eng.com>
Date: Tue Jul 23 11:50:12 2013 -0600
AMD Hudson/Yangtze: Enable support for SATA port multipliers
This patch sets a bit in the Yangtze southbridge to enable
the extra protocol necessary to handle port multiplier chips.
This has been turned on during most of Kabini development
without any notable impact. Olive Hill has an optional daughter
board that incorporates Silicon Image Steel Vines chips. This
change has been tested with and without the daughter board. This
change can be regression tested using any Hudson-based motherboard,
although it has no impact on boards with discreet Hudson/Bolton
southbridges.
This was tested for impact on SATA performance in the absence of
a port multiplier using the IOZone benchmarks within the Phoronix
Test Suite. A SATA 3 hard drive (6.0 Gbps) and an SSD were
connected to the ports on Olive Hill without using the port
multiplier card. The test results contained more run-to-run
variation within the same configuration than was seen in the
aggregate results comparing the interface with and without the
port multiplier protocol additions. In other words, the test
had less accuracy than the impact caused by turning on port
multiplier support.
Change-Id: Ie87873b093f3e2a6a5c83b96ccb6c898d3e25f72
Signed-off-by: Bruce Griffith <bruce.griffith(a)se-eng.com>
Reviewed-by: Martin Roth <martin.roth(a)se-eng.com>
Reviewed-by: Dave Frodin <dave.frodin(a)se-eng.com>
Reviewed-on: http://review.coreboot.org/3808
Tested-by: build bot (Jenkins)
See http://review.coreboot.org/3808 for details.
-gerrit
the following patch was just integrated into master:
commit 915714501bc411f83717b41ebf269fc82a5e9657
Author: Siyuan Wang <wangsiyuanbuaa(a)gmail.com>
Date: Tue Jul 9 17:32:42 2013 +0800
AMD Kabini: Modify Hudson southbridge to support new AMD processor
Yangtze uses Hudson AGESA wrapper code but has some changes.
The changes are necessary and have no effects on Hudson.
Change-Id: Iada90d34fdc2025bd14f566488ee12810a28ac0d
Signed-off-by: Siyuan Wang <SiYuan.Wang(a)amd.com>
Signed-off-by: Siyuan Wang <wangsiyuanbuaa(a)gmail.com>
Signed-off-by: Bruce Griffith <Bruce.Griffith(a)se-eng.com>
Reviewed-by: Dave Frodin <dave.frodin(a)se-eng.com>
Tested-by: Bruce Griffith <bruce.griffith(a)se-eng.com>
Reviewed-on: http://review.coreboot.org/3783
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martin.roth(a)se-eng.com>
See http://review.coreboot.org/3783 for details.
-gerrit
the following patch was just integrated into master:
commit 3e32cc00d1984e7a0d01039577e5aa6ae3b2aa81
Author: Siyuan Wang <wangsiyuanbuaa(a)gmail.com>
Date: Tue Jul 9 17:16:20 2013 +0800
AMD Kabini: Add northbridge AGESA wrapper (new AMD processor)
src/arch/x86/boot/tables.c and src/include/device/pci_ids.h are also
changed because these two files depend on F16kb northbridge macros
Change-Id: Iedc842f0b230826675703fc78ed8001a978319c5
Reviewed-by: Marc Jones <marc.jones(a)se-eng.com>
Signed-off-by: Bruce Griffith <Bruce.Griffith(a)se-eng.com>
Reviewed-by: Bruce Griffith <bruce.griffith(a)se-eng.com>
Tested-by: Bruce Griffith <bruce.griffith(a)se-eng.com>
Reviewed-on: http://review.coreboot.org/3782
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martin.roth(a)se-eng.com>
See http://review.coreboot.org/3782 for details.
-gerrit