After removing code, only one case remains. So use an 'if' instead. Acked-by: Daniel Mack <daniel@xxxxxxxx> Signed-off-by: Mark Hills <mark@xxxxxxxxxxx> --- sound/usb/caiaq/control.c | 6 ++---- 1 files changed, 2 insertions(+), 4 deletions(-) diff --git a/sound/usb/caiaq/control.c b/sound/usb/caiaq/control.c index 70c3866..91c804c 100644 --- a/sound/usb/caiaq/control.c +++ b/sound/usb/caiaq/control.c @@ -95,11 +95,9 @@ static int control_put(struct snd_kcontrol *kcontrol, int pos = kcontrol->private_value; unsigned char cmd = EP1_CMD_WRITE_IO; - switch (dev->chip.usb_id) { - case USB_ID(USB_VID_NATIVEINSTRUMENTS, USB_PID_TRAKTORKONTROLX1): + if (dev->chip.usb_id == + USB_ID(USB_VID_NATIVEINSTRUMENTS, USB_PID_TRAKTORKONTROLX1)) cmd = EP1_CMD_DIMM_LEDS; - break; - } if (pos & CNT_INTVAL) { dev->control_state[pos & ~CNT_INTVAL] -- 1.7.1 _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel