On Thu, 2008-06-19 at 11:59 -0600, Matthew Wilcox wrote:
> Version 2, Jeff pointed out I was missing a check for a sufficiently
> recent version of the specification.
>
> Add support for VPD page b1 to libata
>
> SCSI VPD page b1 reports the nominal rotation speed of the device.
> ATA devices return this information in word 217 of the identify data.
> b1 can also report the physical size of the device (5.25", 3.5", etc)
> but that doesn't currently seem to be defined in the ATA specification.
>
> Signed-off-by: Matthew Wilcox <willy@xxxxxxxxxxxxxxx>
>
> diff --git a/drivers/ata/libata-scsi.c b/drivers/ata/libata-scsi.c
> index 2e6e162..07512d4 100644
> --- a/drivers/ata/libata-scsi.c
> +++ b/drivers/ata/libata-scsi.c
> @@ -1778,7 +1778,9 @@ static unsigned int ata_scsiop_inq_00(struct ata_scsi_args *args, u8 *rbuf)
> const u8 pages[] = {
> 0x00, /* page 0x00, this page */
> 0x80, /* page 0x80, unit serial no page */
> - 0x83 /* page 0x83, device ident page */
> + 0x83, /* page 0x83, device ident page */
> + 0x89, /* page 0x89, ata info page */
> + 0xb1, /* page 0xb1, block device characteristics page */
> };
>
> rbuf[3] = sizeof(pages); /* number of supported VPD pages */
> @@ -1899,6 +1901,18 @@ static unsigned int ata_scsiop_inq_89(struct ata_scsi_args *args, u8 *rbuf)
> return 0;
> }
>
> +static unsigned int ata_scsiop_inq_b1(struct ata_scsi_args *args, u8 *rbuf)
> +{
> + rbuf[1] = 0xb1;
This looks like it can be called for ATA_DEV_ATAPI as well, in which
case it's an illegal page to ask for. Also, shouldn't this be added to
VPD page 0 (which shows the list of available VPD pages ... although it
looks like the supported 0x89 is missing too)?
> + rbuf[3] = 0x3c;
> + if (ata_id_major_version(args->id) > 7) {
> + rbuf[4] = args->id[217] >> 8;
> + rbuf[5] = args->id[217];
> + }
> +
> + return 0;
This would currently set device nominal form factor to unreported. It's
actually available in identify word 168 (bits 0:3) in an identical
format to the one the VPD inquiry is expecting, so why not use it?
James
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
