[coreboot-gerrit] Patch merged into coreboot/master: 5697df2 AMD AGESA: Add missing breaks to switch statement in one file
gerrit at coreboot.org
gerrit at coreboot.org
Tue Aug 6 19:21:12 CEST 2013
the following patch was just integrated into master:
commit 5697df2a84b52f0979d4807fc293dea311e46662
Author: Bruce Griffith <Bruce.Griffith at se-eng.com>
Date: Tue Jun 25 14:14:14 2013 -0600
AMD AGESA: Add missing breaks to switch statement in one file
This is the same patch as an earlier one applied to family 15 [1].
Static analysis often flags case statements that do not include
a terminating "break;" statement. Eclipse's CODAN is an example
of this. This changelist modifies amdlib.c to terminate
case statements with "break;".
[1] e44a89f amd/agesa/f15/Lib/amdlib.c: Add missing breaks ...
Change-Id: Ibd1ae6f2b52fde07de3d978d174975f4d93647ab
Signed-off-by: Bruce Griffith <Bruce.Griffith at se-eng.com>
Reviewed-on: http://review.coreboot.org/3839
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martin at se-eng.com>
See http://review.coreboot.org/3839 for details.
-gerrit
More information about the coreboot-gerrit
mailing list