From: zhong jiang <[email protected]>

dma_pool_destroy() already takes NULL pointer into account so there is
no need to check that again in tb_ctl_free().

Signed-off-by: zhong jiang <[email protected]>
[mw: reword commit log a bit]
Signed-off-by: Mika Westerberg <[email protected]>
---
 drivers/thunderbolt/ctl.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/thunderbolt/ctl.c b/drivers/thunderbolt/ctl.c
index 37a7f4c735d0..e54e84e43ede 100644
--- a/drivers/thunderbolt/ctl.c
+++ b/drivers/thunderbolt/ctl.c
@@ -662,8 +662,7 @@ void tb_ctl_free(struct tb_ctl *ctl)
                tb_ctl_pkg_free(ctl->rx_packets[i]);
 
 
-       if (ctl->frame_pool)
-               dma_pool_destroy(ctl->frame_pool);
+       dma_pool_destroy(ctl->frame_pool);
        kfree(ctl);
 }
 
-- 
2.19.0

Reply via email to