[lkml]   [2007]   [Apr]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    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

    >>>[ 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
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2007-04-17 19:09    [W:0.025 / U:6.484 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site