From: broonie@opensource.wolfsonmicro.com (Mark Brown)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 06/12] imx-ssi sound driver
Date: Mon, 23 Nov 2009 12:00:01 +0000 [thread overview]
Message-ID: <20091123120000.GC24326@rakim.wolfsonmicro.main> (raw)
In-Reply-To: <ed82fe3e0911211812i49342b0eg2afe861c866db90c@mail.gmail.com>
On Sat, Nov 21, 2009 at 08:12:34PM -0600, Timur Tabi wrote:
> Why did you write a new SSI driver when there already is one in sound/soc/fsl?
I can't speak for Sascha here but the i.MX BSPs provided by your
colleagues on the i.MX team also have an independant driver for the SSI
port. The latest public version of the Freescale i.MX code I'm aware of
is in the recent Ubuntu kernels - I've not yet checked but I'd not be
surprised to see substantial code sharing between Sascha's code and the
BSP for the SSI-specific bits.
It would be very much desirable to unify the various drivers, though I
would prefer to avoid making this a blocker for getting i.MX3x "DMA"
support in mainline. Taking a brief glance through the code it looks
like there's some platform-specifics in the drivers - the i.MX one has
optional DMA support due to the whole SDMA issue, while the PowerPC one
is including asm/immap_86xx.h for something. Perhaps pulling the common
code into a library would be a good approach here?
next prev parent reply other threads:[~2009-11-23 12:00 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-19 15:48 i.MX audio support Sascha Hauer
2009-11-19 15:48 ` [PATCH 01/12] mx3: Add SSI pins to iomux table Sascha Hauer
2009-11-19 15:48 ` [PATCH 02/12] mxc: iomux v3: remove resource handling Sascha Hauer
2009-11-19 15:48 ` [PATCH 03/12] add a mc13783 codec driver Sascha Hauer
2009-11-19 15:48 ` [PATCH 04/12] i.MX31 clock: rename SSI clocks to driver name Sascha Hauer
2009-11-19 15:48 ` [PATCH 05/12] mxc: mx1/mx2 DMA: add a possibility to create an endless DMA transfer Sascha Hauer
2009-11-19 15:48 ` [PATCH 06/12] imx-ssi sound driver Sascha Hauer
2009-11-19 15:48 ` [PATCH 07/12] add phycore sound support Sascha Hauer
2009-11-19 15:48 ` [PATCH 08/12] sound/soc/imx: Makefile/Kconfig changes for new driver Sascha Hauer
2009-11-19 15:48 ` [PATCH 09/12] pcm038: add sound support Sascha Hauer
2009-11-19 15:48 ` [PATCH 10/12] pcm043: " Sascha Hauer
2009-11-19 15:48 ` [PATCH 11/12] pca100: " Sascha Hauer
2009-11-19 15:48 ` [PATCH 12/12] pcm037: Add " Sascha Hauer
2009-11-19 19:03 ` [alsa-devel] [PATCH 07/12] add phycore " Mark Brown
2009-11-20 11:11 ` [alsa-devel] [PATCH 06/12] imx-ssi sound driver Mark Brown
2009-11-22 2:12 ` Timur Tabi
2009-11-23 12:00 ` Mark Brown [this message]
2009-11-23 12:13 ` Sascha Hauer
2009-11-28 22:00 ` Timur Tabi
2009-11-23 12:10 ` Sascha Hauer
2009-11-28 19:53 ` Timur Tabi
2009-11-19 19:30 ` [alsa-devel] [PATCH 03/12] add a mc13783 codec driver Mark Brown
2009-11-25 7:46 ` Sascha Hauer
2009-11-25 10:39 ` Mark Brown
2009-11-25 11:08 ` Sascha Hauer
2009-11-25 11:10 ` Mark Brown
2009-11-25 11:30 ` Sascha Hauer
2009-11-25 12:00 ` Mark Brown
2009-11-19 16:28 ` i.MX audio support Mark Brown
2009-11-19 17:53 ` Sascha Hauer
2009-11-19 18:54 ` [alsa-devel] " Mark Brown
2009-11-20 7:51 ` javier Martin
2009-11-20 9:51 ` Sascha Hauer
2009-11-20 10:32 ` javier Martin
2009-11-20 11:10 ` Sascha Hauer
2009-11-19 16:32 ` Mark Brown
2009-11-19 17:47 ` [alsa-devel] " Sascha Hauer
2009-11-19 18:32 ` Mark Brown
2009-11-20 11:17 ` Mark Brown
2009-11-20 12:27 ` Sascha Hauer
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=20091123120000.GC24326@rakim.wolfsonmicro.main \
--to=broonie@opensource.wolfsonmicro.com \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).