On Fri, 23 Jan 2009 17:15:37 +0400 Sergei Shtylyov <sshtylyov@xxxxxxxxxxxxx> wrote: > When checking for CFA feature set support, ata_id_is_cfa() tests bit 2 in word > 82 of the identify data instead the word 83. It also checks the ATA revision > in the word 80 instead of usual validity check for the words 82-83 -- word 83 > bit 14 set, bit 15 cleared. The word 82 validity bit is not sufficient as that bit is itself not defined in ATA < 3. Otherwise the change looks correct. Alan -- 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