coreboot-gerrit August 2015

coreboot-gerrit@coreboot.org

1 participants
814 discussions

Patch merged into coreboot/master: skylake: fix garbled patch from upstream
by gerrit＠coreboot.org 13 Aug '15

13 Aug '15

the following patch was just integrated into master: commit 25477e03a117f3f420146ffde4b5c1eda9f5493b Author: Aaron Durbin <adurbin(a)chromium.org> Date: Thu Jul 30 10:36:34 2015 -0500 skylake: fix garbled patch from upstream In the review process for http://review.coreboot.org/#/c/11052/ the code was mangled and the result was unbuildable code. Fix this. BUG=chrome-os-partner:43419 BRANCH=None TEST=Can actually build bootblock. Original-Change-Id: I5bc63b8c435dbf025f1c334e9a1bc4a9da2b4902 Original-Signed-off-by: Aaron Durbin <adurbin(a)chromium.org> Original-Reviewed-on: https://chromium-review.googlesource.com/289788 Original-Reviewed-by: Robbie Zhang <robbie.zhang(a)intel.com> Original-Reviewed-by: Duncan Laurie <dlaurie(a)chromium.org> Original-Reviewed-by: Patrick Georgi <pgeorgi(a)chromium.org> Change-Id: Id0f67d8b74fa9146bf01990f599d538222f7e0e2 Signed-off-by: Aaron Durbin <adurbin(a)chromium.org> Reviewed-on: http://review.coreboot.org/11167 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi <pgeorgi(a)google.com> See http://review.coreboot.org/11167 for details. -gerrit

1 0

Patch merged into coreboot/master: x86: parameterize asl_template for CBFS inclusion
by gerrit＠coreboot.org 13 Aug '15

13 Aug '15

the following patch was just integrated into master: commit ab454c6b71fd54b61cc90165fe801250890894d2 Author: Aaron Durbin <adurbin(a)chromium.org> Date: Thu Jul 30 12:44:58 2015 -0500 x86: parameterize asl_template for CBFS inclusion The asl_template previously unconditionally included dsdt.aml. However, COMPILE_IN_DSDT=y results in the dsdt.aml being linked directly into ramstage. Thus the information is duplicated. The inclusion of this file unconditionally throws some errors as certain assets need to be included in CBFS. However, as there isn't fine-grained ordering control in how files are added fixed resource requirements for other assets collide result in failure to build. To remedy both things, provide a 2nd argument to asl_template which defaults to 'y' for CBFS addition. In the COMPILE_IN_DSDT=y case pass 'n' so that dsdt.aml is no longer added. BUG=chrome-os-partner:43419 BRANCH=None TEST=For glados: Built with COMPILE_IN_DSDT=y. dsdt.aml not included. Built with COMPILE_IN_DSDT=n. dsdt.aml was included. Original-Change-Id: I4767e5be2915c1732251fe415017f30314c5efc9 Original-Signed-off-by: Aaron Durbin <adurbin(a)chromium.org> Original-Reviewed-on: https://chromium-review.googlesource.com/289840 Original-Reviewed-by: Duncan Laurie <dlaurie(a)chromium.org> Change-Id: Id1828627ba0a034eb05b2fe23be76e19f3040444 Signed-off-by: Aaron Durbin <adurbin(a)chromium.org> Reviewed-on: http://review.coreboot.org/11166 Reviewed-by: Patrick Georgi <pgeorgi(a)google.com> Tested-by: build bot (Jenkins) See http://review.coreboot.org/11166 for details. -gerrit

1 0

Patch merged into coreboot/master: cbfstool: unify actual file creation
by gerrit＠coreboot.org 13 Aug '15

13 Aug '15

the following patch was just integrated into master: commit dd2d3f933a45a2fa48af344bd8ab50bb06c212de Author: Patrick Georgi <pgeorgi(a)google.com> Date: Wed Aug 12 12:29:20 2015 +0200 cbfstool: unify actual file creation After the preparation in earlier commits, it is now possible to handle the more general case of position independent files using the special code path for fixed location files. This leads to a single place where non-empty cbfs file headers are actually written into the image, allowing us to move it up the chain more easily. Change-Id: I8c1fca5e4e81c20971b2960c87690e982aa3e274 Signed-off-by: Patrick Georgi <pgeorgi(a)google.com> Reviewed-on: http://review.coreboot.org/11222 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin <adurbin(a)chromium.org> See http://review.coreboot.org/11222 for details. -gerrit

1 0

Patch merged into coreboot/master: soc/common/intel: Reset is not dependend upon FSP
by gerrit＠coreboot.org 13 Aug '15

13 Aug '15

the following patch was just integrated into master: commit 3432e556f548c0e61c714b880ce04bf1f3f687bd Author: Lee Leahy <leroy.p.leahy(a)intel.com> Date: Mon Jul 20 17:24:44 2015 -0700 soc/common/intel: Reset is not dependend upon FSP Remove dependency of common reset code on FSP BRANCH=none BUG=None TEST=Build and run on Braswell and Skylake Original-Change-Id: I00052f29326f691b6d56d2349f99815cafff5848 Original-Signed-off-by: Lee Leahy <Leroy.P.Leahy(a)intel.com> Original-Reviewed-on: https://chromium-review.googlesource.com/286932 Original-Commit-Queue: Leroy P Leahy <leroy.p.leahy(a)intel.com> Original-Tested-by: Leroy P Leahy <leroy.p.leahy(a)intel.com> Original-Reviewed-by: Aaron Durbin <adurbin(a)chromium.org> Change-Id: I7f59f0aad7dfae92df28cf20fff2d5a684795d22 Signed-off-by: Lee Leahy <Leroy.P.Leahy(a)intel.com> Reviewed-on: http://review.coreboot.org/11165 Tested-by: build bot (Jenkins) Reviewed-by: Leroy P Leahy <leroy.p.leahy(a)intel.com> See http://review.coreboot.org/11165 for details. -gerrit

1 0

Patch merged into coreboot/master: cbfstool: move tests for fixed-location files earlier
by gerrit＠coreboot.org 13 Aug '15

13 Aug '15

the following patch was just integrated into master: commit aa44dbd364bfb36b3cb4eb2fb3f9ee292610ccf4 Author: Patrick Georgi <pgeorgi(a)google.com> Date: Wed Aug 12 12:05:21 2015 +0200 cbfstool: move tests for fixed-location files earlier ... and the assert is gone. The actual action of adding a just-right file can be moved after the tests since it's exactly the condition those tests don't continue or break on. Change-Id: I6d0e829e0158198301136ada9a0de2f168ceee3f Signed-off-by: Patrick Georgi <pgeorgi(a)google.com> Reviewed-on: http://review.coreboot.org/11221 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin <adurbin(a)chromium.org> See http://review.coreboot.org/11221 for details. -gerrit

1 0

Patch merged into coreboot/master: amd: raminit sysinfo offset fix
by gerrit＠coreboot.org 13 Aug '15

13 Aug '15

the following patch was just integrated into master: commit 0dc7354760e889074472ef17652c7f270f3aa529 Author: Aaron Durbin <adurbin(a)chromium.org> Date: Thu Jul 16 16:07:02 2015 -0500 amd: raminit sysinfo offset fix The sysinfo object within the k8 ram init is used to communicate progess/status from all the nodes in the system. However, the code was assuming where the sysinfo object lived in cache-as-ram. The layout of cache-as-ram is dynamic so one needs to do the lookup of the correct address at runtime. The way the amd code is compiled by #include'ing .c files makes the solution a little more complex in that some cache-as-ram support code needed to be refactored. Change-Id: I6500fa7b005dc082c4c0b3382ee2c3a138d9ac31 Signed-off-by: Aaron Durbin <adurbin(a)chromium.org> Reviewed-on: http://review.coreboot.org/10961 Tested-by: build bot (Jenkins) Tested-by: Raptor Engineering Automated Test Stand <noreply(a)raptorengineeringinc.com> Reviewed-by: Patrick Georgi <pgeorgi(a)google.com> See http://review.coreboot.org/10961 for details. -gerrit

1 0

Patch merged into coreboot/master: cbfstool: prepare moving tests earlier
by gerrit＠coreboot.org 13 Aug '15

13 Aug '15

the following patch was just integrated into master: commit db01ddfd49ee6177c738f410496574105e7e5e44 Author: Patrick Georgi <pgeorgi(a)google.com> Date: Wed Aug 12 12:01:01 2015 +0200 cbfstool: prepare moving tests earlier The assert() makes sure the if() holds true. But that assert won't survive for long. Change-Id: Iab7d2bc7bfebb3f3b3ce70dc5bd041902e14bd7a Signed-off-by: Patrick Georgi <pgeorgi(a)google.com> Reviewed-on: http://review.coreboot.org/11220 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin <adurbin(a)chromium.org> See http://review.coreboot.org/11220 for details. -gerrit

1 0

Patch merged into coreboot/master: cbfstool: factor out creating a cbfs file header
by gerrit＠coreboot.org 13 Aug '15

13 Aug '15

the following patch was just integrated into master: commit 57edf16be5ddef5da272ff8e1a9f1d78091377d9 Author: Patrick Georgi <pgeorgi(a)google.com> Date: Wed Aug 12 09:20:11 2015 +0200 cbfstool: factor out creating a cbfs file header We will want to create headers that live outside the final image at some point (eg. to build the file before we even know where to place it). Change-Id: Ie4c0323df8d5be955aec3621b75309e8f11fae49 Signed-off-by: Patrick Georgi <pgeorgi(a)google.com> Reviewed-on: http://review.coreboot.org/11219 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin <adurbin(a)chromium.org> See http://review.coreboot.org/11219 for details. -gerrit

1 0

Patch merged into coreboot/master: cbfstool: Make cbfs_create_empty_entry build a more complete header
by gerrit＠coreboot.org 13 Aug '15

13 Aug '15

the following patch was just integrated into master: commit edf25d9076de79579bafa1010bb5350c4c992158 Author: Patrick Georgi <pgeorgi(a)google.com> Date: Wed Aug 12 09:12:06 2015 +0200 cbfstool: Make cbfs_create_empty_entry build a more complete header Pass the file type into it instead of creating an entry, then modifying the header field again after the fact. Change-Id: I655583218f5085035b0f80efff7f91a66b5b296e Signed-off-by: Patrick Georgi <pgeorgi(a)google.com> Reviewed-on: http://review.coreboot.org/11218 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin <adurbin(a)chromium.org> See http://review.coreboot.org/11218 for details. -gerrit

1 0

Patch merged into coreboot/master: cbfstool: honor larger-than-usual header sizes in cbfs_add_entry_at()
by gerrit＠coreboot.org 13 Aug '15

13 Aug '15

the following patch was just integrated into master: commit 7fd14184fb7cf2a4a713d47b8028155f77a090fc Author: Patrick Georgi <pgeorgi(a)google.com> Date: Tue Aug 11 15:55:16 2015 +0200 cbfstool: honor larger-than-usual header sizes in cbfs_add_entry_at() If an earlier stage built a larger header, cbfs_add_entry_at() shouldn't decide to go with the most boring, least featureful header type (and its size) instead. Change-Id: Icc5dcd9a797a0f3c42f91cddd21b3b3916095b63 Signed-off-by: Patrick Georgi <pgeorgi(a)google.com> Reviewed-on: http://review.coreboot.org/11217 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin <adurbin(a)chromium.org> See http://review.coreboot.org/11217 for details. -gerrit

1 0

← Newer
1
...
53
54
55
56
57
58
59
...
82
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit August 2015