linux-omap.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [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).