GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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 <dforsi@gmail.com>
Reviewed-on: http://review.coreboot.org/6372
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
This commit is contained in:
Daniele Forsi 2014-07-26 10:32:34 +02:00 committed by Patrick Georgi
parent abb381607f
commit b532b12b41
2 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
src/cpu/amd/model_fxx/processor_name.c
View File

@ -235,6 +235,7 @@ int init_processor_name(void)
case 0x30052: case 0x30052:
processor_name_string = processor_name_string =
"AMD Sempron(tm) Processor RR50p"; "AMD Sempron(tm) Processor RR50p";
break;
case 0x30064: case 0x30064:
case 0x30068: case 0x30068:
processor_name_string = processor_name_string =

1
src/southbridge/amd/agesa/hudson/lpc.c
View File

@ -230,6 +230,7 @@ static void hudson_lpc_enable_childrens_resources(device_t dev)
case 0x480: case 0x480:
set_x |= (1 << 17); set_x |= (1 << 17);
rsize = 0x40; rsize = 0x40;
break;
case 0x500: case 0x500:
set_x |= (1 << 18); set_x |= (1 << 18);
rsize = 0x40; rsize = 0x40;