From b532b12b411d315b7799fc00a987bf21deb7e8a3 Mon Sep 17 00:00:00 2001 From: Daniele Forsi Date: Sat, 26 Jul 2014 10:32:34 +0200 Subject: [PATCH] model_fxx/processor_name.c, hudson/lpc.c: add missing break statements Found by Cppcheck 1.65. Fixes: (warning) Variable 'processor_name_string' is reassigned a value before the old one has been used. 'break;' missing? (warning) Variable 'rsize' is reassigned a value before the old one has been used. 'break;' missing? Change-Id: I4a5c947fd5cc5797eb026475ec7036bc5eaf58db Signed-off-by: Daniele Forsi Reviewed-on: http://review.coreboot.org/6372 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel Reviewed-by: Edward O'Callaghan --- src/cpu/amd/model_fxx/processor_name.c | 1 + src/southbridge/amd/agesa/hudson/lpc.c | 1 + 2 files changed, 2 insertions(+) diff --git a/src/cpu/amd/model_fxx/processor_name.c b/src/cpu/amd/model_fxx/processor_name.c index 81cd470a98..0d913d71f3 100644 --- a/src/cpu/amd/model_fxx/processor_name.c +++ b/src/cpu/amd/model_fxx/processor_name.c @@ -235,6 +235,7 @@ int init_processor_name(void) case 0x30052: processor_name_string = "AMD Sempron(tm) Processor RR50p"; + break; case 0x30064: case 0x30068: processor_name_string = diff --git a/src/southbridge/amd/agesa/hudson/lpc.c b/src/southbridge/amd/agesa/hudson/lpc.c index c8051ec42d..20212ef223 100644 --- a/src/southbridge/amd/agesa/hudson/lpc.c +++ b/src/southbridge/amd/agesa/hudson/lpc.c @@ -230,6 +230,7 @@ static void hudson_lpc_enable_childrens_resources(device_t dev) case 0x480: set_x |= (1 << 17); rsize = 0x40; + break; case 0x500: set_x |= (1 << 18); rsize = 0x40;