From: Sergei Shtylyov <sshtylyov@ru.mvista.com>
To: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Cc: linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [git pull] IDE updates #4
Date: Thu, 23 Oct 2008 19:57:14 +0400 [thread overview]
Message-ID: <49009EDA.2030707@ru.mvista.com> (raw)
In-Reply-To: <48FFAFBB.8020709@ru.mvista.com>
Hello, I wrote:
>>>> and number of new submitted patches is < 10 (I'll take
>>>> care of fixing them up, ditto for all other new stuff that will be
>>>> using old
>>>> naming scheme).
>>> Thanks for clarifying this.
>>> This rename only added more uncertainty for my pending patchset
>>> (which had been already dependant on at least TX4939 driver which
>>> keeps being recast by Atsushi and being stale in pata-2.6 series) as
>>> I can't predict when you and Linus will merge the changes and this is
>>> getting on my nerves, as I don't have time on any extra rework and
>>> I'm running out of time with the submission. I know I should have
>>> done this earlier and
>> Maybe some parts could be submitted separately?
>> (so keeping them up-to-date in pata-2.6 would be my task)
> 2 (maybe even 3) out of 4 can be but that doesn't make much sense
> already (and would incur the patch reordering for me) -- the best thing
> you can do is to merge ASAP the last verison of TX4939 which has my ACK.
> I'm not sure about TX4938 driver yet -- will look at it after some sleep...
Still haven't looked at it... too little sleep and incuring headache. :-/
>> Also I didn't know anything about your patchset and its
>> dependency on TX4939, otherwise I'll be pushing things in
> The patchset consists of a large patch moving read_sff_dma_status() to
> its porper place, one small preparatory patch, and 2 followup patches,
> so unfortunately it's dependent on TX4939 in its main patch (worse, the
> relevant part of this driver has changed after your last merged driver
> version)...
>> different order or even skip this pull request if needed
>> (TX493x drivers are new stuff and were still under review,
>> such things can be also submitted after the merge window
>> closes so they were given the lowest priority).
> Unfortunately, that driver has been submitted first back 9/09, long
> before my patchset was even created, so the dependence was just natural.
I could also rip out TX4939 part from the patch and leave Atsushi to deal
with the fallout (though I could give him the ripped out part to simply be
merged to the driver) if you would queue my patchset ahead of the driver.
Though I feel it's too late now for my patchset to get into 2.6.28 the way
things have been happening... :-/
MBR, Sergei
next prev parent reply other threads:[~2008-10-23 15:57 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-10-21 19:00 [git pull] IDE updates #4 Bartlomiej Zolnierkiewicz
2008-10-21 19:18 ` Sergei Shtylyov
2008-10-23 21:13 ` Bartlomiej Zolnierkiewicz
2008-10-22 11:49 ` Sergei Shtylyov
2008-10-22 12:14 ` Bartlomiej Zolnierkiewicz
2008-10-22 12:44 ` Sergei Shtylyov
2008-10-22 21:37 ` Bartlomiej Zolnierkiewicz
2008-10-22 22:56 ` Sergei Shtylyov
2008-10-23 15:57 ` Sergei Shtylyov [this message]
2008-10-23 21:07 ` Bartlomiej Zolnierkiewicz
2008-10-23 22:13 ` Sergei Shtylyov
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=49009EDA.2030707@ru.mvista.com \
--to=sshtylyov@ru.mvista.com \
--cc=bzolnier@gmail.com \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@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).