the following patch was just integrated into master: commit 5697df2a84b52f0979d4807fc293dea311e46662 Author: Bruce Griffith Bruce.Griffith@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@se-eng.com Reviewed-on: http://review.coreboot.org/3839 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth martin@se-eng.com
See http://review.coreboot.org/3839 for details.
-gerrit