Hello, On 10/15/2010 02:32 AM, Robert Hancock wrote: >> + if (qc->tf.protocol == ATA_PROT_PIO && qc->dma_dir == DMA_FROM_DEVICE && >> + !(qc->flags & ATA_QCFLAG_FAILED)) >> + qc->result_tf.command = (rx_fis + RX_FIS_PIO_SETUP)[15]; > > Shouldn't we be reading the entire TF from the PIO setup FIS and then > just replacing the command register value with the E_Status field? It > looks like that's the only field in the TF that's being set here. Hmmm, yeah, probably that would be better. I'll post the updated version soon. Thanks. -- tejun -- 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
