-----Original Message----- From: Mark Brown [mailto:broonie@kernel.org] Sent: Thursday, April 17, 2014 5:12 AM To: Bard Liao Cc: lgirdwood@gmail.com; alsa-devel@alsa-project.org; lars@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.