diff options
| author | Daniele Forsi <dforsi@gmail.com> | 2014-07-26 10:32:34 +0200 |
|---|---|---|
| committer | Patrick Georgi <patrick@georgi-clan.de> | 2014-07-30 10:35:27 +0200 |
| commit | b532b12b411d315b7799fc00a987bf21deb7e8a3 (patch) | |
| tree | d921b4b251db3535156e0b30bd5d1c0fbeb07a5d /src/southbridge | |
| parent | abb381607fabcea0a101273c29a42280064b5f9b (diff) | |
| download | coreboot-b532b12b411d315b7799fc00a987bf21deb7e8a3.tar.xz | |
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>
Diffstat (limited to 'src/southbridge')
| -rw-r--r-- | src/southbridge/amd/agesa/hudson/lpc.c | 1 |
1 files changed, 1 insertions, 0 deletions
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; |