diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-05-30 08:00:34 (GMT) |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-06-03 19:26:18 (GMT) |
commit | 2a1afdbf20ef028d6361c2257b3fef51a8df8b36 (patch) | |
tree | fc2ca5c51a1f5e45aec4cc33784ec9b8ffb5a2e2 /drivers/staging/crystalhd/crystalhd_lnx.c | |
parent | 6441a57887281106ed01222c2b0561ab25f77212 (diff) | |
download | linux-fsl-qoriq-2a1afdbf20ef028d6361c2257b3fef51a8df8b36.tar.xz |
Staging: crystalhd: remove an unneeded NULL check
We already established earlier in the function that "temp" is
non-NULL. We also don't need to set to NULL because it's a stack
variable an we return immediately.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/crystalhd/crystalhd_lnx.c')
-rw-r--r-- | drivers/staging/crystalhd/crystalhd_lnx.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/staging/crystalhd/crystalhd_lnx.c b/drivers/staging/crystalhd/crystalhd_lnx.c index 284b61f..c1f6163 100644 --- a/drivers/staging/crystalhd/crystalhd_lnx.c +++ b/drivers/staging/crystalhd/crystalhd_lnx.c @@ -255,10 +255,7 @@ static int chd_dec_api_cmd(struct crystalhd_adp *adp, unsigned long ua, rc = chd_dec_proc_user_data(adp, temp, ua, 1); } - if (temp) { - chd_dec_free_iodata(adp, temp, 0); - temp = NULL; - } + chd_dec_free_iodata(adp, temp, 0); return rc; } |