From patchwork Mon Sep 11 20:00:57 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 112232 Delivered-To: patch@linaro.org Received: by 10.140.106.117 with SMTP id d108csp4263360qgf; Mon, 11 Sep 2017 13:01:59 -0700 (PDT) X-Google-Smtp-Source: ADKCNb4zHQlBZEJcEg0394cbjkqvTOI0bPPKuSzhl3jBUmR1Ey7bBklE/2T3pwLE/+kZdCd5QzlK X-Received: by 10.99.126.84 with SMTP id o20mr12614835pgn.201.1505160119042; Mon, 11 Sep 2017 13:01:59 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1505160119; cv=none; d=google.com; s=arc-20160816; b=BwIFY41LvldcagUJAMPSkxZx9n7VrbbUJ7DXUapzw8QyQSJDBfDoWTkn7ynNHBlSw4 o/goUKCaom8yxw0ooFXZWg4dD7hv6EjYjEaHHS4il9YFZG3TgqJFFwmz59N4X1FesYy+ JzGnWDnTno5NshC1s7LsWH++1c4XTQAQXw5NTKkFlQy478rMcMNSNsmSPqAHBwlbiARr xvDF6vriNKK21LUPkaENYqlPwb71iMMkiqlF+8a/w4ZqLVMgU9SR++I1vponiaOfv7eB qyDDNWZ73z7YvCzFRzjU4rX0dTCiVsxU/hTI6KrVrx+xPJAJFAO3zecCjDALW4lXks5N JUTw== 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=n3Q7IszT3rIQsh+w9khw5cow/NqVOAOoKf7XfulfQLE=; b=XFsV6n9zOsL/Ui/pI0EBeu0HZmdaSLTIbCnMY68Xgaww2wlUwHgOjsvH2AqRBCdxZo syVhLxiCiE5kwfAwlCQ61snbZdQzySu45fnu4S2Di2MlHim304huD4n8yhqVDaCOclJm hmDUYINn/jYKuTAJy/21L0IHilR2CFmmSpqEh0OkSb0hRppxYhw3pLxwEfCy/Eg71PFT Q9L4R+hyRPL4kRY/lVcLsj+QUxCfiTrqWqHTM7Z2w1sOVc1Aq0XdweHyzKXVHGoJ9G60 9+kVxeb5lAo5OVgrvTnAp14AITOBu/7lXJ/VtHVMtHcZbZ7f4MkZWuZsUmcuMWy9FSuN fhBg== 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 3si5908593plo.278.2017.09.11.13.01.58; Mon, 11 Sep 2017 13:01:59 -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 S1751304AbdIKUB4 (ORCPT + 26 others); Mon, 11 Sep 2017 16:01:56 -0400 Received: from mout.kundenserver.de ([212.227.126.133]:65509 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750782AbdIKUBy (ORCPT ); Mon, 11 Sep 2017 16:01:54 -0400 Received: from wuerfel.lan ([95.208.190.237]) by mrelayeu.kundenserver.de (mreue004 [212.227.15.129]) with ESMTPA (Nemesis) id 0M2pyk-1dY2nb22eU-00sibp; Mon, 11 Sep 2017 22:01:17 +0200 From: Arnd Bergmann To: Russell King , "James E.J. Bottomley" , "Martin K. Petersen" Cc: Arnd Bergmann , Johannes Thumshirn , Hannes Reinecke , linux-arm-kernel@lists.infradead.org, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] scsi: acornscsi: fix build error Date: Mon, 11 Sep 2017 22:00:57 +0200 Message-Id: <20170911200115.3119398-1-arnd@arndb.de> X-Mailer: git-send-email 2.9.0 X-Provags-ID: V03:K0:YgdBwvUfpWZn2KKOdr4fdjzU+tC0f/orB+/McMUzVuvFmGBG9bF sts8OaO+CekMCXtvJHU+yMQpL7/zlFFuPmSi9wxd3z2As05ae72AjMPWyg581ydY0wcFqZW 9pzq9amay1lc5t++muWqVJTcVHYuVy3QVUhjn+AZeG3oG0hbdD4yp0/AuTiUscSYV8UYYZL 0LztwpPFaL3lAmqzbkfsQ== X-UI-Out-Filterresults: notjunk:1; V01:K0:uwPni8ZpXuY=:6bFfYsQhLr/vdcFTbKBKI9 em9uLB/prKvNxGuq1FLwPMYCkyXoXXd6we4ArzcwFVI8vlfrDcGL8dp0XzYk7NKU6ybq7n5Se 0psm8vQDj8w/9T71YYEwBwPPefxaWCIU0c/NGkWeZlquZkMWlQuNREoqaXxJVo8zlO9LVY64y URIQxUZyLubY0APXRgjTQjJAIvybmgZ9Py2/QDd/ZOIs0wn/RXa0M17ls4y6uR5eM/cso4qyK veq4BVcuTnk3nUIjZ9kUEpT8X1L+v0GkVC1e0uwERKXA6WymCEAyJALJ0GkSLyz49TP7SO3BF 5QguEZhf9ywoiHLwBRQQdC37FuLwm47spo0QQYkYDZVFrzQVX9E2lpRTzQWwH5F9JAH6WYjrI 2Ihf4Y7fi7YSturHMpBiZF6ga3embBvmkUUExMJiv4BfWpckdyNbajPwdrAXTmXKgrmVKcE0t sN9vocsmcPn7dKUVIZv4PcVgi0uULePyScmq4liNI5fGoCfI/gidq/NNxU/kGx2YQuFM3w/kt buHRTbJBABaDnSAz/lKbTXPMMABodBiKyPqNRB8Iv5S/iVH7M5hVzxBOKvxCPhl/V7QbiQyQy k3JyKh0ii8x1lVjiGJJ4NdNZ3PBg8HpRm4rIQd9q5RZaLokj7d7kkFPBhwBPpG9mpenjEErsq ar7aGLOm+VaatPHb6Bhn/8NuQR2hY9JEjjruIELeQZqyK+0Yzr9x1dYUUJWP1bSHqLGY2rM/m Ssl/FKCSPezjDh/t3deUVtyI1x1PkGKNDo7pkw== Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org A cleanup patch introduced a fatal typo from inbalanced curly braces: drivers/scsi/arm/acornscsi.c: In function 'acornscsi_host_reset': drivers/scsi/arm/acornscsi.c:2773:1: error: ISO C90 forbids mixed declarations and code [-Werror=declaration-after-statement] drivers/scsi/arm/acornscsi.c:2795:12: error: invalid storage class for function 'acornscsi_show_info' static int acornscsi_show_info(struct seq_file *m, struct Scsi_Host *instance) The same patch incorrectly changed the argument type of the reset handler, as shown by this warning: drivers/scsi/arm/acornscsi.c:2888:27: error: initialization of 'int (*)(struct scsi_cmnd *)' from incompatible pointer type 'int (*)(struct Scsi_Host *)' [-Werror=incompatible-pointer-types] .eh_host_reset_handler = acornscsi_host_reset, This removes one the extraneous opening brace and reverts the argument type change. Fixes: 74fa80ee3fae ("scsi: acornscsi: move bus reset to host reset") Signed-off-by: Arnd Bergmann --- drivers/scsi/arm/acornscsi.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) -- 2.9.0 Reviewed-by: Johannes Thumshirn diff --git a/drivers/scsi/arm/acornscsi.c b/drivers/scsi/arm/acornscsi.c index 690816f3c6af..8d54283a99b2 100644 --- a/drivers/scsi/arm/acornscsi.c +++ b/drivers/scsi/arm/acornscsi.c @@ -2725,9 +2725,9 @@ int acornscsi_abort(struct scsi_cmnd *SCpnt) * Params : SCpnt - command causing reset * Returns : one of SCSI_RESET_ macros */ -int acornscsi_host_reset(struct Scsi_Host *shpnt) +int acornscsi_host_reset(struct scsi_cmnd *SCpnt) { - AS_Host *host = (AS_Host *)shpnt->hostdata; + AS_Host *host = (AS_Host *)SCpnt->device->host->hostdata; struct scsi_cmnd *SCptr; host->stats.resets += 1; @@ -2741,7 +2741,7 @@ int acornscsi_host_reset(struct Scsi_Host *shpnt) printk(KERN_WARNING "acornscsi_reset: "); print_sbic_status(asr, ssr, host->scsi.phase); - for (devidx = 0; devidx < 9; devidx ++) { + for (devidx = 0; devidx < 9; devidx ++) acornscsi_dumplog(host, devidx); } #endif