From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753297AbdJLGkH (ORCPT ); Thu, 12 Oct 2017 02:40:07 -0400 Received: from bhuna.collabora.co.uk ([46.235.227.227]:38110 "EHLO bhuna.collabora.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751682AbdJLGkG (ORCPT ); Thu, 12 Oct 2017 02:40:06 -0400 From: Gabriel Krisman Bertazi To: Wei Yongjun Cc: Xinliang Liu , , , , , , , , , , , , , , Subject: Re: [PATCH] drm: Fix return value check in kirin_drm_platform_probe() Organization: Collabora References: <1507720636-129038-1-git-send-email-weiyongjun1@huawei.com> Date: Thu, 12 Oct 2017 03:39:59 -0300 In-Reply-To: <1507720636-129038-1-git-send-email-weiyongjun1@huawei.com> (Wei Yongjun's message of "Wed, 11 Oct 2017 11:17:16 +0000") Message-ID: <87wp40oon4.fsf@dilma.collabora.co.uk> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Wei Yongjun 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 -- Gabriel Krisman Bertazi