From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: jassi brar <jassisinghbrar@gmail.com>
Cc: alsa-devel@alsa-project.org, Ben Dooks <ben-linux@fluff.org>,
Liam Girdwood <lrg@slimlogic.co.uk>
Subject: Re: ASoC: SAMSUNG: About I2S_v4 support
Date: Wed, 3 Mar 2010 09:54:12 +0000 [thread overview]
Message-ID: <20100303095412.GA2581@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <1b68c6791003030032n7de8a0d8t5395a0a260d92b19@mail.gmail.com>
On Wed, Mar 03, 2010 at 05:32:13PM +0900, jassi brar wrote:
> Otherwise, I am planning to re-factor(as and if needed) s3c-i2s-v2.c,
> s3c2412-i2s.c and s3c64xx-i2s.c
> and add a new s3c-i2s-v4.c to handle I2S_v4 controllers.
As I've said when you've mentioned this in the past I'd prefer an
explicit IISv4 driver (sharing most of the actual code with the core)
since none of the Samsung documentation covers the IISv4 and IISv2
blocks as completely independant blocks. This would make it easier for
people to find what they need to use.
> Kindly ACK/NAK this idea if you are busy with other stuff and I am to
> start working on the issue.
Please also remember that we need to get the clock API changes to allow
the driver to start. I've been chasing patches for these for the past
couple of kernel releases but it's not ideal that I need to do this.
next prev parent reply other threads:[~2010-03-03 9:54 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-03-03 8:32 ASoC: SAMSUNG: About I2S_v4 support jassi brar
2010-03-03 9:54 ` Mark Brown [this message]
2010-03-03 11:10 ` jassi brar
2010-03-03 11:21 ` Mark Brown
2010-03-03 12:53 ` jassi brar
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=20100303095412.GA2581@opensource.wolfsonmicro.com \
--to=broonie@opensource.wolfsonmicro.com \
--cc=alsa-devel@alsa-project.org \
--cc=ben-linux@fluff.org \
--cc=jassisinghbrar@gmail.com \
--cc=lrg@slimlogic.co.uk \
/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).