| Message ID | 1401345049-14779-1-git-send-email-sachin.kamat@linaro.org |
|---|---|
| State | Superseded |
| Headers | show |
On Thursday, May 29, 2014 3:31 PM, Sachin Kamat wrote: > > PTR_ERR_OR_ZERO simplifies the code. > > Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> > Cc: Jingoo Han <jg1.han@samsung.com> Acked-by: Jingoo Han <jg1.han@samsung.com> Best regards, Jingoo Han > --- > drivers/phy/phy-exynos-dp-video.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/phy/phy-exynos-dp-video.c b/drivers/phy/phy-exynos-dp-video.c > index 0786fef842e7..098f822a2fa4 100644 > --- a/drivers/phy/phy-exynos-dp-video.c > +++ b/drivers/phy/phy-exynos-dp-video.c > @@ -9,6 +9,7 @@ > * published by the Free Software Foundation. > */ > > +#include <linux/err.h> > #include <linux/io.h> > #include <linux/kernel.h> > #include <linux/module.h> > @@ -84,10 +85,8 @@ static int exynos_dp_video_phy_probe(struct platform_device *pdev) > phy_set_drvdata(phy, state); > > phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate); > - if (IS_ERR(phy_provider)) > - return PTR_ERR(phy_provider); > > - return 0; > + return PTR_ERR_OR_ZERO(phy_provider); > } > > static const struct of_device_id exynos_dp_video_phy_of_match[] = { > -- > 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Hi Kishon, On Mon, Jun 2, 2014 at 7:56 AM, Jingoo Han <jg1.han@samsung.com> wrote: > On Thursday, May 29, 2014 3:31 PM, Sachin Kamat wrote: >> >> PTR_ERR_OR_ZERO simplifies the code. >> >> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> >> Cc: Jingoo Han <jg1.han@samsung.com> > > Acked-by: Jingoo Han <jg1.han@samsung.com> > > Best regards, > Jingoo Han > >> --- >> drivers/phy/phy-exynos-dp-video.c | 5 ++--- >> 1 file changed, 2 insertions(+), 3 deletions(-) >> >> diff --git a/drivers/phy/phy-exynos-dp-video.c b/drivers/phy/phy-exynos-dp-video.c >> index 0786fef842e7..098f822a2fa4 100644 >> --- a/drivers/phy/phy-exynos-dp-video.c >> +++ b/drivers/phy/phy-exynos-dp-video.c >> @@ -9,6 +9,7 @@ >> * published by the Free Software Foundation. >> */ >> >> +#include <linux/err.h> >> #include <linux/io.h> >> #include <linux/kernel.h> >> #include <linux/module.h> >> @@ -84,10 +85,8 @@ static int exynos_dp_video_phy_probe(struct platform_device *pdev) >> phy_set_drvdata(phy, state); >> >> phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate); >> - if (IS_ERR(phy_provider)) >> - return PTR_ERR(phy_provider); >> >> - return 0; >> + return PTR_ERR_OR_ZERO(phy_provider); >> } >> >> static const struct of_device_id exynos_dp_video_phy_of_match[] = { >> -- >> 1.7.9.5 > > ping on this series.
On Thursday 12 June 2014 03:19 PM, Sachin Kamat wrote: > Hi Kishon, > > On Mon, Jun 2, 2014 at 7:56 AM, Jingoo Han <jg1.han@samsung.com> wrote: >> On Thursday, May 29, 2014 3:31 PM, Sachin Kamat wrote: >>> >>> PTR_ERR_OR_ZERO simplifies the code. >>> >>> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> >>> Cc: Jingoo Han <jg1.han@samsung.com> >> >> Acked-by: Jingoo Han <jg1.han@samsung.com> >> >> Best regards, >> Jingoo Han >> >>> --- >>> drivers/phy/phy-exynos-dp-video.c | 5 ++--- >>> 1 file changed, 2 insertions(+), 3 deletions(-) >>> >>> diff --git a/drivers/phy/phy-exynos-dp-video.c b/drivers/phy/phy-exynos-dp-video.c >>> index 0786fef842e7..098f822a2fa4 100644 >>> --- a/drivers/phy/phy-exynos-dp-video.c >>> +++ b/drivers/phy/phy-exynos-dp-video.c >>> @@ -9,6 +9,7 @@ >>> * published by the Free Software Foundation. >>> */ >>> >>> +#include <linux/err.h> >>> #include <linux/io.h> >>> #include <linux/kernel.h> >>> #include <linux/module.h> >>> @@ -84,10 +85,8 @@ static int exynos_dp_video_phy_probe(struct platform_device *pdev) >>> phy_set_drvdata(phy, state); >>> >>> phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate); >>> - if (IS_ERR(phy_provider)) >>> - return PTR_ERR(phy_provider); >>> >>> - return 0; >>> + return PTR_ERR_OR_ZERO(phy_provider); >>> } >>> >>> static const struct of_device_id exynos_dp_video_phy_of_match[] = { >>> -- >>> 1.7.9.5 >> >> > > ping on this series. I'll queue to -next once -rc1 is tagged. -Kishon > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
On Thu, Jun 12, 2014 at 3:27 PM, Kishon Vijay Abraham I <kishon@ti.com> wrote: > > > On Thursday 12 June 2014 03:19 PM, Sachin Kamat wrote: >> Hi Kishon, >> >> On Mon, Jun 2, 2014 at 7:56 AM, Jingoo Han <jg1.han@samsung.com> wrote: >>> On Thursday, May 29, 2014 3:31 PM, Sachin Kamat wrote: >>>> >>>> PTR_ERR_OR_ZERO simplifies the code. >>>> >>>> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> >>>> Cc: Jingoo Han <jg1.han@samsung.com> >>> >>> Acked-by: Jingoo Han <jg1.han@samsung.com> >>> >>> Best regards, >>> Jingoo Han >>> >>>> --- >>>> drivers/phy/phy-exynos-dp-video.c | 5 ++--- >>>> 1 file changed, 2 insertions(+), 3 deletions(-) >>>> >>>> diff --git a/drivers/phy/phy-exynos-dp-video.c b/drivers/phy/phy-exynos-dp-video.c >>>> index 0786fef842e7..098f822a2fa4 100644 >>>> --- a/drivers/phy/phy-exynos-dp-video.c >>>> +++ b/drivers/phy/phy-exynos-dp-video.c >>>> @@ -9,6 +9,7 @@ >>>> * published by the Free Software Foundation. >>>> */ >>>> >>>> +#include <linux/err.h> >>>> #include <linux/io.h> >>>> #include <linux/kernel.h> >>>> #include <linux/module.h> >>>> @@ -84,10 +85,8 @@ static int exynos_dp_video_phy_probe(struct platform_device *pdev) >>>> phy_set_drvdata(phy, state); >>>> >>>> phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate); >>>> - if (IS_ERR(phy_provider)) >>>> - return PTR_ERR(phy_provider); >>>> >>>> - return 0; >>>> + return PTR_ERR_OR_ZERO(phy_provider); >>>> } >>>> >>>> static const struct of_device_id exynos_dp_video_phy_of_match[] = { >>>> -- >>>> 1.7.9.5 >>> >>> >> >> ping on this series. > > I'll queue to -next once -rc1 is tagged. Thanks Kishon.
On Thu, Jun 12, 2014 at 3:27 PM, Kishon Vijay Abraham I <kishon@ti.com> wrote: > > > On Thursday 12 June 2014 03:19 PM, Sachin Kamat wrote: >> Hi Kishon, >> >> On Mon, Jun 2, 2014 at 7:56 AM, Jingoo Han <jg1.han@samsung.com> wrote: >>> On Thursday, May 29, 2014 3:31 PM, Sachin Kamat wrote: >>>> >>>> PTR_ERR_OR_ZERO simplifies the code. >>>> >>>> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> >>>> Cc: Jingoo Han <jg1.han@samsung.com> >>> >>> Acked-by: Jingoo Han <jg1.han@samsung.com> >>> >>> Best regards, >>> Jingoo Han >>> >>>> --- >>>> drivers/phy/phy-exynos-dp-video.c | 5 ++--- >>>> 1 file changed, 2 insertions(+), 3 deletions(-) >>>> >>>> diff --git a/drivers/phy/phy-exynos-dp-video.c b/drivers/phy/phy-exynos-dp-video.c >>>> index 0786fef842e7..098f822a2fa4 100644 >>>> --- a/drivers/phy/phy-exynos-dp-video.c >>>> +++ b/drivers/phy/phy-exynos-dp-video.c >>>> @@ -9,6 +9,7 @@ >>>> * published by the Free Software Foundation. >>>> */ >>>> >>>> +#include <linux/err.h> >>>> #include <linux/io.h> >>>> #include <linux/kernel.h> >>>> #include <linux/module.h> >>>> @@ -84,10 +85,8 @@ static int exynos_dp_video_phy_probe(struct platform_device *pdev) >>>> phy_set_drvdata(phy, state); >>>> >>>> phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate); >>>> - if (IS_ERR(phy_provider)) >>>> - return PTR_ERR(phy_provider); >>>> >>>> - return 0; >>>> + return PTR_ERR_OR_ZERO(phy_provider); >>>> } >>>> >>>> static const struct of_device_id exynos_dp_video_phy_of_match[] = { >>>> -- >>>> 1.7.9.5 >>> >>> >> >> ping on this series. > > I'll queue to -next once -rc1 is tagged. Gentle ping..
diff --git a/drivers/phy/phy-exynos-dp-video.c b/drivers/phy/phy-exynos-dp-video.c index 0786fef842e7..098f822a2fa4 100644 --- a/drivers/phy/phy-exynos-dp-video.c +++ b/drivers/phy/phy-exynos-dp-video.c @@ -9,6 +9,7 @@ * published by the Free Software Foundation. */ +#include <linux/err.h> #include <linux/io.h> #include <linux/kernel.h> #include <linux/module.h> @@ -84,10 +85,8 @@ static int exynos_dp_video_phy_probe(struct platform_device *pdev) phy_set_drvdata(phy, state); phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate); - if (IS_ERR(phy_provider)) - return PTR_ERR(phy_provider); - return 0; + return PTR_ERR_OR_ZERO(phy_provider); } static const struct of_device_id exynos_dp_video_phy_of_match[] = {
PTR_ERR_OR_ZERO simplifies the code. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Cc: Jingoo Han <jg1.han@samsung.com> --- drivers/phy/phy-exynos-dp-video.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)