Hey,
On Fri, 6 Sep 2019, Pierre-Louis Bossart wrote:
> > + if (!strncmp(card->name, "sof-skl_hda_card", 16)) {
> > + for_each_card_components(card, component) {
> > + modname = module_name(component->dev->driver->owner);
> > + if (!strncmp(component->name, "ehdaudio0D2", 11) &&
> > + !strncmp(modname, "snd_hda_codec_hdmi", 18))
> > + use_common_codec = 1;
> > + }
> > + }
>
> yuk. I am not a big fan of this...
>
> It seems that we could pass information from the SOF side to the machine
> driver using the mach_params argument. we already pass the codec_mask and
> other fields, it wouldn't be too hard to reclaim a field or extend the
> structure to pass the information.
it is ugly, no question about it. :) My reasoning for this was to contain
the ugliness within the machine drivers (especially these that are used
with both SOF and SST). New machine drivers with no legacy to maintain
could just skip it.
If the general concept of compiling both codec drivers in, and selecting
at runtime, is acceptable, I'll check how this would look via mach_params.
Br, Kai
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel
