Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1677125 > unrolled thread

[PATCH] libata: fix build warning in ata_scsi_var_len_cdb_xlat

Started byArnd Bergmann <arnd@arndb.de>
First post2017-06-28 22:10 +0200
Last post2017-06-28 23:30 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] libata: fix build warning in ata_scsi_var_len_cdb_xlat Arnd Bergmann <arnd@arndb.de> - 2017-06-28 22:10 +0200
    Re: [PATCH] libata: fix build warning in ata_scsi_var_len_cdb_xlat Tejun Heo <tj@kernel.org> - 2017-06-28 23:30 +0200

#1677125 — [PATCH] libata: fix build warning in ata_scsi_var_len_cdb_xlat

FromArnd Bergmann <arnd@arndb.de>
Date2017-06-28 22:10 +0200
Subject[PATCH] libata: fix build warning in ata_scsi_var_len_cdb_xlat
Message-ID<tXrrH-1Zp-9@gated-at.bofh.it>
This new harmless warning just showed up:

drivers/ata/libata-scsi.c: In function 'ata_scsi_var_len_cdb_xlat':
drivers/ata/libata-scsi.c:4194:1: error: label 'unspprt_sa' defined but not used [-Werror=unused-label]

The label is obviously unused and can be removed.

Fixes: b1ffbf854e08 ("libata: Support for an ATA PASS-THROUGH(32) command.")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 drivers/ata/libata-scsi.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/ata/libata-scsi.c b/drivers/ata/libata-scsi.c
index 815c6e240aea..9d1b66b4ce7b 100644
--- a/drivers/ata/libata-scsi.c
+++ b/drivers/ata/libata-scsi.c
@@ -4191,7 +4191,6 @@ static unsigned int ata_scsi_var_len_cdb_xlat(struct ata_queued_cmd *qc)
 	if (sa == ATA_32)
 		return ata_scsi_pass_thru(qc);
 
-unspprt_sa:
 	/* unsupported service action */
 	return 1;
 }
-- 
2.9.0

[toc] | [next] | [standalone]


#1677191

FromTejun Heo <tj@kernel.org>
Date2017-06-28 23:30 +0200
Message-ID<tXsH8-rB-25@gated-at.bofh.it>
In reply to#1677125
Hello, Arnd.

On Wed, Jun 28, 2017 at 10:04:03PM +0200, Arnd Bergmann wrote:
> This new harmless warning just showed up:
> 
> drivers/ata/libata-scsi.c: In function 'ata_scsi_var_len_cdb_xlat':
> drivers/ata/libata-scsi.c:4194:1: error: label 'unspprt_sa' defined but not used [-Werror=unused-label]
> 
> The label is obviously unused and can be removed.
> 
> Fixes: b1ffbf854e08 ("libata: Support for an ATA PASS-THROUGH(32) command.")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Already fixed a couple hours ago.  Sorry about the noise.

Thanks.

-- 
tejun

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web