Re: [PATCH v6 2/3] usb: musb: da8xx: Use devm in probe
Hi, On Tue, Sep 06, 2016 at 02:17:03PM -0500, Bin Liu wrote: > Hi, > > On Mon, Sep 05, 2016 at 03:00:32PM -0500, David Lechner wrote: > > Simplify things a bit by using devm functions where possible. > > > > Signed-off-by: David Lechner> > --- > > drivers/usb/musb/da8xx.c | 19 +-- > > 1 file changed, 5 insertions(+), 14 deletions(-) > > > > diff --git a/drivers/usb/musb/da8xx.c b/drivers/usb/musb/da8xx.c > > index b03d3b8..0c1997c 100644 > > --- a/drivers/usb/musb/da8xx.c > > +++ b/drivers/usb/musb/da8xx.c > > @@ -490,20 +490,18 @@ static int da8xx_probe(struct platform_device *pdev) > > struct da8xx_glue *glue; > > struct platform_device_info pinfo; > > struct clk *clk; > > + int ret; > > > > - int ret = -ENOMEM; > > - > > - glue = kzalloc(sizeof(*glue), GFP_KERNEL); > > + glue = devm_kzalloc(>dev, sizeof(*glue), GFP_KERNEL); > > if (!glue) { > > dev_err(>dev, "failed to allocate glue context\n"); > > Greg's usb-next already has a patch which drops this line. You might > have to rebase this patch, so that it can go to v4.9-rc1. I fixed the merge conflict locally. Regards, -Bin. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v6 2/3] usb: musb: da8xx: Use devm in probe
Hi, On Mon, Sep 05, 2016 at 03:00:32PM -0500, David Lechner wrote: > Simplify things a bit by using devm functions where possible. > > Signed-off-by: David Lechner> --- > drivers/usb/musb/da8xx.c | 19 +-- > 1 file changed, 5 insertions(+), 14 deletions(-) > > diff --git a/drivers/usb/musb/da8xx.c b/drivers/usb/musb/da8xx.c > index b03d3b8..0c1997c 100644 > --- a/drivers/usb/musb/da8xx.c > +++ b/drivers/usb/musb/da8xx.c > @@ -490,20 +490,18 @@ static int da8xx_probe(struct platform_device *pdev) > struct da8xx_glue *glue; > struct platform_device_info pinfo; > struct clk *clk; > + int ret; > > - int ret = -ENOMEM; > - > - glue = kzalloc(sizeof(*glue), GFP_KERNEL); > + glue = devm_kzalloc(>dev, sizeof(*glue), GFP_KERNEL); > if (!glue) { > dev_err(>dev, "failed to allocate glue context\n"); Greg's usb-next already has a patch which drops this line. You might have to rebase this patch, so that it can go to v4.9-rc1. Thanks, -Bin. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH v6 2/3] usb: musb: da8xx: Use devm in probe
Simplify things a bit by using devm functions where possible. Signed-off-by: David Lechner--- drivers/usb/musb/da8xx.c | 19 +-- 1 file changed, 5 insertions(+), 14 deletions(-) diff --git a/drivers/usb/musb/da8xx.c b/drivers/usb/musb/da8xx.c index b03d3b8..0c1997c 100644 --- a/drivers/usb/musb/da8xx.c +++ b/drivers/usb/musb/da8xx.c @@ -490,20 +490,18 @@ static int da8xx_probe(struct platform_device *pdev) struct da8xx_glue *glue; struct platform_device_info pinfo; struct clk *clk; + int ret; - int ret = -ENOMEM; - - glue = kzalloc(sizeof(*glue), GFP_KERNEL); + glue = devm_kzalloc(>dev, sizeof(*glue), GFP_KERNEL); if (!glue) { dev_err(>dev, "failed to allocate glue context\n"); - goto err0; + return -ENOMEM; } - clk = clk_get(>dev, "usb20"); + clk = devm_clk_get(>dev, "usb20"); if (IS_ERR(clk)) { dev_err(>dev, "failed to get clock\n"); - ret = PTR_ERR(clk); - goto err3; + return PTR_ERR(clk); } ret = clk_enable(clk); @@ -560,12 +558,7 @@ err5: clk_disable(clk); err4: - clk_put(clk); - -err3: - kfree(glue); -err0: return ret; } @@ -576,8 +569,6 @@ static int da8xx_remove(struct platform_device *pdev) platform_device_unregister(glue->musb); usb_phy_generic_unregister(glue->phy); clk_disable(glue->clk); - clk_put(glue->clk); - kfree(glue); return 0; } -- 2.7.4 -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html