mbox series

[0/3] sh4: Trivial clean-ups for sh_timer.c

Message ID 20201020153935.54315-1-thuth@redhat.com
Headers show
Series sh4: Trivial clean-ups for sh_timer.c | expand

Message

Thomas Huth Oct. 20, 2020, 3:39 p.m. UTC 
Fix coding style, add fallthrough anonotations and remove superfluous
break statements in hw/timer/sh_timer.c.

Thomas Huth (3):
  hw/timer/sh_timer: Coding style clean-up
  hw/timer/sh_timer: Silence warnings about missing fallthrough
    statements
  hw/timer/sh_timer: Remove superfluous "break" statements

 hw/timer/sh_timer.c | 90 +++++++++++++++++++++++++++++----------------
 1 file changed, 58 insertions(+), 32 deletions(-)

Comments

Richard Henderson Oct. 21, 2020, 4:48 p.m. UTC | #1 
On 10/20/20 8:39 AM, Thomas Huth wrote:
> Thomas Huth (3):
>   hw/timer/sh_timer: Coding style clean-up
>   hw/timer/sh_timer: Silence warnings about missing fallthrough
>     statements
>   hw/timer/sh_timer: Remove superfluous "break" statements

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~
Philippe Mathieu-Daudé Oct. 25, 2020, 12:38 a.m. UTC | #2 
On 10/20/20 5:39 PM, Thomas Huth wrote:
> Fix coding style, add fallthrough anonotations and remove superfluous
> break statements in hw/timer/sh_timer.c.
> 
> Thomas Huth (3):
>    hw/timer/sh_timer: Coding style clean-up
>    hw/timer/sh_timer: Silence warnings about missing fallthrough
>      statements
>    hw/timer/sh_timer: Remove superfluous "break" statements
> 
>   hw/timer/sh_timer.c | 90 +++++++++++++++++++++++++++++----------------
>   1 file changed, 58 insertions(+), 32 deletions(-)
> 

Thanks, applied to renesas-next.