> > /* Add SDP3430 specific widgets */ > > @@ -150,6 +194,11 @@ static int sdp3430_twl4030_init(struct > > snd_soc_codec > > *codec) /* Set up SDP3430 specific audio path audio_map */ > > snd_soc_dapm_add_routes(codec, audio_map, > ARRAY_SIZE(audio_map)); > > > > + /* Enable audio interface */ > > + reg = codec->read(codec, TWL4030_REG_AUDIO_IF); > > + reg |= TWL4030_AIF_EN; > > + codec->write(codec, TWL4030_REG_AUDIO_IF, reg); > > + > > Well, this does not do anything (bad or good)... The Audio > interface is permanently enabled by default (by the default > register array in the twl4030.c). > I have plans to do it dynamically, but it needs some more > thinking, since the digital loopback on the audio path needs > also the audio interface to be enabled. > > Is there a reason to do this at all? I thought in enabling the audio interface in machine driver and clean AUDIO_IF reg in register cache, but that would require to enable it in all other machine drivers. However, I didn't consider the digital loopback scenario. So, I'll remove this from machine driver and resend it. _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
