Re: [PATCH v2 3/8] ASoC: qdsp6: q6afe-dai: add support to pcm port dais
From: Adam Serbinski
Date: Mon Feb 10 2020 - 12:22:33 EST
On 2020-02-10 12:13, Srinivas Kandagatla wrote:
Few minor comments
+static int q6afe_tdm_set_sysclk(struct snd_soc_dai *dai,
+ int clk_id, unsigned int freq, int dir)
+{
Why are we adding exactly duplicate function of q6afe_mi2s_set_sysclk
here?
It isn't an exact duplicate.
The reason I split off the new function is because the clock IDs for PCM
overlap/duplicate the clock IDs for TDM, yet the parameters to
q6afe_port_set_sysclk are not the same for PCM and TDM.
+ SND_SOC_DAPM_AIF_IN("QUAT_PCM_RX", NULL,
+ 0, 0, 0, 0),
This can be in single line, same for below
I will adjust these.