Browse Source

qla2xxx: Remove redundant NULL check before release_firmware() call.

release_firmware() checks for NULL pointers internally so checking
before calling it is redundant.

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Acked-by: Chad Dupuis <chad.dupuis@qlogic.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Jesper Juhl 13 years ago
parent
commit
cf92549f1c
1 changed files with 1 additions and 2 deletions
  1. 1 2
      drivers/scsi/qla2xxx/qla_os.c

+ 1 - 2
drivers/scsi/qla2xxx/qla_os.c

@@ -4106,8 +4106,7 @@ qla2x00_release_firmware(void)
 
 	mutex_lock(&qla_fw_lock);
 	for (idx = 0; idx < FW_BLOBS; idx++)
-		if (qla_fw_blobs[idx].fw)
-			release_firmware(qla_fw_blobs[idx].fw);
+		release_firmware(qla_fw_blobs[idx].fw);
 	mutex_unlock(&qla_fw_lock);
 }