Re: [PATCH] n64cart: fix return value check in n64cart_probe()

From: Chaitanya Kulkarni
Date: Thu Sep 09 2021 - 12:17:29 EST


On 9/9/21 2:06 AM, Yang Yingliang wrote:
> External email: Use caution opening links or attachments
>
>
> In case of error, the function devm_platform_ioremap_resource()
> returns ERR_PTR() and never returns NULL. The NULL test in the
> return value check should be replaced with IS_ERR().
>
> Fixes: d9b2a2bbbb4d ("block: Add n64 cart driver")
> Reported-by: Hulk Robot <hulkci@xxxxxxxxxx>
> Signed-off-by: Yang Yingliang <yangyingliang@xxxxxxxxxx>

Looks good.

Reviewed-by: Chaitanya Kulkarni <kch@xxxxxxxxxx>