Commit 76b1cf21 authored by Julia Lawall's avatar Julia Lawall Committed by Daniel Vetter

drm: i915: drop null test before destroy functions

Remove unneeded NULL test.

The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)

// <smpl>
@@ expression x; @@
-if (x != NULL)
  \(kmem_cache_destroy\|mempool_destroy\|dma_pool_destroy\)(x);
// </smpl>
Signed-off-by: default avatarJulia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent 5507faeb
...@@ -1074,12 +1074,9 @@ int i915_driver_load(struct drm_device *dev, unsigned long flags) ...@@ -1074,12 +1074,9 @@ int i915_driver_load(struct drm_device *dev, unsigned long flags)
put_bridge: put_bridge:
pci_dev_put(dev_priv->bridge_dev); pci_dev_put(dev_priv->bridge_dev);
free_priv: free_priv:
if (dev_priv->requests) kmem_cache_destroy(dev_priv->requests);
kmem_cache_destroy(dev_priv->requests); kmem_cache_destroy(dev_priv->vmas);
if (dev_priv->vmas) kmem_cache_destroy(dev_priv->objects);
kmem_cache_destroy(dev_priv->vmas);
if (dev_priv->objects)
kmem_cache_destroy(dev_priv->objects);
kfree(dev_priv); kfree(dev_priv);
return ret; return ret;
} }
...@@ -1170,13 +1167,9 @@ int i915_driver_unload(struct drm_device *dev) ...@@ -1170,13 +1167,9 @@ int i915_driver_unload(struct drm_device *dev)
if (dev_priv->regs != NULL) if (dev_priv->regs != NULL)
pci_iounmap(dev->pdev, dev_priv->regs); pci_iounmap(dev->pdev, dev_priv->regs);
if (dev_priv->requests) kmem_cache_destroy(dev_priv->requests);
kmem_cache_destroy(dev_priv->requests); kmem_cache_destroy(dev_priv->vmas);
if (dev_priv->vmas) kmem_cache_destroy(dev_priv->objects);
kmem_cache_destroy(dev_priv->vmas);
if (dev_priv->objects)
kmem_cache_destroy(dev_priv->objects);
pci_dev_put(dev_priv->bridge_dev); pci_dev_put(dev_priv->bridge_dev);
kfree(dev_priv); kfree(dev_priv);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment