Ignore:
Timestamp:
Sep 26, 2021, 6:18:40 PM (4 years ago)
Author:
David Azarewicz
Message:

Merge changes from next branch.

Location:
GPL/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • GPL/trunk

  • GPL/trunk/alsa-kernel/pci/cmipci.c

    r695 r703  
    12351235        rate = subs->runtime->rate;
    12361236
    1237         if (up && do_ac3)
    1238                 if ((err = save_mixer_state(cm)) < 0)
     1237        if (up && do_ac3) {
     1238                err = save_mixer_state(cm);
     1239                if (err < 0)
    12391240                        return err;
     1241        }
    12401242
    12411243        spin_lock_irq(&cm->reg_lock);
    … …  
    12861288        if (do_spdif && cm->can_ac3_hw)
    12871289                do_ac3 = cm->dig_pcm_status & IEC958_AES0_NONAUDIO;
    1288         if ((err = setup_spdif_playback(cm, substream, do_spdif, do_ac3)) < 0)
     1290        err = setup_spdif_playback(cm, substream, do_spdif, do_ac3);
     1291        if (err < 0)
    12891292                return err;
    12901293        return snd_cmipci_pcm_prepare(cm, &cm->channel[CM_CH_PLAY], substream);
    … …  
    13011304        else
    13021305                do_ac3 = 1; /* doesn't matter */
    1303         if ((err = setup_spdif_playback(cm, substream, 1, do_ac3)) < 0)
     1306        err = setup_spdif_playback(cm, substream, 1, do_ac3);
     1307        if (err < 0)
    13041308                return err;
    13051309        return snd_cmipci_pcm_prepare(cm, &cm->channel[CM_CH_PLAY], substream);
    … …  
    16491653        int err;
    16501654
    1651         if ((err = open_device_check(cm, CM_OPEN_PLAYBACK, substream)) < 0)
     1655        err = open_device_check(cm, CM_OPEN_PLAYBACK, substream);
     1656        if (err < 0)
    16521657                return err;
    16531658        runtime->hw = snd_cmipci_playback;
    … …  
    16751680        int err;
    16761681
    1677         if ((err = open_device_check(cm, CM_OPEN_CAPTURE, substream)) < 0)
     1682        err = open_device_check(cm, CM_OPEN_CAPTURE, substream);
     1683        if (err < 0)
    16781684                return err;
    16791685        runtime->hw = snd_cmipci_capture;
    … …  
    16991705        int err;
    17001706
    1701         if ((err = open_device_check(cm, CM_OPEN_PLAYBACK2, substream)) < 0) /* use channel B */
     1707        /* use channel B */
     1708        err = open_device_check(cm, CM_OPEN_PLAYBACK2, substream);
     1709        if (err < 0)
    17021710                return err;
    17031711        runtime->hw = snd_cmipci_playback2;
    … …  
    17371745        int err;
    17381746
    1739         if ((err = open_device_check(cm, CM_OPEN_SPDIF_PLAYBACK, substream)) < 0) /* use channel A */
     1747        /* use channel A */
     1748        err = open_device_check(cm, CM_OPEN_SPDIF_PLAYBACK, substream);
     1749        if (err < 0)
    17401750                return err;
    17411751        if (cm->can_ac3_hw) {
    … …  
    17641774        int err;
    17651775
    1766         if ((err = open_device_check(cm, CM_OPEN_SPDIF_CAPTURE, substream)) < 0) /* use channel B */
     1776        /* use channel B */
     1777        err = open_device_check(cm, CM_OPEN_SPDIF_CAPTURE, substream);
     1778        if (err < 0)
    17671779                return err;
    17681780        runtime->hw = snd_cmipci_capture_spdif;
    … …  
    26712683                                continue;
    26722684                }
    2673                 if ((err = snd_ctl_add(card, snd_ctl_new1(&snd_cmipci_mixers[idx], cm))) < 0)
     2685                err = snd_ctl_add(card, snd_ctl_new1(&snd_cmipci_mixers[idx], cm));
     2686                if (err < 0)
    26742687                        return err;
    26752688        }
    … …  
    26962709                }
    26972710                if (cm->can_ac3_hw) {
    2698                         if ((err = snd_ctl_add(card, kctl = snd_ctl_new1(&snd_cmipci_spdif_default, cm))) < 0)
     2711                        kctl = snd_ctl_new1(&snd_cmipci_spdif_default, cm);
     2712                        err = snd_ctl_add(card, kctl);
     2713                        if (err < 0)
    26992714                                return err;
    27002715                        kctl->id.device = pcm_spdif_device;
    2701                         if ((err = snd_ctl_add(card, kctl = snd_ctl_new1(&snd_cmipci_spdif_mask, cm))) < 0)
     2716                        kctl = snd_ctl_new1(&snd_cmipci_spdif_mask, cm);
     2717                        err = snd_ctl_add(card, kctl);
     2718                        if (err < 0)
    27022719                                return err;
    27032720                        kctl->id.device = pcm_spdif_device;
    2704                         if ((err = snd_ctl_add(card, kctl = snd_ctl_new1(&snd_cmipci_spdif_stream, cm))) < 0)
     2721                        kctl = snd_ctl_new1(&snd_cmipci_spdif_stream, cm);
     2722                        err = snd_ctl_add(card, kctl);
     2723                        if (err < 0)
    27052724                                return err;
    27062725                        kctl->id.device = pcm_spdif_device;
    … …  
    29762995                }
    29772996        }
    2978         if ((err = snd_opl3_hwdep_new(opl3, 0, 1, NULL)) < 0) {
     2997        err = snd_opl3_hwdep_new(opl3, 0, 1, NULL);
     2998        if (err < 0) {
    29792999                dev_err(cm->card->dev, "cannot create OPL3 hwdep\n");
    29803000                return err;
    … …  
    30083028        *rcmipci = NULL;
    30093029
    3010         if ((err = pci_enable_device(pci)) < 0)
     3030        err = pci_enable_device(pci);
     3031        if (err < 0)
    30113032                return err;
    30123033
    … …  
    30273048        cm->channel[0].is_dac = cm->channel[1].is_dac = 1; /* dual DAC mode */
    30283049
    3029         if ((err = pci_request_regions(pci, card->driver)) < 0) {
     3050        err = pci_request_regions(pci, card->driver);
     3051        if (err < 0) {
    30303052                kfree(cm);
    30313053                pci_disable_device(pci);
    … …  
    31413163                card->shortname, modelstr, cm->iobase, cm->irq);
    31423164
    3143         if ((err = snd_device_new(card, SNDRV_DEV_LOWLEVEL, cm, &ops)) < 0) {
     3165        err = snd_device_new(card, SNDRV_DEV_LOWLEVEL, cm, &ops);
     3166        if (err < 0) {
    31443167                snd_cmipci_free(cm);
    31453168                return err;
    … …  
    31933216        /* create pcm devices */
    31943217        pcm_index = pcm_spdif_index = 0;
    3195         if ((err = snd_cmipci_pcm_new(cm, pcm_index)) < 0)
     3218        err = snd_cmipci_pcm_new(cm, pcm_index);
     3219        if (err < 0)
    31963220                return err;
    31973221        pcm_index++;
    3198         if ((err = snd_cmipci_pcm2_new(cm, pcm_index)) < 0)
     3222        err = snd_cmipci_pcm2_new(cm, pcm_index);
     3223        if (err < 0)
    31993224                return err;
    32003225        pcm_index++;
    32013226        if (cm->can_ac3_hw || cm->can_ac3_sw) {
    32023227                pcm_spdif_index = pcm_index;
    3203                 if ((err = snd_cmipci_pcm_spdif_new(cm, pcm_index)) < 0)
     3228                err = snd_cmipci_pcm_spdif_new(cm, pcm_index);
     3229                if (err < 0)
    32043230                        return err;
    32053231        }
    32063232
    32073233        /* create mixer interface & switches */
    3208         if ((err = snd_cmipci_mixer_new(cm, pcm_spdif_index)) < 0)
     3234        err = snd_cmipci_mixer_new(cm, pcm_spdif_index);
     3235        if (err < 0)
    32093236                return err;
    32103237
    32113238        if (iomidi > 0) {
    3212                 if ((err = snd_mpu401_uart_new(card, 0, MPU401_HW_CMIPCI,
    3213                                                iomidi,
    3214                                                (integrated_midi ?
    3215                                                 MPU401_INFO_INTEGRATED : 0) |
    3216                                                MPU401_INFO_IRQ_HOOK,
    3217                                                -1, &cm->rmidi)) < 0) {
     3239                err = snd_mpu401_uart_new(card, 0, MPU401_HW_CMIPCI,
     3240                                          iomidi,
     3241                                          (integrated_midi ?
     3242                                           MPU401_INFO_INTEGRATED : 0) |
     3243                                          MPU401_INFO_IRQ_HOOK,
     3244                                          -1, &cm->rmidi);
     3245                if (err < 0)
    32183246                        dev_err(cm->card->dev,
    32193247                                "no UART401 device at 0x%lx\n", iomidi);
    3220                 }
    32213248        }
    32223249
Note: See TracChangeset for help on using the changeset viewer.