diff options
| author | Jesper Juhl <jj@chaosbits.net> | 2012-07-30 12:50:56 +0800 |
|---|---|---|
| committer | Andy Green <andy.green@linaro.org> | 2012-07-30 12:50:56 +0800 |
| commit | e4d819a9239ed0c8be5aac8ed4112d0177e421a6 (patch) | |
| tree | dd23c130ce391087f892427d2a373b174ccec2a2 | |
| parent | d7e3487966e5194098a2782989dbe55691507ed8 (diff) | |
remoteproc: remove redundant NULL check before release_firmware()
release_firmware deals gracefully with NULL pointers, so checking
first is redundant.
Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
| -rw-r--r-- | drivers/remoteproc/remoteproc_core.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/remoteproc/remoteproc_core.c b/drivers/remoteproc/remoteproc_core.c index 23ae71ad785..79c3a1c1345 100644 --- a/drivers/remoteproc/remoteproc_core.c +++ b/drivers/remoteproc/remoteproc_core.c @@ -1394,8 +1394,7 @@ static void rproc_fw_config_virtio(const struct firmware *fw, void *context) goto out; out: - if (fw) - release_firmware(fw); + release_firmware(fw); /* allow rproc_unregister() contexts, if any, to proceed */ complete_all(&rproc->firmware_loading_complete); } |