Re: [PATCH 1/3] phy: exynos-dp-video: Use PTR_ERR_OR_ZERO

From: Kishon Vijay Abraham I
Date: Thu Jun 12 2014 - 05:57:50 EST




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@xxxxxxxxxxx> 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@xxxxxxxxxx>
>>> Cc: Jingoo Han <jg1.han@xxxxxxxxxxx>
>>
>> Acked-by: Jingoo Han <jg1.han@xxxxxxxxxxx>
>>
>> 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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/