Jonathan Benson wrote: > >>Could you please give a try to the attached patch and post the content > of word 62? >>If word 62 + SiI 3611 looks good, we can turn on the DMADIR > automatically. Thanks! > > Per Albert's request, I've tested his patch to check the status of word > 62 with the SiI 3611 bridge chip. I used kernel version 2.6.17- rc1, as > 2.6.16 reported fuzz when I applied the patch. 2.6.17-rc1 reported an > offset, but applied without any errors. I've tested with the > ATAPI_ENABLE_DMADIR workaround enabled, and disabled. Debugging is > disabled. Looks like word 62 returns 0 as its value, dmesg with the > relevant parts attached below. > > If you need any more information, let me know and I'll be happy to > provide it. > > Jonathan Benson > > (snip) > > ata2: dev 0 ATA-7, max UDMA/133, 240121728 sectors: LBA > ata2(0): applying bridge limits > ata2: dev 0 configured for UDMA/100 > scsi1 : sata_sil > ata3: SATA link up 1.5 Gbps (SStatus 113) > ata3: dev 0 cfg 49:2f00 82:7c6b 83:7b09 84:4003 85:7c69 86:3a01 87:4003 > 88:007f > ata3: dev 0 ATA-7, max UDMA/133, 240121728 sectors: LBA > ata3(0): applying bridge limits > ata3: dev 0 configured for UDMA/100 > scsi2 : sata_sil > ata4: SATA link up 1.5 Gbps (SStatus 113) > ata4: dev 0 cfg 49:0b00 82:0210 83:1000 84:0000 85:0000 86:0000 87:0000 > 88:0007 > ata4: dev 0 word 62 [0] > ata4: dev 0 ATAPI, max UDMA/33 > ata4: dev 0 word 62 [0] > ata4: dev 0 configured for UDMA/33 Word 62 is also zero on my JMicron 20330 bridge. Since JMicron doesn't need the DMA DIR bit, it looks consistent (maybe by coincidence) with the ATA-7 spec. For SiI 3611, it was made before the ATA-7 spec is finalized. So, it is reasonable that SiI 3611 doesn't implement word 62 and we have to manually enable the DMA DIR support as work around unless SiImage provides other solution to identify the SiI 3611 bridge. Thanks for the test result. :) -- albert - : 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
