Hi!
I'll fix it in v2. Thanks for your review!
Cyrille
Le 29/09/2015 10:42, Lars-Peter Clausen a écrit :
On 09/29/2015 10:36 AM, Cyrille Pitchen wrote:
The AD1934 codec has no DAC feature. Hence it register mapping is slightly different from the register mapping of other members of the AD193x family.
Some ASoC controls and widgets are related to the DAC feature so are not relevant in the case of an AD1934 codec.
Looks mostly good, thanks.
diff --git a/sound/soc/codecs/ad193x-i2c.c b/sound/soc/codecs/ad193x-i2c.c index df3a1a415825..8aa50d35662f 100644 --- a/sound/soc/codecs/ad193x-i2c.c +++ b/sound/soc/codecs/ad193x-i2c.c @@ -15,8 +15,8 @@ #include "ad193x.h"
static const struct i2c_device_id ad193x_id[] = {
- { "ad1936", 0 },
- { "ad1937", 0 },
- { "ad1936", AD193x },
- { "ad1937", AD193x },
Should be a upper case X.
{ } };
[...] [...]
@@ -91,18 +97,33 @@ static const struct snd_soc_dapm_route audio_paths[] = { { "DAC", NULL, "SYSCLK" }, { "DAC Output", NULL, "DAC" }, { "DAC Output", NULL, "VMID" },
- { "ADC", NULL, "SYSCLK" },
- { "DAC", NULL, "ADC_PWR" },
- { "ADC", NULL, "ADC_PWR" }, { "DAC1OUT", NULL, "DAC Output" }, { "DAC2OUT", NULL, "DAC Output" }, { "DAC3OUT", NULL, "DAC Output" }, { "DAC4OUT", NULL, "DAC Output" },
- { "SYSCLK", NULL, "PLL_PWR" },
+};
+static const struct snd_soc_dapm_route adc_audio_paths[] = {
this should have the ad193x_ prefix as well.
- { "ADC", NULL, "SYSCLK" },
- { "ADC", NULL, "ADC_PWR" }, { "ADC", NULL, "ADC1IN" }, { "ADC", NULL, "ADC2IN" },
- { "SYSCLK", NULL, "PLL_PWR" },
};