All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ASoC: rnsd: fix build regression without CONFIG_OF
@ 2015-04-10 22:18 ` Arnd Bergmann
  0 siblings, 0 replies; 5+ messages in thread
From: Arnd Bergmann @ 2015-04-10 22:18 UTC (permalink / raw)
  To: broonie
  Cc: alsa-devel, lgirdwood, Kuninori Morimoto, linux-kernel, linux-arm-kernel

The r-car sound driver only works when CONFIG_OF is set, and
after a recent change has a compile-time dependency as well:

sound/built-in.o: In function `rsnd_dma_request_channel':
:(.text+0x9fb84): undefined reference to `of_dma_request_slave_channel'

This could be fixed either by adding a static inline wrapper
for the function, or by adding a Kconfig dependency. This
implements the second approach, which seems appropriate
because the driver in fact has a hard dependency.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Fixes: 72adc61f4637aa3 ("ASoC: rsnd: 1st DMAC dma-names cares subnode")

diff --git a/sound/soc/sh/Kconfig b/sound/soc/sh/Kconfig
index 2b3030415573..07114b0b0dc1 100644
--- a/sound/soc/sh/Kconfig
+++ b/sound/soc/sh/Kconfig
@@ -36,6 +36,7 @@ config SND_SOC_SH4_SIU
 
 config SND_SOC_RCAR
 	tristate "R-Car series SRU/SCU/SSIU/SSI support"
+	depends on DMA_OF
 	select SND_SIMPLE_CARD
 	select REGMAP_MMIO
 	help


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

* [PATCH] ASoC: rnsd: fix build regression without CONFIG_OF
@ 2015-04-10 22:18 ` Arnd Bergmann
  0 siblings, 0 replies; 5+ messages in thread
From: Arnd Bergmann @ 2015-04-10 22:18 UTC (permalink / raw)
  To: broonie
  Cc: linux-arm-kernel, alsa-devel, lgirdwood, Kuninori Morimoto, linux-kernel

The r-car sound driver only works when CONFIG_OF is set, and
after a recent change has a compile-time dependency as well:

sound/built-in.o: In function `rsnd_dma_request_channel':
:(.text+0x9fb84): undefined reference to `of_dma_request_slave_channel'

This could be fixed either by adding a static inline wrapper
for the function, or by adding a Kconfig dependency. This
implements the second approach, which seems appropriate
because the driver in fact has a hard dependency.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Fixes: 72adc61f4637aa3 ("ASoC: rsnd: 1st DMAC dma-names cares subnode")

diff --git a/sound/soc/sh/Kconfig b/sound/soc/sh/Kconfig
index 2b3030415573..07114b0b0dc1 100644
--- a/sound/soc/sh/Kconfig
+++ b/sound/soc/sh/Kconfig
@@ -36,6 +36,7 @@ config SND_SOC_SH4_SIU
 
 config SND_SOC_RCAR
 	tristate "R-Car series SRU/SCU/SSIU/SSI support"
+	depends on DMA_OF
 	select SND_SIMPLE_CARD
 	select REGMAP_MMIO
 	help

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

* [PATCH] ASoC: rnsd: fix build regression without CONFIG_OF
@ 2015-04-10 22:18 ` Arnd Bergmann
  0 siblings, 0 replies; 5+ messages in thread
From: Arnd Bergmann @ 2015-04-10 22:18 UTC (permalink / raw)
  To: linux-arm-kernel

The r-car sound driver only works when CONFIG_OF is set, and
after a recent change has a compile-time dependency as well:

sound/built-in.o: In function `rsnd_dma_request_channel':
:(.text+0x9fb84): undefined reference to `of_dma_request_slave_channel'

This could be fixed either by adding a static inline wrapper
for the function, or by adding a Kconfig dependency. This
implements the second approach, which seems appropriate
because the driver in fact has a hard dependency.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Fixes: 72adc61f4637aa3 ("ASoC: rsnd: 1st DMAC dma-names cares subnode")

diff --git a/sound/soc/sh/Kconfig b/sound/soc/sh/Kconfig
index 2b3030415573..07114b0b0dc1 100644
--- a/sound/soc/sh/Kconfig
+++ b/sound/soc/sh/Kconfig
@@ -36,6 +36,7 @@ config SND_SOC_SH4_SIU
 
 config SND_SOC_RCAR
 	tristate "R-Car series SRU/SCU/SSIU/SSI support"
+	depends on DMA_OF
 	select SND_SIMPLE_CARD
 	select REGMAP_MMIO
 	help

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

* Re: [PATCH] ASoC: rnsd: fix build regression without CONFIG_OF
  2015-04-10 22:18 ` Arnd Bergmann
@ 2015-04-11 22:05   ` Mark Brown
  -1 siblings, 0 replies; 5+ messages in thread
From: Mark Brown @ 2015-04-11 22:05 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: alsa-devel, lgirdwood, Kuninori Morimoto, linux-kernel, linux-arm-kernel

[-- Attachment #1: Type: text/plain, Size: 207 bytes --]

On Sat, Apr 11, 2015 at 12:18:37AM +0200, Arnd Bergmann wrote:
> The r-car sound driver only works when CONFIG_OF is set, and
> after a recent change has a compile-time dependency as well:

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

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

* [PATCH] ASoC: rnsd: fix build regression without CONFIG_OF
@ 2015-04-11 22:05   ` Mark Brown
  0 siblings, 0 replies; 5+ messages in thread
From: Mark Brown @ 2015-04-11 22:05 UTC (permalink / raw)
  To: linux-arm-kernel

On Sat, Apr 11, 2015 at 12:18:37AM +0200, Arnd Bergmann wrote:
> The r-car sound driver only works when CONFIG_OF is set, and
> after a recent change has a compile-time dependency as well:

Applied, thanks.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150411/8a6fae94/attachment.sig>

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

end of thread, other threads:[~2015-04-11 22:05 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-10 22:18 [PATCH] ASoC: rnsd: fix build regression without CONFIG_OF Arnd Bergmann
2015-04-10 22:18 ` Arnd Bergmann
2015-04-10 22:18 ` Arnd Bergmann
2015-04-11 22:05 ` Mark Brown
2015-04-11 22:05   ` 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.