* [PATCH 06/15] ASoC: RX-51 audio needs I2C
[not found] <1398770316-19715-1-git-send-email-kaixu.xia@linaro.org>
@ 2014-04-29 11:18 ` Xia Kaixu
2014-05-01 17:59 ` Mark Brown
2014-04-29 11:18 ` [PATCH 14/15] ASoC: Amstrad E3 needs TTY support for codec Xia Kaixu
1 sibling, 1 reply; 4+ messages in thread
From: Xia Kaixu @ 2014-04-29 11:18 UTC (permalink / raw)
To: linux-kernel
Cc: linaro-kernel, arnd, kaixu.xia, Mark Brown, Liam Girdwood,
Peter Ujfalusi, Jarkko Nikula, alsa-devel, linux-omap
From: Arnd Bergmann <arnd@arndb.de>
The codec requires I2C to be enabled, so any other option
that selects it should also depend on I2C.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Xia Kaixu <kaixu.xia@linaro.org>
Cc: Mark Brown <broonie@kernel.org>
Cc: Liam Girdwood <lgirdwood@gmail.com>
Cc: Peter Ujfalusi <peter.ujfalusi@ti.com>
Cc: Jarkko Nikula <jarkko.nikula@bitmer.com>
Cc: alsa-devel@alsa-project.org
Cc: linux-omap@vger.kernel.org
---
sound/soc/omap/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sound/soc/omap/Kconfig b/sound/soc/omap/Kconfig
index e006593..2796af9 100644
--- a/sound/soc/omap/Kconfig
+++ b/sound/soc/omap/Kconfig
@@ -26,7 +26,7 @@ config SND_OMAP_SOC_N810
config SND_OMAP_SOC_RX51
tristate "SoC Audio support for Nokia RX-51"
- depends on SND_OMAP_SOC && ARM && (MACH_NOKIA_RX51 || COMPILE_TEST)
+ depends on SND_OMAP_SOC && ARM && (MACH_NOKIA_RX51 || COMPILE_TEST) && I2C
select SND_OMAP_SOC_MCBSP
select SND_SOC_TLV320AIC3X
select SND_SOC_TPA6130A2
--
1.7.9.5
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH 14/15] ASoC: Amstrad E3 needs TTY support for codec
[not found] <1398770316-19715-1-git-send-email-kaixu.xia@linaro.org>
2014-04-29 11:18 ` [PATCH 06/15] ASoC: RX-51 audio needs I2C Xia Kaixu
@ 2014-04-29 11:18 ` Xia Kaixu
2014-05-01 20:31 ` Mark Brown
1 sibling, 1 reply; 4+ messages in thread
From: Xia Kaixu @ 2014-04-29 11:18 UTC (permalink / raw)
To: linux-kernel
Cc: linaro-kernel, arnd, kaixu.xia, Mark Brown, Liam Girdwood,
Peter Ujfalusi, Jarkko Nikula, linux-omap, alsa-devel
From: Arnd Bergmann <arnd@arndb.de>
The cx20442 codec driver used here requires the TTY layer to
be enabled, or we get a link error:
sound/built-in.o: In function `cx20442_codec_remove':
cx20442.c:398: undefined reference to `tty_hangup'
sound/built-in.o: In function `ams_delta_remove':
ams-delta.c:613: undefined reference to `tty_unregister_ldisc'
sound/built-in.o: In function `ams_delta_cx20442_init':
ams-delta.c:559: undefined reference to `tty_register_ldisc'
This adds the missing dependency in the E3 configuration, there
was already one for the codec.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Xia Kaixu <kaixu.xia@linaro.org>
Cc: Mark Brown <broonie@kernel.org>
Cc: Liam Girdwood <lgirdwood@gmail.com>
Cc: Peter Ujfalusi <peter.ujfalusi@ti.com>
Cc: Jarkko Nikula <jarkko.nikula@bitmer.com>
Cc: linux-omap@vger.kernel.org
Cc: alsa-devel@alsa-project.org
---
sound/soc/omap/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sound/soc/omap/Kconfig b/sound/soc/omap/Kconfig
index 2796af9..d44463a 100644
--- a/sound/soc/omap/Kconfig
+++ b/sound/soc/omap/Kconfig
@@ -37,7 +37,7 @@ config SND_OMAP_SOC_RX51
config SND_OMAP_SOC_AMS_DELTA
tristate "SoC Audio support for Amstrad E3 (Delta) videophone"
- depends on SND_OMAP_SOC && MACH_AMS_DELTA
+ depends on SND_OMAP_SOC && MACH_AMS_DELTA && TTY
select SND_OMAP_SOC_MCBSP
select SND_SOC_CX20442
help
--
1.7.9.5
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 06/15] ASoC: RX-51 audio needs I2C
2014-04-29 11:18 ` [PATCH 06/15] ASoC: RX-51 audio needs I2C Xia Kaixu
@ 2014-05-01 17:59 ` Mark Brown
0 siblings, 0 replies; 4+ messages in thread
From: Mark Brown @ 2014-05-01 17:59 UTC (permalink / raw)
To: Xia Kaixu
Cc: linux-kernel, linaro-kernel, arnd, Liam Girdwood, Peter Ujfalusi,
Jarkko Nikula, alsa-devel, linux-omap
[-- Attachment #1: Type: text/plain, Size: 230 bytes --]
On Tue, Apr 29, 2014 at 07:18:27PM +0800, Xia Kaixu wrote:
> From: Arnd Bergmann <arnd@arndb.de>
>
> The codec requires I2C to be enabled, so any other option
> that selects it should also depend on I2C.
Applied, thanks.
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 14/15] ASoC: Amstrad E3 needs TTY support for codec
2014-04-29 11:18 ` [PATCH 14/15] ASoC: Amstrad E3 needs TTY support for codec Xia Kaixu
@ 2014-05-01 20:31 ` Mark Brown
0 siblings, 0 replies; 4+ messages in thread
From: Mark Brown @ 2014-05-01 20:31 UTC (permalink / raw)
To: Xia Kaixu
Cc: linux-kernel, linaro-kernel, arnd, Liam Girdwood, Peter Ujfalusi,
Jarkko Nikula, linux-omap, alsa-devel
[-- Attachment #1: Type: text/plain, Size: 226 bytes --]
On Tue, Apr 29, 2014 at 07:18:35PM +0800, Xia Kaixu wrote:
> From: Arnd Bergmann <arnd@arndb.de>
>
> The cx20442 codec driver used here requires the TTY layer to
> be enabled, or we get a link error:
Applied, thanks.
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-05-01 20:31 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1398770316-19715-1-git-send-email-kaixu.xia@linaro.org>
2014-04-29 11:18 ` [PATCH 06/15] ASoC: RX-51 audio needs I2C Xia Kaixu
2014-05-01 17:59 ` Mark Brown
2014-04-29 11:18 ` [PATCH 14/15] ASoC: Amstrad E3 needs TTY support for codec Xia Kaixu
2014-05-01 20:31 ` Mark Brown
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).