From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Cc: Rafael <rjw@sisk.pl>, Linux-ALSA <alsa-devel@alsa-project.org>,
Paul Mundt <lethal@linux-sh.org>, Liam Girdwood <lrg@ti.com>,
Kuninori Morimoto <kuninori.morimoto.gx@gmail.com>
Subject: Re: [PATCH 5/5] ASoC: sh: fsi: add FSI-WM8978 support
Date: Tue, 3 Apr 2012 13:31:17 +0100 [thread overview]
Message-ID: <20120403123116.GD3192@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <874nt1k1yf.wl%kuninori.morimoto.gx@renesas.com>
[-- Attachment #1.1: Type: text/plain, Size: 1074 bytes --]
On Mon, Apr 02, 2012 at 07:02:03PM -0700, Kuninori Morimoto wrote:
> If board selects their CODEC driver, the driver will be forcefully included to kernel.
> And some of these are depend on extra config like I2C.
> I'm afraid about this kind of "forcefully select".
> Of course we can create selectable Kconfig for each boards.
> but it is just complex..
I guess, yes. Though thinking about this further perhaps what we really
want to do is provide a way to list all the CODEC drivers which can be
supported by this driver and then have the driver select them all
together, that way we don't have to keep adding Kconfig options for each
individual device.
The driver isn't really particularly FSI specific either... the main
thing I want to avoid is having to end up doing lots of this sort of
little update on lots of platforms - it feels like we should be able to
avoid having to add a Kconfig entry for every single combination of CPU
and CODEC unless there's specific code to support that combination
somehow (like when the CODEC has fancy clocking arrangements).
[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
[-- Attachment #2: Type: text/plain, Size: 0 bytes --]
next prev parent reply other threads:[~2012-04-03 12:31 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-02 1:31 [PATCH 0/5] ASoC: sh: fsi-codec Kuninori Morimoto
2012-04-02 1:32 ` [PATCH 1/5] ASoC: sh: fsi: add fsi-codec card support Kuninori Morimoto
2012-04-02 21:39 ` Mark Brown
2012-04-03 0:14 ` Kuninori Morimoto
2012-04-03 2:01 ` Kuninori Morimoto
2012-04-03 12:36 ` Mark Brown
2012-04-02 1:32 ` [PATCH 2/5] ASoC: sh: fsi: remove fsi-ak4642 card driver Kuninori Morimoto
2012-04-02 1:33 ` [PATCH 3/5] ASoC: sh: fsi: remove fsi-hdmi " Kuninori Morimoto
2012-04-02 1:33 ` [PATCH 4/5] ASoC: sh: fsi: remove fsi-da7210 " Kuninori Morimoto
2012-04-02 1:33 ` [PATCH 5/5] ASoC: sh: fsi: add FSI-WM8978 support Kuninori Morimoto
2012-04-02 21:42 ` Mark Brown
2012-04-03 0:50 ` Kuninori Morimoto
2012-04-03 2:02 ` Kuninori Morimoto
2012-04-03 12:31 ` Mark Brown [this message]
2012-04-03 12:37 ` Mark Brown
2012-04-04 5:04 ` [PATCH 0/4 v2] ASoC: sh: fsi-codec Kuninori Morimoto
2012-04-04 5:06 ` [PATCH 1/4 v2] ASoC: sh: fsi: add fsi-common-codec card support Kuninori Morimoto
2012-04-06 17:58 ` Mark Brown
2012-04-09 0:01 ` Kuninori Morimoto
2012-04-09 4:12 ` [PATCH 0/5 v3] ASoC: add simple-card support Kuninori Morimoto
2012-04-09 4:17 ` [PATCH 1/5 v3] ASoC: add generic " Kuninori Morimoto
2012-04-13 10:35 ` Mark Brown
2012-04-16 7:11 ` Kuninori Morimoto
2012-04-16 8:22 ` Mark Brown
2012-04-16 8:30 ` Kuninori Morimoto
2012-04-16 9:48 ` Takashi Iwai
2012-04-16 10:05 ` Mark Brown
2012-04-16 10:16 ` Takashi Iwai
2012-04-19 2:54 ` Kuninori Morimoto
2012-04-19 5:41 ` Takashi Iwai
2012-04-19 7:00 ` [PATCH] ALSA: workaround: change the timing of alsa_sound_last_init() Kuninori Morimoto
2012-04-19 11:43 ` Mark Brown
2012-04-19 11:52 ` Takashi Iwai
2012-04-09 4:18 ` [PATCH 2/5 v3] ASoC: sh: fsi: use simple-card instead of fsi-ak4642 Kuninori Morimoto
2012-04-09 4:18 ` [PATCH 3/5 v3] ASoC: sh: fsi: use simple-card instead of fsi-hdmi Kuninori Morimoto
2012-04-09 4:19 ` [PATCH 4/5 v3] ASoC: sh: fsi: use simple-card instead of fsi-da7210 Kuninori Morimoto
2012-04-09 4:20 ` [PATCH 5/5 v3] ASoC: sh: fsi: select simple-card on Kconfig Kuninori Morimoto
2012-04-04 5:07 ` [PATCH 2/4 v2] ASoC: sh: fsi: use fsi-common-codec instead of fsi-ak4642 Kuninori Morimoto
2012-04-04 5:08 ` [PATCH 3/4 v2] ASoC: sh: fsi: use fsi-common-codec instead of fsi-hdmi Kuninori Morimoto
2012-04-04 5:09 ` [PATCH 4/4 v2] ASoC: sh: fsi: use fsi-common-codec instead of fsi-da7210 Kuninori Morimoto
2012-04-18 10:47 ` [PATCH 0/4 v2] ASoC: sh: fsi-codec Rafael J. Wysocki
2012-04-18 16:58 ` Mark Brown
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=20120403123116.GD3192@opensource.wolfsonmicro.com \
--to=broonie@opensource.wolfsonmicro.com \
--cc=alsa-devel@alsa-project.org \
--cc=kuninori.morimoto.gx@gmail.com \
--cc=kuninori.morimoto.gx@renesas.com \
--cc=lethal@linux-sh.org \
--cc=lrg@ti.com \
--cc=rjw@sisk.pl \
/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).