Update libata's SMART-related ioctl handlers to match the current ATA command pass-through specification (T10/04-262r7). Also change related SCSI op-code definition to match current spec.
Signed-off-by: John W. Linville <[EMAIL PROTECTED]> --- Contact w/ spec author (Curtis Stevens @ Western Digital) indicates that while a revision 8 of the spec is expected, that it is really only a re-formatting of the text to match T10 requirements. According to Stevens, revision 8 is expected to be the last version of the spec. drivers/scsi/libata-scsi.c | 6 ++++-- include/scsi/scsi.h | 6 +++--- 2 files changed, 7 insertions(+), 5 deletions(-) --- sata-smart-2.6/drivers/scsi/libata-scsi.c.orig 2005-02-01 16:24:01.687622085 -0500 +++ sata-smart-2.6/drivers/scsi/libata-scsi.c 2005-02-01 16:49:18.213876086 -0500 @@ -109,14 +109,16 @@ int ata_cmd_ioctl(struct scsi_device *sc return -ENOMEM; scsi_cmd[1] = (4 << 1); /* PIO Data-in */ + scsi_cmd[2] = 0x0e; /* no off.line or cc, read from dev, + block count in sector count field */ sreq->sr_data_direction = DMA_FROM_DEVICE; } else { scsi_cmd[1] = (3 << 1); /* Non-data */ + /* scsi_cmd[2] is already 0 -- no off.line, cc, or data xfer */ sreq->sr_data_direction = DMA_NONE; } scsi_cmd[0] = ATA_16; - scsi_cmd[2] = 0x1f; /* no off.line or cc, yes all registers */ scsi_cmd[4] = args[2]; if (args[0] == WIN_SMART) { /* hack -- ide driver does this too... */ @@ -179,7 +181,7 @@ int ata_task_ioctl(struct scsi_device *s memset(scsi_cmd, 0, sizeof(scsi_cmd)); scsi_cmd[0] = ATA_16; scsi_cmd[1] = (3 << 1); /* Non-data */ - scsi_cmd[2] = 0x1f; /* no off.line or cc, yes all registers */ + /* scsi_cmd[2] is already 0 -- no off.line, cc, or data xfer */ scsi_cmd[4] = args[1]; scsi_cmd[6] = args[2]; scsi_cmd[8] = args[3]; --- sata-smart-2.6/include/scsi/scsi.h.orig 2005-02-01 16:22:12.390234346 -0500 +++ sata-smart-2.6/include/scsi/scsi.h 2005-02-01 16:23:02.828491161 -0500 @@ -113,9 +113,9 @@ extern const char *const scsi_device_typ /* values for service action in */ #define SAI_READ_CAPACITY_16 0x10 -/* Temporary values for T10/04-262 until official values are allocated */ -#define ATA_16 0x85 /* 16-byte pass-thru [0x85 == unused]*/ -#define ATA_12 0xb3 /* 12-byte pass-thru [0xb3 == obsolete set limits command] */ +/* Values for T10/04-262r7 */ +#define ATA_16 0x85 /* 16-byte pass-thru */ +#define ATA_12 0xa1 /* 12-byte pass-thru */ /* * SCSI Architecture Model (SAM) Status codes. Taken from SAM-3 draft -- John W. Linville [EMAIL PROTECTED] - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/