Jacob Garber has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/32180
Change subject: vc/amd/agesa/f14: Add missing break statement ......................................................................
vc/amd/agesa/f14: Add missing break statement
We do not want to ASSERT(FALSE).
Found-by: Coverity Scan, CID 1241850 (MISSING_BREAK) Signed-off-by: Jacob Garber jgarber1@ualberta.ca Change-Id: Ia08bb519cdb5ef5d2a79898706c7fac7e58adf3f --- M src/vendorcode/amd/agesa/f14/Proc/Mem/NB/mnS3.c 1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/80/32180/1
diff --git a/src/vendorcode/amd/agesa/f14/Proc/Mem/NB/mnS3.c b/src/vendorcode/amd/agesa/f14/Proc/Mem/NB/mnS3.c index 7fb195d..daf529c 100644 --- a/src/vendorcode/amd/agesa/f14/Proc/Mem/NB/mnS3.c +++ b/src/vendorcode/amd/agesa/f14/Proc/Mem/NB/mnS3.c @@ -792,6 +792,7 @@ break; case AccessS3SaveWidth32: RegValue = *(UINT32 *) Value; + break; default: ASSERT (FALSE); }