From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932469AbdEKLr3 (ORCPT ); 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 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 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(-) 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); -- 2.9.0