diff options
| author | Jesper Juhl <jj@chaosbits.net> | 2012-04-09 16:51:30 -0400 |
|---|---|---|
| committer | Jiri Kosina <jkosina@suse.cz> | 2012-04-30 07:22:58 -0400 |
| commit | ee51f4432e375d07c29758e592c8c1ad1a343208 (patch) | |
| tree | b92887aaa7e8e88702e3edec2d9997b816efdf8a /drivers/scsi/aic94xx/aic94xx_seq.c | |
| parent | 53c0ad522e215251f55623e80b921129aafa0d48 (diff) | |
aic94xx: Get rid of redundant NULL check before release_firmware() call
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>
Diffstat (limited to 'drivers/scsi/aic94xx/aic94xx_seq.c')
| -rw-r--r-- | drivers/scsi/aic94xx/aic94xx_seq.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/scsi/aic94xx/aic94xx_seq.c b/drivers/scsi/aic94xx/aic94xx_seq.c index 390168f62a13..5fdca93892ad 100644 --- a/drivers/scsi/aic94xx/aic94xx_seq.c +++ b/drivers/scsi/aic94xx/aic94xx_seq.c | |||
| @@ -1228,8 +1228,7 @@ static int asd_seq_start_lseq(struct asd_ha_struct *asd_ha, int lseq) | |||
| 1228 | 1228 | ||
| 1229 | int asd_release_firmware(void) | 1229 | int asd_release_firmware(void) |
| 1230 | { | 1230 | { |
| 1231 | if (sequencer_fw) | 1231 | release_firmware(sequencer_fw); |
| 1232 | release_firmware(sequencer_fw); | ||
| 1233 | return 0; | 1232 | return 0; |
| 1234 | } | 1233 | } |
| 1235 | 1234 | ||