From mboxrd@z Thu Jan 1 00:00:00 1970 From: kbuild test robot Subject: [PATCH] thermal: cpu_cooling: fix ptr_ret.cocci warnings Date: Thu, 5 Mar 2015 04:34:02 +0800 Message-ID: <20150304203402.GA168719@athens.lkp.intel.com> References: <201503050401.EJ55ouQ7%fengguang.wu@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mga03.intel.com ([134.134.136.65]:29844 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932907AbbCDUeU (ORCPT ); Wed, 4 Mar 2015 15:34:20 -0500 Content-Disposition: inline In-Reply-To: <201503050401.EJ55ouQ7%fengguang.wu@intel.com> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: Javi Merino Cc: kbuild-all@01.org, Eduardo Valentin , Kapileshwar Singh , Punit Agrawal , Zhang Rui , linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org drivers/thermal/cpu_cooling.c:463:18-24: WARNING: PTR_ERR_OR_ZERO can be used Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR Generated by: scripts/coccinelle/api/ptr_ret.cocci CC: Javi Merino Signed-off-by: Fengguang Wu --- cpu_cooling.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/drivers/thermal/cpu_cooling.c +++ b/drivers/thermal/cpu_cooling.c @@ -460,7 +460,7 @@ static int get_static_power(struct cpufr if (voltage == 0) { dev_warn_ratelimited(cpufreq_device->cpu_dev, "Failed to get voltage for frequency %lu: %ld\n", - freq_hz, IS_ERR(opp) ? PTR_ERR(opp) : 0); + freq_hz, PTR_ERR_OR_ZERO(opp)); return -EINVAL; }