Hello.
Alan Cox wrote:
bit scan). Note that words 64-127 were marked reserved, and must be set
to 0 since ATA-1. Have your say please -- do I still need to recast the
patch?
"Must be" but were not.
Let's see what ATA-3 rev. 7b has to say on this matter:
7.7.38 Words 82-83: Command sets supported
Words 82 and 83 indicate command sets supported. The values 0000h and
FFFFh in these words indicate
that command set support is not indicated. Bits 4 through 15 of Word 82
are reserved. Bits 0 through 13 of
Word 83 are reserved. Bit 14 of Word 83 shall be set to one. Bit 15 of
Word 83 shall be cleared to zero.
ATA/PI-4 rev. 18 further clarifies:
8.12.45 Words 82-84: Features/command sets supported
Words 82, 83, and 84 shall indicate features/command sets supported. The
value 0000h or FFFFh was
placed in each of these words by devices prior to ATA-3 and shall be
interpreted by the host as meaning
that features/command sets supported are not indicated. Bits 1 through
13 of word 83 and bits 0 through 13
of word 84 are reserved. Bit 14 of word 83 and word 84 shall be set to
one and bit 15 of word 83 and word
84 shall be cleared to zero to provide indication that the
features/command sets supported words are valid.
Since checking bits 14-15 filters out both 0 and 0xFFFF, I don't see
why we have to check for ATA revision prior to that -- unless we're
trying to guard against pre ATA-3 values other than 0 or 0xFFFF.
MBR, Sergei
--
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