Messages in this thread Patch in this message | | | From | Bartlomiej Zolnierkiewicz <> | Subject | Re: [PATCH 4/5] pata: Update experimental tags | Date | Thu, 19 Nov 2009 18:38:11 +0100 |
| |
On Thursday 19 November 2009 17:27:38 Alan Cox wrote: > > > > already have a buggy cable detection (since ->pre_reset ignores the mandatory > > > > > > Wrong. > > > > You cannot know it unless you know how chip operates internally. That's it. > > Read the code. How the chip operates is irrelevant.
lol, there is a genuine cable detection bug in pata_hpt3x2n on closer look...
Jeff, please apply the patch below.
From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> Subject: [PATCH] pata_hpt3x2n: fix cable detection
The detection was reversed between primary and secondary ports.
Fix it to match hpt366 and the vendor driver.
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> --- drivers/ata/pata_hpt3x2n.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Index: b/drivers/ata/pata_hpt3x2n.c =================================================================== --- a/drivers/ata/pata_hpt3x2n.c +++ b/drivers/ata/pata_hpt3x2n.c @@ -133,7 +133,7 @@ static int hpt3x2n_cable_detect(struct a /* Restore state */ pci_write_config_byte(pdev, 0x5B, scr2); - if (ata66 & (1 << ap->port_no)) + if (ata66 & (2 >> ap->port_no)) return ATA_CBL_PATA40; else return ATA_CBL_PATA80;
| |