RE: [PATCH -next] irqchip/ompic: fix return value check in ompic_of_init()

From: weiyongjun (A)
Date: Wed Jan 03 2018 - 22:57:09 EST


On Wed, Jan 03, 2018 10:41 PM, Stafford Horne wrote:
> On Tue, Jan 02, 2018 at 11:47:19AM +0000, Wei Yongjun wrote:
> > In case of error, the function ioremap() returns NULL pointer not
> > ERR_PTR(). The IS_ERR() test in the return value check should be
> > replaced with NULL test.
>
> Thanks, I am not sure why I was using the *ERR*() macros. 2 questions:
>
> - How did you notice this?

This is detected by Coccinelle semantic patch.

> - Should I pull this in through my tree or do you have another route? i.e.
> Marc?

Feel free to pull this through your tree.