* [PATCH] soc/sh: Fix I2C dependency for SND_FSI_AK4642
@ 2010-06-14 15:39 ` Peter Huewe
0 siblings, 0 replies; 22+ messages in thread
From: Peter Huewe @ 2010-06-14 15:39 UTC (permalink / raw)
To: Liam Girdwood
Cc: linux-sh, Mark Brown, Jaroslav Kysela, Takashi Iwai,
Kuninori Morimoto, Guennadi Liakhovetski, alsa-devel,
linux-kernel
From: Peter Huewe <peterhuewe@gmx.de>
The config option SND_FSI_AK4642 selects SND_SOC_AK464 which in turn
enables the compilation of ak4642.c - however this codec uses I2C to
communicate with the HW.
Consequently when I2C is not set, the compilation fails [1]
This patch fixes this issues.
Signed-off-by: Peter Huewe <peterhuewe@gmx.de>
---
References:
[1] http://kisskb.ellerman.id.au/kisskb/buildresult/2709408/
KernelVersion: linux-next 20100614
sound/soc/sh/Kconfig | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/sound/soc/sh/Kconfig b/sound/soc/sh/Kconfig
index a1d14bc..d26b815 100644
--- a/sound/soc/sh/Kconfig
+++ b/sound/soc/sh/Kconfig
@@ -50,6 +50,7 @@ config SND_FSI_AK4642
bool "FSI-AK4642 sound support"
depends on SND_SOC_SH4_FSI
select SND_SOC_AK4642
+ select I2C
help
This option enables generic sound support for the
FSI - AK4642 unit
--
1.6.4.4
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH] soc/sh: Fix I2C dependency for SND_FSI_AK4642
@ 2010-06-14 15:39 ` Peter Huewe
0 siblings, 0 replies; 22+ messages in thread
From: Peter Huewe @ 2010-06-14 15:39 UTC (permalink / raw)
To: Liam Girdwood
Cc: linux-sh, Mark Brown, Jaroslav Kysela, Takashi Iwai,
Kuninori Morimoto, Guennadi Liakhovetski, alsa-devel,
linux-kernel
From: Peter Huewe <peterhuewe@gmx.de>
The config option SND_FSI_AK4642 selects SND_SOC_AK464 which in turn
enables the compilation of ak4642.c - however this codec uses I2C to
communicate with the HW.
Consequently when I2C is not set, the compilation fails [1]
This patch fixes this issues.
Signed-off-by: Peter Huewe <peterhuewe@gmx.de>
---
References:
[1] http://kisskb.ellerman.id.au/kisskb/buildresult/2709408/
KernelVersion: linux-next 20100614
sound/soc/sh/Kconfig | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/sound/soc/sh/Kconfig b/sound/soc/sh/Kconfig
index a1d14bc..d26b815 100644
--- a/sound/soc/sh/Kconfig
+++ b/sound/soc/sh/Kconfig
@@ -50,6 +50,7 @@ config SND_FSI_AK4642
bool "FSI-AK4642 sound support"
depends on SND_SOC_SH4_FSI
select SND_SOC_AK4642
+ select I2C
help
This option enables generic sound support for the
FSI - AK4642 unit
--
1.6.4.4
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH] soc/sh: Fix I2C dependency for SND_FSI_AK4642
2010-06-14 15:39 ` Peter Huewe
(?)
@ 2010-06-14 16:10 ` Mark Brown
-1 siblings, 0 replies; 22+ messages in thread
From: Mark Brown @ 2010-06-14 16:10 UTC (permalink / raw)
To: Peter Huewe
Cc: alsa-devel, linux-sh, Takashi Iwai, linux-kernel,
Kuninori Morimoto, Guennadi Liakhovetski, Liam Girdwood
On Mon, Jun 14, 2010 at 05:39:41PM +0200, Peter Huewe wrote:
> bool "FSI-AK4642 sound support"
> depends on SND_SOC_SH4_FSI
> select SND_SOC_AK4642
> + select I2C
I'd have thought a depends was more appropriate. Arguably you want to
depend on the SH I2C driver since the driver is useless without the
relevant I2C controller.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] soc/sh: Fix I2C dependency for SND_FSI_AK4642
@ 2010-06-14 16:10 ` Mark Brown
0 siblings, 0 replies; 22+ messages in thread
From: Mark Brown @ 2010-06-14 16:10 UTC (permalink / raw)
To: Peter Huewe
Cc: Liam Girdwood, linux-sh, Jaroslav Kysela, Takashi Iwai,
Kuninori Morimoto, Guennadi Liakhovetski, alsa-devel,
linux-kernel
On Mon, Jun 14, 2010 at 05:39:41PM +0200, Peter Huewe wrote:
> bool "FSI-AK4642 sound support"
> depends on SND_SOC_SH4_FSI
> select SND_SOC_AK4642
> + select I2C
I'd have thought a depends was more appropriate. Arguably you want to
depend on the SH I2C driver since the driver is useless without the
relevant I2C controller.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] soc/sh: Fix I2C dependency for SND_FSI_AK4642
@ 2010-06-14 16:10 ` Mark Brown
0 siblings, 0 replies; 22+ messages in thread
From: Mark Brown @ 2010-06-14 16:10 UTC (permalink / raw)
To: Peter Huewe
Cc: alsa-devel, linux-sh, Takashi Iwai, linux-kernel,
Kuninori Morimoto, Guennadi Liakhovetski, Liam Girdwood
On Mon, Jun 14, 2010 at 05:39:41PM +0200, Peter Huewe wrote:
> bool "FSI-AK4642 sound support"
> depends on SND_SOC_SH4_FSI
> select SND_SOC_AK4642
> + select I2C
I'd have thought a depends was more appropriate. Arguably you want to
depend on the SH I2C driver since the driver is useless without the
relevant I2C controller.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] soc/sh: Fix I2C dependency for SND_FSI_AK4642
2010-06-14 16:10 ` Mark Brown
@ 2010-06-15 14:49 ` Peter Hüwe
-1 siblings, 0 replies; 22+ messages in thread
From: Peter Hüwe @ 2010-06-15 14:49 UTC (permalink / raw)
To: Mark Brown
Cc: Liam Girdwood, linux-sh, Jaroslav Kysela, Takashi Iwai,
Kuninori Morimoto, Guennadi Liakhovetski, alsa-devel,
linux-kernel
Am Montag 14 Juni 2010 18:10:34 schrieb Mark Brown:
> On Mon, Jun 14, 2010 at 05:39:41PM +0200, Peter Huewe wrote:
> > bool "FSI-AK4642 sound support"
> > depends on SND_SOC_SH4_FSI
> > select SND_SOC_AK4642
> > + select I2C
>
> I'd have thought a depends was more appropriate. Arguably you want to
> depend on the SH I2C driver since the driver is useless without the
> relevant I2C controller.
>
Thanks for pointing this out - I had also a closer look at the codecs and the
other FSI related codec (da7210.c) depends on I2C too.
So maybe even move the dependency up to SND_SOC_SH4_FSI
since both codecs rely on I2C?
So something like this would be more appropriate?
config SND_SOC_SH4_FSI
tristate "SH4 FSI support"
+ depends on CONFIG_I2C_SH_MOBILE
help
This option enables FSI sound support
If you think this patch sounds correct, I can create a new 'official' patch.
Peter
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] soc/sh: Fix I2C dependency for SND_FSI_AK4642
@ 2010-06-15 14:49 ` Peter Hüwe
0 siblings, 0 replies; 22+ messages in thread
From: Peter Hüwe @ 2010-06-15 14:49 UTC (permalink / raw)
To: Mark Brown
Cc: Liam Girdwood, linux-sh, Jaroslav Kysela, Takashi Iwai,
Kuninori Morimoto, Guennadi Liakhovetski, alsa-devel,
linux-kernel
Am Montag 14 Juni 2010 18:10:34 schrieb Mark Brown:
> On Mon, Jun 14, 2010 at 05:39:41PM +0200, Peter Huewe wrote:
> > bool "FSI-AK4642 sound support"
> > depends on SND_SOC_SH4_FSI
> > select SND_SOC_AK4642
> > + select I2C
>
> I'd have thought a depends was more appropriate. Arguably you want to
> depend on the SH I2C driver since the driver is useless without the
> relevant I2C controller.
>
Thanks for pointing this out - I had also a closer look at the codecs and the
other FSI related codec (da7210.c) depends on I2C too.
So maybe even move the dependency up to SND_SOC_SH4_FSI
since both codecs rely on I2C?
So something like this would be more appropriate?
config SND_SOC_SH4_FSI
tristate "SH4 FSI support"
+ depends on CONFIG_I2C_SH_MOBILE
help
This option enables FSI sound support
If you think this patch sounds correct, I can create a new 'official' patch.
Peter
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] soc/sh: Fix I2C dependency for SND_FSI_AK4642
2010-06-15 14:49 ` Peter Hüwe
(?)
@ 2010-06-15 15:19 ` Mark Brown
-1 siblings, 0 replies; 22+ messages in thread
From: Mark Brown @ 2010-06-15 15:19 UTC (permalink / raw)
To: Peter Hüwe
Cc: alsa-devel, linux-sh, Takashi Iwai, linux-kernel,
Kuninori Morimoto, Guennadi Liakhovetski, Liam Girdwood
On Tue, Jun 15, 2010 at 04:49:07PM +0200, Peter Hüwe wrote:
> So maybe even move the dependency up to SND_SOC_SH4_FSI
> since both codecs rely on I2C?
No, while the two boards in mainline currently may need I2C obviously
other board designs with other control interfaces are possible so it
should be on the individual boards.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] soc/sh: Fix I2C dependency for SND_FSI_AK4642
@ 2010-06-15 15:19 ` Mark Brown
0 siblings, 0 replies; 22+ messages in thread
From: Mark Brown @ 2010-06-15 15:19 UTC (permalink / raw)
To: Peter Hüwe
Cc: Liam Girdwood, linux-sh, Jaroslav Kysela, Takashi Iwai,
Kuninori Morimoto, Guennadi Liakhovetski, alsa-devel,
linux-kernel
On Tue, Jun 15, 2010 at 04:49:07PM +0200, Peter Hüwe wrote:
> So maybe even move the dependency up to SND_SOC_SH4_FSI
> since both codecs rely on I2C?
No, while the two boards in mainline currently may need I2C obviously
other board designs with other control interfaces are possible so it
should be on the individual boards.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] soc/sh: Fix I2C dependency for SND_FSI_AK4642
@ 2010-06-15 15:19 ` Mark Brown
0 siblings, 0 replies; 22+ messages in thread
From: Mark Brown @ 2010-06-15 15:19 UTC (permalink / raw)
To: Peter Hüwe
Cc: alsa-devel, linux-sh, Takashi Iwai, linux-kernel,
Kuninori Morimoto, Guennadi Liakhovetski, Liam Girdwood
On Tue, Jun 15, 2010 at 04:49:07PM +0200, Peter Hüwe wrote:
> So maybe even move the dependency up to SND_SOC_SH4_FSI
> since both codecs rely on I2C?
No, while the two boards in mainline currently may need I2C obviously
other board designs with other control interfaces are possible so it
should be on the individual boards.
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH v2] soc/sh: Fix I2C dependency for SND_FSI_AK4642 and SND_FSI_DA7210
2010-06-15 15:19 ` Mark Brown
@ 2010-06-15 15:38 ` Peter Huewe
-1 siblings, 0 replies; 22+ messages in thread
From: Peter Huewe @ 2010-06-15 15:38 UTC (permalink / raw)
To: Mark Brown
Cc: Liam Girdwood, linux-sh, Jaroslav Kysela, Takashi Iwai,
Kuninori Morimoto, Guennadi Liakhovetski, alsa-devel,
linux-kernel
From: Peter Huewe <peterhuewe@gmx.de>
The config option SND_FSI_AK4642 selects SND_SOC_AK4642 which in turn
enables the compilation of ak4642.c - however this codec uses I2C to
communicate with the HW.
Same applies to DA7210.
Consequently when I2C is not set, the compilation fails [1]
This patch fixes this issues, by adding a depencdency on the related HW-
controller.
Signed-off-by: Peter Huewe <peterhuewe@gmx.de>
---
How about this one?
References:
[1] http://kisskb.ellerman.id.au/kisskb/buildresult/2709408/
KernelVersion: linux-next 20100615
sound/soc/sh/Kconfig | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/soc/sh/Kconfig b/sound/soc/sh/Kconfig
index a1d14bc..52d7e8e 100644
--- a/sound/soc/sh/Kconfig
+++ b/sound/soc/sh/Kconfig
@@ -48,7 +48,7 @@ config SND_SH7760_AC97
config SND_FSI_AK4642
bool "FSI-AK4642 sound support"
- depends on SND_SOC_SH4_FSI
+ depends on SND_SOC_SH4_FSI && I2C_SH_MOBILE
select SND_SOC_AK4642
help
This option enables generic sound support for the
@@ -56,7 +56,7 @@ config SND_FSI_AK4642
config SND_FSI_DA7210
bool "FSI-DA7210 sound support"
- depends on SND_SOC_SH4_FSI
+ depends on SND_SOC_SH4_FSI && I2C_SH_MOBILE
select SND_SOC_DA7210
help
This option enables generic sound support for the
--
1.6.4.4
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH v2] soc/sh: Fix I2C dependency for SND_FSI_AK4642 and SND_FSI_DA7210
@ 2010-06-15 15:38 ` Peter Huewe
0 siblings, 0 replies; 22+ messages in thread
From: Peter Huewe @ 2010-06-15 15:38 UTC (permalink / raw)
To: Mark Brown
Cc: Liam Girdwood, linux-sh, Jaroslav Kysela, Takashi Iwai,
Kuninori Morimoto, Guennadi Liakhovetski, alsa-devel,
linux-kernel
From: Peter Huewe <peterhuewe@gmx.de>
The config option SND_FSI_AK4642 selects SND_SOC_AK4642 which in turn
enables the compilation of ak4642.c - however this codec uses I2C to
communicate with the HW.
Same applies to DA7210.
Consequently when I2C is not set, the compilation fails [1]
This patch fixes this issues, by adding a depencdency on the related HW-
controller.
Signed-off-by: Peter Huewe <peterhuewe@gmx.de>
---
How about this one?
References:
[1] http://kisskb.ellerman.id.au/kisskb/buildresult/2709408/
KernelVersion: linux-next 20100615
sound/soc/sh/Kconfig | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/soc/sh/Kconfig b/sound/soc/sh/Kconfig
index a1d14bc..52d7e8e 100644
--- a/sound/soc/sh/Kconfig
+++ b/sound/soc/sh/Kconfig
@@ -48,7 +48,7 @@ config SND_SH7760_AC97
config SND_FSI_AK4642
bool "FSI-AK4642 sound support"
- depends on SND_SOC_SH4_FSI
+ depends on SND_SOC_SH4_FSI && I2C_SH_MOBILE
select SND_SOC_AK4642
help
This option enables generic sound support for the
@@ -56,7 +56,7 @@ config SND_FSI_AK4642
config SND_FSI_DA7210
bool "FSI-DA7210 sound support"
- depends on SND_SOC_SH4_FSI
+ depends on SND_SOC_SH4_FSI && I2C_SH_MOBILE
select SND_SOC_DA7210
help
This option enables generic sound support for the
--
1.6.4.4
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH v2] soc/sh: Fix I2C dependency for SND_FSI_AK4642 and SND_FSI_DA7210
2010-06-15 15:38 ` Peter Huewe
@ 2010-06-16 3:54 ` Paul Mundt
-1 siblings, 0 replies; 22+ messages in thread
From: Paul Mundt @ 2010-06-16 3:54 UTC (permalink / raw)
To: Peter Huewe
Cc: Mark Brown, Liam Girdwood, linux-sh, Jaroslav Kysela,
Takashi Iwai, Kuninori Morimoto, Guennadi Liakhovetski,
alsa-devel, linux-kernel
On Tue, Jun 15, 2010 at 05:38:55PM +0200, Peter Huewe wrote:
> The config option SND_FSI_AK4642 selects SND_SOC_AK4642 which in turn
> enables the compilation of ak4642.c - however this codec uses I2C to
> communicate with the HW.
> Same applies to DA7210.
>
It uses I2C, but it doesn't matter which I2C master it's interfaced with.
A simple depends on I2C will suffice for this purpose just fine.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] soc/sh: Fix I2C dependency for SND_FSI_AK4642 and SND_FSI_DA7210
@ 2010-06-16 3:54 ` Paul Mundt
0 siblings, 0 replies; 22+ messages in thread
From: Paul Mundt @ 2010-06-16 3:54 UTC (permalink / raw)
To: Peter Huewe
Cc: Mark Brown, Liam Girdwood, linux-sh, Jaroslav Kysela,
Takashi Iwai, Kuninori Morimoto, Guennadi Liakhovetski,
alsa-devel, linux-kernel
On Tue, Jun 15, 2010 at 05:38:55PM +0200, Peter Huewe wrote:
> The config option SND_FSI_AK4642 selects SND_SOC_AK4642 which in turn
> enables the compilation of ak4642.c - however this codec uses I2C to
> communicate with the HW.
> Same applies to DA7210.
>
It uses I2C, but it doesn't matter which I2C master it's interfaced with.
A simple depends on I2C will suffice for this purpose just fine.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] soc/sh: Fix I2C dependency for SND_FSI_AK4642 and SND_FSI_DA7210
2010-06-16 3:54 ` Paul Mundt
@ 2010-06-16 7:43 ` Mark Brown
-1 siblings, 0 replies; 22+ messages in thread
From: Mark Brown @ 2010-06-16 7:43 UTC (permalink / raw)
To: Paul Mundt
Cc: Peter Huewe, Liam Girdwood, linux-sh, Jaroslav Kysela,
Takashi Iwai, Kuninori Morimoto, Guennadi Liakhovetski,
alsa-devel, linux-kernel
On 16 Jun 2010, at 04:54, Paul Mundt wrote:
> On Tue, Jun 15, 2010 at 05:38:55PM +0200, Peter Huewe wrote:
>> The config option SND_FSI_AK4642 selects SND_SOC_AK4642 which in turn
>> enables the compilation of ak4642.c - however this codec uses I2C to
>> communicate with the HW.
>> Same applies to DA7210.
> It uses I2C, but it doesn't matter which I2C master it's interfaced with.
> A simple depends on I2C will suffice for this purpose just fine.
Since these are board specific drivers they know which I2C controller they are
hooked up to and so can usefully depend on a specific controller - without the
correct controller driver they will not function.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] soc/sh: Fix I2C dependency for SND_FSI_AK4642 and SND_FSI_DA7210
@ 2010-06-16 7:43 ` Mark Brown
0 siblings, 0 replies; 22+ messages in thread
From: Mark Brown @ 2010-06-16 7:43 UTC (permalink / raw)
To: Paul Mundt
Cc: Peter Huewe, Liam Girdwood, linux-sh, Jaroslav Kysela,
Takashi Iwai, Kuninori Morimoto, Guennadi Liakhovetski,
alsa-devel, linux-kernel
On 16 Jun 2010, at 04:54, Paul Mundt wrote:
> On Tue, Jun 15, 2010 at 05:38:55PM +0200, Peter Huewe wrote:
>> The config option SND_FSI_AK4642 selects SND_SOC_AK4642 which in turn
>> enables the compilation of ak4642.c - however this codec uses I2C to
>> communicate with the HW.
>> Same applies to DA7210.
> It uses I2C, but it doesn't matter which I2C master it's interfaced with.
> A simple depends on I2C will suffice for this purpose just fine.
Since these are board specific drivers they know which I2C controller they are
hooked up to and so can usefully depend on a specific controller - without the
correct controller driver they will not function.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] soc/sh: Fix I2C dependency for SND_FSI_AK4642 and SND_FSI_DA7210
2010-06-16 7:43 ` Mark Brown
@ 2010-06-16 7:49 ` Paul Mundt
-1 siblings, 0 replies; 22+ messages in thread
From: Paul Mundt @ 2010-06-16 7:49 UTC (permalink / raw)
To: Mark Brown
Cc: Peter Huewe, Liam Girdwood, linux-sh, Jaroslav Kysela,
Takashi Iwai, Kuninori Morimoto, Guennadi Liakhovetski,
alsa-devel, linux-kernel
On Wed, Jun 16, 2010 at 08:43:33AM +0100, Mark Brown wrote:
> On 16 Jun 2010, at 04:54, Paul Mundt wrote:
>
> > On Tue, Jun 15, 2010 at 05:38:55PM +0200, Peter Huewe wrote:
> >> The config option SND_FSI_AK4642 selects SND_SOC_AK4642 which in turn
> >> enables the compilation of ak4642.c - however this codec uses I2C to
> >> communicate with the HW.
> >> Same applies to DA7210.
>
> > It uses I2C, but it doesn't matter which I2C master it's interfaced with.
> > A simple depends on I2C will suffice for this purpose just fine.
>
> Since these are board specific drivers they know which I2C controller they are
> hooked up to and so can usefully depend on a specific controller - without the
> correct controller driver they will not function.
I don't really mind one way or the other. Having it depend on the
controller is fine, but it's not really much of a stretch to expect it to
show up on other boards with other controllers as well.
Very few SH CPUs use the same controller, so in this case it's really
more the exception than the norm. As soon as another board takes up one
of these blocks then we're forced to either switch to an I2C depends or
start making a dependency list for multiple controllers. It's the desire
to avoid the latter why I generally prefer the former, but again, I'm
fairly impartial regardless.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] soc/sh: Fix I2C dependency for SND_FSI_AK4642 and SND_FSI_DA7210
@ 2010-06-16 7:49 ` Paul Mundt
0 siblings, 0 replies; 22+ messages in thread
From: Paul Mundt @ 2010-06-16 7:49 UTC (permalink / raw)
To: Mark Brown
Cc: Peter Huewe, Liam Girdwood, linux-sh, Jaroslav Kysela,
Takashi Iwai, Kuninori Morimoto, Guennadi Liakhovetski,
alsa-devel, linux-kernel
On Wed, Jun 16, 2010 at 08:43:33AM +0100, Mark Brown wrote:
> On 16 Jun 2010, at 04:54, Paul Mundt wrote:
>
> > On Tue, Jun 15, 2010 at 05:38:55PM +0200, Peter Huewe wrote:
> >> The config option SND_FSI_AK4642 selects SND_SOC_AK4642 which in turn
> >> enables the compilation of ak4642.c - however this codec uses I2C to
> >> communicate with the HW.
> >> Same applies to DA7210.
>
> > It uses I2C, but it doesn't matter which I2C master it's interfaced with.
> > A simple depends on I2C will suffice for this purpose just fine.
>
> Since these are board specific drivers they know which I2C controller they are
> hooked up to and so can usefully depend on a specific controller - without the
> correct controller driver they will not function.
I don't really mind one way or the other. Having it depend on the
controller is fine, but it's not really much of a stretch to expect it to
show up on other boards with other controllers as well.
Very few SH CPUs use the same controller, so in this case it's really
more the exception than the norm. As soon as another board takes up one
of these blocks then we're forced to either switch to an I2C depends or
start making a dependency list for multiple controllers. It's the desire
to avoid the latter why I generally prefer the former, but again, I'm
fairly impartial regardless.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] soc/sh: Fix I2C dependency for SND_FSI_AK4642 and SND_FSI_DA7210
2010-06-16 7:49 ` Paul Mundt
@ 2010-06-16 7:58 ` Mark Brown
-1 siblings, 0 replies; 22+ messages in thread
From: Mark Brown @ 2010-06-16 7:58 UTC (permalink / raw)
To: Paul Mundt
Cc: Peter Huewe, Liam Girdwood, linux-sh, Jaroslav Kysela,
Takashi Iwai, Kuninori Morimoto, Guennadi Liakhovetski,
alsa-devel, linux-kernel
On 16 Jun 2010, at 08:49, Paul Mundt wrote:
> On Wed, Jun 16, 2010 at 08:43:33AM +0100, Mark Brown wrote:
>> Since these are board specific drivers they know which I2C controller they are
>> hooked up to and so can usefully depend on a specific controller - without the
>> correct controller driver they will not function.
> I don't really mind one way or the other. Having it depend on the
> controller is fine, but it's not really much of a stretch to expect it to
> show up on other boards with other controllers as well.
This is the driver for the board itself and its wiring - it's not a chip driver, it's
the driver that says how the chips on a given board are connected. For new
boards I'd expect you'd end up with a new driver.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] soc/sh: Fix I2C dependency for SND_FSI_AK4642 and SND_FSI_DA7210
@ 2010-06-16 7:58 ` Mark Brown
0 siblings, 0 replies; 22+ messages in thread
From: Mark Brown @ 2010-06-16 7:58 UTC (permalink / raw)
To: Paul Mundt
Cc: Peter Huewe, Liam Girdwood, linux-sh, Jaroslav Kysela,
Takashi Iwai, Kuninori Morimoto, Guennadi Liakhovetski,
alsa-devel, linux-kernel
On 16 Jun 2010, at 08:49, Paul Mundt wrote:
> On Wed, Jun 16, 2010 at 08:43:33AM +0100, Mark Brown wrote:
>> Since these are board specific drivers they know which I2C controller they are
>> hooked up to and so can usefully depend on a specific controller - without the
>> correct controller driver they will not function.
> I don't really mind one way or the other. Having it depend on the
> controller is fine, but it's not really much of a stretch to expect it to
> show up on other boards with other controllers as well.
This is the driver for the board itself and its wiring - it's not a chip driver, it's
the driver that says how the chips on a given board are connected. For new
boards I'd expect you'd end up with a new driver.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] soc/sh: Fix I2C dependency for SND_FSI_AK4642 and
2010-06-15 15:38 ` Peter Huewe
@ 2010-06-16 15:43 ` Mark Brown
-1 siblings, 0 replies; 22+ messages in thread
From: Mark Brown @ 2010-06-16 15:43 UTC (permalink / raw)
To: Peter Huewe
Cc: Liam Girdwood, linux-sh, Jaroslav Kysela, Takashi Iwai,
Kuninori Morimoto, Guennadi Liakhovetski, alsa-devel,
linux-kernel
On Tue, Jun 15, 2010 at 05:38:55PM +0200, Peter Huewe wrote:
> From: Peter Huewe <peterhuewe@gmx.de>
>
> The config option SND_FSI_AK4642 selects SND_SOC_AK4642 which in turn
> enables the compilation of ak4642.c - however this codec uses I2C to
> communicate with the HW.
> Same applies to DA7210.
Applied, thanks.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v2] soc/sh: Fix I2C dependency for SND_FSI_AK4642 and SND_FSI_DA7210
@ 2010-06-16 15:43 ` Mark Brown
0 siblings, 0 replies; 22+ messages in thread
From: Mark Brown @ 2010-06-16 15:43 UTC (permalink / raw)
To: Peter Huewe
Cc: Liam Girdwood, linux-sh, Jaroslav Kysela, Takashi Iwai,
Kuninori Morimoto, Guennadi Liakhovetski, alsa-devel,
linux-kernel
On Tue, Jun 15, 2010 at 05:38:55PM +0200, Peter Huewe wrote:
> From: Peter Huewe <peterhuewe@gmx.de>
>
> The config option SND_FSI_AK4642 selects SND_SOC_AK4642 which in turn
> enables the compilation of ak4642.c - however this codec uses I2C to
> communicate with the HW.
> Same applies to DA7210.
Applied, thanks.
^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2010-06-16 15:44 UTC | newest]
Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-06-14 15:39 [PATCH] soc/sh: Fix I2C dependency for SND_FSI_AK4642 Peter Huewe
2010-06-14 15:39 ` Peter Huewe
2010-06-14 16:10 ` Mark Brown
2010-06-14 16:10 ` Mark Brown
2010-06-14 16:10 ` Mark Brown
2010-06-15 14:49 ` Peter Hüwe
2010-06-15 14:49 ` Peter Hüwe
2010-06-15 15:19 ` Mark Brown
2010-06-15 15:19 ` Mark Brown
2010-06-15 15:19 ` Mark Brown
2010-06-15 15:38 ` [PATCH v2] soc/sh: Fix I2C dependency for SND_FSI_AK4642 and SND_FSI_DA7210 Peter Huewe
2010-06-15 15:38 ` Peter Huewe
2010-06-16 3:54 ` Paul Mundt
2010-06-16 3:54 ` Paul Mundt
2010-06-16 7:43 ` Mark Brown
2010-06-16 7:43 ` Mark Brown
2010-06-16 7:49 ` Paul Mundt
2010-06-16 7:49 ` Paul Mundt
2010-06-16 7:58 ` Mark Brown
2010-06-16 7:58 ` Mark Brown
2010-06-16 15:43 ` [PATCH v2] soc/sh: Fix I2C dependency for SND_FSI_AK4642 and Mark Brown
2010-06-16 15:43 ` [PATCH v2] soc/sh: Fix I2C dependency for SND_FSI_AK4642 and SND_FSI_DA7210 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.