Date 8.1.2013 12:37, Takashi Iwai wrote:
Hi,
this is a large series of patches to implement the really generic HD-audio codec parser usable for various codecs. It's based on the Realtek codec parser, so I started hacking patch_realtek.c for more generalization and improvements. Then it's merged to the existing generic parser code, hda_generic.c, and let Realtek parser just uses this new parser code. In the end, patch_realtek.c contains only the device specific fixups and the call of generic parser functions.
Great step forward. Thanks.
In my opinion, it would be great to handle all fixups in some "user space" compatible mode. I mean, do not use the C code for fixups, but use an universal description code which can be loaded from the user space probably as ASCII text (firmware like behaviour), too. This code can be buildin to the kernel modules in a default version. I don't mean the current implemented patch processing, but full codec initialization. I think that it may be last change to bring the HDA code to more maintenable state. Just an idea to avoid further HDA code rewrites which happen periodically. And now (with your rewrite) it's really good time to propose also this change.
Please, provide the GIT URL with these changes.
Thanks, Jaroslav