diff options
author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2012-10-23 01:24:51 -0400 |
---|---|---|
committer | Felipe Balbi <balbi@ti.com> | 2012-10-26 04:09:04 -0400 |
commit | 562915153140292b8c59bcab12f039b3aef78cb5 (patch) | |
tree | 24126db45e2d85baed07f25388ccf11902882bd6 | |
parent | 0e7090a626eb6205c123f735a879065702a08cb8 (diff) |
usb: musb: am35x: use platform_device_unregister in am35x_remove()
platform_device_unregister() only calls platform_device_del() and
platform_device_put(), thus use platform_device_unregister() to
simplify the code.
Also the documents in platform.c shows that platform_device_del
and platform_device_put must _only_ be externally called in error
cases. All other usage is a bug.
dpatch engine is used to auto generate this patch.
(https://github.com/weiyj/dpatch)
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: Felipe Balbi <balbi@ti.com>
-rw-r--r-- | drivers/usb/musb/am35x.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/usb/musb/am35x.c b/drivers/usb/musb/am35x.c index 89b128bdbca4..fdfd779c35b3 100644 --- a/drivers/usb/musb/am35x.c +++ b/drivers/usb/musb/am35x.c | |||
@@ -572,8 +572,7 @@ static int __devexit am35x_remove(struct platform_device *pdev) | |||
572 | struct am35x_glue *glue = platform_get_drvdata(pdev); | 572 | struct am35x_glue *glue = platform_get_drvdata(pdev); |
573 | 573 | ||
574 | musb_put_id(&pdev->dev, glue->musb->id); | 574 | musb_put_id(&pdev->dev, glue->musb->id); |
575 | platform_device_del(glue->musb); | 575 | platform_device_unregister(glue->musb); |
576 | platform_device_put(glue->musb); | ||
577 | clk_disable(glue->clk); | 576 | clk_disable(glue->clk); |
578 | clk_disable(glue->phy_clk); | 577 | clk_disable(glue->phy_clk); |
579 | clk_put(glue->clk); | 578 | clk_put(glue->clk); |