Jarkko,
On 12/23/2016 08:27 PM, Jarkko Nikula wrote:
> Hi
>
> On 12/23/2016 11:21 AM, Peter Ujfalusi wrote:
>> The RESET register only have one self clearing bit and it should not be
>> cached. If it is cached, when we sync the registers back to the chip we
>> will initiate a software reset as well, which is not desirable.
>>
>> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@xxxxxx>
>> ---
>> sound/soc/codecs/tlv320aic3x.c | 13 +++++++++++++
>> 1 file changed, 13 insertions(+)
>>
>> diff --git a/sound/soc/codecs/tlv320aic3x.c b/sound/soc/codecs/tlv320aic3x.c
>> index 216f74084c6a..7b6924e19021 100644
>> --- a/sound/soc/codecs/tlv320aic3x.c
>> +++ b/sound/soc/codecs/tlv320aic3x.c
>> @@ -126,6 +126,16 @@ static const struct reg_default aic3x_reg[] = {
>> { 108, 0x00 }, { 109, 0x00 },
>> };
>>
>> +static bool aic3x_volatile_reg(struct device *dev, unsigned int reg)
>> +{
>> + switch (reg) {
>> + case AIC3X_RESET:
>> + return true;
>> + default:
>> + return false;
>> + }
>> +}
>> +
>
> You mentioned offline you tracked this into my commit 9fb352b18b11
> ("ASoC: tlv320aic3x: Do soft reset to codec when going to bias off
> state") but was it by bisecting or by debugging? I think I tried to
> cover it in a commit before 508b76864c18 ("ASoC: tlv320aic3x: Don't sync
> first two registers from register cache").
>
> If you found it by debugging can it be that pop noise came because of
> 2a6fedec195b ("ASoC: tlv320aic3x: Convert to direct regmap API usage")?
Noticed this when debugging. The conversion to regmap did introduced this.
> Just thinking if there's a need to have this into stable.
Right, it might be a good idea.
> Reviewed-by: Jarkko Nikula <jarkko.nikula@xxxxxxxxxx>
>
--
Péter
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
