From patchwork Tue Sep 5 07:51:29 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 111632 Delivered-To: patch@linaro.org Received: by 10.140.94.166 with SMTP id g35csp2503505qge; Tue, 5 Sep 2017 00:52:35 -0700 (PDT) X-Google-Smtp-Source: ADKCNb4dS0DP35WKwMe2pLtLYljxPUppXlSOpN9tM48OlcyvgIXilZkXxBpR4nttKSZ9cZSBic88 X-Received: by 10.84.132.42 with SMTP id 39mr3373422ple.243.1504597955331; Tue, 05 Sep 2017 00:52:35 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1504597955; cv=none; d=google.com; s=arc-20160816; b=lV78CZjGIHM9Dt0KqahdPpqBAbFIfE5S4CtanTZ1Tm9Usme/cNiJZOjCrYWNB5dBi/ 8gk/zX4AHUOZwMbzsdNMUGXXODusog/r/jQ7cf8PQMAcw5EJly0HuoLtVpWiWzkHsO6/ TbDGQUEbjNaJsjSF/aVChB5zCQNmIJQ7EmE1yTgwDm0zTVuMgatyAnx8cnp+flwj3u/R FBwpmkh+7hceCB5ubwP5Qz6ianUduqu2o874Yf0T2iNBkc8sDUf9vgSpL+Wk625BSnkL jeLt9FpDJ65PB355lPHvEBcfc8y1VQXeZHKCC1mmgXswQO4nAgQ+kBt1SeR52Jt0wnAO ETBA== 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=tzILgjroBEq5DmFIKpaUgMbSjMouHgLL/RKTTEqCyQA=; b=lGyyQ30DVlbqLYr05tmm08aLkJWivcoSiCv8l17VombKuxi3CnC/x+sOEbbfQBdin8 p+IAS89W21Bc4XXSu9NPYibv6rSPDgLhz8+MK69mQp7caPb5XF4lb5AnbeQ1wNJJgV1A vyawE1xJujw3ls2H3inObcQDWXIF7bulFySmSXHWohNBlKeA6s0DOxyq3/JFvwklmBbx eK1IiQS8vfNuHGWnpWM01ZOj3Qgm4YANF6KrlXjGndLFewT5EdYyR7pD780H+UbPXiV8 JMij9dQBy08rt2e3RTXYTP2+Um46cSCnueARaDwdDSBIeSBcWUq1oHOwMveyEEm/H4aT wUBw== 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 h15si79085pln.748.2017.09.05.00.52.35; Tue, 05 Sep 2017 00:52:35 -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 S1751479AbdIEHwc (ORCPT + 26 others); Tue, 5 Sep 2017 03:52:32 -0400 Received: from mout.kundenserver.de ([217.72.192.74]:61183 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750810AbdIEHwb (ORCPT ); Tue, 5 Sep 2017 03:52:31 -0400 Received: from wuerfel.lan ([95.208.190.237]) by mrelayeu.kundenserver.de (mreue104 [212.227.15.145]) with ESMTPA (Nemesis) id 0LoZ6A-1dHiVR22dm-00gWlr; Tue, 05 Sep 2017 09:51:43 +0200 From: Arnd Bergmann To: GOTO Masanori , YOKOTA Hiroshi , "James E.J. Bottomley" , "Martin K. Petersen" Cc: Arnd Bergmann , Hannes Reinecke , Johannes Thumshirn , linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] scsi: nsp32: fix logic bug in error handling Date: Tue, 5 Sep 2017 09:51:29 +0200 Message-Id: <20170905075139.518406-1-arnd@arndb.de> X-Mailer: git-send-email 2.9.0 X-Provags-ID: V03:K0:Sl2UeSe8+KXFIyM3nzG40gC1Zvc+nN+r91LFaPJwWmoNT09h6QO 1mlZ1iXwo+Y9brYowKaxFUCKY3tHaTti/ZOXElqU3Vue/xY99S3dZMIeCo0hnwsVGzINfh1 6QJt1vp+J52n6F5PsNjEePFY2aJQddZoaSf2c5CFzhVa1x+3MEe1gAPiFLJAUv1qGKVVFi4 IKjTpJGfp2zhBIc/aP5Hg== X-UI-Out-Filterresults: notjunk:1; V01:K0:CDVVtqVCWRk=:2U+MfW3JBJsLSWDsT3OEYo a5GEo+c/vqRelhP2EfbE/HjuWz2JN0hvrwN8aQbClEH14BKE6wmAF1nAszQnYvV13uoCytVP+ RYG9/tfl6EnQrlmC+IbF8UJxQlqSQ9S5+jXMQVSMvpXlHrN1VgLOaYTfEj26wHC69Voi5raXI KfOnQMUXqhxHjalYLjW1MIGkJzqeV8ig1y3B0C40fl+0clJbpmwLXYu+Zc/3jHT3EKCTVE49o zC6onzqTRmU9tQcK7zfgHm+RlVU92sF7LTrq48TOQFxWV3vlLOBKOxft4TyUkaQpbhA1/zy4b yGM96UGMaJ0Ji9rt1O2zYoStVkZIx/cB8cvbJN3Q2J5dtJLhTLVyijA+c4pZ0D7jdvZ4YbnNs V/M7A3iX8hsCvZLc08i8K5b2RxRFHglzbmN93jLVfJ3BajoFC3fYctjFFg6JKdSGLy3j7jplU BIIY2k/2G+0nX+iTghIYsfEf3onsvcTDD8EoA9lnc+imrY9h8Shn6F99+vzalPEsyqlv5OgQp lvPN+6mhSgQWLHSlxpn1+bD3CSkk4hL0vBTM+m9syF6RsCp9+kM+g6rUyBV8z0SY3vFFHhtRD 3+zivtYqMe4vkvachHF/vgiluPj4GH+fmvGX1wIcM+nNWDHDt9QU5LeBkz5doRj5WYXOI6gjR H6dId9YVU3/mqDKpn6Omoe9QK6ggVmC65PQ5tCCsP3z+UqRyxmp/F9uomCSPJPqrwgs7Xu/3q Z2NWmX0oXFHut+xZiwx/ZVF6MhTy/c3AlDq7Sg== Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org gcc-8 points out a logic error that has existed since the start of the git history: drivers/scsi/nsp32.c: In function 'nsp32_selection_autoscsi': drivers/scsi/nsp32.c:607:27: error: bitwise comparison always evaluates to false [-Werror=tautological-compare] if(((phase & BUSMON_BSY) == 1) || (phase & BUSMON_SEL) == 1) { ^~ Presumably the author intended to check if one of two bits was set, so that's what I'm changing the code to. This will obviously change the behavior of the code, hopefully to do the right thing, but I have not tested this or checked if the new "(phase & BUSMON_BSY) || (phase & BUSMON_SEL)" condition should indeed be treated as a fatal error. Signed-off-by: Arnd Bergmann --- drivers/scsi/nsp32.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.9.0 Signed-off-by: GOTO Masanori diff --git a/drivers/scsi/nsp32.c b/drivers/scsi/nsp32.c index 107e191bf023..8620ac5d6e41 100644 --- a/drivers/scsi/nsp32.c +++ b/drivers/scsi/nsp32.c @@ -604,7 +604,7 @@ static int nsp32_selection_autoscsi(struct scsi_cmnd *SCpnt) * check bus line */ phase = nsp32_read1(base, SCSI_BUS_MONITOR); - if(((phase & BUSMON_BSY) == 1) || (phase & BUSMON_SEL) == 1) { + if ((phase & BUSMON_BSY) || (phase & BUSMON_SEL)) { nsp32_msg(KERN_WARNING, "bus busy"); SCpnt->result = DID_BUS_BUSY << 16; status = 1;