[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4624FE88.2010908@ru.mvista.com>
Date: Tue, 17 Apr 2007 21:06:16 +0400
From: Sergei Shtylyov <sshtylyov@...mvista.com>
To: John Stoffel <john@...ffel.org>
CC: linux-kernel@...r.kernel.org, linux-ide@...r.kernel.org
Subject: Re: 2.6.21-rc6-mm1 ATA HPT37x regression
John Stoffel wrote:
>>>I was just testing out 2.6.21-rc6-mm1 to test some Cyclades patches
>>>and I noticed that my HPT302 (rev1) controller with a pair of 120gb WD
>>>disks are not longer detected and I get the following in the dmesg
>>>logs:
>>>[ 148.121490] hpt37x: DPLL did not stabilize.
>>>Where before, under 2.6.21-rc6 I got the following:
> Sergei> For the moment I thought you're reporting another hpt366.c
> Sergei> driver breakage but then noticed the next line:
> I used to think that I wanted to use that driver too, but it's not
> going to work. It's because I've got the version 1 of the chip, not
> the newer version 2 or higher version.
So what?
> It looks like you really mean for me to use the hpt3x2.c driver
> instead, since that's the only other one which mentions the
> PCI_DEVICE_ID_TTI_HPT302 device id in it's init method.
I actually meant drivers/ide/pci/hpt366.c. :-)
>>>[ 173.749349] pata_hpt37x: BIOS has not set timing clocks.
>>>[ 173.752949] hpt37x: HPT302: Bus clock 33MHz.
> Sergei> Obviously this was bacause of switching from PCI to DPLL
> Sergei> clock. The old and new drivers are different in how they
> Sergei> handle DPLL calibration though. Could you try hpt366.c
> Sergei> (there's been report about its failure too recently -- not
> Sergei> enought details yet)?
> I don't think it will work here, it won't even touch the card since
> the pci_device_id won't match at all. The htp366.c only looks at
> HPT366 ids, while mine is an HPT302 id.
The driver I've named should work (unless something is broken somewhere, that is).
Basically, it uses wider range to calibrate DPLL.
> As a quick note, 2.6.21-rc7 is working just fine here.
Yeah, it's still using PCI clock.
> Ok, so do I need to do anything special with the next -mm release and
> the next version?
Well, let Alan decide that (2Alan: and I said that HPT code is bogus :-).
> Thanks,
> John
MBR, Sergei
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists