[alsa-devel] [PATCH 2/2] ASoC: da7210: Add 11025/22050/44100/88200 rate support

[Mark Brown]

On Wed, Mar 10, 2010 at 11:45:52AM +0900, Kuninori Morimoto wrote:

> This chip doesn't support asymmetric sample rates.
> Is there a method of prohibiting it on ALSA ?
> Or should I add any lock ?

Set symmetric_rates in the DAI.

> > > +	/* As suggested by Dialog */
> > > +	da7210_write(codec, DA7210_A_HID_UNLOCK,	0x8B);
> > > +	da7210_write(codec, DA7210_A_TEST_UNLOCK,	0xB4);
> > > +	da7210_write(codec, DA7210_A_PLL1,		0x01);
> > > +	da7210_write(codec, DA7210_A_CP_MODE,		0x7C);

> > I suspect you want to disable the HID and test keys when you're done
> > fiddling with the configuration here...

> Ohh sorry.
> This XXX_UNLOCK doesn't mean "disable"..
> In default, we can not change value on some register
> because it is locked.
> So, this driver unlock it here.

> Should I add some comment ?

No, what I'm saying is that since the registers are supposed to be
locked in normal operation you should reset the lock registers so that
unintended register writes don't occur in normal operation.  The locks
will cover more than you're updating here.