From: Tony Lindgren <tony@atomide.com>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Randy Dunlap <randy.dunlap@oracle.com>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/90] Post 2.6.21 OMAP update
Date: Thu, 5 Apr 2007 09:21:48 -0400 [thread overview]
Message-ID: <20070405132144.GC24297@atomide.com> (raw)
In-Reply-To: <20070404224448.0ff60ed2@the-village.bc.nu>
* Alan Cox <alan@lxorguk.ukuu.org.uk> [070404 17:42]:
> > > That's only a problem when there are <too many> patches to be
> > > reviewed. Hence, do more frequent postings of fewer patches per series.
> > > That will likely also get you more reviewers.
> >
> > Yeah, I agree.. We've already missed two merge windows because of the
> > large patch set :( The problem is that patch set just keeps growing.
>
> The other problem is that GIT patch sets are completely useless for
> merging and reviewing if you simply fire off a blast of 70 changesets.
> Each time you do it at least one gets queried or a NAK and the lot go in
> the bin again. Repeat 50 times...
Yeah, it is also big pain to maintain patchset this large.
> It would be enormously helpful if instead we got say 15 small patch sets
> which are independant so that a query or NAK means 14 sets get applied
> this time around not zero.
I was thinking about merging the patches together more, but the end result
would not be very readable, and we would lose the original author info.
Also trying to do this 15 small patches at time would make it hard
to have it all integrated within the two week merge window.
It would be nice to get to a situation where we only have 10 - 20 small
linux-omap patches ready for every merge window, and then send fixes
during the -rc series.
Regards,
Tony
next prev parent reply other threads:[~2007-04-05 13:22 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-04-04 17:46 [PATCH 0/90] Post 2.6.21 OMAP update Tony Lindgren
2007-04-04 17:46 ` [PATCH 1/90] ARM: OMAP: Place SMS and SDRC into smart idle mode Tony Lindgren
2007-04-04 17:46 ` [PATCH 2/90] ARM: OMAP: Force APLLs always active Tony Lindgren
2007-04-04 17:46 ` [PATCH 3/90] ARM: OMAP: Add DMA IRQ sanity checks Tony Lindgren
2007-04-04 17:46 ` [PATCH 4/90] ARM: OMAP: Enable 24xx GPIO autoidling Tony Lindgren
2007-04-04 17:46 ` [PATCH 5/90] ARM: OMAP: Add function to print clock usecounts Tony Lindgren
2007-04-04 17:46 ` [PATCH 6/90] ARM: OMAP: Enable serial idling and wakeup features Tony Lindgren
2007-04-04 17:46 ` [PATCH 7/90] ARM: OMAP: Optimize INTC register accesses and enable autoidling Tony Lindgren
2007-04-04 17:46 ` [PATCH 8/90] ARM: OMAP: FB: add controller platform data Tony Lindgren
2007-04-04 18:27 ` [PATCH 0/90] Post 2.6.21 OMAP update Tony Lindgren
2007-04-04 18:39 ` Randy Dunlap
2007-04-04 18:44 ` Tony Lindgren
2007-04-04 21:44 ` Alan Cox
2007-04-05 13:21 ` Tony Lindgren [this message]
2007-04-05 16:31 ` Roland Dreier
2007-04-05 18:03 ` Tony Lindgren
2007-04-05 18:29 ` Roland Dreier
2007-04-09 21:45 ` Tony Lindgren
2007-04-05 20:27 ` Alan Cox
2007-04-06 7:18 ` Russell King
2007-04-09 21:49 ` Tony Lindgren
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=20070405132144.GC24297@atomide.com \
--to=tony@atomide.com \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=linux-kernel@vger.kernel.org \
--cc=randy.dunlap@oracle.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