From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752597AbdCFAqV (ORCPT ); Sun, 5 Mar 2017 19:46:21 -0500 Received: from mga03.intel.com ([134.134.136.65]:51040 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752500AbdCFAqS (ORCPT ); Sun, 5 Mar 2017 19:46:18 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.35,250,1484035200"; d="scan'208";a="64527465" Date: Mon, 6 Mar 2017 08:45:53 +0800 From: kbuild test robot To: Rik van Riel Cc: kbuild-all@01.org, linux-kernel@vger.kernel.org, tipbuild@zytor.com, Ingo Molnar , Thomas Gleixner , "H. Peter Anvin" , x86@kernel.org, Dave Hansen , Yu-cheng Yu , Fenghua Yu , Borislav Petkov Subject: [PATCH] x86/fpu: fix boolreturn.cocci warnings Message-ID: <20170306004553.GA25764@lkp-wsm-ep1> References: <201703060848.Q2LHSMc4%fengguang.wu@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <201703060848.Q2LHSMc4%fengguang.wu@intel.com> User-Agent: Mutt/1.5.23 (2014-03-12) X-SA-Exim-Connect-IP: X-SA-Exim-Mail-From: fengguang.wu@intel.com X-SA-Exim-Scanned: No (on bee); SAEximRunCond expanded to false Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org arch/x86/kernel/fpu/xstate.c:931:9-10: WARNING: return of 0/1 in function 'xfeatures_mxcsr_quirk' with return type bool Return statements in functions returning bool should use true/false instead of 1/0. Generated by: scripts/coccinelle/misc/boolreturn.cocci CC: Rik van Riel Signed-off-by: Fengguang Wu --- xstate.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) --- a/arch/x86/kernel/fpu/xstate.c +++ b/arch/x86/kernel/fpu/xstate.c @@ -928,12 +928,12 @@ int arch_set_user_pkey_access(struct tas static inline bool xfeatures_mxcsr_quirk(u64 xfeatures) { if (!(xfeatures & (XFEATURE_MASK_SSE|XFEATURE_MASK_YMM))) - return 0; + return false; if (xfeatures & XFEATURE_MASK_FP) - return 0; + return false; - return 1; + return true; } /*