[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