On 8/10/19 12:08 AM, Joe Perches wrote:
> On Sat, 2019-08-10 at 00:01 -0500, Gustavo A. R. Silva wrote:
>> Mark switch cases where we are expecting to fall through.
>>
>> Fix the following warnings (Building: defconfig sh):
> []
>> diff --git a/arch/sh/kernel/disassemble.c b/arch/sh/kernel/disassemble.c
> []
>> @@ -477,6 +477,7 @@ static void print_sh_insn(u32 memaddr, u16 insn)
>> case FD_REG_N:
>> if (0)
>> goto d_reg_n;
>
> Might as well remove this if (0) goto,
> remove the added comment
>
You're right. I'll respin and stop working this Friday midnight.
>> + /* else, fall through */
>> case F_REG_N:
>> printk("fr%d", rn);
>> break;
>> @@ -488,6 +489,7 @@ static void print_sh_insn(u32 memaddr, u16 insn)
>> printk("xd%d", rn & ~1);
>> break;
>> }
>> + /* else, fall through */
>> d_reg_n:
>
> and remove this only use of d_reg_n
>
Sure.
Thanks
--
Gustavo
>> case D_REG_N:
>> printk("dr%d", rn);
>> @@ -497,6 +499,7 @@ static void print_sh_insn(u32 memaddr, u16 insn)
>> printk("xd%d", rm & ~1);
>> break;
>> }
>> + /* else, fall through */
>> case D_REG_M:
>> printk("dr%d", rm);
>> break;
>
