[alsa-devel] [RFD] Add hw_params support for hostless stream BE DAIs

[Mark Brown]

On Fri, Mar 11, 2016 at 02:27:03AM +0530, Jeeja KP wrote:
> On Wed, Mar 02, 2016 at 11:02:15AM +0900, Mark Brown wrote:
> > On Tue, Feb 16, 2016 at 09:47:47PM +0530, Jeeja KP wrote:

> > Like I keep saying if you're thinking about this in terms of DPCM you're
> > solving the wrong problem - we already support supplying parameters for
> > device<->device links, we already have systems that need to configure
> > things for device<->device links so we can already see that if we're
> > doing something that only works for things that are on SoC and can use
> > DPCM then we're not solving the problem.

> No this is not exactly about DPCM. It is about the device<->device links that
> assume you can have only codecs as hostless and thus change the direction
> for the link. Since we are rendering from SoC we don't need direction change.

You're routing them via DSP though?

> > This is clearly confused, if we can't figure out what the two devices we
> > are connecting are based on just looking at the devices then that's
> > really not a good sign that our interfaces are sensible and easy to work
> > with.

> Right now dai_link assumes device<->device links based on params so we
> cannot specify params for anything where we dont want direction change and would
> like to treat as normal direction dai_link. So we propose this to move to
> codec_link flag and use params only for creating controls based on params
> provided.

What I'm telling you is that you shouldn't be writing hacks like this to
bodge things but coming up with scalable and maintainable solutions.  I
already discussed this with Vinod several times, we've discussed the
idea that if you're having trouble with the asymmetries in the
definitions of DAI links you should make the DAI links symmetric so
things like this just work.  We need clear, comprehensible abstractions
which work robustly not some mess of special case handling.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20160311/d87651fd/attachment.sig>