[PATCH] ASoC: fsl_micfil: Add DC output remover control

Mark Brown broonie at kernel.org
Tue Apr 21 23:00:56 AEST 2026


On Tue, Apr 21, 2026 at 03:40:12PM +0800, Shengjiu Wang wrote:
> On Mon, Apr 20, 2026 at 8:50 PM Mark Brown <broonie at kernel.org> wrote:

> > > +     /* Update DC Remover mode for all channels */
> > > +     ret = snd_soc_component_update_bits(comp, REG_MICFIL_DC_OUT_CTRL,
> > > +                                         MICFIL_DC_CTRL_CONFIG, reg_val);

> > This will return 0 not 1 when the value changes, meaning event
> > generation is missed.  The mixer-test selftest should report this.

> snd_soc_component_update_bits() will return 1 if the value is changed.
> and the mixer-test pass

Ah, so it will - I'm used to the regmap one which doesn't do that and
instead has a separate function to get a bool back for changes.  The
code is fine
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://lists.ozlabs.org/pipermail/linuxppc-dev/attachments/20260421/b436d179/attachment.sig>


More information about the Linuxppc-dev mailing list