Received: by 2002:a25:8b91:0:0:0:0:0 with SMTP id j17csp14411240ybl; Mon, 30 Dec 2019 09:36:38 -0800 (PST) X-Google-Smtp-Source: APXvYqzFNInuPKg0RngvCGMkrCpL9vfGw+89EWd5uASk1nQ+8dg44k9dPEt6l+PhnrQvImhy6qnE X-Received: by 2002:a9d:6045:: with SMTP id v5mr70593910otj.252.1577727398866; Mon, 30 Dec 2019 09:36:38 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1577727398; cv=none; d=google.com; s=arc-20160816; b=s3I9QEBCgJAEqUSO1mr+fs8N15MDS4XOYIpcNu/gO17LO7udZ5Po+3KMcSu7jDdHRp OQerd8XqOB80CiHSFmmbE+YLO7kEhuHXIPeAEdTjMEenthzJnCjDT8q0iYoDxZfuWvaw YVmRDb7Y4lueYfNRWd15R82laR9x+IUJ3Z4EtIfK1owTM398Aeq4urpjt8QMC189NJRK GIvrkY805SDN6fX6qZYiCjOyON/dGV/o4dciQ4Dk3wMekKhHpqKJ90AU5l0yckcpdNQO Qmmcnb0dYCf3i1FlDslE30WUojcMDSfKzgyIhevmIufSOYHOgc81Mu0u6PFDjQZsLeOQ /Xcg== 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=I1cJoKs8LlFvu6w5X36p4uZCgACG8vHddU4ZddhxihA=; b=T1Ai4lO0uhpN/UBrbTmBhXwntBk/mzNk/hnT/N/xumYMS5RouMbEOEiSCYYzjgQ+Y3 SpUKvUJMscXOTCV1FebB8i9Xo9tHVxtjONntyMRFWCM5KI4QlXpA9yDlnT7f1XCKrenG AYqGUuGT/m2yaObEVrzgQ4tDQEXx1yUPvh4VjeT29jfxaI+9sk4ZpAO+zwWfqsKyonO/ KRsKOnERqeKOYeuHxg+rvVE/d0TtcYXO8OLV3fq6lpWnV07apOYMp7fs+GUIPvxvrjEv w2udCc07OwRi6DOhLqf5RcxX0d7M9uvapRB/78FcL5vnowVpZwSq3j+3RIqgKN9y5fcv J01A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=l2pi9946; 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 u12si23635978otq.51.2019.12.30.09.36.26; Mon, 30 Dec 2019 09:36:38 -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=l2pi9946; 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 S1727558AbfL3Rfm (ORCPT + 99 others); Mon, 30 Dec 2019 12:35:42 -0500 Received: from mail.kernel.org ([198.145.29.99]:40700 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726602AbfL3Rfm (ORCPT ); Mon, 30 Dec 2019 12:35:42 -0500 Received: from mail-lj1-f170.google.com (mail-lj1-f170.google.com [209.85.208.170]) (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 6CE832080A; Mon, 30 Dec 2019 17:35:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1577727340; bh=YjZsLJP99nDwq2E3J4ovBTUTnTY/uRPMv60bV8MttJk=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=l2pi99460cTcBY4y9/wxHor/yeLmALERgqd9fiCZPcmgvFQXSUyw1EmP01I5vNlf2 3wR6zLvyb6oLzP79GH9JbBWEQggDkypZq1sLy3ZpRX4wBaYtDUAcWqGU1WCXgKxJPN NqKRFP+WOci6rj6qPx03rD6Xog+rEYpJfYIlZmoo= Received: by mail-lj1-f170.google.com with SMTP id l2so34003132lja.6; Mon, 30 Dec 2019 09:35:40 -0800 (PST) X-Gm-Message-State: APjAAAUibNS1VuQZOP05QOsJMgXT9cX9Irl5M8wf9/hyjmitZn2Zxw0M KLoGJeRe/zEaJwS5rX9UOb33SB+YB1pCykYMoj8= X-Received: by 2002:a05:651c:106f:: with SMTP id y15mr13370428ljm.63.1577727338476; Mon, 30 Dec 2019 09:35:38 -0800 (PST) MIME-Version: 1.0 References: <20191230084731.15776-1-yuehaibing@huawei.com> In-Reply-To: <20191230084731.15776-1-yuehaibing@huawei.com> From: Chanwoo Choi Date: Tue, 31 Dec 2019 02:35:02 +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 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 > + return ret; > + } > priv->dram_pll = devm_clk_get(dev, "pll"); > + if (IS_ERR(priv->dram_pll)) { > + ret = PTR_ERR(priv->dram_pll); > + dev_err(dev, "failed to fetch pll clock: %d\n", ret); ditto. > + return ret; > + } > priv->dram_alt = devm_clk_get(dev, "alt"); > + if (IS_ERR(priv->dram_alt)) { > + ret = PTR_ERR(priv->dram_alt); > + dev_err(dev, "failed to fetch alt clock: %d\n", ret); ditto. > + return ret; > + } > priv->dram_apb = devm_clk_get(dev, "apb"); > - if (IS_ERR(priv->dram_core) || > - IS_ERR(priv->dram_pll) || > - IS_ERR(priv->dram_alt) || > - IS_ERR(priv->dram_apb)) { > - ret = PTR_ERR(priv->devfreq); > - dev_err(dev, "failed to fetch clocks: %d\n", ret); > + if (IS_ERR(priv->dram_apb)) { > + ret = PTR_ERR(priv->dram_apb); > + dev_err(dev, "failed to fetch apb clock: %d\n", ret); ditto. > return ret; > } > > -- > 2.7.4 > > -- Best Regards, Chanwoo Choi