July 2016 - coreboot-gerrit - mail.coreboot.org

Patch merged into coreboot/master: src/soc: Capitalize CPU, ACPI, RAM and ROM
by gerrit＠coreboot.org July 31, 2016

July 31, 2016

the following patch was just integrated into master: commit 038e7247dc9705ff2d47dd90ec9a807f6feb52ba Author: Elyes HAOUAS <ehaouas(a)noos.fr> Date: Fri Jul 29 18:31:16 2016 +0200 src/soc: Capitalize CPU, ACPI, RAM and ROM Change-Id: I7f0d3400126d593bad8e78f95e6b9a378463b4ce Signed-off-by: Elyes HAOUAS <ehaouas(a)noos.fr> Reviewed-on: https://review.coreboot.org/15963 Tested-by: build bot (Jenkins) Reviewed-by: Omar Pakker Reviewed-by: Martin Roth <martinroth(a)google.com> See https://review.coreboot.org/15963 for details. -gerrit

1 0

Patch merged into coreboot/master: src/acpi: Capitalize ACPI and SATA
by gerrit＠coreboot.org July 31, 2016

July 31, 2016

the following patch was just integrated into master: commit f9e7d1b0ca7282a0d51313a68f90e9298c0c46c6 Author: Elyes HAOUAS <ehaouas(a)noos.fr> Date: Fri Jul 29 18:11:23 2016 +0200 src/acpi: Capitalize ACPI and SATA Change-Id: I7a4f1bd5c9e3c8152ebdd9118adddbc526d03a53 Signed-off-by: Elyes HAOUAS <ehaouas(a)noos.fr> Reviewed-on: https://review.coreboot.org/15959 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> Reviewed-by: Duncan Laurie <dlaurie(a)chromium.org> See https://review.coreboot.org/15959 for details. -gerrit

1 0

Patch merged into coreboot/master: sunw/ultra40m2: Fix handling non-existence of a device
by gerrit＠coreboot.org July 31, 2016

July 31, 2016

the following patch was just integrated into master: commit 2660a1321aad07efe14b739e15c39e5c0c886a85 Author: Patrick Georgi <pgeorgi(a)chromium.org> Date: Sat Jul 30 13:10:44 2016 +0200 sunw/ultra40m2: Fix handling non-existence of a device This probably never happens, but since we already test for the presence of the device, it makes no sense to try to configure it after its absense was determined. Change-Id: I9877dcd15819fb7949fa08a0954b05780df66316 Signed-off-by: Patrick Georgi <pgeorgi(a)chromium.org> Found-by: Coverity Scan #1347362 Reviewed-on: https://review.coreboot.org/15982 Tested-by: build bot (Jenkins) Reviewed-by: Jonathan A. Kollasch <jakllsch(a)kollasch.net> Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> Reviewed-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com> See https://review.coreboot.org/15982 for details. -gerrit

1 0

Patch merged into coreboot/master: intelvbttool: cope with errors in open()
by gerrit＠coreboot.org July 31, 2016

July 31, 2016

the following patch was just integrated into master: commit 1fcebe7fd952d8ea50d421bbc0bd02f23cbf760b Author: Patrick Georgi <pgeorgi(a)chromium.org> Date: Fri Jul 29 18:28:42 2016 +0200 intelvbttool: cope with errors in open() Change-Id: I9fee87b7331ee05d4e984cb024f124abb2c97a69 Signed-off-by: Patrick Georgi <pgeorgi(a)chromium.org> Found-by: Coverity Scan #1347357 Reviewed-on: https://review.coreboot.org/15962 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> Reviewed-by: Duncan Laurie <dlaurie(a)chromium.org> See https://review.coreboot.org/15962 for details. -gerrit

1 0

Patch merged into coreboot/master: sis/sis966: fix typo
by gerrit＠coreboot.org July 31, 2016

July 31, 2016

the following patch was just integrated into master: commit c09160436343ff7f7ce70015f4cca5a03b315f18 Author: Patrick Georgi <pgeorgi(a)chromium.org> Date: Fri Jul 29 18:23:32 2016 +0200 sis/sis966: fix typo temp8 & (!0x10) == temp8 & 0 == 0, which is certainly not intended. Change-Id: Ie9f735d31eedbec171f82929a147fc1b2e30b45a Signed-off-by: Patrick Georgi <pgeorgi(a)chromium.org> Found-by: Coverity Scan #1229587 Reviewed-on: https://review.coreboot.org/15961 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> Reviewed-by: Duncan Laurie <dlaurie(a)chromium.org> See https://review.coreboot.org/15961 for details. -gerrit

1 0

Patch merged into coreboot/master: sis/sis966: don't store a 32bit value in a 16bit variable
by gerrit＠coreboot.org July 31, 2016

July 31, 2016

the following patch was just integrated into master: commit 21ce6efe289ecc8eb6f047ceb3c3c25303e7272e Author: Patrick Georgi <pgeorgi(a)chromium.org> Date: Fri Jul 29 18:19:46 2016 +0200 sis/sis966: don't store a 32bit value in a 16bit variable That only makes deviceid == 0 (because shifting a 16bit value by 16 bits, well...) Change-Id: Iddca1de20760f92f70fe2d05886b488e5b48313d Signed-off-by: Patrick Georgi <pgeorgi(a)chromium.org> Found-by: Coverity Scan #1229558 Reviewed-on: https://review.coreboot.org/15960 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> Reviewed-by: Duncan Laurie <dlaurie(a)chromium.org> See https://review.coreboot.org/15960 for details. -gerrit

1 0

Patch merged into coreboot/master: libpayload: Drop superfluous "continue"
by gerrit＠coreboot.org July 31, 2016

July 31, 2016

the following patch was just integrated into master: commit e8c413318f6c4d4297b774ca129cd0a63cec4422 Author: Patrick Georgi <pgeorgi(a)chromium.org> Date: Fri Jul 29 16:33:48 2016 +0200 libpayload: Drop superfluous "continue" Change-Id: I5a1d1ce8ba268b08d1275f392f0b9e602860c6ab Signed-off-by: Patrick Georgi <pgeorgi(a)chromium.org> Found-by: Coverity Scan #1260729 Reviewed-on: https://review.coreboot.org/15957 Tested-by: build bot (Jenkins) Reviewed-by: Vadim Bendebury <vbendeb(a)chromium.org> Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> Reviewed-by: Omar Pakker See https://review.coreboot.org/15957 for details. -gerrit

1 0

Patch merged into coreboot/master: util/inteltool: fix memory leak
by gerrit＠coreboot.org July 31, 2016

July 31, 2016

the following patch was just integrated into master: commit 030c73031defb881fe379c002729b55919c5be3a Author: Patrick Georgi <pgeorgi(a)chromium.org> Date: Fri Jul 29 22:30:20 2016 +0200 util/inteltool: fix memory leak A struct pci_dev was allocated but not freed. Change-Id: I6a8bbef6a118fc1f0aa7037e72c4d0dda9208f4b Signed-off-by: Patrick Georgi <pgeorgi(a)chromium.org> Found-by: Coverity Scan #1353037 Reviewed-on: https://review.coreboot.org/15971 Tested-by: build bot (Jenkins) Reviewed-by: Duncan Laurie <dlaurie(a)chromium.org> Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> See https://review.coreboot.org/15971 for details. -gerrit

1 0

Patch merged into coreboot/master: util/cbmem: Initialize variable
by gerrit＠coreboot.org July 31, 2016

July 31, 2016

the following patch was just integrated into master: commit be8f0fa2bb43f80915d0635c06fbfda0285f113f Author: Patrick Georgi <pgeorgi(a)chromium.org> Date: Fri Jul 29 21:55:44 2016 +0200 util/cbmem: Initialize variable There can be cases where "found" wasn't initialized, do so. Change-Id: Ifef8d61daa70e27ec39b7a8f3481d2316dfaa36e Signed-off-by: Patrick Georgi <pgeorgi(a)chromium.org> Found-by: Coverity Scan #1347334 Reviewed-on: https://review.coreboot.org/15969 Tested-by: build bot (Jenkins) Reviewed-by: Duncan Laurie <dlaurie(a)chromium.org> Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> See https://review.coreboot.org/15969 for details. -gerrit

1 0

Patch merged into coreboot/master: nvramtool: Don't consider reserved regions to be "out of range"
by gerrit＠coreboot.org July 31, 2016

July 31, 2016

the following patch was just integrated into master: commit 8260650cde3db83e8faf9dad66eac3dc5f0937eb Author: Patrick Georgi <pgeorgi(a)chromium.org> Date: Fri Jul 29 21:01:19 2016 +0200 nvramtool: Don't consider reserved regions to be "out of range" Reserved regions showed different behavior for debug and regular builds. Debug output was unfriendly, regular was wrong. Print a proper error message and exit instead. Change-Id: I9842ff61f7d554800e2041e8c4c607f22b2df79f Signed-off-by: Patrick Georgi <pgeorgi(a)chromium.org> Found-by: Coverity Scan #1287076 Reviewed-on: https://review.coreboot.org/15968 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> Reviewed-by: Duncan Laurie <dlaurie(a)chromium.org> See https://review.coreboot.org/15968 for details. -gerrit

1 0