June 2014 - coreboot-gerrit - mail.coreboot.org

Patch merged into coreboot/master: 480790b Makefile: HOSTCC set too late in clang builds
by gerrit＠coreboot.org June 29, 2014

June 29, 2014

the following patch was just integrated into master: commit 480790b593607c5e5d472c1aa45813b08f580cf9 Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com> Date: Fri Jun 27 18:14:47 2014 +1000 Makefile: HOSTCC set too late in clang builds Currently we set HOSTCC=clang a little late meaning some minor bits (utils/kconfig) are built with GCC. Move the assignment up the Makefile. Change-Id: Ic72ad808eba0c0bf508bde34fb9bf0390c0b1d4d Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com> Reviewed-on: http://review.coreboot.org/6140 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> Reviewed-by: Idwer Vollering <vidwer(a)gmail.com> See http://review.coreboot.org/6140 for details. -gerrit

1 0

Patch merged into coreboot/master: d2a7523 northbridge/amd/gx2/raminit.c Halt func needs noreturn attrib
by gerrit＠coreboot.org June 29, 2014

June 29, 2014

the following patch was just integrated into master: commit d2a7523cb18e804e71eee16656deab364dbf6603 Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com> Date: Sun Jun 29 02:27:19 2014 +1000 northbridge/amd/gx2/raminit.c Halt func needs noreturn attrib Missing "__attribute__((noreturn))" on halt function. This sync's the implementation to be the same as that of amd/lx thereby avoiding compiler warnings. Change-Id: Iead16125805eb36ff875fba767cf8d4e5aa86715 Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com> Reviewed-on: http://review.coreboot.org/6157 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi <patrick(a)georgi-clan.de> See http://review.coreboot.org/6157 for details. -gerrit

1 0

Patch merged into coreboot/master: ec79d7a drivers/pc80/mc146818rtc_early.c: Silence unused func complaints
by gerrit＠coreboot.org June 29, 2014

June 29, 2014

the following patch was just integrated into master: commit ec79d7a66ec18874f3f0ac6a8e6e84f4d23be1d9 Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com> Date: Thu Jun 26 18:14:04 2014 +1000 drivers/pc80/mc146818rtc_early.c: Silence unused func complaints Clang complains these functions are unused since they find their way into the bootblock of ROMCC boards by #including the .c file. These static inlines should probably be moved into a header in reality. Change-Id: I9d82a6befb0ac99afab6265f9d3649e419f2887d Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com> Reviewed-on: http://review.coreboot.org/6122 Tested-by: build bot (Jenkins) Reviewed-by: David Hendricks <dhendrix(a)chromium.org> See http://review.coreboot.org/6122 for details. -gerrit

1 0

Patch merged into coreboot/master: 78d33b6 southbridge/intel/ibexpeak/me.c: Silence warns about unused func
by gerrit＠coreboot.org June 29, 2014

June 29, 2014

the following patch was just integrated into master: commit 78d33b649e96dcb5a49f33464e234d1227e88384 Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com> Date: Thu Jun 26 22:13:15 2014 +1000 southbridge/intel/ibexpeak/me.c: Silence warns about unused func Move some __SMM__ functions under the #if preprocessor condition to avoid warnings about unused functions. Change-Id: I7f6fbc6a577032bc4e4635d91e8e94aecb517bd3 Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com> Reviewed-on: http://review.coreboot.org/6127 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi <patrick(a)georgi-clan.de> See http://review.coreboot.org/6127 for details. -gerrit

1 0

Patch merged into coreboot/master: caac5ef northbridge/amd: Remove some extraneous parentheses from if-statements
by gerrit＠coreboot.org June 29, 2014

June 29, 2014

the following patch was just integrated into master: commit caac5ef86d1164d3f12c0486d0a2e469d0ebb252 Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com> Date: Sat Jun 28 17:52:34 2014 +1000 northbridge/amd: Remove some extraneous parentheses from if-statements Spotted by Clang. Change-Id: I38832da7b93d4ee18b8de3e80dc39513a8910221 Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com> Reviewed-on: http://review.coreboot.org/6149 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi <patrick(a)georgi-clan.de> See http://review.coreboot.org/6149 for details. -gerrit

1 0

Patch merged into coreboot/master: 4d7539e cpu/x86/pae/pgtbl.c: Unsigned comparison < 0 always false
by gerrit＠coreboot.org June 29, 2014

June 29, 2014

the following patch was just integrated into master: commit 4d7539eefdedc4297e1529ee94df7165e2076fa1 Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com> Date: Sat Jun 28 15:33:30 2014 +1000 cpu/x86/pae/pgtbl.c: Unsigned comparison < 0 always false Comparison of unsigned expression < 0 is always false. Change-Id: Idf4e7846b50f4376a5d33515681efbd773d1caca Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com> Reviewed-on: http://review.coreboot.org/6146 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi <patrick(a)georgi-clan.de> See http://review.coreboot.org/6146 for details. -gerrit

1 0

Patch merged into coreboot/master: 2f9b3af AMD boards: Fix comment style and typos
by gerrit＠coreboot.org June 29, 2014

June 29, 2014

the following patch was just integrated into master: commit 2f9b3afc84de76c75dd783adc3876da3c600e6d8 Author: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Date: Thu Jun 26 05:30:54 2014 +0300 AMD boards: Fix comment style and typos Change-Id: Id630cc46b79a39e1786d42adbc21f3b9c3a051aa Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Reviewed-on: http://review.coreboot.org/6118 Tested-by: build bot (Jenkins) Reviewed-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com> See http://review.coreboot.org/6118 for details. -gerrit

1 0

Patch merged into coreboot/master: 25b56c3 build: remove -ccopts mechanism
by gerrit＠coreboot.org June 29, 2014

June 29, 2014

the following patch was just integrated into master: commit 25b56c3af514faa8a730d56fe14cae4960ac83aa Author: Patrick Georgi <patrick(a)georgi-clan.de> Date: Sun May 25 00:11:35 2014 +0200 build: remove -ccopts mechanism We now use the slightly more familiar CFLAGS_* and CPPFLAGS_* for the same purpose. Change-Id: Ifd2bd13f67f71fa0a15611a6d11a6a4c7994271b Signed-off-by: Patrick Georgi <patrick(a)georgi-clan.de> Reviewed-on: http://review.coreboot.org/5875 Tested-by: build bot (Jenkins) Reviewed-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com> Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> See http://review.coreboot.org/5875 for details. -gerrit

1 0

Patch merged into coreboot/master: d638c2b include/pc80/mc146818rtc.h: Inconsequential, comment ifdef maze
by gerrit＠coreboot.org June 29, 2014

June 29, 2014

the following patch was just integrated into master: commit d638c2b34bae222ad16670c72ad7c9a8841a3ec9 Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com> Date: Thu Jun 26 18:11:07 2014 +1000 include/pc80/mc146818rtc.h: Inconsequential, comment ifdef maze Change-Id: Ie1ec8dbcdbbe0f2b05fdb10b1dca43cfee2a58cb Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com> Reviewed-on: http://review.coreboot.org/6120 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi <patrick(a)georgi-clan.de> See http://review.coreboot.org/6120 for details. -gerrit

1 0

Patch merged into coreboot/master: 95b0c3d arch/x86/include/bootblock_common.h: Sanitize header inclusion
by gerrit＠coreboot.org June 29, 2014

June 29, 2014

the following patch was just integrated into master: commit 95b0c3d75a40cea40a15e8f7d6f56af2ebe2db77 Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com> Date: Thu Jun 26 18:09:34 2014 +1000 arch/x86/include/bootblock_common.h: Sanitize header inclusion Sanitize the inclusion of mc146818rtc.h in bootblock_common.h Change-Id: I37d9ffd1375aedbf1f3eaa4ddce27e16166ce0b9 Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com> Reviewed-on: http://review.coreboot.org/6119 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi <patrick(a)georgi-clan.de> Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> See http://review.coreboot.org/6119 for details. -gerrit

1 0