[alsa-devel] [PATCH v4 1/7] ASoC: hda - add ASoC HDA codec match function
Mark Brown
broonie at kernel.org
Fri May 29 19:35:03 CEST 2015
On Fri, May 29, 2015 at 01:45:44PM +0530, Vinod Koul wrote:
> well userspace parsing is only for the pin mapping and changes which
> users want to do on their systems. For basic things the default graph
> will be read by existing HDA parser and codec driver will create
> widgets based on that. The generic HDA codec library should do most and
> the HDA ASoC codecs should do device specfic work (like existing
> patch_xxx)
So we *are* planning to continue to use the existing CODEC support but
with some different interfacing on top of it?
> > And it's the reason I suggested for pending merge until the codec side
> > is ready. We need the topology stuff merged at first, then HDA-codec
> > driver together with user-space side.
> I dont mind if code doesnt get merged into next merge window and put
> in one after that. But in needs to get accepted is some topic branch
> so that subsequent series can be psted and reviewed, merged. Btw
> users will not see unless we have machine driver. I will add the
> disable feature after this series, so no user conflicts occur
I need to have another look through. TBH I suspect it would be easier
with something that was more of a minimum viable prototype than with a
part of the system - with something like this it's as much the overall
shape of the solution that's being reviewed as anything else.
> Now that we have debated this and I guess all have the big picture
> here, how do we go about this. Do you guys need changes in this series
> or this is good for now?
Like I say I really need to review it again, and I'm still not entirely
convinced I understand what's going on well enough.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20150529/c5565096/attachment-0001.sig>
More information about the Alsa-devel
mailing list