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:
4 edited

Legend:

Unmodified
Added
Removed

  • GPL/trunk

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

    r695 r703  
    7878        if (err < 0)
    7979                return err;
    80         if ((err = snd_cs46xx_create(card, pci,
    81                                      external_amp[dev], thinkpad[dev],
    82                                      &chip)) < 0) {
     80        err = snd_cs46xx_create(card, pci,
     81                                external_amp[dev], thinkpad[dev],
     82                                &chip);
     83        if (err < 0) {
    8384                snd_card_free(card);
    8485                return err;
    … …  
    8687        card->private_data = chip;
    8788        chip->accept_valid = mmap_valid[dev];
    88         if ((err = snd_cs46xx_pcm(chip, 0)) < 0) {
     89        err = snd_cs46xx_pcm(chip, 0);
     90        if (err < 0) {
    8991                snd_card_free(card);
    9092                return err;
    9193        }
    9294#ifdef CONFIG_SND_CS46XX_NEW_DSP
    93         if ((err = snd_cs46xx_pcm_rear(chip, 1)) < 0) {
     95        err = snd_cs46xx_pcm_rear(chip, 1);
     96        if (err < 0) {
    9497                snd_card_free(card);
    9598                return err;
    9699        }
    97         if ((err = snd_cs46xx_pcm_iec958(chip, 2)) < 0) {
     100        err = snd_cs46xx_pcm_iec958(chip, 2);
     101        if (err < 0) {
    98102                snd_card_free(card);
    99103                return err;
    100104        }
    101105#endif
    102         if ((err = snd_cs46xx_mixer(chip, 2)) < 0) {
     106        err = snd_cs46xx_mixer(chip, 2);
     107        if (err < 0) {
    103108                snd_card_free(card);
    104109                return err;
    … …  
    106111#ifdef CONFIG_SND_CS46XX_NEW_DSP
    107112        if (chip->nr_ac97_codecs ==2) {
    108                 if ((err = snd_cs46xx_pcm_center_lfe(chip, 3)) < 0) {
     113                err = snd_cs46xx_pcm_center_lfe(chip, 3);
     114                if (err < 0) {
    109115                        snd_card_free(card);
    110116                        return err;
    … …  
    112118        }
    113119#endif
    114         if ((err = snd_cs46xx_midi(chip, 0)) < 0) {
     120        err = snd_cs46xx_midi(chip, 0);
     121        if (err < 0) {
    115122                snd_card_free(card);
    116123                return err;
    117124        }
    118         if ((err = snd_cs46xx_start_dsp(chip)) < 0) {
     125        err = snd_cs46xx_start_dsp(chip);
     126        if (err < 0) {
    119127                snd_card_free(card);
    120128                return err;
    … …  
    132140                chip->irq);
    133141
    134         if ((err = snd_card_register(card)) < 0) {
     142        err = snd_card_register(card);
     143        if (err < 0) {
    135144                snd_card_free(card);
    136145                return err;

  • GPL/trunk/alsa-kernel/pci/cs46xx/cs46xx_lib.c

    r695 r703  
    10631063                cs46xx_dsp_destroy_pcm_channel (chip,cpcm->pcm_channel);
    10641064
    1065                 if ( (cpcm->pcm_channel = cs46xx_dsp_create_pcm_channel (chip, sample_rate, cpcm,
    1066                                                                          cpcm->hw_buf.addr,
    1067                                                                          cpcm->pcm_channel_id)) == NULL) {
     1065                cpcm->pcm_channel = cs46xx_dsp_create_pcm_channel(chip, sample_rate, cpcm,
     1066                                                                  cpcm->hw_buf.addr,
     1067                                                                  cpcm->pcm_channel_id);
     1068                if (!cpcm->pcm_channel) {
    10681069                        dev_err(chip->card->dev,
    10691070                                "failed to re-create virtual PCM channel\n");
    … …  
    11521153                        runtime->dma_bytes = 0;
    11531154                }
    1154                 if ((err = snd_pcm_lib_malloc_pages(substream, params_buffer_bytes(hw_params))) < 0) {
     1155                err = snd_pcm_lib_malloc_pages(substream, params_buffer_bytes(hw_params));
     1156                if (err < 0) {
    11551157#ifdef CONFIG_SND_CS46XX_NEW_DSP
    11561158                        mutex_unlock(&chip->spos_mutex);
    … …  
    13001302                        runtime->dma_bytes = 0;
    13011303                }
    1302                 if ((err = snd_pcm_lib_malloc_pages(substream, params_buffer_bytes(hw_params))) < 0)
     1304                err = snd_pcm_lib_malloc_pages(substream, params_buffer_bytes(hw_params));
     1305                if (err < 0)
    13031306                        return err;
    13041307                substream->ops = &snd_cs46xx_capture_indirect_ops;
    … …  
    17651768        int err;
    17661769
    1767         if ((err = snd_pcm_new(chip->card, "CS46xx", device, MAX_PLAYBACK_CHANNELS, 1, &pcm)) < 0)
     1770        err = snd_pcm_new(chip->card, "CS46xx", device, MAX_PLAYBACK_CHANNELS, 1, &pcm);
     1771        if (err < 0)
    17681772                return err;
    17691773
    … …  
    17921796        int err;
    17931797
    1794         if ((err = snd_pcm_new(chip->card, "CS46xx - Rear", device, MAX_PLAYBACK_CHANNELS, 0, &pcm)) < 0)
     1798        err = snd_pcm_new(chip->card, "CS46xx - Rear", device, MAX_PLAYBACK_CHANNELS, 0, &pcm);
     1799        if (err < 0)
    17951800                return err;
    17961801
    … …  
    18161821        int err;
    18171822
    1818         if ((err = snd_pcm_new(chip->card, "CS46xx - Center LFE", device, MAX_PLAYBACK_CHANNELS, 0, &pcm)) < 0)
     1823        err = snd_pcm_new(chip->card, "CS46xx - Center LFE", device, MAX_PLAYBACK_CHANNELS, 0, &pcm);
     1824        if (err < 0)
    18191825                return err;
    18201826
    … …  
    18401846        int err;
    18411847
    1842         if ((err = snd_pcm_new(chip->card, "CS46xx - IEC958", device, 1, 0, &pcm)) < 0)
     1848        err = snd_pcm_new(chip->card, "CS46xx - IEC958", device, 1, 0, &pcm);
     1849        if (err < 0)
    18431850                return err;
    18441851
    … …  
    24192426                /* test if we can write to the record gain volume register */
    24202427                snd_ac97_write(ac97, AC97_REC_GAIN, 0x8a05);
    2421                 if ((err = snd_ac97_read(ac97, AC97_REC_GAIN)) == 0x8a05)
     2428                err = snd_ac97_read(ac97, AC97_REC_GAIN);
     2429                if (err == 0x8a05)
    24222430                        return;
    24232431
    … …  
    24812489        chip->nr_ac97_codecs = 0;
    24822490        dev_dbg(chip->card->dev, "detecting primary codec\n");
    2483         if ((err = snd_ac97_bus(card, 0, &ops, chip, &chip->ac97_bus)) < 0)
     2491        err = snd_ac97_bus(card, 0, &ops, chip, &chip->ac97_bus);
     2492        if (err < 0)
    24842493                return err;
    24852494        chip->ac97_bus->private_free = snd_cs46xx_mixer_free_ac97_bus;
    … …  
    25022511                if (kctl && kctl->id.iface == SNDRV_CTL_ELEM_IFACE_PCM)
    25032512                        kctl->id.device = spdif_device;
    2504                 if ((err = snd_ctl_add(card, kctl)) < 0)
     2513                err = snd_ctl_add(card, kctl);
     2514                if (err < 0)
    25052515                        return err;
    25062516        }
    … …  
    26892699        int err;
    26902700
    2691         if ((err = snd_rawmidi_new(chip->card, "CS46XX", device, 1, 1, &rmidi)) < 0)
     2701        err = snd_rawmidi_new(chip->card, "CS46XX", device, 1, 1, &rmidi);
     2702        if (err < 0)
    26922703                return err;
    26932704        strcpy(rmidi->name, "CS46XX");
    … …  
    35313542
    35323543                kctl = snd_ctl_new1(&snd_hercules_controls[idx], chip);
    3533                 if ((err = snd_ctl_add(card, kctl)) < 0) {
     3544                err = snd_ctl_add(card, kctl);
     3545                if (err < 0) {
    35343546                        dev_err(card->dev,
    35353547                                "failed to initialize Hercules mixer (%d)\n",
    … …  
    38763888
    38773889        /* enable PCI device */
    3878         if ((err = pci_enable_device(pci)) < 0)
     3890        err = pci_enable_device(pci);
     3891        if (err < 0)
    38793892                return err;
    38803893
    … …  
    39703983        for (idx = 0; idx < 5; idx++) {
    39713984                region = &chip->region.idx[idx];
    3972                 if ((region->resource = request_mem_region(region->base, region->size,
    3973                                                            region->name)) == NULL) {
     3985                region->resource = request_mem_region(region->base, region->size,
     3986                                                      region->name);
     3987                if (!region->resource) {
    39743988                        dev_err(chip->card->dev,
    39753989                                "unable to request memory region 0x%lx-0x%lx\n",
    … …  
    40104024        }
    40114025
    4012         if ((err = snd_device_new(card, SNDRV_DEV_LOWLEVEL, chip, &ops)) < 0) {
     4026        err = snd_device_new(card, SNDRV_DEV_LOWLEVEL, chip, &ops);
     4027        if (err < 0) {
    40134028                snd_cs46xx_free(chip);
    40144029                return err;

  • GPL/trunk/alsa-kernel/pci/cs46xx/dsp_spos.c

    r679 r703  
    618618                }
    619619
    620                 if ( (symbol = cs46xx_dsp_lookup_symbol_addr (chip,i / sizeof(u32), SYMBOL_PARAMETER)) != NULL) {
     620                symbol = cs46xx_dsp_lookup_symbol_addr(chip, i / sizeof(u32), SYMBOL_PARAMETER);
     621                if (symbol) {
    621622                        col = 0;
    622623                        snd_iprintf (buffer,"\n%s:\n",symbol->symbol_name);
Note: See TracChangeset for help on using the changeset viewer.