[PATCH] drm: Fix return value check in kirin_drm_platform_probe()

Sean Paul seanpaul at chromium.org
Thu Oct 12 14:42:55 UTC 2017


On Thu, Oct 12, 2017 at 2:39 AM, Gabriel Krisman Bertazi
<krisman at collabora.co.uk> wrote:
> Wei Yongjun <weiyongjun1 at huawei.com> writes:
>
>> In case of error, the function of_graph_get_remote_node() returns NULL
>> pointer not ERR_PTR(). The IS_ERR() test in the return value check
>> should be replaced with NULL test..
>
> Looks good.
>
> Reviewed-by: Gabriel Krisman Bertazi <krisman at collabora.co.uk>
>

FYI, I just noticed Dan Carpenter sent an identical patch last week
"<20171005125751.jvtjms62vbtxuvak at mwanda> [PATCH] drm/kirin: Checking
for IS_ERR() instead of NULL".

I'll add my R-b there, since it was first.

Sean


> --
> Gabriel Krisman Bertazi


More information about the dri-devel mailing list