Commit 97163873 authored by Sergei Shtylyov's avatar Sergei Shtylyov Committed by Jeff Garzik

pata_legacy: correctly mask recovery field for HT6560B

According to the HT6560H datasheet, the recovery timing field is 4-bit wide,
with a value of 0 meaning 16 cycles. Correct obvious thinko in the recovery
field mask.
Signed-off-by: default avatarSergei Shtylyov <sshtylyov@ru.mvista.com>
Cc: stable@kernel.org
Signed-off-by: default avatarJeff Garzik <jgarzik@redhat.com>
parent 78140cfe
...@@ -401,8 +401,7 @@ static void ht6560b_set_piomode(struct ata_port *ap, struct ata_device *adev) ...@@ -401,8 +401,7 @@ static void ht6560b_set_piomode(struct ata_port *ap, struct ata_device *adev)
ata_timing_compute(adev, adev->pio_mode, &t, 20000, 1000); ata_timing_compute(adev, adev->pio_mode, &t, 20000, 1000);
active = clamp_val(t.active, 2, 15); active = clamp_val(t.active, 2, 15);
recover = clamp_val(t.recover, 2, 16); recover = clamp_val(t.recover, 2, 16) & 0x0F;
recover &= 0x15;
inb(0x3E6); inb(0x3E6);
inb(0x3E6); inb(0x3E6);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment