ALSA: hda: tas2781-i2c: Remove unnecessary NULL check before release_firmware()
authorChen Ni <nichen@iscas.ac.cn>
Tue, 25 Mar 2025 08:49:39 +0000 (16:49 +0800)
committerTakashi Iwai <tiwai@suse.de>
Tue, 25 Mar 2025 10:54:44 +0000 (11:54 +0100)
release_firmware() checks for NULL pointers internally.
Remove unneeded NULL check for fmw here.

Signed-off-by: Chen Ni <nichen@iscas.ac.cn>
Link: https://patch.msgid.link/20250325084939.801117-1-nichen@iscas.ac.cn
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/pci/hda/tas2781_hda_i2c.c

index 50c5e5f2658953fdb2b0bb4211717ff1cd433d0f..9ed49b0dbe6bc8b866b5f13a6ee1693f5ce1b811 100644 (file)
@@ -753,8 +753,7 @@ static void tasdev_fw_ready(const struct firmware *fmw, void *context)
 
 out:
        mutex_unlock(&tas_hda->priv->codec_lock);
-       if (fmw)
-               release_firmware(fmw);
+       release_firmware(fmw);
        pm_runtime_mark_last_busy(tas_hda->dev);
        pm_runtime_put_autosuspend(tas_hda->dev);
 }