On Thu, Aug 22, 2024 at 03:57:22PM +0200, Cezary Rojewski wrote: > You've shared many scenarios, I do not think we can cover all of them here > and while I could agree that current FE/BE (DPCM?) design did not age well, > we're entering "rewrite how-to-pcm-in-linux" area. > If general opinion is: > it's too much, we have to rewrite for the framework to scale > into the next 20 years of audio in Linux > then my thoughts regarding current review are: > if the avs-driver needs sideband interface, so be it, but do it > locally rather than polluting entire framework. Switch to the > framework-solution once its rewritten. On this bit as I mentioned in the prior reply there's been ideas for redesigning how we tackle digital audio which I think there's general agreement would be the best way forwards. DPCM is very fragile and creaking at the seams, it can't really represent scenarios like the sideband case you've mentioned well. OTOH a redesign is a very big lift and there's never really a point where it seems constructive to actually block things on it so long as everyone involved is OK with what's going on. The upshot is that while I'd be *really* happy to see investment in the framework side of things I probably wouldn't block a driver specific or DPCM solution simply on the grounds that it's messy. DPCM would need buy in from other people using DPCM of course, and hopefully at some point someone with one of these issues will find that the cost of maintaining a bodge is going to be enough to push them to do the work (or someone will have free time to just work on the issue).
Attachment:
signature.asc
Description: PGP signature
- Follow-Ups:
- Re: [PATCH] [RFC] ASoC: Conditional PCM support
- From: Cezary Rojewski
- Re: [PATCH] [RFC] ASoC: Conditional PCM support
- References:
- [PATCH] [RFC] ASoC: Conditional PCM support
- From: Cezary Rojewski
- Re: [PATCH] [RFC] ASoC: Conditional PCM support
- From: Pierre-Louis Bossart
- Re: [PATCH] [RFC] ASoC: Conditional PCM support
- From: Cezary Rojewski
- [PATCH] [RFC] ASoC: Conditional PCM support
- Prev by Date: Re: [PATCH] ASoC: meson: Remove unused declartion in header file
- Next by Date: [PATCH] ASoC: tlv320aic31xx: Fix typos
- Previous by thread: Re: [PATCH] [RFC] ASoC: Conditional PCM support
- Next by thread: Re: [PATCH] [RFC] ASoC: Conditional PCM support
- Index(es):