[alsa-devel] [PATCH v6] ASoC: add RT286 CODEC driver

[Bard Liao]

> -----Original Message-----
> From: Mark Brown [mailto:broonie at kernel.org]
> Sent: Thursday, April 17, 2014 5:12 AM
> To: Bard Liao
> Cc: lgirdwood at gmail.com; alsa-devel at alsa-project.org; lars at metafoo.de; Flove;
> Oder Chiou; Gustaw Lewandowski
> Subject: Re: [PATCH v6] ASoC: add RT286 CODEC driver
> 
> On Wed, Apr 16, 2014 at 01:26:41PM +0000, Bard Liao wrote:
> 
> > > *MUST* use regmap to do this.  What's not been clear is if this
> > > device has a control interface that looks enough like a register map
> > > to be implemented like one but since we're just guessing based on a
> > > combination of the code and the HDA spec it's not entirely clear.
> 
> > I am a little bit confused about it. I thought you told me not using regmap.
> > Actually, I don't know how to use dapm without implementing register I/O.
> > Can I just convert rt286_hw_read/write to regmap?
> 
> As far as we've been able to tell thus far (based on inferring stuff from the HDA
> spec) you shouldn't be doing register I/O at all, the device doesn't really have
> things that look enough like registers.
> 
> You can implement non-register DAPM widgets - look at the virtual controls in the
> core, or things using SND_SOC_NOPM.

It means dapm widget will not control the codec directly, right? 
I.e. Everything will be done by _event?
For simplest the driver, can I not define .dapm_widgets and .dapm_routes?
All controls will be done in .controls.

> 
> ------Please consider the environment before printing this e-mail.