linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andrei Martynov <andrei.martynov@web.de>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: linux-ide@vger.kernel.org
Subject: Re[2]: porting IDE patches to PATA driver
Date: Mon, 29 Jun 2009 14:01:57 +0200	[thread overview]
Message-ID: <596875090.20090629140157@web.de> (raw)
In-Reply-To: <20090626172143.3fe6a59c@lxorguk.ukuu.org.uk>

[-- Attachment #1: Type: text/plain, Size: 918 bytes --]

Friday, June 26, 2009, 6:21:43 PM, Alan wrote:

> The source you posted has MWDMA undefined which makes me
> wonder if it matches the binary your dmesg is from.

> It's also possible some of the other fixes in that code are relevant to MWDMA - certainly
> your driver looks sane.

you are right. I'm working with multiple sources from different sites
and missed the fact the original driver ide-jasper.c published at
uclinux.org has BMDMA undefined. Dmegs output was produced using a
slightly modified version of the file that has BMDMA defined, but
there are no other changes in it that would explain unexpected DMA
interrupt mystery, it should be one of the patches done to ide and
cdrom subsystem.

Here is an updated patch, could you please check it? I don't see
anything unusual except the "never disable DMA" logic in ide_dma.c.

-- 
Best regards,
 Andrei                            mailto:andrei.martynov@web.de

[-- Attachment #2: sigma.patch.gz --]
[-- Type: application/x-gzip, Size: 14691 bytes --]

  parent reply	other threads:[~2009-06-29 12:02 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-25  8:09 porting IDE patches to PATA driver Andrei Martynov
2009-06-25  8:23 ` Alan Cox
2009-06-25  8:38   ` Re[2]: " Andrei Martynov
2009-06-25  9:05     ` Alan Cox
2009-06-25 10:30       ` Re[2]: " Andrei Martynov
2009-06-25 11:29         ` Alan Cox
2009-06-26 13:24           ` Re[2]: " Andrei Martynov
2009-06-26 13:46             ` Alan Cox
2009-06-26 14:40               ` Re[2]: " Andrei Martynov
2009-06-26 16:21                 ` Alan Cox
2009-06-28 20:33                   ` Sergei Shtylyov
2009-06-29 12:31                     ` Alan Cox
2009-06-29 12:01                   ` Andrei Martynov [this message]
2009-06-29 14:53                     ` Alan Cox
2009-07-03  9:50                       ` Re[2]: " Andrei Martynov
2009-07-04  1:17                         ` Robert Hancock

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=596875090.20090629140157@web.de \
    --to=andrei.martynov@web.de \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=linux-ide@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).