From mboxrd@z Thu Jan 1 00:00:00 1970 From: Albert Lee Subject: Re: [PATCH 1/1] libata: pata_pdc2027x PLL input clock fix Date: Wed, 11 Jul 2007 10:45:35 +0800 Message-ID: <4694444F.3050909@tw.ibm.com> References: <200707102314.l6ANEWq2029195@harpo.it.uu.se> Reply-To: albertl@mail.com Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Received: from e36.co.us.ibm.com ([32.97.110.154]:37903 "EHLO e36.co.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754673AbXGKCpq (ORCPT ); Tue, 10 Jul 2007 22:45:46 -0400 Received: from d03relay04.boulder.ibm.com (d03relay04.boulder.ibm.com [9.17.195.106]) by e36.co.us.ibm.com (8.13.8/8.13.8) with ESMTP id l6B2jjPA016558 for ; Tue, 10 Jul 2007 22:45:45 -0400 Received: from d03av03.boulder.ibm.com (d03av03.boulder.ibm.com [9.17.195.169]) by d03relay04.boulder.ibm.com (8.13.8/8.13.8/NCO v8.3) with ESMTP id l6B2jjwm139944 for ; Tue, 10 Jul 2007 20:45:45 -0600 Received: from d03av03.boulder.ibm.com (loopback [127.0.0.1]) by d03av03.boulder.ibm.com (8.12.11.20060308/8.13.3) with ESMTP id l6B2jijC032078 for ; Tue, 10 Jul 2007 20:45:45 -0600 In-Reply-To: <200707102314.l6ANEWq2029195@harpo.it.uu.se> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Mikael Pettersson Cc: alan@lxorguk.ukuu.org.uk, bahadir.balban@gmail.com, dwm@enoyolf.org, jeff@garzik.org, linux-ide@vger.kernel.org, linuxppc-dev@ozlabs.org, sshtylyov@ru.mvista.com Mikael Pettersson wrote: > > > 2.6.22 + this prints the following on my G3: > > pata_pdc2027x 0000:00:0e.0: version 0.9 > usec_elapsed for mdelay(37) [35431] > start time: [1184112028]s [775333]us > end time: [1184112028]s [810764]us > pata_pdc2027x 0000:00:0e.0: PLL input clock 1691741 kHz > pata_pdc2027x: Invalid PLL input clock 1691741kHz, give up! > My x86 box got this: Jul 11 10:02:17 p4ht-s kernel: [ 260.746980] ACPI: PCI Interrupt 0000:02:05.0[A] -> Link [LNK1] -> GSI 10 (level, low) -> IRQ 10 Jul 11 10:02:17 p4ht-s kernel: [ 260.882905] usec_elapsed for mdelay(37) [36734] Jul 11 10:02:17 p4ht-s kernel: [ 260.882911] start time: [1184119336]s [999802]us Jul 11 10:02:17 p4ht-s kernel: [ 260.882914] end time: [1184119337]s [36536]us Jul 11 10:02:17 p4ht-s kernel: [ 260.882917] pata_pdc2027x 0000:02:05.0: PLL input clock 16829 kHz So, it seems both mdelay(37) and do_gettimeofday() are working properly on PowerMac G3. Maybe the calculated wrong PLL input is due to wrong reading of the counter register? Could you please try the attached debug patch for more clue, thanks. -- albert diff -Nrup 00_libata-dev/drivers/ata/pata_pdc2027x.c 01_debug/drivers/ata/pata_pdc2027x.c --- 00_libata-dev/drivers/ata/pata_pdc2027x.c 2007-07-07 09:58:55.000000000 +0800 +++ 01_debug/drivers/ata/pata_pdc2027x.c 2007-07-11 10:41:41.000000000 +0800 @@ -574,8 +574,8 @@ retry: counter = (bccrh << 15) | bccrl; - PDPRINTK("bccrh [%X] bccrl [%X]\n", bccrh, bccrl); - PDPRINTK("bccrhv[%X] bccrlv[%X]\n", bccrhv, bccrlv); + printk(KERN_ERR "bccrh [%X] bccrl [%X]\n", bccrh, bccrl); + printk(KERN_ERR "bccrhv[%X] bccrlv[%X]\n", bccrhv, bccrlv); /* * The 30-bit decreasing counter are read by 2 pieces. @@ -584,7 +584,7 @@ retry: */ if (retry && !(bccrh == bccrhv && bccrl >= bccrlv)) { retry--; - PDPRINTK("rereading counter\n"); + printk(KERN_ERR "rereading counter\n"); goto retry; } @@ -722,8 +722,21 @@ static long pdc_detect_pll_input_clock(s pll_clock = (start_count - end_count) / 100 * (100000000 / usec_elapsed); - PDPRINTK("start[%ld] end[%ld] \n", start_count, end_count); - PDPRINTK("PLL input clock[%ld]Hz\n", pll_clock); + printk(KERN_ERR "start[%ld] end[%ld] \n", start_count, end_count); + printk(KERN_ERR "usec_elapsed for mdelay(100) [%ld]\n", usec_elapsed); + printk(KERN_ERR "start time: [%ld]s [%ld]us \n", start_time.tv_sec, start_time.tv_usec); + printk(KERN_ERR "end time: [%ld]s [%ld]us \n", end_time.tv_sec, end_time.tv_usec); + printk(KERN_ERR "PLL input clock[%ld]Hz\n", pll_clock); + + /* mdelay(37) for comparison */ + do_gettimeofday(&start_time); + mdelay(37); + do_gettimeofday(&end_time); + usec_elapsed = (end_time.tv_sec - start_time.tv_sec) * 1000000 + + (end_time.tv_usec - start_time.tv_usec); + printk(KERN_ERR "usec_elapsed for mdelay(37) [%ld]\n", usec_elapsed); + printk(KERN_ERR "start time: [%ld]s [%ld]us \n", start_time.tv_sec, start_time.tv_usec); + printk(KERN_ERR "end time: [%ld]s [%ld]us \n", end_time.tv_sec, end_time.tv_usec); return pll_clock; }