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
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
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
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
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
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
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
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