| Message ID | 1347352949-17515-1-git-send-email-sachin.kamat@linaro.org |
|---|---|
| State | Superseded |
| Headers | show |
On Tuesday, September 11, 2012 5:42 PM Sachin Kamat wrote > > This file already makes use of device managed functions. > Convert pwm_get() too to use it. > > Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> CC'ed Thierry Reding. Acked-by: Jingoo Han <jg1.han@samsung.com> > --- > drivers/video/backlight/pwm_bl.c | 7 ++----- > 1 files changed, 2 insertions(+), 5 deletions(-) > > diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c > index 4965408..0c91023 100644 > --- a/drivers/video/backlight/pwm_bl.c > +++ b/drivers/video/backlight/pwm_bl.c > @@ -218,7 +218,7 @@ static int pwm_backlight_probe(struct platform_device *pdev) > pb->exit = data->exit; > pb->dev = &pdev->dev; > > - pb->pwm = pwm_get(&pdev->dev, NULL); > + pb->pwm = devm_pwm_get(&pdev->dev, NULL); > if (IS_ERR(pb->pwm)) { > dev_err(&pdev->dev, "unable to request PWM, trying legacy API\n"); > > @@ -251,7 +251,7 @@ static int pwm_backlight_probe(struct platform_device *pdev) > if (IS_ERR(bl)) { > dev_err(&pdev->dev, "failed to register backlight\n"); > ret = PTR_ERR(bl); > - goto err_bl; > + goto err_alloc; > } > > bl->props.brightness = data->dft_brightness; > @@ -260,8 +260,6 @@ static int pwm_backlight_probe(struct platform_device *pdev) > platform_set_drvdata(pdev, bl); > return 0; > > -err_bl: > - pwm_put(pb->pwm); > err_alloc: > if (data->exit) > data->exit(&pdev->dev); > @@ -276,7 +274,6 @@ static int pwm_backlight_remove(struct platform_device *pdev) > backlight_device_unregister(bl); > pwm_config(pb->pwm, 0, pb->period); > pwm_disable(pb->pwm); > - pwm_put(pb->pwm); > if (pb->exit) > pb->exit(&pdev->dev); > return 0; > -- > 1.7.4.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wed, Sep 12, 2012 at 09:55:48AM +0900, Jingoo Han wrote: > On Tuesday, September 11, 2012 5:42 PM Sachin Kamat wrote > > > > This file already makes use of device managed functions. > > Convert pwm_get() too to use it. > > > > Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> > > CC'ed Thierry Reding. > > Acked-by: Jingoo Han <jg1.han@samsung.com> Hi Sachin, As discussed elsewhere I'm taking over maintenance of the pwm-backlight driver as part of the PWM subsystem. I hadn't been Cc'ed on your first send and I'm not subscribed to linux-fbdev so I didn't receive a copy of the patch. Thanks, Thierry
Hi Thierry, On 17 September 2012 11:34, Thierry Reding <thierry.reding@avionic-design.de> wrote: > On Wed, Sep 12, 2012 at 09:55:48AM +0900, Jingoo Han wrote: >> On Tuesday, September 11, 2012 5:42 PM Sachin Kamat wrote >> > >> > This file already makes use of device managed functions. >> > Convert pwm_get() too to use it. >> > >> > Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> >> >> CC'ed Thierry Reding. >> >> Acked-by: Jingoo Han <jg1.han@samsung.com> > > Hi Sachin, > > As discussed elsewhere I'm taking over maintenance of the pwm-backlight > driver as part of the PWM subsystem. That is really nice :) I hadn't been Cc'ed on your first > send and I'm not subscribed to linux-fbdev so I didn't receive a copy of > the patch. Yes. I obtained the list from get_maintainers.pl, so missed out CCing you. I will re-send the patch with you in CC now. > > Thanks, > Thierry
On Mon, Sep 17, 2012 at 11:43:11AM +0530, Sachin Kamat wrote: > Hi Thierry, > > On 17 September 2012 11:34, Thierry Reding > <thierry.reding@avionic-design.de> wrote: > > On Wed, Sep 12, 2012 at 09:55:48AM +0900, Jingoo Han wrote: > >> On Tuesday, September 11, 2012 5:42 PM Sachin Kamat wrote > >> > > >> > This file already makes use of device managed functions. > >> > Convert pwm_get() too to use it. > >> > > >> > Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> > >> > >> CC'ed Thierry Reding. > >> > >> Acked-by: Jingoo Han <jg1.han@samsung.com> > > > > Hi Sachin, > > > > As discussed elsewhere I'm taking over maintenance of the pwm-backlight > > driver as part of the PWM subsystem. > > That is really nice :) > > I hadn't been Cc'ed on your first > > send and I'm not subscribed to linux-fbdev so I didn't receive a copy of > > the patch. > > Yes. I obtained the list from get_maintainers.pl, so missed out CCing you. > I will re-send the patch with you in CC now. Yes, I've contacted Richard to get his approval as the backlight subsystem maintainer. If he's okay I'll send a patch shortly to add the corresponding files to MAINTAINERS so that next time I'll wind up in that list. Thierry
diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c index 4965408..0c91023 100644 --- a/drivers/video/backlight/pwm_bl.c +++ b/drivers/video/backlight/pwm_bl.c @@ -218,7 +218,7 @@ static int pwm_backlight_probe(struct platform_device *pdev) pb->exit = data->exit; pb->dev = &pdev->dev; - pb->pwm = pwm_get(&pdev->dev, NULL); + pb->pwm = devm_pwm_get(&pdev->dev, NULL); if (IS_ERR(pb->pwm)) { dev_err(&pdev->dev, "unable to request PWM, trying legacy API\n"); @@ -251,7 +251,7 @@ static int pwm_backlight_probe(struct platform_device *pdev) if (IS_ERR(bl)) { dev_err(&pdev->dev, "failed to register backlight\n"); ret = PTR_ERR(bl); - goto err_bl; + goto err_alloc; } bl->props.brightness = data->dft_brightness; @@ -260,8 +260,6 @@ static int pwm_backlight_probe(struct platform_device *pdev) platform_set_drvdata(pdev, bl); return 0; -err_bl: - pwm_put(pb->pwm); err_alloc: if (data->exit) data->exit(&pdev->dev); @@ -276,7 +274,6 @@ static int pwm_backlight_remove(struct platform_device *pdev) backlight_device_unregister(bl); pwm_config(pb->pwm, 0, pb->period); pwm_disable(pb->pwm); - pwm_put(pb->pwm); if (pb->exit) pb->exit(&pdev->dev); return 0;
This file already makes use of device managed functions. Convert pwm_get() too to use it. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> --- drivers/video/backlight/pwm_bl.c | 7 ++----- 1 files changed, 2 insertions(+), 5 deletions(-)