All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ASoC: Make SND_SOC_MX27VIS_AIC32X4 depend on I2C
@ 2011-12-04  8:11 Axel Lin
  2011-12-04 23:40 ` Mark Brown
  0 siblings, 1 reply; 2+ messages in thread
From: Axel Lin @ 2011-12-04  8:11 UTC (permalink / raw)
  To: alsa-devel
  Cc: Sascha Hauer, Mark Brown, Arnaud Patard, Eric Bénard,
	Liam Girdwood

SND_SOC_MX27VIS_AIC32X4 selects SND_SOC_TLV320AIC32X4,
but SND_SOC_TLV320AIC32X4 needs CONFIG_I2C.
So we need to make SND_SOC_MX27VIS_AIC32X4 depend on I2C.
otherwise I got below build error if CONFIG_I2C is not selected.

  CC      sound/soc/codecs/tlv320aic32x4.o
sound/soc/codecs/tlv320aic32x4.c: In function 'aic32x4_read':
sound/soc/codecs/tlv320aic32x4.c:323: error: implicit declaration of function 'i2c_smbus_read_byte_data'
sound/soc/codecs/tlv320aic32x4.c: In function 'aic32x4_probe':
sound/soc/codecs/tlv320aic32x4.c:641: error: 'i2c_master_send' undeclared (first use in this function)
sound/soc/codecs/tlv320aic32x4.c:641: error: (Each undeclared identifier is reported only once
sound/soc/codecs/tlv320aic32x4.c:641: error: for each function it appears in.)
sound/soc/codecs/tlv320aic32x4.c: In function 'aic32x4_modinit':
sound/soc/codecs/tlv320aic32x4.c:763: error: implicit declaration of function 'i2c_add_driver'
sound/soc/codecs/tlv320aic32x4.c: In function 'aic32x4_exit':
sound/soc/codecs/tlv320aic32x4.c:774: error: implicit declaration of function 'i2c_del_driver'
make[3]: *** [sound/soc/codecs/tlv320aic32x4.o] Error 1
make[2]: *** [sound/soc/codecs] Error 2
make[1]: *** [sound/soc] Error 2
make: *** [sound] Error 2

Signed-off-by: Axel Lin <axel.lin@gmail.com>
---
 sound/soc/imx/Kconfig |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/sound/soc/imx/Kconfig b/sound/soc/imx/Kconfig
index b133bfc..7383917 100644
--- a/sound/soc/imx/Kconfig
+++ b/sound/soc/imx/Kconfig
@@ -28,7 +28,7 @@ config SND_MXC_SOC_WM1133_EV1
 
 config SND_SOC_MX27VIS_AIC32X4
 	tristate "SoC audio support for Visstrim M10 boards"
-	depends on MACH_IMX27_VISSTRIM_M10
+	depends on MACH_IMX27_VISSTRIM_M10 && I2C
 	select SND_SOC_TLV320AIC32X4
 	select SND_MXC_SOC_MX2
 	help
-- 
1.7.5.4

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] ASoC: Make SND_SOC_MX27VIS_AIC32X4 depend on I2C
  2011-12-04  8:11 [PATCH] ASoC: Make SND_SOC_MX27VIS_AIC32X4 depend on I2C Axel Lin
@ 2011-12-04 23:40 ` Mark Brown
  0 siblings, 0 replies; 2+ messages in thread
From: Mark Brown @ 2011-12-04 23:40 UTC (permalink / raw)
  To: Axel Lin
  Cc: Arnaud Patard, alsa-devel, Sascha Hauer, Eric Bénard,
	Liam Girdwood

On Sun, Dec 04, 2011 at 04:11:16PM +0800, Axel Lin wrote:
> SND_SOC_MX27VIS_AIC32X4 selects SND_SOC_TLV320AIC32X4,
> but SND_SOC_TLV320AIC32X4 needs CONFIG_I2C.
> So we need to make SND_SOC_MX27VIS_AIC32X4 depend on I2C.
> otherwise I got below build error if CONFIG_I2C is not selected.

Applied, thanks.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-12-04 23:40 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-12-04  8:11 [PATCH] ASoC: Make SND_SOC_MX27VIS_AIC32X4 depend on I2C Axel Lin
2011-12-04 23:40 ` Mark Brown

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.