Patchwork [v3,1/1,libata] SCSI: really use SATL version in VPD

login
register
mail settings
Submitter Baruch Even
Date April 5, 2013, 4:11 p.m.
Message ID <1365178306-17363-1-git-send-email-baruch@ev-en.org>
Download mbox | patch
Permalink /patch/234190/
State Not Applicable
Delegated to: David Miller
Headers show

Comments

Baruch Even - April 5, 2013, 4:11 p.m.
As per c78968bb by Jeff Garzik ([libata] SCSI: simulator version, not device
version, belongs in VPD) We need to provide the SATL driver version and not the
disk firmware version but the code overwrites the driver version with the disk
version.

Signed-off-by: Baruch Even <baruch@ev-en.org>
---
 drivers/ata/libata-scsi.c |    1 -
 1 file changed, 1 deletion(-)
Jeff Garzik - April 11, 2013, 11:45 p.m.
On 04/05/2013 12:11 PM, Baruch Even wrote:
> As per c78968bb by Jeff Garzik ([libata] SCSI: simulator version, not device
> version, belongs in VPD) We need to provide the SATL driver version and not the
> disk firmware version but the code overwrites the driver version with the disk
> version.
>
> Signed-off-by: Baruch Even <baruch@ev-en.org>
> ---
>   drivers/ata/libata-scsi.c |    1 -
>   1 file changed, 1 deletion(-)

applied



--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff --git a/drivers/ata/libata-scsi.c b/drivers/ata/libata-scsi.c
index 318b413..e61a054 100644
--- a/drivers/ata/libata-scsi.c
+++ b/drivers/ata/libata-scsi.c
@@ -2126,7 +2126,6 @@  static unsigned int ata_scsiop_inq_89(struct ata_scsi_args *args, u8 *rbuf)
 	memcpy(&rbuf[8], "linux   ", 8);
 	memcpy(&rbuf[16], "libata          ", 16);
 	memcpy(&rbuf[32], DRV_VERSION, 4);
-	ata_id_string(args->id, &rbuf[32], ATA_ID_FW_REV, 4);
 
 	/* we don't store the ATA device signature, so we fake it */