All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeff Garzik <jeff@garzik.org>
To: Sergey Matyukevich <geomatsi@gmail.com>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>,
	linux-ide@vger.kernel.org, Andrew Victor <linux@maxim.org.za>,
	Sergey Lapin <slapin@ossfans.org>,
	Andrew Victor <avictor.za@gmail.com>
Subject: Re: [PATCH] PATA driver for CF interface on AT91SAM9260 SoC
Date: Sat, 20 Jun 2009 13:09:47 -0400	[thread overview]
Message-ID: <4A3D17DB.30001@garzik.org> (raw)
In-Reply-To: <20090620165439.6314f9ef@realm>

Sergey Matyukevich wrote:
> On Fri, 19 Jun 2009 08:27:40 +0400
> Sergey Matyukevich <geomatsi@gmail.com> wrote:
> 
>> On Thu, 18 Jun 2009 19:37:20 +0100
>> Alan Cox <alan@lxorguk.ukuu.org.uk> wrote:
>>
>>>> Concerning the first point, this module parameter is used in
>>>> driver probing function: 
>>>> 	ap->pio_mask = pio_mask;
>>>>
>>>> I would suggest to keep pio_mask as a module parameter, since it
>>>> might be useful to modify it in order to сut down PIO speed, e.g.
>>>> for testing purposes. 
>>> Libata already has general support for that. (Arguably we should
>>> take it out of the pata_legacy driver in the same way)
>>
>> Patch update: module param for pio_mask removed.
>>
>>
> 
> Hello,
> Is there any chance for this driver to be accepted during current merge
> window ?

Absolutely...  is this the last revision of the driver (for the moment)?

	Jeff




  reply	other threads:[~2009-06-20 17:09 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-16 19:39 [PATCH] PATA driver for CF interface on AT91SAM9260 SoC geomatsi
2009-06-17 20:20 ` Sergey Matyukevich
2009-06-17 22:53   ` Alan Cox
2009-06-18 18:24     ` Sergey Matyukevich
2009-06-18 18:37       ` Alan Cox
2009-06-19  4:27         ` Sergey Matyukevich
2009-06-20 12:54           ` Sergey Matyukevich
2009-06-20 17:09             ` Jeff Garzik [this message]
2009-06-21 16:51               ` Sergey Matyukevich
2009-06-23  5:56           ` Jeff Garzik

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=4A3D17DB.30001@garzik.org \
    --to=jeff@garzik.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=avictor.za@gmail.com \
    --cc=geomatsi@gmail.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux@maxim.org.za \
    --cc=slapin@ossfans.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.