On Fri, Feb 07, 2014 at 07:27:44AM +0000, Bard Liao wrote:
- if (rt286->pdata.irq_en)
rt286_index_update_bits(codec,NODE_ID_VENDOR_REGISTERS, 0x33, 0x0002, 0x0002);
Why would the system ever want to supply an interrupt and not use it?
The codec support a configurable irq pin which can be disable or enable. Some customers may not use it. So, I add a flag in platform data to configure it.
Should it not be possible to do this based on if an interrupt is specified - why is a separate configuration option needed for this when the user also needs to specify if an interrupt is used and which interrupt is used?
+static int rt286_hp_event(struct snd_soc_dapm_widget *w,
struct snd_kcontrol *kcontrol, int event) {- struct snd_soc_codec *codec = w->codec;
- unsigned int val;
- switch (event) {
- case SND_SOC_DAPM_POST_PMU:
val = snd_soc_read(codec, NODE_ID_HP_OUT) & 0x8080;switch (val) {case 0x0:rt286_write(codec, AC_VERB_SET_AMP_GAIN_MUTE,NODE_ID_HP_OUT, 0xb000);break;case 0x8000:rt286_write(codec, AC_VERB_SET_AMP_GAIN_MUTE,NODE_ID_HP_OUT, 0x9000);break;case 0x0080:rt286_write(codec, AC_VERB_SET_AMP_GAIN_MUTE,NODE_ID_HP_OUT, 0xa000);break;}This is rather unclear. What is it doing?
SOC_DOUBLE_EXT("Headphone Playback Switch", NODE_ID_HP_OUT, 15, 8, 1, 1, rt286_playback_switch_get, rt286_playback_switch_put), We use NODE_ID_HP_OUT bit 15, 8 to store the mute/unmute status for headphone L/R channel. rt286_hp_event will mute both channels when power down. And it will read the status from RT286_HP_OUT to unmute the corresponding channel(s) when power on.
So how does the mute control work when the headphone is powered up then? You may be looking for a SOC_DAPM_SINGLE_AUTODISABLE() control here, I think that's what you're implementing here.