On Thu, 07 Nov 2024 14:33:48 +0100, Colin Ian King wrote: > > After an initial range change on the insigned int alt being > 1 > the only possible values for alt are 0 or 1. Therefore the else > statement for values other than 0 or 1 is redundant and can be > removed. Replace the else if (all == 1) check with just an else. > > Signed-off-by: Colin Ian King <colin.i.king@xxxxxxxxx> Reviewed-by: Takashi Iwai <tiwai@xxxxxxx> Also worth to put the original discussion thread: Link: https://lore.kernel.org/5f54ffd0-b5fe-4203-a626-c166becad362@xxxxxxxxx thanks, Takashi > --- > drivers/usb/gadget/function/f_midi2.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/usb/gadget/function/f_midi2.c b/drivers/usb/gadget/function/f_midi2.c > index 8285df9ed6fd..5f3f6e7700c7 100644 > --- a/drivers/usb/gadget/function/f_midi2.c > +++ b/drivers/usb/gadget/function/f_midi2.c > @@ -1282,16 +1282,14 @@ static int f_midi2_set_alt(struct usb_function *fn, unsigned int intf, > > if (intf != midi2->midi_if || alt > 1) > return 0; > > if (alt == 0) > op_mode = MIDI_OP_MODE_MIDI1; > - else if (alt == 1) > + else > op_mode = MIDI_OP_MODE_MIDI2; > - else > - op_mode = MIDI_OP_MODE_UNSET; > > if (midi2->operation_mode == op_mode) > return 0; > > midi2->operation_mode = op_mode; > > -- > 2.39.5 >
