Received: by 2002:a25:8b91:0:0:0:0:0 with SMTP id j17csp14441749ybl; Mon, 30 Dec 2019 10:10:44 -0800 (PST) X-Google-Smtp-Source: APXvYqygPoiTabTZl+oK8xLuM49sYiERi9zJVk+gzO4v0Znal8jyKJn0nw1C9eVzPqds5jFsw6bu X-Received: by 2002:a05:6830:596:: with SMTP id c22mr73296329oth.188.1577729443949; Mon, 30 Dec 2019 10:10:43 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1577729443; cv=none; d=google.com; s=arc-20160816; b=pAVyoSvmF1xO4Og8vMbe2d8Q0W6ZtplZXGkEAoPoahImJWs4J6X6CKZQESQkZPFkip QGheSrvjka6+Bi2hdXz1PNzfQKNjGjTgRNZ3opahKEFjHkxsSmWbm+c3w+Am/ge8dBZh 93phLZiq97eZQ+zaF3yIhKtGJ5PgF8RJz+9yw4GGOyqCquAL8rE5u5WlTsNKZs0wBS5r xG82/pAyrT9I9mBHFgW4bbpro6dLVcpyKtKJ24BYQLUkEqNYa1NEP+iiHs3Myd/ji9mp JX5ZAqG3pm75sSeqQQHY9RbWYTAuHCY7Q352esFB9u2h1RH+YTDtgG1Sc8852zs8Kieq SnZQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=GOaff4LHD4NmKb6lQIR8oW5HbE18P3xyEGYkhGmt1GY=; b=TvPixfCTme29V8M4a988eItnu7jWfbKrXsZ2tCJag9NeIz+cVJo/L/SNcDwL8kGxth 0RixPu4FVq/g8JqkVND+HUra8bNTlZdOQSzebIGRmAxnfaUg+6toxLKd3BswaWyj0NJH GdMFUujH/ftpJyFRBKWANJWjP3jV0uuDWG30V31q5HFxGHx4N0XRbC3nCZWI8j1dP2oc jfRAOUnzAuPjs/VcWTU91/uNnGZW1ZU/1m1aEaejzLbTdu5U4FVGIPX0g/8I0SfjRh4c 6+FhnGao8Qo23jJfflRbKdntOTSIr2cptJIlMx7HG2IdGiwy+rC0tZmQpCAMBsIVeQnp jdQw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=vlN+7jez; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id l14si25044053otk.225.2019.12.30.10.10.33; Mon, 30 Dec 2019 10:10:43 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=vlN+7jez; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727624AbfL3SJd (ORCPT + 99 others); Mon, 30 Dec 2019 13:09:33 -0500 Received: from mail.kernel.org ([198.145.29.99]:38324 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727318AbfL3SJc (ORCPT ); Mon, 30 Dec 2019 13:09:32 -0500 Received: from mail-lj1-f169.google.com (mail-lj1-f169.google.com [209.85.208.169]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 257D021D7D; Mon, 30 Dec 2019 18:09:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1577729372; bh=CIxky4h0LVYFSbOfjb+Wx3SNAnNGLt1D4UQk5UmNk1c=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=vlN+7jezkIv4FXDqX/nz1VaDVfc0s34AOc7VRoXixFBvyBf2NS7UAvciKl5f4JrS2 COyw+4rmt4XdgJO7vLxLwTnzzegkh9+Of/s9YxdiQYIZmhvTVWwkokqMsc0fFhJJxn v3XwBo0Id+PIVVPcAWYLHjWEznXovcBggYYfbJzU= Received: by mail-lj1-f169.google.com with SMTP id j26so34086104ljc.12; Mon, 30 Dec 2019 10:09:32 -0800 (PST) X-Gm-Message-State: APjAAAX1r2MxcVz5HRDCf84rVfVONk6OlhIApDFY8J6k1TXsCV3Fbssq p7RgpnjCLOe3l4+Nhme5x0PnztDLwNkgyIRrb7s= X-Received: by 2002:a2e:a361:: with SMTP id i1mr37666609ljn.29.1577729370158; Mon, 30 Dec 2019 10:09:30 -0800 (PST) MIME-Version: 1.0 References: <20191230084731.15776-1-yuehaibing@huawei.com> In-Reply-To: From: Chanwoo Choi Date: Tue, 31 Dec 2019 03:08:53 +0900 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH -next] PM / devfreq: imx8m-ddrc: Fix inconsistent IS_ERR and PTR_ERR To: YueHaibing Cc: MyungJoo Ham , Kyungmin Park , Chanwoo Choi , Shawn Guo , s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, dl-linux-imx , Linux PM list , linux-arm-kernel , linux-kernel Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Dec 31, 2019 at 2:35 AM Chanwoo Choi wrote: > > On Mon, Dec 30, 2019 at 5:58 PM YueHaibing wrote: > > > > Fix inconsistent IS_ERR and PTR_ERR in imx8m_ddrc_probe(). > > Detected using Coccinelle. > > > > Signed-off-by: YueHaibing > > --- > > drivers/devfreq/imx8m-ddrc.c | 24 ++++++++++++++++++------ > > 1 file changed, 18 insertions(+), 6 deletions(-) > > > > diff --git a/drivers/devfreq/imx8m-ddrc.c b/drivers/devfreq/imx8m-ddrc.c > > index 53df792..bc82d36 100644 > > --- a/drivers/devfreq/imx8m-ddrc.c > > +++ b/drivers/devfreq/imx8m-ddrc.c > > @@ -395,15 +395,27 @@ static int imx8m_ddrc_probe(struct platform_device *pdev) > > } > > > > priv->dram_core = devm_clk_get(dev, "core"); > > + if (IS_ERR(priv->dram_core)) { > > + ret = PTR_ERR(priv->dram_core); > > + dev_err(dev, "failed to fetch core clock: %d\n", ret); > > If there is no special reason, just use 'get' instead of ' fetch' word. > s/fetch/get Usually, use 'verb' included in function name to show the debug message. But, Again thinking. the original comment used 'fetch' word. Please ignore my comment Applied it. (snip) Best Regards, Chanwoo Choi