ata_if_xfermask() word 51 fix
authorLennert Buytenhek <buytenh@wantstofly.org>
Mon, 29 Jan 2007 12:28:47 +0000 (13:28 +0100)
committerJeff Garzik <jeff@garzik.org>
Tue, 30 Jan 2007 14:14:56 +0000 (09:14 -0500)
If word 53 bit 1 isn't set, the maximum PIO mode is indicated by
the upper 8 bits of word 51, not the lower 8 bits.  Fixes PIO mode
detection on old Compact Flash cards.

Signed-off-by: Lennert Buytenhek <buytenh@wantstofly.org>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
drivers/ata/libata-core.c

index a388a8df00431ec92cfe1001b82e8f96b32ec50b..cf707029352e29ac034a118518b1632cc303c0c5 100644 (file)
@@ -1037,7 +1037,7 @@ static unsigned int ata_id_xfermask(const u16 *id)
                 * the PIO timing number for the maximum. Turn it into
                 * a mask.
                 */
-               u8 mode = id[ATA_ID_OLD_PIO_MODES] & 0xFF;
+               u8 mode = (id[ATA_ID_OLD_PIO_MODES] >> 8) & 0xFF;
                if (mode < 5)   /* Valid PIO range */
                        pio_mask = (2 << mode) - 1;
                else