All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tony Lindgren <tony@atomide.com>
To: Carlos Aguiar <carlos.aguiar@indt.org.br>
Cc: omap-linux <linux-omap-open-source@linux.omap.com>
Subject: Re: [PATCH 00/17] MMC: OMAP: Add MMC multislot support
Date: Tue, 27 Nov 2007 18:03:00 -0800	[thread overview]
Message-ID: <20071128020300.GB11825@atomide.com> (raw)
In-Reply-To: <474AEDBB.1070006@indt.org.br>

Hi,

* Carlos Aguiar <carlos.aguiar@indt.org.br> [071126 08:03]:
> Hi Tony and folks,
> 
> This is the latest 'MMC: OMAP: Add MMC multislot support' patch series
> with changes, corrections and improvements based on recent comments and
> suggestions from community and coworkers.
> 
> Just to remind you, this patch series adds the MMC multislot support for
> N800 and TI OMAP H2 and H3 boards, and Siemens SX1. It's based on Juha's and Jarkko's
> earlier implementation and it can be applied against most updated
> linux-omap tree (2.6.24-rc3-omap1, on top of commit 3e1251dd443d07f5a5298646adf746e4add89cf3).
>
> After recent merging of linux omap tree with linus tree, such support was broken for N800.
> After investigating it, I realize the source of the problem comes from DMA, from a line of 
> Anand Gadiyar' earlier patch committed [1].
>
> In order this support work properly on N800, please apply the patch I proposed at [2].

Thanks, pushed your DMA fix.

> I've tested with MMC, SD and SDHC cards and looks fine for the boards
> mentioned above (SX1 is compiled test only).
> 
> Testing, comments and suggestions are welcome.

Well, I've pushed most of your patches. I ended up splitting your
patch 02/17 into something that's easier to read and also compiles.
Basically I temporarily removed all the cover switch handling and
fixed the header.

I did not push patches 15-17 as the power up should be done the same
way as for other boards. Can you please refresh those patches?

Regards,

Tony

  reply	other threads:[~2007-11-28  2:03 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-11-26 16:00 [PATCH 00/17] MMC: OMAP: Add MMC multislot support Carlos Aguiar
2007-11-28  2:03 ` Tony Lindgren [this message]
2007-11-28  8:06   ` Kyungmin Park
2007-11-28 18:05     ` Tony Lindgren
2007-11-28 23:24       ` Kyungmin Park
2007-11-29  4:49         ` Kyungmin Park
2007-11-29  7:19           ` Kyungmin Park
2007-11-29 17:52   ` [PATCH 0/2] MMC: OMAP: Adds MMC multislot support for Siemens SX1 mobile phones Carlos Aguiar
2007-12-01  0:09     ` Tony Lindgren
2007-11-29 17:52   ` [PATCH 1/2] MMC: OMAP: Supporting MMC multislot structures for Siemens SX1 board Carlos Aguiar
2007-11-29 17:52   ` [PATCH 2/2] MMC: OMAP: Make board-sx1.c uses new board-sx1-mmc.c code Carlos Aguiar

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=20071128020300.GB11825@atomide.com \
    --to=tony@atomide.com \
    --cc=carlos.aguiar@indt.org.br \
    --cc=linux-omap-open-source@linux.omap.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 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.