All of lore.kernel.org
 help / color / mirror / Atom feed
From: Haavard Skinnemoen <haavard.skinnemoen@atmel.com>
To: Haavard Skinnemoen <haavard.skinnemoen@atmel.com>
Cc: Dan Williams <dan.j.williams@intel.com>,
	Pierre Ossman <drzeus-list@drzeus.cx>,
	linux-kernel@vger.kernel.org, linux-embedded@vger.kernel.org,
	kernel@avr32linux.org, shannon.nelson@intel.com,
	David Brownell <david-b@pacbell.net>
Subject: Re: [PATCH v4 6/6] Atmel MCI: Driver for Atmel on-chip MMC controllers
Date: Fri, 27 Jun 2008 21:10:14 +0200	[thread overview]
Message-ID: <20080627211014.0e80db04@siona.local> (raw)
In-Reply-To: <1214486603-23655-7-git-send-email-haavard.skinnemoen@atmel.com>

On Thu, 26 Jun 2008 15:23:23 +0200
Haavard Skinnemoen <haavard.skinnemoen@atmel.com> wrote:

> This driver can also use PIO transfers when no DMA channels are
> available, and for transfers where using DMA may be difficult or
> impractical for some reason (e.g. the DMA setup overhead is usually
> not worth it for very short transfers, and badly aligned buffers or
> lengths are difficult to handle.)

Btw, it's probably not that hard to rip the DMA bits out and post them
as a separate patch. This would mean that:
  * Pierre can merge the driver independently of the other 5 patches
  * A separate patch adding DMA support would make it clearer how the
    DMA slave interface is used.
  * The chances of having MMC support out of the box on avr32 boards in
    2.6.27 become greater, and many people have been asking about that
    (including Pierre and David.)

The driver is surprisingly fast with DMA turned off (2-3 MiB/s), but
the CPU usage is of course horrible.

If that sounds like a good plan to you, I'll split the driver tomorrow.

This driver has been out of tree for way too long. I'm hoping we can
get it in before 2.6.27.

Haavard

  reply	other threads:[~2008-06-27 19:10 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-26 13:23 [PATCH v4 0/6] dmaengine/mmc: DMA slave interface and two new drivers Haavard Skinnemoen
2008-06-26 13:23 ` [PATCH v4 1/6] dmaengine: Add dma_client parameter to device_alloc_chan_resources Haavard Skinnemoen
2008-06-26 13:23   ` [PATCH v4 2/6] dmaengine: Add dma_chan_is_in_use() function Haavard Skinnemoen
2008-06-26 13:23     ` [PATCH v4 3/6] dmaengine: Add slave DMA interface Haavard Skinnemoen
2008-06-26 13:23       ` [PATCH v4 4/6] dmaengine: Make DMA Engine menu visible for AVR32 users Haavard Skinnemoen
2008-06-26 13:23         ` [PATCH v4 5/6] dmaengine: Driver for the Synopsys DesignWare DMA controller Haavard Skinnemoen
2008-06-26 13:23           ` [PATCH v4 6/6] Atmel MCI: Driver for Atmel on-chip MMC controllers Haavard Skinnemoen
2008-06-27 19:10             ` Haavard Skinnemoen [this message]
2008-06-27 19:56               ` Pierre Ossman
2008-06-28 12:43                 ` Haavard Skinnemoen
2008-06-28 13:31                   ` Haavard Skinnemoen
2008-06-29 16:49                     ` Pierre Ossman
2008-06-28 13:45                   ` Pierre Ossman
2008-06-28 14:01                     ` Haavard Skinnemoen
2008-06-28 14:11                       ` Pierre Ossman
2008-06-27 21:31               ` Dan Williams
2008-06-28 12:47                 ` Haavard Skinnemoen
2008-07-04  0:40           ` dmaengine skip unmap (was: Re: [PATCH v4 5/6] dmaengine: Driver for the Synopsys DesignWare DMA controller) Dan Williams
2008-07-04 14:47             ` Haavard Skinnemoen
2008-06-26 14:15         ` [PATCH v4 4/6] dmaengine: Make DMA Engine menu visible for AVR32 users Adrian Bunk
2008-06-26 14:46           ` Haavard Skinnemoen
2008-06-27  0:59             ` Dan Williams
2008-06-27 16:37               ` David Brownell
2008-06-27 17:44                 ` Adrian Bunk
2008-06-27 18:24                   ` David Brownell
2008-06-27 18:29                     ` Adrian Bunk
2008-06-27 18:31                     ` Dan Williams
2008-06-27 18:13                 ` Haavard Skinnemoen
2008-06-26 20:04         ` David Brownell
2008-06-26 13:32       ` [PATCH v4 3/6] dmaengine: Add slave DMA interface Haavard Skinnemoen
2008-06-28 12:29       ` Haavard Skinnemoen
2008-07-02  1:31     ` [PATCH v4 2/6] dmaengine: Add dma_chan_is_in_use() function Dan Williams
2008-07-02  2:00       ` Dan Williams
2008-07-02  7:59         ` Haavard Skinnemoen
2008-07-02  7:56       ` Haavard Skinnemoen
2008-07-04  1:06 ` [PATCH v4 0/6] dmaengine/mmc: DMA slave interface and two new drivers Dan Williams
2008-07-04 15:13   ` Haavard Skinnemoen

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=20080627211014.0e80db04@siona.local \
    --to=haavard.skinnemoen@atmel.com \
    --cc=dan.j.williams@intel.com \
    --cc=david-b@pacbell.net \
    --cc=drzeus-list@drzeus.cx \
    --cc=kernel@avr32linux.org \
    --cc=linux-embedded@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=shannon.nelson@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 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.