On Monday 03 August 2009 07:29:14 ext sean.mcneil@xxxxxx wrote:
> From: Sean McNeil <sean.mcneil@xxxxxx>
>
>
> Signed-off-by: Sean McNeil <sean.mcneil@xxxxxx>
I'm not sure what is the rationale behind..
Can you explain what are you trying to achieve with these changes?
Also, it would be better if you split the patch to a series with smaller patches
(and write some explanation what you are doing and why).
As Mark already pointed out, please use the latest code for basing your patches
(see the first comment below).
Few comments:
> 0x00, /* REG_AVTXL2PGA (0xC) */
> 0x00, /* REG_AVTXR2PGA (0xD) */
> 0x01, /* REG_AUDIO_IF (0xE) */
> - 0x00, /* REG_VOICE_IF (0xF) */
> + 0x04, /* REG_VOICE_IF (0xF) */
The VOICE_IF:VIF_TRI_EN (and the AUDIO_IF:AIF_TRI_EN) bit has been already used
in the latest code (see twl4030_set_tristate and twl4030_voice_set_tristate
functions).
> struct snd_soc_dai twl4030_dai[] = {
> {
> .name = "twl4030",
> .playback = {
> .stream_name = "HiFi Playback",
> - .channels_min = 2,
> + .channels_min = 1,
The HiFi (Audio interface in twl terms) either supports 2 or 4 channels, it does
not support mono audio.
> .channels_max = 4,
> .rates = TWL4030_RATES | SNDRV_PCM_RATE_96000,
> - .formats = TWL4030_FORMATS,},
> + .formats = TWL4030_FORMATS,
> + },
> .capture = {
> .stream_name = "Capture",
> - .channels_min = 2,
> + .channels_min = 1,
Same here, mono is not supported by the codec.
> .channels_max = 4,
> .rates = TWL4030_RATES,
> - .formats = TWL4030_FORMATS,},
> + .formats = TWL4030_FORMATS,
> + },
> .ops = &twl4030_dai_ops,
> },
> {
> @@ -2092,17 +2356,37 @@ struct snd_soc_dai twl4030_dai[] = {
> .playback = {
> .stream_name = "Voice Playback",
> .channels_min = 1,
> - .channels_max = 1,
> + .channels_max = 2,
The Voice interface only supports mono playback. Stereo is only supported on the
capture path.
> .rates = SNDRV_PCM_RATE_8000 | SNDRV_PCM_RATE_16000,
> - .formats = SNDRV_PCM_FMTBIT_S16_LE,},
> + .formats = SNDRV_PCM_FMTBIT_S16_LE,
> + },
--
Péter
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
