From: plagnioj@jcrosoft.com (Jean-Christophe PLAGNIOL-VILLARD)
To: linux-arm-kernel@lists.infradead.org
Subject: [PULL] at91 init factorize
Date: Sun, 31 Jul 2011 05:44:17 +0200 [thread overview]
Message-ID: <20110731034417.GA6255@game.jcrosoft.org> (raw)
In-Reply-To: <alpine.LFD.2.00.1107301434160.12766@xanadu.home>
On 14:39 Sat 30 Jul , Nicolas Pitre wrote:
> On Sat, 30 Jul 2011, Jean-Christophe PLAGNIOL-VILLARD wrote:
>
> > On 17:58 Thu 28 Jul , Arnd Bergmann wrote:
> > > On Thursday 28 July 2011, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > > > > The patches look good, but come at an inconvenient time. We're still
> > > > > in the merge window, so I don't want to add stuff to linux-next yet
> > > > > that is destined for 3.2, and I have already sent out all the arm-soc
> > > > > patches for the 3.1 merge window, so I don't really want to send another
> > > > > round of patches at the last minute.
> > > > this are waiting ofr 2 release already
> > > >
> > > > and was inthe next last release for more than 1 month
> > > >
> > > > can we have them merge this time
> > >
> > > What I don't understand at all is why you are waiting instead of sending
> > > a pull request for all that time then. I've repeatedly given announcements
> > > about the state of the arm-soc tree and asked people to send patches
> > > they want in 3.1, and you actually sent bug fixes that way earlier
> > >
> > > You've had more than enough time before the merge window, and would even
> > > have made an exception if you had sent your stuff a few hours earlier,
> > > before I sent everything to Linus.
> > > Also, the branch you sent me was created on the same day, meaning that
> > > it can't possibly have had a lot of testing (though it looks harmless
> > > enough). When you send a pull request, the patches should always be
> > > based on an -rc or main release to simplify the merge history, and
> > > it's better not to rebase to the latest one if you already have the
> > > patches.
> > sorry but those patch are 4 motnhs old and yes I rebase the branch to the
> > current linus tree before send the pull request.
>
> Please don't do that. It is best if you keep the same branch content
> identical to what has been tested and validated for a while.
except I send time to retest it
>
> > so as the merge is still open and the patches was in the next for more than
> > one month there no reason to do not pull them for this relese
>
> Yes there is a reason: you were invited to submit that pull request much
> sooner i.e. _before_ the merge window opened. Why didn't you do it a
> month ago?
work on other cleanup for this merge but can not finish them in time
so this work can go other work are pending as they depend on it
let this pull go and theere was no announch that we can not send pull during
the merge windows so if there is such rule we must write a patch and put it in
the ARM tree to specified the ARM merge window otherwise the merge normal
merge window prime
Best Regards,
J.
next prev parent reply other threads:[~2011-07-31 3:44 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-27 12:11 [PULL] at91 init factorize Jean-Christophe PLAGNIOL-VILLARD
2011-07-27 14:47 ` Arnd Bergmann
2011-07-27 23:34 ` Jean-Christophe PLAGNIOL-VILLARD
2011-07-28 15:58 ` Arnd Bergmann
2011-07-30 7:46 ` Jean-Christophe PLAGNIOL-VILLARD
2011-07-30 18:39 ` Nicolas Pitre
2011-07-31 3:44 ` Jean-Christophe PLAGNIOL-VILLARD [this message]
2011-07-31 15:12 ` Russell King - ARM Linux
2011-08-02 12:38 ` Jean-Christophe PLAGNIOL-VILLARD
2011-07-31 21:42 ` Nicolas Ferre
2011-08-02 12:40 ` Jean-Christophe PLAGNIOL-VILLARD
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=20110731034417.GA6255@game.jcrosoft.org \
--to=plagnioj@jcrosoft.com \
--cc=linux-arm-kernel@lists.infradead.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).