hc
2024-01-03 2f7c68cb55ecb7331f2381deb497c27155f32faf
kernel/sound/pci/ymfpci/ymfpci_main.c
....@@ -292,7 +292,8 @@
292292 struct snd_ymfpci_pcm *ypcm;
293293 u32 pos, delta;
294294
295
- if ((ypcm = voice->ypcm) == NULL)
295
+ ypcm = voice->ypcm;
296
+ if (!ypcm)
296297 return;
297298 if (ypcm->substream == NULL)
298299 return;
....@@ -628,7 +629,8 @@
628629 struct snd_ymfpci_pcm *ypcm = runtime->private_data;
629630 int err;
630631
631
- if ((err = snd_ymfpci_pcm_voice_alloc(ypcm, params_channels(hw_params))) < 0)
632
+ err = snd_ymfpci_pcm_voice_alloc(ypcm, params_channels(hw_params));
633
+ if (err < 0)
632634 return err;
633635 return 0;
634636 }
....@@ -932,7 +934,8 @@
932934 struct snd_ymfpci_pcm *ypcm;
933935 int err;
934936
935
- if ((err = snd_ymfpci_playback_open_1(substream)) < 0)
937
+ err = snd_ymfpci_playback_open_1(substream);
938
+ if (err < 0)
936939 return err;
937940 ypcm = runtime->private_data;
938941 ypcm->output_front = 1;
....@@ -954,7 +957,8 @@
954957 struct snd_ymfpci_pcm *ypcm;
955958 int err;
956959
957
- if ((err = snd_ymfpci_playback_open_1(substream)) < 0)
960
+ err = snd_ymfpci_playback_open_1(substream);
961
+ if (err < 0)
958962 return err;
959963 ypcm = runtime->private_data;
960964 ypcm->output_front = 0;
....@@ -982,7 +986,8 @@
982986 struct snd_ymfpci_pcm *ypcm;
983987 int err;
984988
985
- if ((err = snd_ymfpci_playback_open_1(substream)) < 0)
989
+ err = snd_ymfpci_playback_open_1(substream);
990
+ if (err < 0)
986991 return err;
987992 ypcm = runtime->private_data;
988993 ypcm->output_front = 0;
....@@ -1124,7 +1129,8 @@
11241129 struct snd_pcm *pcm;
11251130 int err;
11261131
1127
- if ((err = snd_pcm_new(chip->card, "YMFPCI", device, 32, 1, &pcm)) < 0)
1132
+ err = snd_pcm_new(chip->card, "YMFPCI", device, 32, 1, &pcm);
1133
+ if (err < 0)
11281134 return err;
11291135 pcm->private_data = chip;
11301136
....@@ -1157,7 +1163,8 @@
11571163 struct snd_pcm *pcm;
11581164 int err;
11591165
1160
- if ((err = snd_pcm_new(chip->card, "YMFPCI - PCM2", device, 0, 1, &pcm)) < 0)
1166
+ err = snd_pcm_new(chip->card, "YMFPCI - PCM2", device, 0, 1, &pcm);
1167
+ if (err < 0)
11611168 return err;
11621169 pcm->private_data = chip;
11631170
....@@ -1190,7 +1197,8 @@
11901197 struct snd_pcm *pcm;
11911198 int err;
11921199
1193
- if ((err = snd_pcm_new(chip->card, "YMFPCI - IEC958", device, 1, 0, &pcm)) < 0)
1200
+ err = snd_pcm_new(chip->card, "YMFPCI - IEC958", device, 1, 0, &pcm);
1201
+ if (err < 0)
11941202 return err;
11951203 pcm->private_data = chip;
11961204
....@@ -1230,7 +1238,8 @@
12301238 struct snd_pcm *pcm;
12311239 int err;
12321240
1233
- if ((err = snd_pcm_new(chip->card, "YMFPCI - Rear", device, 1, 0, &pcm)) < 0)
1241
+ err = snd_pcm_new(chip->card, "YMFPCI - Rear", device, 1, 0, &pcm);
1242
+ if (err < 0)
12341243 return err;
12351244 pcm->private_data = chip;
12361245
....@@ -1785,7 +1794,8 @@
17851794 .read = snd_ymfpci_codec_read,
17861795 };
17871796
1788
- if ((err = snd_ac97_bus(chip->card, 0, &ops, chip, &chip->ac97_bus)) < 0)
1797
+ err = snd_ac97_bus(chip->card, 0, &ops, chip, &chip->ac97_bus);
1798
+ if (err < 0)
17891799 return err;
17901800 chip->ac97_bus->private_free = snd_ymfpci_mixer_free_ac97_bus;
17911801 chip->ac97_bus->no_vra = 1; /* YMFPCI doesn't need VRA */
....@@ -1793,7 +1803,8 @@
17931803 memset(&ac97, 0, sizeof(ac97));
17941804 ac97.private_data = chip;
17951805 ac97.private_free = snd_ymfpci_mixer_free_ac97;
1796
- if ((err = snd_ac97_mixer(chip->ac97_bus, &ac97, &chip->ac97)) < 0)
1806
+ err = snd_ac97_mixer(chip->ac97_bus, &ac97, &chip->ac97);
1807
+ if (err < 0)
17971808 return err;
17981809
17991810 /* to be sure */
....@@ -1801,7 +1812,8 @@
18011812 AC97_EA_VRA|AC97_EA_VRM, 0);
18021813
18031814 for (idx = 0; idx < ARRAY_SIZE(snd_ymfpci_controls); idx++) {
1804
- if ((err = snd_ctl_add(chip->card, snd_ctl_new1(&snd_ymfpci_controls[idx], chip))) < 0)
1815
+ err = snd_ctl_add(chip->card, snd_ctl_new1(&snd_ymfpci_controls[idx], chip));
1816
+ if (err < 0)
18051817 return err;
18061818 }
18071819 if (chip->ac97->ext_id & AC97_EI_SDAC) {
....@@ -1814,27 +1826,37 @@
18141826 /* add S/PDIF control */
18151827 if (snd_BUG_ON(!chip->pcm_spdif))
18161828 return -ENXIO;
1817
- if ((err = snd_ctl_add(chip->card, kctl = snd_ctl_new1(&snd_ymfpci_spdif_default, chip))) < 0)
1829
+ kctl = snd_ctl_new1(&snd_ymfpci_spdif_default, chip);
1830
+ err = snd_ctl_add(chip->card, kctl);
1831
+ if (err < 0)
18181832 return err;
18191833 kctl->id.device = chip->pcm_spdif->device;
1820
- if ((err = snd_ctl_add(chip->card, kctl = snd_ctl_new1(&snd_ymfpci_spdif_mask, chip))) < 0)
1834
+ kctl = snd_ctl_new1(&snd_ymfpci_spdif_mask, chip);
1835
+ err = snd_ctl_add(chip->card, kctl);
1836
+ if (err < 0)
18211837 return err;
18221838 kctl->id.device = chip->pcm_spdif->device;
1823
- if ((err = snd_ctl_add(chip->card, kctl = snd_ctl_new1(&snd_ymfpci_spdif_stream, chip))) < 0)
1839
+ kctl = snd_ctl_new1(&snd_ymfpci_spdif_stream, chip);
1840
+ err = snd_ctl_add(chip->card, kctl);
1841
+ if (err < 0)
18241842 return err;
18251843 kctl->id.device = chip->pcm_spdif->device;
18261844 chip->spdif_pcm_ctl = kctl;
18271845
18281846 /* direct recording source */
1829
- if (chip->device_id == PCI_DEVICE_ID_YAMAHA_754 &&
1830
- (err = snd_ctl_add(chip->card, kctl = snd_ctl_new1(&snd_ymfpci_drec_source, chip))) < 0)
1831
- return err;
1847
+ if (chip->device_id == PCI_DEVICE_ID_YAMAHA_754) {
1848
+ kctl = snd_ctl_new1(&snd_ymfpci_drec_source, chip);
1849
+ err = snd_ctl_add(chip->card, kctl);
1850
+ if (err < 0)
1851
+ return err;
1852
+ }
18321853
18331854 /*
18341855 * shared rear/line-in
18351856 */
18361857 if (rear_switch) {
1837
- if ((err = snd_ctl_add(chip->card, snd_ctl_new1(&snd_ymfpci_rear_shared, chip))) < 0)
1858
+ err = snd_ctl_add(chip->card, snd_ctl_new1(&snd_ymfpci_rear_shared, chip));
1859
+ if (err < 0)
18381860 return err;
18391861 }
18401862
....@@ -1847,7 +1869,8 @@
18471869 kctl->id.device = chip->pcm->device;
18481870 kctl->id.subdevice = idx;
18491871 kctl->private_value = (unsigned long)substream;
1850
- if ((err = snd_ctl_add(chip->card, kctl)) < 0)
1872
+ err = snd_ctl_add(chip->card, kctl);
1873
+ if (err < 0)
18511874 return err;
18521875 chip->pcm_mixer[idx].left = 0x8000;
18531876 chip->pcm_mixer[idx].right = 0x8000;
....@@ -1928,7 +1951,8 @@
19281951 tid.card = chip->card->number;
19291952 tid.device = device;
19301953 tid.subdevice = 0;
1931
- if ((err = snd_timer_new(chip->card, "YMFPCI", &tid, &timer)) >= 0) {
1954
+ err = snd_timer_new(chip->card, "YMFPCI", &tid, &timer);
1955
+ if (err >= 0) {
19321956 strcpy(timer->name, "YMFPCI timer");
19331957 timer->private_data = chip;
19341958 timer->hw = snd_ymfpci_timer_hw;
....@@ -2140,7 +2164,7 @@
21402164 chip->work_base = ptr;
21412165 chip->work_base_addr = ptr_addr;
21422166
2143
- snd_BUG_ON(ptr + chip->work_size !=
2167
+ snd_BUG_ON(ptr + PAGE_ALIGN(chip->work_size) !=
21442168 chip->work_ptr.area + chip->work_ptr.bytes);
21452169
21462170 snd_ymfpci_writel(chip, YDSXGR_PLAYCTRLBASE, chip->bank_base_playback_addr);
....@@ -2334,7 +2358,8 @@
23342358 *rchip = NULL;
23352359
23362360 /* enable PCI device */
2337
- if ((err = pci_enable_device(pci)) < 0)
2361
+ err = pci_enable_device(pci);
2362
+ if (err < 0)
23382363 return err;
23392364
23402365 chip = kzalloc(sizeof(*chip), GFP_KERNEL);
....@@ -2357,7 +2382,8 @@
23572382 pci_set_master(pci);
23582383 chip->src441_used = -1;
23592384
2360
- if ((chip->res_reg_area = request_mem_region(chip->reg_area_phys, 0x8000, "YMFPCI")) == NULL) {
2385
+ chip->res_reg_area = request_mem_region(chip->reg_area_phys, 0x8000, "YMFPCI");
2386
+ if (!chip->res_reg_area) {
23612387 dev_err(chip->card->dev,
23622388 "unable to grab memory region 0x%lx-0x%lx\n",
23632389 chip->reg_area_phys, chip->reg_area_phys + 0x8000 - 1);