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

Legend:

Unmodified
Added
Removed

  • GPL/trunk

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

    r695 r703  
    7777                if (io_port == 1) {
    7878                        /* auto-detect */
    79                         if (!(io_port = pci_resource_start(chip->pci, 2)))
     79                        io_port = pci_resource_start(chip->pci, 2);
     80                        if (!io_port)
    8081                                return -ENODEV;
    8182                }
    … …  
    8687                                if (io_port == 0x203)
    8788                                        continue;
    88                                 if ((r = request_region(io_port, 1, "YMFPCI gameport")) != NULL)
     89                                r = request_region(io_port, 1, "YMFPCI gameport");
     90                                if (r)
    8991                                        break;
    9092                        }
    … …  
    107109        }
    108110
    109         if (!r && !(r = request_region(io_port, 1, "YMFPCI gameport"))) {
    110                 dev_err(chip->card->dev,
    111                         "joystick port %#x is in use.\n", io_port);
    112                 return -EBUSY;
     111        if (!r) {
     112                r = request_region(io_port, 1, "YMFPCI gameport");
     113                if (!r) {
     114                        dev_err(chip->card->dev,
     115                                "joystick port %#x is in use.\n", io_port);
     116                        return -EBUSY;
     117                }
    113118        }
    114119
    … …  
    198203                        fm_port[dev] = pci_resource_start(pci, 1);
    199204                }
    200                 if (fm_port[dev] > 0 &&
    201                     (fm_res = request_region(fm_port[dev], 4, "YMFPCI OPL3")) != NULL) {
     205                if (fm_port[dev] > 0)
     206                        fm_res = request_region(fm_port[dev], 4, "YMFPCI OPL3");
     207                if (fm_res) {
    202208                        legacy_ctrl |= YMFPCI_LEGACY_FMEN;
    203209                        pci_write_config_word(pci, PCIR_DSXG_FMBASE, fm_port[dev]);
    … …  
    207213                        mpu_port[dev] = pci_resource_start(pci, 1) + 0x20;
    208214                }
    209                 if (mpu_port[dev] > 0 &&
    210                     (mpu_res = request_region(mpu_port[dev], 2, "YMFPCI MPU401")) != NULL) {
     215                if (mpu_port[dev] > 0)
     216                        mpu_res = request_region(mpu_port[dev], 2, "YMFPCI MPU401");
     217                if (mpu_res) {
    211218                        legacy_ctrl |= YMFPCI_LEGACY_MEN;
    212219                        pci_write_config_word(pci, PCIR_DSXG_MPU401BASE, mpu_port[dev]);
    … …  
    220227                default: fm_port[dev] = 0; break;
    221228                }
    222                 if (fm_port[dev] > 0 &&
    223                     (fm_res = request_region(fm_port[dev], 4, "YMFPCI OPL3")) != NULL) {
     229                if (fm_port[dev] > 0)
     230                        fm_res = request_region(fm_port[dev], 4, "YMFPCI OPL3");
     231                if (fm_res) {
    224232                        legacy_ctrl |= YMFPCI_LEGACY_FMEN;
    225233                } else {
    … …  
    234242                default: mpu_port[dev] = 0; break;
    235243                }
    236                 if (mpu_port[dev] > 0 &&
    237                     (mpu_res = request_region(mpu_port[dev], 2, "YMFPCI MPU401")) != NULL) {
     244                if (mpu_port[dev] > 0)
     245                        mpu_res = request_region(mpu_port[dev], 2, "YMFPCI MPU401");
     246                if (mpu_res) {
    238247                        legacy_ctrl |= YMFPCI_LEGACY_MEN;
    239248                } else {
    … …  
    249258        pci_write_config_word(pci, PCIR_DSXG_LEGACY, legacy_ctrl);
    250259        pci_write_config_word(pci, PCIR_DSXG_ELEGACY, legacy_ctrl2);
    251         if ((err = snd_ymfpci_create(card, pci,
    252                                      old_legacy_ctrl,
    253                                      &chip)) < 0) {
     260        err = snd_ymfpci_create(card, pci, old_legacy_ctrl, &chip);
     261        if (err  < 0) {
    254262                release_and_free_resource(mpu_res);
    255263                release_and_free_resource(fm_res);
    … …  
    292300
    293301        if (chip->mpu_res) {
    294                 if ((err = snd_mpu401_uart_new(card, 0, MPU401_HW_YMFPCI,
    295                                                mpu_port[dev],
    296                                                MPU401_INFO_INTEGRATED |
    297                                                MPU401_INFO_IRQ_HOOK,
    298                                                -1, &chip->rawmidi)) < 0) {
     302                err = snd_mpu401_uart_new(card, 0, MPU401_HW_YMFPCI,
     303                                          mpu_port[dev],
     304                                          MPU401_INFO_INTEGRATED |
     305                                          MPU401_INFO_IRQ_HOOK,
     306                                          -1, &chip->rawmidi);
     307                if (err < 0) {
    299308                        dev_warn(card->dev,
    300309                                 "cannot initialize MPU401 at 0x%lx, skipping...\n",
    … …  
    305314        }
    306315        if (chip->fm_res) {
    307                 if ((err = snd_opl3_create(card,
    308                                            fm_port[dev],
    309                                            fm_port[dev] + 2,
    310                                            OPL3_HW_OPL3, 1, &opl3)) < 0) {
     316                err = snd_opl3_create(card,
     317                                      fm_port[dev],
     318                                      fm_port[dev] + 2,
     319                                      OPL3_HW_OPL3, 1, &opl3);
     320                if (err < 0) {
    311321                        dev_warn(card->dev,
    312322                                 "cannot initialize FM OPL3 at 0x%lx, skipping...\n",
    … …  
    314324                        legacy_ctrl &= ~YMFPCI_LEGACY_FMEN;
    315325                        pci_write_config_word(pci, PCIR_DSXG_LEGACY, legacy_ctrl);
    316                 } else if ((err = snd_opl3_hwdep_new(opl3, 0, 1, NULL)) < 0) {
    317                         dev_err(card->dev, "cannot create opl3 hwdep\n");
    318                         goto free_card;
     326                } else {
     327                        err = snd_opl3_hwdep_new(opl3, 0, 1, NULL);
     328                        if (err < 0) {
     329                                dev_err(card->dev, "cannot create opl3 hwdep\n");
     330                                goto free_card;
     331                        }
    319332                }
    320333        }

  • GPL/trunk/alsa-kernel/pci/ymfpci/ymfpci_main.c

    r679 r703  
    297297        u32 pos, delta;
    298298       
    299         if ((ypcm = voice->ypcm) == NULL)
     299        ypcm = voice->ypcm;
     300        if (!ypcm)
    300301                return;
    301302        if (ypcm->substream == NULL)
    … …  
    633634        int err;
    634635
    635         if ((err = snd_ymfpci_pcm_voice_alloc(ypcm, params_channels(hw_params))) < 0)
     636        err = snd_ymfpci_pcm_voice_alloc(ypcm, params_channels(hw_params));
     637        if (err < 0)
    636638                return err;
    637639        return 0;
    … …  
    937939        int err;
    938940       
    939         if ((err = snd_ymfpci_playback_open_1(substream)) < 0)
     941        err = snd_ymfpci_playback_open_1(substream);
     942        if (err < 0)
    940943                return err;
    941944        ypcm = runtime->private_data;
    … …  
    959962        int err;
    960963       
    961         if ((err = snd_ymfpci_playback_open_1(substream)) < 0)
     964        err = snd_ymfpci_playback_open_1(substream);
     965        if (err < 0)
    962966                return err;
    963967        ypcm = runtime->private_data;
    … …  
    987991        int err;
    988992       
    989         if ((err = snd_ymfpci_playback_open_1(substream)) < 0)
     993        err = snd_ymfpci_playback_open_1(substream);
     994        if (err < 0)
    990995                return err;
    991996        ypcm = runtime->private_data;
    … …  
    11291134        int err;
    11301135
    1131         if ((err = snd_pcm_new(chip->card, "YMFPCI", device, 32, 1, &pcm)) < 0)
     1136        err = snd_pcm_new(chip->card, "YMFPCI", device, 32, 1, &pcm);
     1137        if (err < 0)
    11321138                return err;
    11331139        pcm->private_data = chip;
    … …  
    11621168        int err;
    11631169
    1164         if ((err = snd_pcm_new(chip->card, "YMFPCI - PCM2", device, 0, 1, &pcm)) < 0)
     1170        err = snd_pcm_new(chip->card, "YMFPCI - PCM2", device, 0, 1, &pcm);
     1171        if (err < 0)
    11651172                return err;
    11661173        pcm->private_data = chip;
    … …  
    11951202        int err;
    11961203
    1197         if ((err = snd_pcm_new(chip->card, "YMFPCI - IEC958", device, 1, 0, &pcm)) < 0)
     1204        err = snd_pcm_new(chip->card, "YMFPCI - IEC958", device, 1, 0, &pcm);
     1205        if (err < 0)
    11981206                return err;
    11991207        pcm->private_data = chip;
    … …  
    12351243        int err;
    12361244
    1237         if ((err = snd_pcm_new(chip->card, "YMFPCI - Rear", device, 1, 0, &pcm)) < 0)
     1245        err = snd_pcm_new(chip->card, "YMFPCI - Rear", device, 1, 0, &pcm);
     1246        if (err < 0)
    12381247                return err;
    12391248        pcm->private_data = chip;
    … …  
    17901799        };
    17911800
    1792         if ((err = snd_ac97_bus(chip->card, 0, &ops, chip, &chip->ac97_bus)) < 0)
     1801        err = snd_ac97_bus(chip->card, 0, &ops, chip, &chip->ac97_bus);
     1802        if (err < 0)
    17931803                return err;
    17941804        chip->ac97_bus->private_free = snd_ymfpci_mixer_free_ac97_bus;
    … …  
    17981808        ac97.private_data = chip;
    17991809        ac97.private_free = snd_ymfpci_mixer_free_ac97;
    1800         if ((err = snd_ac97_mixer(chip->ac97_bus, &ac97, &chip->ac97)) < 0)
     1810        err = snd_ac97_mixer(chip->ac97_bus, &ac97, &chip->ac97);
     1811        if (err < 0)
    18011812                return err;
    18021813
    … …  
    18061817
    18071818        for (idx = 0; idx < ARRAY_SIZE(snd_ymfpci_controls); idx++) {
    1808                 if ((err = snd_ctl_add(chip->card, snd_ctl_new1(&snd_ymfpci_controls[idx], chip))) < 0)
     1819                err = snd_ctl_add(chip->card, snd_ctl_new1(&snd_ymfpci_controls[idx], chip));
     1820                if (err < 0)
    18091821                        return err;
    18101822        }
    … …  
    18191831        if (snd_BUG_ON(!chip->pcm_spdif))
    18201832                return -ENXIO;
    1821         if ((err = snd_ctl_add(chip->card, kctl = snd_ctl_new1(&snd_ymfpci_spdif_default, chip))) < 0)
     1833        kctl = snd_ctl_new1(&snd_ymfpci_spdif_default, chip);
     1834        err = snd_ctl_add(chip->card, kctl);
     1835        if (err < 0)
    18221836                return err;
    18231837        kctl->id.device = chip->pcm_spdif->device;
    1824         if ((err = snd_ctl_add(chip->card, kctl = snd_ctl_new1(&snd_ymfpci_spdif_mask, chip))) < 0)
     1838        kctl = snd_ctl_new1(&snd_ymfpci_spdif_mask, chip);
     1839        err = snd_ctl_add(chip->card, kctl);
     1840        if (err < 0)
    18251841                return err;
    18261842        kctl->id.device = chip->pcm_spdif->device;
    1827         if ((err = snd_ctl_add(chip->card, kctl = snd_ctl_new1(&snd_ymfpci_spdif_stream, chip))) < 0)
     1843        kctl = snd_ctl_new1(&snd_ymfpci_spdif_stream, chip);
     1844        err = snd_ctl_add(chip->card, kctl);
     1845        if (err < 0)
    18281846                return err;
    18291847        kctl->id.device = chip->pcm_spdif->device;
    … …  
    18311849
    18321850        /* direct recording source */
    1833         if (chip->device_id == PCI_DEVICE_ID_YAMAHA_754 &&
    1834             (err = snd_ctl_add(chip->card, kctl = snd_ctl_new1(&snd_ymfpci_drec_source, chip))) < 0)
    1835                 return err;
     1851        if (chip->device_id == PCI_DEVICE_ID_YAMAHA_754) {
     1852                kctl = snd_ctl_new1(&snd_ymfpci_drec_source, chip);
     1853                err = snd_ctl_add(chip->card, kctl);
     1854                if (err < 0)
     1855                        return err;
     1856        }
    18361857
    18371858        /*
    … …  
    18391860         */
    18401861        if (rear_switch) {
    1841                 if ((err = snd_ctl_add(chip->card, snd_ctl_new1(&snd_ymfpci_rear_shared, chip))) < 0)
     1862                err = snd_ctl_add(chip->card, snd_ctl_new1(&snd_ymfpci_rear_shared, chip));
     1863                if (err < 0)
    18421864                        return err;
    18431865        }
    … …  
    18521874                kctl->id.subdevice = idx;
    18531875                kctl->private_value = (unsigned long)substream;
    1854                 if ((err = snd_ctl_add(chip->card, kctl)) < 0)
     1876                err = snd_ctl_add(chip->card, kctl);
     1877                if (err < 0)
    18551878                        return err;
    18561879                chip->pcm_mixer[idx].left = 0x8000;
    … …  
    19331956        tid.device = device;
    19341957        tid.subdevice = 0;
    1935         if ((err = snd_timer_new(chip->card, "YMFPCI", &tid, &timer)) >= 0) {
     1958        err = snd_timer_new(chip->card, "YMFPCI", &tid, &timer);
     1959        if (err >= 0) {
    19361960                strcpy(timer->name, "YMFPCI timer");
    19371961                timer->private_data = chip;
    … …  
    23392363
    23402364        /* enable PCI device */
    2341         if ((err = pci_enable_device(pci)) < 0)
     2365        err = pci_enable_device(pci);
     2366        if (err < 0)
    23422367                return err;
    23432368
    … …  
    23622387        chip->src441_used = -1;
    23632388
    2364         if ((chip->res_reg_area = request_mem_region(chip->reg_area_phys, 0x8000, "YMFPCI")) == NULL) {
     2389        chip->res_reg_area = request_mem_region(chip->reg_area_phys, 0x8000, "YMFPCI");
     2390        if (!chip->res_reg_area) {
    23652391                dev_err(chip->card->dev,
    23662392                        "unable to grab memory region 0x%lx-0x%lx\n",
Note: See TracChangeset for help on using the changeset viewer.