OpenE2K/qemu-e2k
13
4
Fork 0
Code Issues 4 Pull Requests Projects 2 Releases Activity

hw/timer/sh_timer: Remove superfluous "break" statements

Browse Source 
hw_error() is marked as QEMU_NORETURN, so the "break" statements
after this function are just dead code.

Signed-off-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20201020153935.54315-4-thuth@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
This commit is contained in:
Thomas Huth 2020-10-20 17:39:35 +02:00 committed by Philippe Mathieu-Daudé
parent 97edd8ba4b
commit 45514b48df
1 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
hw/timer/sh_timer.c
View File

@ -125,7 +125,6 @@ static void sh_timer_write(void *opaque, hwaddr offset,
/* fallthrough */ /* fallthrough */
default: default:
hw_error("sh_timer_write: Reserved TPSC value\n"); hw_error("sh_timer_write: Reserved TPSC value\n");
break;
} }
switch ((value & TIMER_TCR_CKEG) >> 3) { switch ((value & TIMER_TCR_CKEG) >> 3) {
case 0: case 0:
@ -139,7 +138,6 @@ static void sh_timer_write(void *opaque, hwaddr offset,
/* fallthrough */ /* fallthrough */
default: default:
hw_error("sh_timer_write: Reserved CKEG value\n"); hw_error("sh_timer_write: Reserved CKEG value\n");
break;
} }
switch ((value & TIMER_TCR_ICPE) >> 6) { switch ((value & TIMER_TCR_ICPE) >> 6) {
case 0: case 0:
@ -152,7 +150,6 @@ static void sh_timer_write(void *opaque, hwaddr offset,
/* fallthrough */ /* fallthrough */
default: default:
hw_error("sh_timer_write: Reserved ICPE value\n"); hw_error("sh_timer_write: Reserved ICPE value\n");
break;
} }
if ((value & TIMER_TCR_UNF) == 0) { if ((value & TIMER_TCR_UNF) == 0) {
s->int_level = 0; s->int_level = 0;