public inbox for linux-mmc@vger.kernel.org
 help / color / mirror / Atom feed
From: Chris Ball <cjb@laptop.org>
To: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Cc: linux-sh@vger.kernel.org, linux-mmc@vger.kernel.org,
	Ian Molton <ian@mnementh.co.uk>,
	Magnus Damm <magnus.damm@gmail.com>,
	Samuel Ortiz <sameo@linux.intel.com>,
	Paul Mundt <lethal@linux-sh.org>
Subject: Re: outstanding tmio patches
Date: Thu, 24 Mar 2011 22:05:14 -0400	[thread overview]
Message-ID: <m3k4folztx.fsf@pullcord.laptop.org> (raw)
In-Reply-To: <Pine.LNX.4.64.1103231505450.6836@axis700.grange> (Guennadi Liakhovetski's message of "Wed, 23 Mar 2011 15:15:43 +0100 (CET)")

Hi Guennadi,

On Wed, Mar 23 2011, Guennadi Liakhovetski wrote:
> I thought it might help to list all the recent patches in the order they 
> should apply to make it a bit easier...
>
> [1/2] mmc: tmio: Improve DMA stability on sh-mobile

Some serious merge conflicts with Linus W's dmaengine cleanup here.

> [2/2] mmc: tmio: use PIO for short transfers
> [1/6,v2] mmc: tmio: split core functionality, DMA and MFD glue

And again here.  These flag-day patches that move around thousands of
lines of code are extremely difficult to pull off without announcing
them far ahead of time and getting everyone else to hold off from
merging code first; you're essentially saying that development is closed
and everyone else has to wait for your patch to be merged and then
rebase onto it.

I'm open to suggestions on what to do next -- if you find it easier
to prepare a git tree with everything applied for me to pull from
I'd be happy to do that, although I think we should be preparing this
much churn long before the merge window opens if it's supposed to go
into mainline for the current cycle; the merge window should be for
merging linux-next into mainline.

I think my own suggestion, without knowing too much about tmio, would be
to apply the four patches that provide hardware fixes now, for -rc2:

[1/4,v3] mmc: tmio: only access registers above 0xff, if available
[2/4,v3] ARM: mach-shmobile: fix SDHI IO address-range
[3/4,v3] sh: fix SDHI IO address-range
[4/4,v3] mmc: tmio: remove work-around for unmasked SDIO interrupts

and push the module split and clock gating work out to the start of the
next merge window.

Let me know what you think, thanks,

- Chris.
-- 
Chris Ball   <cjb@laptop.org>   <http://printf.net/>
One Laptop Per Child

  parent reply	other threads:[~2011-03-25  2:05 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-23 14:15 outstanding tmio patches Guennadi Liakhovetski
2011-03-24  7:38 ` Guennadi Liakhovetski
2011-03-25  2:05 ` Chris Ball [this message]
2011-03-25  8:04   ` Guennadi Liakhovetski
2011-03-25 13:09     ` Chris Ball
2011-03-25 13:35       ` Guennadi Liakhovetski
2011-03-25 14:01   ` Chris Ball
2011-03-25 14:19     ` Guennadi Liakhovetski
2011-03-25 14:54       ` Chris Ball
2011-03-27  0:20         ` Simon Horman
2011-03-29 22:18       ` Chris Ball
2011-03-29 22:33         ` Guennadi Liakhovetski

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=m3k4folztx.fsf@pullcord.laptop.org \
    --to=cjb@laptop.org \
    --cc=g.liakhovetski@gmx.de \
    --cc=ian@mnementh.co.uk \
    --cc=lethal@linux-sh.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=linux-sh@vger.kernel.org \
    --cc=magnus.damm@gmail.com \
    --cc=sameo@linux.intel.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