Hi, On 7/13/20 2:34 AM, Yi Wang wrote: > From: Liao Pingfang <[email protected]> > > Remove superfluous breaks, as there is a "return" before them. > > Signed-off-by: Liao Pingfang <[email protected]>
This patch (and various other you posted recently) misses your S-o-b tag, see: https://wiki.qemu.org/Contribute/SubmitAPatch#Patch_emails_must_include_a_Signed-off-by:_line When reposting with your S-o-b please amend: Reviewed-by: Philippe Mathieu-Daudé <[email protected]> > --- > target/openrisc/sys_helper.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/target/openrisc/sys_helper.c b/target/openrisc/sys_helper.c > index d9fe6c5..d9691d0 100644 > --- a/target/openrisc/sys_helper.c > +++ b/target/openrisc/sys_helper.c > @@ -289,10 +289,8 @@ target_ulong HELPER(mfspr)(CPUOpenRISCState *env, > target_ulong rd, > > case TO_SPR(5, 1): /* MACLO */ > return (uint32_t)env->mac; > - break; > case TO_SPR(5, 2): /* MACHI */ > return env->mac >> 32; > - break; > > case TO_SPR(8, 0): /* PMR */ > return env->pmr; >
