On Thu, Feb 20, 2020 at 11:27:21AM -0600, Dan Murphy wrote: > + /* Set the Digital volume to -110dB */ > + ret = snd_soc_component_write(component, TAS2562_DVC_CFG4, 0x00); > + if (ret) > + return ret; > + ret = snd_soc_component_write(component, TAS2562_DVC_CFG3, 0x00); > + if (ret) > + return ret; > + ret = snd_soc_component_write(component, TAS2562_DVC_CFG2, 0x0d); > + if (ret) > + return ret; > + ret = snd_soc_component_write(component, TAS2562_DVC_CFG1, 0x43); > + if (ret) > + return ret; Is there a reason not to use the chip default here? Otherwise this looks good.
Attachment:
signature.asc
Description: PGP signature