diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2017-08-12 09:18:56 -0400 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2017-08-12 17:27:21 -0400 |
commit | 2cb98d010fdc7a545856baf32bce096fa431d250 (patch) | |
tree | 4304386376aaf7026083a0d10acad1a108d80df2 /sound/pci/rme96.c | |
parent | d72a8010ae60551ad14f1385e637be101cf741b5 (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.c | 12 |
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; |