lkml.org 
[lkml]   [2007]   [Apr]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: 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@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-04-17 19:09    [W:0.415 / U:0.320 seconds]
©2003-2011 Jasper Spaans. Advertise on this site