Re: [PATCH v2 2/6] ASoC: samsung: Add Samsung Low Power Audio Subsystem driver

[Mark Brown <broonie@kernel.org>]

[-- Attachment #1.1: Type: text/plain, Size: 1150 bytes --]

On Thu, Jun 30, 2016 at 07:16:46PM +0200, Sylwester Nawrocki wrote:

> Sorry, I didn't notice this e-mail earlier.  With previous Exynos versions
> the LPASS (or AudioSS) was mainly about the embedded audio DSP processor
> (at least WRT to SFRs), which was not required for boards supported in 
> the mainline kernel. Since e.g. Exynos5433 the LPASS SFR region contains 
> also entries related to other IP blocks, like I2S or DMAC.  Even though 
> functionality covered by these registers is still rather trivial, like 
> SW resets and unmasking interrupts, it's essential for the whole audio 
> subsystem operation. The intention was to have in future the LPASS driver 
> covering any functionality provided by the embedded audio dedicated MCU.
> I'm afraid we have to handle those power sequences in central place to
> ensure proper SoC operation. I would also rather avoid adding any exynos 
> quirks to the PL330 DMAC driver. 

Hrm.  This is sounding a bit like a combination of a power domain and
an interrupt controller, would something like that fit in perhaps?
Depends on how many of these trivial bits there are I think...

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]