aic94xx: Get rid of redundant NULL check before release_firmware() call
authorJesper Juhl <jj@chaosbits.net>
Mon, 9 Apr 2012 20:51:30 +0000 (22:51 +0200)
committerJiri Kosina <jkosina@suse.cz>
Mon, 30 Apr 2012 11:22:58 +0000 (13:22 +0200)
release_firmware() checks for NULL pointers internally, so checking
before calling the function is redundant.

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
drivers/scsi/aic94xx/aic94xx_seq.c

index 390168f62a13f5dc79316818a4322306ebdc63a5..5fdca93892ad4e2fda61c85eaefa4ebf36aaec76 100644 (file)
@@ -1228,8 +1228,7 @@ static int asd_seq_start_lseq(struct asd_ha_struct *asd_ha, int lseq)
 
 int asd_release_firmware(void)
 {
-       if (sequencer_fw)
-               release_firmware(sequencer_fw);
+       release_firmware(sequencer_fw);
        return 0;
 }