[coreboot-gerrit] Patch merged into coreboot/master: 5d74156 AMD Inagua: broadcom.c: Add missing prototype for `broadcom_init()`
gerrit at coreboot.org
gerrit at coreboot.org
Fri Mar 29 21:10:50 CET 2013
the following patch was just integrated into master:
commit 5d7415673f93ddac743de71fb2272fccdd7300fb
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date: Fri Mar 29 11:07:22 2013 +0100
AMD Inagua: broadcom.c: Add missing prototype for `broadcom_init()`
Building the AMD Inagua board, the following warning is thrown.
CC mainboard/amd/inagua/get_bus_conf.ramstage.o
src/mainboard/amd/inagua/broadcom.c:319:6: warning: no previous prototype for 'broadcom_init' [-Wmissing-prototypes]
This warning was introduced by commit 3926b4c5.
commit 3926b4c520e74da9dc22e3d136a8a178483e0d25
Author: Jens Rottmann <JRottmann at LiPPERTembedded.de>
Date: Fri Mar 1 19:41:41 2013 +0100
AMD Inagua: add GEC firmware, document Broadcom BCM57xx Selfboot Patch format
Reviewed-on: http://review.coreboot.org/2831
Adding the prototype to `broadcom.c` and removing it from
`mainboard.c` fixes the warning.
Change-Id: I1da0c4e972e129047dd8230d573f1c43fd71eb20
Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
Reviewed-on: http://review.coreboot.org/2952
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>
Build-Tested: build bot (Jenkins) at Fri Mar 29 11:36:26 2013, giving +1
Reviewed-By: Stefan Reinauer <stefan.reinauer at coreboot.org> at Fri Mar 29 21:10:34 2013, giving +2
See http://review.coreboot.org/2952 for details.
-gerrit
More information about the coreboot-gerrit
mailing list