From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jeff Garzik <jgarzik@pobox.com>
Subject: Re: [PATCH 2.6.9-libata1-dev1] libata-scsi: hdparm support
Date: Sat, 30 Oct 2004 08:43:26 -0400
Sender: linux-ide-owner@vger.kernel.org
Message-ID: <41838C6E.6050702@pobox.com>
References: <1098487253.28916.12.camel@server.lorenz.priv>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit
Return-path: <linux-ide-owner@vger.kernel.org>
Received: from parcelfarce.linux.theplanet.co.uk ([195.92.249.252]:18322 "EHLO
	www.linux.org.uk") by vger.kernel.org with ESMTP id S261151AbUJ3Mnj
	(ORCPT <rfc822;linux-ide@vger.kernel.org>);
	Sat, 30 Oct 2004 08:43:39 -0400
In-Reply-To: <1098487253.28916.12.camel@server.lorenz.priv>
List-Id: linux-ide@vger.kernel.org
To: Tobias Lorenz <tobias.lorenz@gmx.net>
Cc: linux-ide@vger.kernel.org

Tobias Lorenz wrote:
> Hi,
> 
> this patch adds support for transfering drive informations via the
> hd_driveid structure to the hdparm utility. At the moment, only
> cylinders, sectors, heads, model and firmware version are transfered.
> 
> Our intention was to display the supported and the used UDMA mode(s),
> but we didn't find the correct structures yet to get these infos...
> 
> Signed-off-by: Tobias Lorenz <tobias.lorenz@gmx.net>
> 
>>--- a/drivers/scsi/libata-scsi.c        2004-10-20 22:07:50.000000000 +0200
>>+++ b/drivers/scsi/libata-scsi.c        2004-10-20 19:25:15.000000000 +0200
>>@@ -215,6 +215,17 @@
>>        struct ata_port *ap;
>>        struct ata_device *dev;
>>        int val = -EINVAL, rc = -EINVAL;
>>+       struct hd_driveid drv_id = {
>>+               .cyls           = 0,
>>+               .sectors        = 0,
>>+               .heads          = 0,
>>+               .fw_rev         = "",
>>+               .model          = "",
>>+               .cur_cyls       = 0,
>>+               .cur_heads      = 0,
>>+               .cur_sectors    = 0,
>>+       };
>>+       int geom[3];
>>
>>        ap = (struct ata_port *) &scsidev->host->hostdata[0];
>>        if (!ap)
>>@@ -249,7 +260,25 @@
>>                        return -EACCES;
>>                return ata_task_ioctl(scsidev, arg);
>>
>>+       case HDIO_GET_IDENTITY:
>>+               ata_std_bios_param(scsidev, NULL, dev->n_sectors, geom);
>>+               drv_id.cur_heads        = drv_id.heads          = geom[0];
>>+               drv_id.cur_sectors      = drv_id.sectors        = geom[1];
>>+               drv_id.cur_cyls         = drv_id.cyls           = geom[2];
>>+               strncpy((char *) &drv_id.model, scsidev->model, sizeof(drv_id.model));
>>+               strncpy((char *) &drv_id.fw_rev, scsidev->rev, sizeof(drv_id.fw_rev));
>>+               if(copy_to_user((char *) arg, (char *) &drv_id,
>>+                                sizeof(drv_id)))
>>+                       return(-EFAULT);
>>+               return 0;
>>+
>>        default:
>>                rc = -EOPNOTSUPP;
>>                break;

This patch doesn't apply, since it is prefixed with "> " ...

	Jeff