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.