[alsa-devel] [PATCH v6 3/9] ASoC: Intel: skl-hda-dsp-generic: use snd-hda-codec-hdmi

[Kai Vehmanen]

Hi,

On Thu, 17 Oct 2019, Takashi Iwai wrote:
> Kai Vehmanen wrote:
> > +static inline struct snd_pcm *hda_dsp_hdmi_pcm_handle(struct snd_soc_card *card,
> > +						      int hdmi_idx)
> > +{
[...]
> > +static inline int hda_dsp_hdmi_build_controls(struct snd_soc_card *card,
> > +					      struct snd_soc_component *comp)
>
> These additions look a bit too large for inline functions.
> Wouldn't it be difficult to make them non-inline?

ack, shouldn't be too difficult. I'll make the update and also rebase on 
top of Mark's tree.

My rationale for inlining was that these functions are used by machine 
drivers only, so each machine driver got a copy anyways in the end, plus 
I wanted to avoid modifying the Makefile rules for all the machine drivers
I touched. But neither is a hard requirement, so let's put the new helpers
into their own compilation unit.

Thanks for the review!

Br, Kai