ALSA: firewire-tascam: fix an LED bug
authorDan Carpenter <dan.carpenter@oracle.com>
Thu, 15 Oct 2015 18:16:30 +0000 (21:16 +0300)
committerTakashi Iwai <tiwai@suse.de>
Sat, 17 Oct 2015 10:04:16 +0000 (12:04 +0200)
We recently tried to add some new code to support turning the LED on and
off but the code in snd_tscm_transaction_reregister() is unreachable.

Fixes: e65e2cb99e44 ('ALSA: firewire-tascam: Turn on/off FireWire LED')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/firewire/tascam/tascam-transaction.c

index 1c9a88be55c83b2bdab066717092ba339083248d..45c3ce3d6020c985c89cf94c995d736a2aff8375 100644 (file)
@@ -249,7 +249,7 @@ int snd_tscm_transaction_reregister(struct snd_tscm *tscm)
 
        /* Turn on messaging. */
        reg = cpu_to_be32(0x00000001);
-       return snd_fw_transaction(tscm->unit, TCODE_WRITE_QUADLET_REQUEST,
+       err = snd_fw_transaction(tscm->unit, TCODE_WRITE_QUADLET_REQUEST,
                                  TSCM_ADDR_BASE + TSCM_OFFSET_MIDI_TX_ON,
                                  &reg, sizeof(reg), 0);
        if (err < 0)