Re: [PATCH] libata atapi work #2.1

[Jeff Garzik <jgarzik@pobox.com>]

Pat LaVarre wrote:
>>Please let me know what patches (if any
>>besides #define) are needed for setup, after
>>you include patch #2.2 and #2.3 in the pile.
> 
> 
> I will revert to -bk3 vanilla, then apply 2.* from linux-ide, then
> report back.

Last night's -bk4 should include everything except the obvious patch to 
include/linux/libata.h to define ATAPI_ENABLE_DMADIR.


> I will have to discover how to auto sense only after a plain failure,
> not inappropriately after a timeout.  Last I checked, the code was
> written to see any failure as if it were a timeout.

Good guess, very close:  the code is written to see any timeout as if it 
were a failure.  Calling ata_eng_timeout() function is unfortunately a 
bit misnamed.


>>* in INQUIRY output, simulate compliance with
>>MMC-3.  Linux SCSI stack is not aware that all
>>ATAPI devices report zeroes in the SCSI version
>>field (as do some USB storage devices).  So,
>>we fake it.
> 
> 
> I will try to munge the otherwise transparent pass thru of op x12
> "INQUIRY" after the pattern of:
> 
> http://lxr.linux.no/source/drivers/usb/storage/protocol.c?v=2.6.5#L62
> 
> 62 static void fix_inquiry_data(Scsi_Cmnd *srb)
> ...
> 82         /* Change the SCSI revision number */
> 83         databuf[2] = (databuf[2] & ~7) | 2;

Yes, something like this is needed.

We need to indicated MMC-3...

	Jeff