diff options
| author | Takashi Iwai <tiwai@suse.de> | 2017-12-22 04:45:07 -0500 |
|---|---|---|
| committer | Takashi Iwai <tiwai@suse.de> | 2017-12-22 04:56:56 -0500 |
| commit | a36c2638380c0a4676647a1f553b70b20d3ebce1 (patch) | |
| tree | 27f06b3f900466d299e7f40a79a529ec038cf109 | |
| parent | 8da5bbfc7cbba909f4f32d5e1dda3750baa5d853 (diff) | |
ALSA: hda: Drop useless WARN_ON()
Since the commit 97cc2ed27e5a ("ALSA: hda - Fix yet another i915
pointer leftover in error path") cleared hdac_acomp pointer, the
WARN_ON() non-NULL check in snd_hdac_i915_register_notifier() may give
a false-positive warning, as the function gets called no matter
whether the component is registered or not. For fixing it, let's get
rid of the spurious WARN_ON().
Fixes: 97cc2ed27e5a ("ALSA: hda - Fix yet another i915 pointer leftover in error path")
Cc: <stable@vger.kernel.org>
Reported-by: Kouta Okamoto <kouta.okamoto@toshiba.co.jp>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
| -rw-r--r-- | sound/hda/hdac_i915.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/hda/hdac_i915.c b/sound/hda/hdac_i915.c index 038a180d3f81..cbe818eda336 100644 --- a/sound/hda/hdac_i915.c +++ b/sound/hda/hdac_i915.c | |||
| @@ -325,7 +325,7 @@ static int hdac_component_master_match(struct device *dev, void *data) | |||
| 325 | */ | 325 | */ |
| 326 | int snd_hdac_i915_register_notifier(const struct i915_audio_component_audio_ops *aops) | 326 | int snd_hdac_i915_register_notifier(const struct i915_audio_component_audio_ops *aops) |
| 327 | { | 327 | { |
| 328 | if (WARN_ON(!hdac_acomp)) | 328 | if (!hdac_acomp) |
| 329 | return -ENODEV; | 329 | return -ENODEV; |
| 330 | 330 | ||
| 331 | hdac_acomp->audio_ops = aops; | 331 | hdac_acomp->audio_ops = aops; |
