Re: sata_promise SATA300TX4 intermittent problems

2007-03-07 Thread Mikael Pettersson
Peter Favrholdt writes: Hi, I've seen intermittent problems with Promise SATA300 TX4 controllers and Linux kernel 2.6.19 (through 2.6.20-rc2 with some additional patches). Sometimes the TX4 will loose a port - a reboot brings the drive back up again. I'm quite sure the

[PATCH 2.6.21-rc2] pmac ide: fix pmac_ide_dma_host_on() declaration

2007-03-03 Thread Mikael Pettersson
of non-void function These occur because -dma_host_on should be a function returning void, but pmac.c bogusly declares it as returning int, and furthermore fails to return a value from it. Correcting the return type to void fixes both warnings. Signed-off-by: Mikael Pettersson [EMAIL PROTECTED

[PATCH 2.6.21-rc2] pata_legacy: fix io/irq mismatch

2007-03-03 Thread Mikael Pettersson
486. Signed-off-by: Mikael Pettersson [EMAIL PROTECTED] --- linux-2.6.21-rc2/drivers/ata/pata_legacy.c.~1~ 2007-02-28 13:32:46.0 +0100 +++ linux-2.6.21-rc2/drivers/ata/pata_legacy.c 2007-03-03 18:40:37.0 +0100 @@ -69,7 +69,7 @@ #define NR_HOST 6 static int legacy_port

[BUG] pata_legacy reserves too many io/irq resources

2007-03-03 Thread Mikael Pettersson
pata_legacy probes 6 io/ctr/irq combinations for devices. Or rather, it loops, claims an io/ctr/irq combination, does an ata_device_add(), and libata then does the probing. However, the io/ctr/irq resource combinations aren't released after failed probes (as in: no devices found). The result is

Re: [RFT] sata_promise: intermittent errors bug hunting

2007-03-02 Thread Mikael Pettersson
On Fri, 2 Mar 2007 13:51:19 +0200 (EET), Tomi Orava wrote: There have been some reports of intermittent errors with sata_promise, especially with newer disks. My theory is that the driver isn't initialisating the controller properly for SATAII 3Gbps transfer speeds. I did a test run

[RFT] sata_promise: decode and report error reasons

2007-02-28 Thread Mikael Pettersson
This is a preliminary patch to add error reason decoding and reporting to sata_promise. It's fairly simplistic but should log all error info the controller is able to provide. Works for me, but my test rig hardly ever throws any errors. Testing by those that experience frequent errors would be

[RFT] sata_promise: intermittent errors bug hunting

2007-02-27 Thread Mikael Pettersson
There have been some reports of intermittent errors with sata_promise, especially with newer disks. My theory is that the driver isn't initialisating the controller properly for SATAII 3Gbps transfer speeds. It would be helpful if people seeing intermittent errors with sata_promise could do the

[PATCH 2.6.21-rc1] sata_promise: fix 20619 new EH merge error

2007-02-25 Thread Mikael Pettersson
When I merged my 20619 new EH conversion with #libata-upstream I had to manually resolve a conflict, and inadvertently lost pdc_pata_ops' -post_internal_cmd binding. Corrected by this patch. Signed-off-by: Mikael Pettersson [EMAIL PROTECTED] --- drivers/ata/sata_promise.c |1 + 1 files

[PATCH 2.6.21-rc1] sata_promise: simplify port setup

2007-02-25 Thread Mikael Pettersson
of the PATA-only 20619 redundantly set up scr_addr for the ports. Remove this. Tested on 20619, 20575, and 20775 chips. Signed-off-by: Mikael Pettersson [EMAIL PROTECTED] --- drivers/ata/sata_promise.c | 29 ++--- 1 files changed, 10 insertions(+), 19 deletions(-) diff

Re: sata_promise: random/intermittent errors

2007-02-19 Thread Mikael Pettersson
On Mon, 19 Feb 2007 12:43:50 +0800, Marc Marais wrote: I've decided to post this to the linux-ide list to see if I can get to the bottom of this problem I'm experiencing with sata_promise and my PATA drives. I've pasted a thread from the linux-raid list where I was trying to

Re: [PATCH] (pata-2.6 fix queue) cmd64x: remove broken SW/MW DMA support

2007-02-16 Thread Mikael Pettersson
On Thu, 8 Feb 2007 09:58:45 +0100 (MET), Mikael Pettersson wrote: On Thu, 8 Feb 2007 00:00:32 +0300, Sergei Shtylyov wrote: Remove the bogus code pretending to set SW/MW DMA timings -- I wonder whether its author really thought that he could achieve that wrtiting to BMIDE status registers? Stop

[PATCH] sata_promise: new EH conversion for 20619 chips, take 2

2007-02-11 Thread Mikael Pettersson
and writing CDs. Signed-off-by: Mikael Pettersson [EMAIL PROTECTED] --- Changes since first version: pdc_pata_cbl_detect() is not removed since it's now called from pdc_error_handler() via pdc_pre_reset(). drivers/ata/sata_promise.c | 55 +++-- 1 files

Re: [PATCH] (2.6.20) cmd64x: fix PIO mode setup (take 3)

2007-02-06 Thread Mikael Pettersson
Sergei Shtylyov writes: The driver's tuneproc() method fails to set the drive's own speed -- fix this by renaming the function to cmd64x_tune_pio(), making it return the mode set, and wrapping the new tuneproc() method around it; while at it, also get rid of the non-working prefetch

[PATCH] sata_promise: new EH conversion for 20619 chips

2007-02-04 Thread Mikael Pettersson
in public documentation, so for now I just wrapped it with #if 0/#endif Tested on a TX4000 with both modern working disks and old/quirky disks. Also used a CD-RW drive to test reading and writing CDs. Signed-off-by: Mikael Pettersson [EMAIL PROTECTED] --- drivers/ata/sata_promise.c | 58

Re: Implement the technote about promise/maxtor drives

2007-02-02 Thread Mikael Pettersson
On Fri, 02 Feb 2007 11:49:17 -0500, Jeff Garzik wrote: Alan wrote: I don't have the hardware combination to test this one so would appreciate people testing it before it goes anywhere further I wonder if sata_promise PATA support needs this too? Hmm, I saw Alan's post but ignored it because I

Re: [PATCH 2/4] libata-core.c: add another IRQ calls

2007-01-17 Thread Mikael Pettersson
On Tue, 16 Jan 2007 17:04:27 -0500, Jeff Garzik wrote: Alan wrote: Jeff - at some point we could eliminate a lot of the NULL checks like these by having the registration code fill in the defaults where appropriate ? libata policy up until this point has been to require all drivers fill in the

Re: [PATCH 2/4] libata-core.c: add another IRQ calls

2007-01-17 Thread Mikael Pettersson
Jeff Garzik writes: Mikael Pettersson wrote: If there is a highly common case, the code for that case is available, and the code is cheap to execute (doesn't require too many additional variables), then having an explicit test + inline code is preferred. If any of these conditions

Re: ICH7m problem using libata

2007-01-16 Thread Mikael Pettersson
On Tue, 16 Jan 2007 15:53:14 +0200, Jan Gutter wrote: On Tue, 2007-01-16 at 20:42 +0900, Tejun Heo wrote: Can you try 2.6.20-rc5? It has better error reporting and will tell us which SCSI command is timing out. Here you go: attaching dmesg ... nvidia: module license 'NVIDIA' taints

Re: [TEST] PATA Promise 20378

2007-01-14 Thread Mikael Pettersson
Vincent Legoll writes: Hello, I tested libata-dev#promise-sata-pata branch, then I saw Mikael Pettersson's patch, which kind of obsoleted the testing. So I did libata-dev#upstream too. Nothig bad happened yet, after about 1h30 of test for each kernel. bonnie++, dd, untar,

[PATCH 1/2] sata_promise: issue ATAPI commands as normal packets

2007-01-13 Thread Mikael Pettersson
- pdc_issue_atai_pkt_cmd() and its two helper procedures pdc_wait_for_drq() and pdc_wait_on_busy() are removed Tested on first- and second-generation chips, SATAPI and PATAPI, with no observable regressions. Signed-off-by: Mikael Pettersson [EMAIL PROTECTED] --- drivers/ata/sata_promise.c

[PATCH 2/2] sata_promise: handle ATAPI_NODATA ourselves

2007-01-13 Thread Mikael Pettersson
and PATAPI, with no observable regressions. Signed-off-by: Mikael Pettersson [EMAIL PROTECTED] --- drivers/ata/sata_promise.c | 30 +++--- 1 files changed, 23 insertions(+), 7 deletions(-) --- linux-2.6.20-rc5/drivers/ata/sata_promise.c.~1~ 2007-01-13 16:45:54.0

[PATCH 2.6.20-rc4] MAINTAINERS: maintainer for sata_promise

2007-01-10 Thread Mikael Pettersson
This patch adds myself as maintainer of the sata_promise libata driver. Signed-off-by: Mikael Pettersson [EMAIL PROTECTED] --- linux-2.6.20-rc4/MAINTAINERS.~1~2007-01-08 20:42:55.0 +0100 +++ linux-2.6.20-rc4/MAINTAINERS2007-01-10 01:04:23.0 +0100 @@ -2622,6 +2622,12

[PATCH 2.6.20-rc4 1/2] sata_promise: TX2plus PATA support

2007-01-09 Thread Mikael Pettersson
the exceptional per-port flags array in the driver's private host structure, and uses it in -port_start() to finalise the port's flags. This patch obsoletes the #promise-sata-pata branch included in the #all branch. Signed-off-by: Mikael Pettersson [EMAIL PROTECTED] --- Changes since the RFC version

Re: [PATCH libata #promise-sata-pata] sata_promise: unbreak 20619

2007-01-08 Thread Mikael Pettersson
On Sun, 07 Jan 2007 20:46:13 -0500, Jeff Garzik wrote: Given that I agree with your RFC, this means I can drop all these #promise-sata-pata patches, and kill the #promise-sata-pata branch soon, right? Yes /Mikael - To unsubscribe from this list: send the line unsubscribe linux-ide in the

Re: [PATCH] preliminary sata_promise ATAPI support

2007-01-01 Thread Mikael Pettersson
On Sat, 30 Dec 2006 18:37:18 +0100 (MET), Mikael Pettersson wrote: This patch against 2.6.20-rc2 adds partial ATAPI support to the sata_promise driver. This patch is preliminary and for review only. ... As to why CD-writing fails with DMA enabled, I suspect that I either need to blacklist specific

[PATCH] preliminary sata_promise ATAPI support

2006-12-30 Thread Mikael Pettersson
This patch against 2.6.20-rc2 adds partial ATAPI support to the sata_promise driver. This patch is preliminary and for review only. Current Status: - Tested with PATA optical devices on 20378 and 20575 chips. - Mounting and reading CD/DVD discs in DMA mode works fine. - Using cdrecord to write

Re: SATA Promise

2006-12-19 Thread Mikael Pettersson
On Tue, 19 Dec 2006 08:54:07 -0700, Gary Thomas wrote: I'm trying to use the sata_promise driver (1.04) with a CDROM and having little luck. I'm using this chipset: 00:0b.0 Mass storage controller: Promise Technology, Inc. PDC40775 (SATA 300 TX2plus) (rev 02) If I plug in a SATAPI

Re: OOPS: deref 0x14 at pdc_port_start+0x82 [Was: 2.6.20-rc1-mm1]

2006-12-15 Thread Mikael Pettersson
On Fri, 15 Dec 2006 11:24:12 -0800, Andrew Morton wrote: On Fri, 15 Dec 2006 15:45:55 +0059 Jiri Slaby [EMAIL PROTECTED] wrote: Andrew Morton wrote: Temporarily at http://userweb.kernel.org/~akpm/2.6.20-rc1-mm1/ Will appear later at

Re: [PATCH 2.6.19 2/3] sata_promise: new EH conversion

2006-12-06 Thread Mikael Pettersson
On Sun, 03 Dec 2006 22:19:35 +0900, Tejun Heo wrote: Jeff Garzik wrote: Tejun Heo wrote: Hello, Mikael. Thanks for doing this. Mikael Pettersson wrote: [--snip--] +static void pdc_freeze(struct ata_port *ap) +{ +void __iomem *mmio = (void __iomem *) ap-ioaddr.cmd_addr; +u32 tmp

[PATCH 2.6.19] sata_promise: cleanups, take 2

2006-12-06 Thread Mikael Pettersson
-dev #upstream, cleaned up hotplug_offset computation based on Tejun's comments, expanded hotplug_offset removal rationale. This patch does not depend on the pending new EH conversion patch. Signed-off-by: Mikael Pettersson [EMAIL PROTECTED] --- linux-2.6.19/drivers/ata/sata_promise.c.~1~ 2006-12

[PATCH 2.6.19 1/3] sata_promise: PHYMODE4 fixup

2006-12-01 Thread Mikael Pettersson
this patch does. This patch depends on the sata_promise SATAII updates patch I sent recently. Signed-off-by: Mikael Pettersson [EMAIL PROTECTED] diff -rupN linux-2.6.19.sata_promise-1-genII-fixes/drivers/ata/sata_promise.c linux-2.6.19.sata_promise-2-PHYMODE4-fixup/drivers/ata/sata_promise.c --- linux

[PATCH 2.6.19 3/3] sata_promise: cleanups

2006-12-01 Thread Mikael Pettersson
to compute hotplug_offset in pdc_host_init(). This patch depends on the sata_promise SATAII updates patch I sent recently. Signed-off-by: Mikael Pettersson [EMAIL PROTECTED] diff -rupN linux-2.6.19.sata_promise-3-new_EH/drivers/ata/sata_promise.c linux-2.6.19.sata_promise-4-cleanups/drivers/ata

<    1   2