[PATCH 1/1] ARM: imx: use PTR_ERR_OR_ZERO

Sachin Kamat spk.linux at gmail.com
Mon Jul 7 23:19:09 PDT 2014


On Tue, Jul 8, 2014 at 10:49 AM, Shawn Guo <shawn.guo at freescale.com> wrote:
> On Tue, Jul 08, 2014 at 09:18:56AM +0530, Sachin Kamat wrote:
>> Hi Shawn,
>>
>> On Mon, Jul 7, 2014 at 11:25 AM, Shawn Guo <shawn.guo at freescale.com> wrote:
>> > On Fri, Jul 04, 2014 at 09:03:10PM +0200, Fabian Frederick wrote:
>> >> replace IS_ERR/PTR_ERR
>> >>
>> >> Cc: Shawn Guo <shawn.guo at freescale.com>
>> >> Cc: Sascha Hauer <kernel at pengutronix.de>
>> >> Cc: linux-arm-kernel at lists.infradead.org
>> >> Signed-off-by: Fabian Frederick <fabf at skynet.be>
>> >
>> > Applied, thanks.
>>
>>
>> I had sent a similar series [1] in May which you rejected. Just
>> curious if something
>> changed between then and now?
>
> The main difference is this is the second attempt from a different
> person to "clean up" the code.  I'm still not fond of the change, but
> I'm getting tied to tell people about that, so just applied the change,
> since after all it at least gives some good diffstat.
>
> I would prefer Fabian's patch over yours because it's really good
> enough to have one patch than 5 patches in a series to do the change for
> IMX platform.

Doesn't really matter whose patches are in. I was just curious to know
the change
in logic at this point of time. Thanks for clarification.

-- 
Regards,
Sachin.



More information about the linux-arm-kernel mailing list