linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Bart Hartgers <bart.hartgers@gmail.com>
To: Jeff Garzik <jgarzik@pobox.com>
Cc: Tejun Heo <htejun@gmail.com>,
	linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org,
	juergen.metzdorf@telelev-dsl.de, markpschool@hotmail.com,
	sporadic.crash@gmail.com, apopov@sirma.bg, david@coomber.co.za,
	jay4mail@gmail.com
Subject: Re: [patch 2/2] sata_via: Delay on vt6420 when starting ATAPI DMA write
Date: Sun, 14 Feb 2010 12:20:14 +0100	[thread overview]
Message-ID: <4B77DC6E.8070902@gmail.com> (raw)
In-Reply-To: <4B772C19.90501@pobox.com>

Jeff Garzik wrote:
> On 01/20/2010 01:55 AM, Tejun Heo wrote:
>> Hello,
>>
>> On 01/20/2010 03:44 PM, Bart Hartgers wrote:
>>> Yes, you're right. I'll drop the printk_once and send another patch
>>> for inclusion. However, for testing I found it very useful to make
>>> sure that I got the right module loaded. So I figured it could be
>>> helpful for the interpretation of success/failure reports.
>>
>> Oh yeah, for testing, having it there is a good idea.
>>
>>> Assuming that this patch works for other people as well, what is
>>> prefered: resending both patches or just to make a new #2/2 (the
>>> vt6420 one)?
>>
>> I think just re-doing the second patch should be enough.
>
> Agreed...  ping, Bart?
>
Hi Jeff,

I was waiting for some feedback on whether this patch actually solves 
the problem for others as well. Unfortunately, I didn't get any response 
so far, that's why I didn't resend the patch yet. But I guess it doesn't 
make sense to wait any longer, so I'll redo the patch and send it in a 
separate mail in a few minutes.

(I also tried searching for people suffering from this bug in the past 3 
months, and couldn't find any. I _did_ see some recent posts on various 
bug-lists by Tejun asking to test my patch, but again no response.)

Groeten,
Bart

      reply	other threads:[~2010-02-14 11:20 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-16 23:56 [patch 0/2] sata_via: bugfixes for vt6420 and vt6421 Bart Hartgers
2010-01-16 23:56 ` [patch 1/2] sata_via: Correctly setup PIO/DMA for pata slave on vt6421 Bart Hartgers
2010-01-20  3:33   ` Tejun Heo
2010-01-20 19:35   ` Jeff Garzik
2010-01-16 23:56 ` [patch 2/2] sata_via: Delay on vt6420 when starting ATAPI DMA write Bart Hartgers
2010-01-20  3:30   ` Tejun Heo
2010-01-20  4:53     ` Robert Hancock
2010-01-20  6:33       ` Bart Hartgers
2010-01-20  6:54         ` Tejun Heo
2010-01-20 10:43           ` Bart Hartgers
2010-01-20  6:44     ` Bart Hartgers
2010-01-20  6:55       ` Tejun Heo
2010-02-13 22:47         ` Jeff Garzik
2010-02-14 11:20           ` Bart Hartgers [this message]

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=4B77DC6E.8070902@gmail.com \
    --to=bart.hartgers@gmail.com \
    --cc=apopov@sirma.bg \
    --cc=david@coomber.co.za \
    --cc=htejun@gmail.com \
    --cc=jay4mail@gmail.com \
    --cc=jgarzik@pobox.com \
    --cc=juergen.metzdorf@telelev-dsl.de \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=markpschool@hotmail.com \
    --cc=sporadic.crash@gmail.com \
    /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).