Re: [PATCH v5 3/4] ASoC: Intel - add makefile support for SKL driver

[Mark Brown <broonie@kernel.org>]

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

On Mon, Jun 15, 2015 at 06:42:39PM +0200, Takashi Iwai wrote:
> Mark Brown wrote:
> > On Thu, Jun 11, 2015 at 10:03:57PM +0530, Vinod Koul wrote:

> > > +	  HD-audio driver.  A larger buffer (e.g. 2048) is preferred
> > > +	  for systems using PulseAudio.  The default 64 is chosen just
> > > +	  for compatibility reasons.

> > What are those compatibility reasons - why should users have to worry
> > about this?

> ... if a user wants to reduce the memory foot print.
> 2MB buffers for each stream are significant size, supposing that it
> can have 16 streams.

That doesn't sound like a compatibility thing though?  Compatibility
makes it sound like things will break rather than we'll just use too
much RAM.  Is there anything else going on?

> > Should this be a module parameter or something?

> The buffers can be reallocated dynamically via procfs in general, but
> it's not practical and intuitive.  That's why the similar option was
> provided for HDA legacy driver.  It was the time when PA started
> accepted slowly.  I don't know whether it makes sense for SKL ASoC
> driver, though -- it's up to you guys.

I'm not against having configuration, I just want it to be configuration
that people can understand and it seems better to make the default be
that for PulseAudio given how much of the common case it is now.
Something like "make this smaller to reduce the default memory
footprint" for example.

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

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