[alsa-devel] [PATCH] ASoC: Writing register default value for the reset register

Dimitris Papastamos dp at opensource.wolfsonmicro.com
Tue Oct 11 16:16:31 CEST 2011


On Tue, Oct 11, 2011 at 02:31:18PM +0100, Mark Brown wrote:
> On Tue, Oct 11, 2011 at 01:33:05PM +0100, Dimitris Papastamos wrote:
> > On Tue, Oct 11, 2011 at 05:55:00PM +0800, Axel Lin wrote:
> > > The WM8983 can be reset by performing a write of any value to
> > > the software reset register.
> 
> > > -	ret = snd_soc_write(codec, WM8983_SOFTWARE_RESET, 0x8983);
> > > +	ret = snd_soc_write(codec, WM8983_SOFTWARE_RESET, 0);
> 
> > We should probably just handle the reset register specially during
> > resume.
> 
> I really don't think it's worth the code complexity to add special
> handling like that, this is a trivial patch which avoids the need for
> any extra infrastructure.

Aw I thought this code was using the old style open coded syncing
functions, which would be trivial and perhaps idiomatic to simply
check for the reset register and continue.

Thanks,
Dimitris


More information about the Alsa-devel mailing list