linux.git/drivers/ata/libata-core.c, branch v2.6.22

libata: fix assigned IRQ reporting

2007-07-03T14:06:35+00:00

host->irq and host->irq2 should be set before ata_host_register() for
IRQ reporting to work.  Move up host->irq assignment in
ata_host_activate() and add it to ata_pci_init_one() native path and
pata_cs5520.

The port info printing in ata_host_register() doesn't fit all the
different controllers.  It should probably be moved out to LLDs with
some helpers in the future.

Signed-off-by: Tejun Heo 
Signed-off-by: Jeff Garzik

libata: add HTS541616J9SA00 to NCQ blacklist

2007-07-02T14:14:02+00:00

Another member of HTS5416* family doing spurious NCQ completion.

Signed-off-by: Tejun Heo 
Cc: Enrico Sardi 
Signed-off-by: Jeff Garzik

libata: remove reading alt_status from ata_hsm_qc_complete()

2007-07-02T14:12:34+00:00

In ata_hsm_qc_complete():
Calling ata_altstatus() after the qc is completed might race with next qc. Remove it.

Signed-off-by: Albert Lee 
Signed-off-by: Jeff Garzik

libata: kill ATA_HORKAGE_DMA_RW_ONLY

2007-06-27T06:50:08+00:00

ATA_HORKAGE_DMA_RW_ONLY for TORiSAN is verified to be subset of using
DMA for ATAPI commands which aren't aligned to 16 bytes.  As libata
now doesn't use DMA for unaligned ATAPI commands, the horkage is
redundant.  Kill it.

Signed-off-by: Tejun Heo 
Signed-off-by: Jeff Garzik

libata: use PIO for non-16 byte aligned ATAPI commands

2007-06-27T06:50:08+00:00

The IDE driver used DMA for ATAPI commands if READ/WRITE command is
multiple of sector size or sg command is multiple of 16 bytes.  For
libata, READ/WRITE sector alignment is guaranteed by the high level
driver (sr), so we only have to worry about the 16 byte alignment.

This patch makes ata_check_atapi_dma() always request PIO for all data
transfer commands which are not multiple of 16 bytes.

The following reports are related to this problem.

http://bugzilla.kernel.org/show_bug.cgi?id=8605		(confirmed)
http://thread.gmane.org/gmane.linux.kernel/476620	(confirmed)
https://bugzilla.novell.com/show_bug.cgi?id=229260	(probably)

Albert first pointed out the difference between IDE and libata.  Kudos
to him.

Signed-off-by: Tejun Heo 
Cc: Albert Lee 
Signed-off-by: Jeff Garzik

libata: fix ata_dev_disable()

2007-06-27T06:44:21+00:00

Fix silly condition check bug in ata_dev_disable().

Signed-off-by: Tejun Heo 
Signed-off-by: Jeff Garzik

libata: be less verbose about hpa

2007-06-27T06:44:19+00:00

There's no reason to print out hpa related messages when HPA is not
active.  Kill the unconditional message and add a warning message
which is printed if HPA size is smaller than the current size.

Signed-off-by: Tejun Heo 
Signed-off-by: Jeff Garzik

kerneldoc fix in libata

2007-06-20T23:56:21+00:00

Fix parameter name from ata_dev_reread_id() in libata-core.c for kerneldoc.

Signed-off-by: Henrik Kretzschmar 
Signed-off-by: Jeff Garzik

libata: more NONCQ devices

2007-06-20T23:56:21+00:00

More for the NCQ blacklist.  One hitachi and one raptor.  Other
members of these families of drives are already on the list, so no
surprises.

Signed-off-by: Tejun Heo 
Signed-off-by: Jeff Garzik

libata: limit post SRST nsect/lbal wait to ~100ms

2007-06-11T04:52:53+00:00

After SRST, libata used to wait for nsect/lbal to be set to 1/1 for
the slave device.  However, some ATAPI devices don't set nsect/lbal
after SRST and the wait itself isn't too useful as we're gonna wait
for !BSY right after that anyway.

Before reset-seq update, nsect/lbal wait failure used to be ignored
and caused 30sec delay during detection.  After reset-seq, all
timeouts are considered error conditions making libata fail to detect
such ATAPI devices.

This patch limits nsect/lbal wait to around 100ms.  This should give
acceptable behavior to such ATAPI devices while not disturbing the
heavily used code path too much.

Signed-off-by: Tejun Heo 
Signed-off-by: Jeff Garzik