From patchwork Thu May 11 11:46:44 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 99618 Delivered-To: patch@linaro.org Received: by 10.140.96.100 with SMTP id j91csp687164qge; Thu, 11 May 2017 04:47:36 -0700 (PDT) X-Received: by 10.98.34.5 with SMTP id i5mr12004281pfi.228.1494503256847; Thu, 11 May 2017 04:47:36 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1494503256; cv=none; d=google.com; s=arc-20160816; b=Hgo00XZcg7KfY2r2QSCxGVDsG931XxqYjSVlo3cAhBeGcoeoWkl5+HLmi2cBR5Tx2j oRle9zRJpPBi2SGEmaCq883pryh8+PiUliZhNwsjcCD4p7YGK03Yz+0Zl50HHFofdkQp wNQIMibjeqCZrUakNbwbHzaSBd4iVFLitNA7MlcWVPHPAemqjYk/QsiyzXjzux/+FACB L8VfUM4HzZTI0pd7bIN19fmlHNTwO20yRq6o1FvJS5KC9fThJpVcpDyDfxIVoIWMRVoc qkauQVmj9yCTTIEzhcpTu8tHh9edX8ncrIlYjt/3cQZCczt8iit/Z685MqJSVRBAzODq u25A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:cc:to:from :arc-authentication-results; bh=yab1wEOAZh548cpseNPP4pPXRzAInPHLgHertYJg95E=; b=BVjgIvFEdmyo7dnGMavmD9DbW0jy7jCFYtxzmMLjmhuw0i6og7arU1NFK5GhlJCQ9/ wI8IvmxSU7kEKHY/64dHv3aljYFRMeQ/kEH6XWl+5ONKr7oO/1vL1mPwI6m3TTbgrDMe iltqgnqfPkbuAKMs4MQZHTkklflSgqSxfMqs1o0Y+kf2n6TH7wgCiyqKGwG4nbxvKTE+ yzxiJu0cQ/1l6ABYMtuGZtjITmgkev8mkEaCAAokd3Q8RP0HnuTp2Ekq5LxARoQFqawY lp3IYldMowMrRBLcRiK+0WykwhHTF9Y3SP91muQbNa7xs4b6bGM49xFcO7oLMkibSM9T QQeA== ARC-Authentication-Results: i=1; mx.google.com; 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id f5si1689323plm.45.2017.05.11.04.47.36; Thu, 11 May 2017 04:47:36 -0700 (PDT) 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; 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932470AbdEKLr3 (ORCPT + 25 others); Thu, 11 May 2017 07:47:29 -0400 Received: from mout.kundenserver.de ([212.227.126.187]:50227 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932250AbdEKLr1 (ORCPT ); Thu, 11 May 2017 07:47:27 -0400 Received: from wuerfel.lan ([78.42.17.5]) by mrelayeu.kundenserver.de (mreue004 [212.227.15.129]) with ESMTPA (Nemesis) id 0LbO0s-1dtguR2zz0-00kz58; Thu, 11 May 2017 13:47:13 +0200 From: Arnd Bergmann To: Mauro Carvalho Chehab Cc: Arnd Bergmann , Sean Young , Hans Verkuil , Wolfram Sang , Geliang Tang , Daniel Wagner , Sakari Ailus , linux-media@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] V4L/DVB: ir-core: fix gcc-7 warning on bool arithmetic Date: Thu, 11 May 2017 13:46:44 +0200 Message-Id: <20170511114711.2916593-1-arnd@arndb.de> X-Mailer: git-send-email 2.9.0 X-Provags-ID: V03:K0:PWBL3xtehzpCngzcNbkOYmilynTlPkkh9Cw8QsInMPgh3CyivzC HYZ+zUdCMdnKDAIMBn4y/5rr9McJvBms4odFPrUj6Wcg4p/OIZeUx3khZO4SKNDYyPknb9W NqpvhfL26TxNd5pzowv1MaiqHbbhryRUf7tHqNneTyN0YXDY7qEq+3piAu72CWDiP8uOWbG JGwuVXKlSNeMz1FxGtaig== X-UI-Out-Filterresults: notjunk:1; V01:K0:U/DXAdtLUQ0=:Lg2GrjDlLNw1qBRFDh4Zfw vYPFiiv+8qYoVAHwPmWIQa14tcriW1EZMfk1A5PR/454jNFVyYT+793qLZUGf3uDgDLQJcc07 WtJARwsMfAfqV7USXfFtBFmcJEQvvAoMDTAhOqKMgEMbXUvHbcEtNCeaHiKAdTcC98FKf5TsP IcSSlt0IMJiRDYX/UaaCwGaJfA/6MVHK9qGae/py/ddnH/pD57SewPSBXmX3AQ+5HnlBnu1zK C2rHKS08NfGeihKwJpPiVSAQcQXE585OMjC5pUJdx/4TTFWDhOFm+pIpVh7+Ki6Hog3dAOSQ0 kFHUGjoGm231Myr+CgbBR/EEK5XlYeAPXlcik/R70YoaLtFNjEvKWc22xptIkPcTtIEByCxGt Ijtr3ltUwZdAJOfQmbXPKqf9niImbd9BAKsWj44J0S+naDfbHoZpRI5uUXpH5+K645D0FF6Y2 pm8NCxmmNirt++XutEEt7indGE1HdhUcJnIhGYZw9WmzI7NRwFiAr5ycZ2IQ04mMnLyeJLW1g dHQxosESfdTdm39SByHemEfrFCKVXtsjU/tZsw4vhSBGTHiqw6ZG+O5YBqKmkhaFNpPoVb3QO 0Uh5xaAsO9a0X69MHM9Ix5qUpdMrIv2Niqxf3v3I9zHLwQYmWm0q81uEPSAeb5Zq40K8Q25xs ZpMhehwXYkXbg7hiSuRgZEMQlcCNl8rgKowjls8ogxJL/3ZE01n+AbrlA3NoSxN5md1U= Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org gcc-7 suggests that an expression using a bitwise not and a bitmask on a 'bool' variable is better written using boolean logic: drivers/media/rc/imon.c: In function 'imon_incoming_scancode': drivers/media/rc/imon.c:1725:22: error: '~' on a boolean expression [-Werror=bool-operation] ictx->pad_mouse = ~(ictx->pad_mouse) & 0x1; ^ drivers/media/rc/imon.c:1725:22: note: did you mean to use logical not? I agree. Fixes: 21677cfc562a ("V4L/DVB: ir-core: add imon driver") Signed-off-by: Arnd Bergmann --- drivers/media/rc/imon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.9.0 diff --git a/drivers/media/rc/imon.c b/drivers/media/rc/imon.c index 3489010601b5..bd76534a2749 100644 --- a/drivers/media/rc/imon.c +++ b/drivers/media/rc/imon.c @@ -1722,7 +1722,7 @@ static void imon_incoming_scancode(struct imon_context *ictx, if (kc == KEY_KEYBOARD && !ictx->release_code) { ictx->last_keycode = kc; if (!nomouse) { - ictx->pad_mouse = ~(ictx->pad_mouse) & 0x1; + ictx->pad_mouse = !ictx->pad_mouse; dev_dbg(dev, "toggling to %s mode\n", ictx->pad_mouse ? "mouse" : "keyboard"); spin_unlock_irqrestore(&ictx->kc_lock, flags);