Re: [alsa-devel] [PATCH v3] ASoC: nau8825: fix issue that pop noise when start playback

On Fri, Feb 05, 2016 at 04:32:45PM +0800, John Hsu wrote:

switch (event) { case SND_SOC_DAPM_POST_PMU: /* Prevent startup click by letting charge pump to ramp up */ msleep(10);

• regmap_update_bits(nau8825->regmap, NAU8825_REG_CHARGE_PUMP,
  

• 	NAU8825_JAMNODCLOW, NAU8825_JAMNODCLOW);
  

• break;
  

• case SND_SOC_DAPM_PRE_PMD:

• regmap_update_bits(nau8825->regmap, NAU8825_REG_CLASSG_CTRL,
  

• 	NAU8825_CLASSG_EN, 0);
  

• break;
  

• case SND_SOC_DAPM_POST_PMD:

• regmap_update_bits(nau8825->regmap, NAU8825_REG_BOOST,
  

• 	NAU8825_HP_BOOST_DIS, 0);
  

  break;

This is really hard to follow, the enables and disables here don't match up (I see that at least some of these are handled by other widgets on the other path). I at least need some words explaining what the intended effect of the change is.