]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
ALSA: usb-audio: Follow standard coding style
authorTakashi Iwai <tiwai@suse.de>
Sun, 27 May 2018 13:18:22 +0000 (15:18 +0200)
committerTakashi Iwai <tiwai@suse.de>
Mon, 28 May 2018 12:52:40 +0000 (14:52 +0200)
Avoid if ((err = ...) style and expand to multiple lines instead.
No change in the end result, but just the beautification.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/usb/card.c
sound/usb/clock.c
sound/usb/mixer.c
sound/usb/mixer_quirks.c
sound/usb/pcm.c

index f6c3c1cd591e98600d5c81a041e77a4ce7fd49f8..54c77d407a6d362d0c817ca0d6b42aa09ed461f1 100644 (file)
@@ -508,7 +508,8 @@ static int snd_usb_audio_create(struct usb_interface *intf,
        INIT_LIST_HEAD(&chip->midi_list);
        INIT_LIST_HEAD(&chip->mixer_list);
 
-       if ((err = snd_device_new(card, SNDRV_DEV_LOWLEVEL, chip, &ops)) < 0) {
+       err = snd_device_new(card, SNDRV_DEV_LOWLEVEL, chip, &ops);
+       if (err < 0) {
                snd_usb_audio_free(chip);
                snd_card_free(card);
                return err;
index 17673f37fcc8c1d4e7dfda23b9d6aceafd604e98..c79749613fa601703850d2f75bdaff0ff8fadf7e 100644 (file)
@@ -443,10 +443,11 @@ static int set_sample_rate_v1(struct snd_usb_audio *chip, int iface,
        data[0] = rate;
        data[1] = rate >> 8;
        data[2] = rate >> 16;
-       if ((err = snd_usb_ctl_msg(dev, usb_sndctrlpipe(dev, 0), UAC_SET_CUR,
-                                  USB_TYPE_CLASS | USB_RECIP_ENDPOINT | USB_DIR_OUT,
-                                  UAC_EP_CS_ATTR_SAMPLE_RATE << 8, ep,
-                                  data, sizeof(data))) < 0) {
+       err = snd_usb_ctl_msg(dev, usb_sndctrlpipe(dev, 0), UAC_SET_CUR,
+                             USB_TYPE_CLASS | USB_RECIP_ENDPOINT | USB_DIR_OUT,
+                             UAC_EP_CS_ATTR_SAMPLE_RATE << 8, ep,
+                             data, sizeof(data));
+       if (err < 0) {
                dev_err(&dev->dev, "%d:%d: cannot set freq %d to ep %#x\n",
                        iface, fmt->altsetting, rate, ep);
                return err;
@@ -460,10 +461,11 @@ static int set_sample_rate_v1(struct snd_usb_audio *chip, int iface,
        if (chip->sample_rate_read_error > 2)
                return 0;
 
-       if ((err = snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0), UAC_GET_CUR,
-                                  USB_TYPE_CLASS | USB_RECIP_ENDPOINT | USB_DIR_IN,
-                                  UAC_EP_CS_ATTR_SAMPLE_RATE << 8, ep,
-                                  data, sizeof(data))) < 0) {
+       err = snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0), UAC_GET_CUR,
+                             USB_TYPE_CLASS | USB_RECIP_ENDPOINT | USB_DIR_IN,
+                             UAC_EP_CS_ATTR_SAMPLE_RATE << 8, ep,
+                             data, sizeof(data));
+       if (err < 0) {
                dev_err(&dev->dev, "%d:%d: cannot get freq at ep %#x\n",
                        iface, fmt->altsetting, ep);
                chip->sample_rate_read_error++;
index bf74e7edc92b011a054d939b9853f4194b4ef78b..898afd3001ea0bb64f4fa9237df32c41bc68b2c2 100644 (file)
@@ -598,7 +598,8 @@ int snd_usb_mixer_add_control(struct usb_mixer_elem_list *list,
 
        while (snd_ctl_find_id(mixer->chip->card, &kctl->id))
                kctl->id.index++;
-       if ((err = snd_ctl_add(mixer->chip->card, kctl)) < 0) {
+       err = snd_ctl_add(mixer->chip->card, kctl);
+       if (err < 0) {
                usb_audio_dbg(mixer->chip, "cannot add control (err = %d)\n",
                              err);
                return err;
@@ -1850,7 +1851,8 @@ static int parse_audio_feature_unit(struct mixer_build *state, int unitid,
        }
 
        /* parse the source unit */
-       if ((err = parse_audio_unit(state, hdr->bSourceID)) < 0)
+       err = parse_audio_unit(state, hdr->bSourceID);
+       if (err < 0)
                return err;
 
        /* determine the input source type and name */
@@ -2270,7 +2272,8 @@ static int build_audio_procunit(struct mixer_build *state, int unitid,
        }
 
        for (i = 0; i < num_ins; i++) {
-               if ((err = parse_audio_unit(state, desc->baSourceID[i])) < 0)
+               err = parse_audio_unit(state, desc->baSourceID[i]);
+               if (err < 0)
                        return err;
        }
 
@@ -2483,7 +2486,8 @@ static int parse_audio_selector_unit(struct mixer_build *state, int unitid,
        }
 
        for (i = 0; i < desc->bNrInPins; i++) {
-               if ((err = parse_audio_unit(state, desc->baSourceID[i])) < 0)
+               err = parse_audio_unit(state, desc->baSourceID[i]);
+               if (err < 0)
                        return err;
        }
 
@@ -3310,11 +3314,16 @@ int snd_usb_create_mixer(struct snd_usb_audio *chip, int ctrlif,
 
        if (mixer->protocol == UAC_VERSION_3 &&
                        chip->badd_profile >= UAC3_FUNCTION_SUBCLASS_GENERIC_IO) {
-               if ((err = snd_usb_mixer_controls_badd(mixer, ctrlif)) < 0)
+               err = snd_usb_mixer_controls_badd(mixer, ctrlif);
+               if (err < 0)
+                       goto _error;
+       } else {
+               err = snd_usb_mixer_controls(mixer);
+               if (err < 0)
+                       goto _error;
+               err = snd_usb_mixer_status_create(mixer);
+               if (err < 0)
                        goto _error;
-       } else if ((err = snd_usb_mixer_controls(mixer)) < 0 ||
-                       (err = snd_usb_mixer_status_create(mixer)) < 0) {
-               goto _error;
        }
        err = create_keep_iface_ctl(mixer);
        if (err < 0)
index 1b94387e18b693c99e0d4b14fe2d54b2f1eb6c17..4149543f613ea984ecc5be1e53bf6de0b9503619 100644 (file)
@@ -1824,7 +1824,8 @@ int snd_usb_mixer_apply_create_quirk(struct usb_mixer_interface *mixer)
        int err = 0;
        struct snd_info_entry *entry;
 
-       if ((err = snd_usb_soundblaster_remote_init(mixer)) < 0)
+       err = snd_usb_soundblaster_remote_init(mixer);
+       if (err < 0)
                return err;
 
        switch (mixer->chip->usb_id) {
index a66bc717b952a0eb45a307fea4317510698ce012..897a2cbef6de146f56a6cebc8917b74cad429de0 100644 (file)
@@ -163,10 +163,11 @@ static int init_pitch_v1(struct snd_usb_audio *chip, int iface,
        ep = get_endpoint(alts, 0)->bEndpointAddress;
 
        data[0] = 1;
-       if ((err = snd_usb_ctl_msg(dev, usb_sndctrlpipe(dev, 0), UAC_SET_CUR,
-                                  USB_TYPE_CLASS|USB_RECIP_ENDPOINT|USB_DIR_OUT,
-                                  UAC_EP_CS_ATTR_PITCH_CONTROL << 8, ep,
-                                  data, sizeof(data))) < 0) {
+       err = snd_usb_ctl_msg(dev, usb_sndctrlpipe(dev, 0), UAC_SET_CUR,
+                             USB_TYPE_CLASS|USB_RECIP_ENDPOINT|USB_DIR_OUT,
+                             UAC_EP_CS_ATTR_PITCH_CONTROL << 8, ep,
+                             data, sizeof(data));
+       if (err < 0) {
                usb_audio_err(chip, "%d:%d: cannot set enable PITCH\n",
                              iface, ep);
                return err;
@@ -184,10 +185,11 @@ static int init_pitch_v2(struct snd_usb_audio *chip, int iface,
        int err;
 
        data[0] = 1;
-       if ((err = snd_usb_ctl_msg(dev, usb_sndctrlpipe(dev, 0), UAC2_CS_CUR,
-                                  USB_TYPE_CLASS | USB_RECIP_ENDPOINT | USB_DIR_OUT,
-                                  UAC2_EP_CS_PITCH << 8, 0,
-                                  data, sizeof(data))) < 0) {
+       err = snd_usb_ctl_msg(dev, usb_sndctrlpipe(dev, 0), UAC2_CS_CUR,
+                             USB_TYPE_CLASS | USB_RECIP_ENDPOINT | USB_DIR_OUT,
+                             UAC2_EP_CS_PITCH << 8, 0,
+                             data, sizeof(data));
+       if (err < 0) {
                usb_audio_err(chip, "%d:%d: cannot set enable PITCH (v2)\n",
                              iface, fmt->altsetting);
                return err;