All of lore.kernel.org
 help / color / mirror / Atom feed
From: broonie@kernel.org (Mark Brown)
To: linux-arm-kernel@lists.infradead.org
Subject: Porting plat-pxa to the MMP PDMA driver
Date: Mon, 7 Apr 2014 15:19:18 +0100	[thread overview]
Message-ID: <20140407141918.GH14763@sirena.org.uk> (raw)
In-Reply-To: <5342B20C.4070608@zonque.org>

On Mon, Apr 07, 2014 at 04:11:24PM +0200, Daniel Mack wrote:
> On 04/07/2014 04:02 PM, Mark Brown wrote:

> > Is it not possible to do what we're doing with the Samsung
> > platforms and have both platform and generic DMA code in the
> > source, selected via a Kconfig symbol?  That way you don't have to
> > have a flag day.

> Not sure whether I'm following, but on PXA, the DMA controller can
> either be used by the old framework or the new driver, and so users
> can't choose which one to use. I thought about a macro hack to make
> the transition easier, but that seems impossible to do.

That's exactly what Samsung have too - an old API and dmaengine as
either/or options.

> Sure, we could keep implementations for both environments around, but
> chances are that we'll never get rid of the old cruft at all then.
> Also, for most drivers that means lots of code duplication.

At least with Samsung the transition has mostly happened, it's been
taking a long time but there's progress and it means that platforms that
are actively being worked on get to use the new stuff.  The code
duplication is a pain point but it's less painful than having to
transition the entire platform at once when it's difficult to get people
to care.  It makes it easier to maintian forward motion.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140407/20c91c88/attachment.sig>

  reply	other threads:[~2014-04-07 14:19 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-07 13:12 Porting plat-pxa to the MMP PDMA driver Laurent Pinchart
2014-04-07 13:25 ` Daniel Mack
2014-04-07 14:02   ` Mark Brown
2014-04-07 14:11     ` Daniel Mack
2014-04-07 14:19       ` Mark Brown [this message]
2014-04-08  1:46   ` Laurent Pinchart

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=20140407141918.GH14763@sirena.org.uk \
    --to=broonie@kernel.org \
    --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 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.