summaryrefslogtreecommitdiffstats
path: root/sound/pci/rme96.c
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2017-08-12 09:18:56 -0400
committerTakashi Iwai <tiwai@suse.de>2017-08-12 17:27:21 -0400
commit2cb98d010fdc7a545856baf32bce096fa431d250 (patch)
tree4304386376aaf7026083a0d10acad1a108d80df2 /sound/pci/rme96.c
parentd72a8010ae60551ad14f1385e637be101cf741b5 (diff)
ALSA: rme96: Adjust five checks for null pointers
The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written … Thus fix the affected source code places. Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/pci/rme96.c')
-rw-r--r--sound/pci/rme96.c12
1 files changed, 6 insertions, 6 deletions
diff --git a/sound/pci/rme96.c b/sound/pci/rme96.c
index 0cdfd53b7796..b488d74c3b99 100644
--- a/sound/pci/rme96.c
+++ b/sound/pci/rme96.c
@@ -1199,7 +1199,7 @@ snd_rme96_playback_spdif_open(struct snd_pcm_substream *substream)
1199 1199
1200 snd_pcm_set_sync(substream); 1200 snd_pcm_set_sync(substream);
1201 spin_lock_irq(&rme96->lock); 1201 spin_lock_irq(&rme96->lock);
1202 if (rme96->playback_substream != NULL) { 1202 if (rme96->playback_substream) {
1203 spin_unlock_irq(&rme96->lock); 1203 spin_unlock_irq(&rme96->lock);
1204 return -EBUSY; 1204 return -EBUSY;
1205 } 1205 }
@@ -1248,7 +1248,7 @@ snd_rme96_capture_spdif_open(struct snd_pcm_substream *substream)
1248 } 1248 }
1249 1249
1250 spin_lock_irq(&rme96->lock); 1250 spin_lock_irq(&rme96->lock);
1251 if (rme96->capture_substream != NULL) { 1251 if (rme96->capture_substream) {
1252 spin_unlock_irq(&rme96->lock); 1252 spin_unlock_irq(&rme96->lock);
1253 return -EBUSY; 1253 return -EBUSY;
1254 } 1254 }
@@ -1268,7 +1268,7 @@ snd_rme96_playback_adat_open(struct snd_pcm_substream *substream)
1268 1268
1269 snd_pcm_set_sync(substream); 1269 snd_pcm_set_sync(substream);
1270 spin_lock_irq(&rme96->lock); 1270 spin_lock_irq(&rme96->lock);
1271 if (rme96->playback_substream != NULL) { 1271 if (rme96->playback_substream) {
1272 spin_unlock_irq(&rme96->lock); 1272 spin_unlock_irq(&rme96->lock);
1273 return -EBUSY; 1273 return -EBUSY;
1274 } 1274 }
@@ -1315,7 +1315,7 @@ snd_rme96_capture_adat_open(struct snd_pcm_substream *substream)
1315 } 1315 }
1316 1316
1317 spin_lock_irq(&rme96->lock); 1317 spin_lock_irq(&rme96->lock);
1318 if (rme96->capture_substream != NULL) { 1318 if (rme96->capture_substream) {
1319 spin_unlock_irq(&rme96->lock); 1319 spin_unlock_irq(&rme96->lock);
1320 return -EBUSY; 1320 return -EBUSY;
1321 } 1321 }
@@ -1578,9 +1578,9 @@ snd_rme96_free(void *private_data)
1578{ 1578{
1579 struct rme96 *rme96 = (struct rme96 *)private_data; 1579 struct rme96 *rme96 = (struct rme96 *)private_data;
1580 1580
1581 if (rme96 == NULL) { 1581 if (!rme96)
1582 return; 1582 return;
1583 } 1583
1584 if (rme96->irq >= 0) { 1584 if (rme96->irq >= 0) {
1585 snd_rme96_trigger(rme96, RME96_STOP_BOTH); 1585 snd_rme96_trigger(rme96, RME96_STOP_BOTH);
1586 rme96->areg &= ~RME96_AR_DAC_EN; 1586 rme96->areg &= ~RME96_AR_DAC_EN;