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