diff options
| author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2012-11-08 16:27:33 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-11-14 12:14:11 -0800 |
| commit | 70847780dca19f3707d9fa433f68edf8cbaf91f2 (patch) | |
| tree | abcda945fba6a857359dbde13587cbc8b3056deb /drivers | |
| parent | b240add62c7c0e15bfaba59fa37bd44480e31681 (diff) | |
| download | blackbird-op-linux-70847780dca19f3707d9fa433f68edf8cbaf91f2.tar.gz blackbird-op-linux-70847780dca19f3707d9fa433f68edf8cbaf91f2.zip | |
usb: host: tegra: remove pointless NULL check in tegra_ehci_remove()
Test for tegra and hcd in tegra_ehci_remove() look like potential
NULL pointer dereference, but in fact those tests are not needed,
so remove these pointless tests entirely.
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Acked-by: Alan Stern <stern@rowland.harvard.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/usb/host/ehci-tegra.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/usb/host/ehci-tegra.c b/drivers/usb/host/ehci-tegra.c index 94ee3212094e..ef0a6ef7875b 100644 --- a/drivers/usb/host/ehci-tegra.c +++ b/drivers/usb/host/ehci-tegra.c @@ -775,9 +775,6 @@ static int tegra_ehci_remove(struct platform_device *pdev) struct tegra_ehci_hcd *tegra = platform_get_drvdata(pdev); struct usb_hcd *hcd = ehci_to_hcd(tegra->ehci); - if (tegra == NULL || hcd == NULL) - return -EINVAL; - pm_runtime_get_sync(&pdev->dev); pm_runtime_disable(&pdev->dev); pm_runtime_put_noidle(&pdev->dev); |
