firmware-gsmi: Delete an unnecessary check before the function call "dma_pool_destroy"
The dma_pool_destroy() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Cc: Greg KH <gregkh@linuxfoundation.org> Cc: Julia Lawall <julia.lawall@lip6.fr> Cc: Mike Waychison <mikew@google.com> Cc: Michel Lespinasse <walken@google.com> Signed-off-by: Matt Fleming <matt@codeblueprint.co.uk>
This commit is contained in:
parent
ac0e94b63e
commit
d520dd1f34
|
@ -910,8 +910,7 @@ out_err:
|
||||||
gsmi_buf_free(gsmi_dev.param_buf);
|
gsmi_buf_free(gsmi_dev.param_buf);
|
||||||
gsmi_buf_free(gsmi_dev.data_buf);
|
gsmi_buf_free(gsmi_dev.data_buf);
|
||||||
gsmi_buf_free(gsmi_dev.name_buf);
|
gsmi_buf_free(gsmi_dev.name_buf);
|
||||||
if (gsmi_dev.dma_pool)
|
dma_pool_destroy(gsmi_dev.dma_pool);
|
||||||
dma_pool_destroy(gsmi_dev.dma_pool);
|
|
||||||
platform_device_unregister(gsmi_dev.pdev);
|
platform_device_unregister(gsmi_dev.pdev);
|
||||||
pr_info("gsmi: failed to load: %d\n", ret);
|
pr_info("gsmi: failed to load: %d\n", ret);
|
||||||
return ret;
|
return ret;
|
||||||
|
|
Loading…
Reference in New Issue