On Thu, 2022-04-28 at 13:13 +0100, Mark Brown wrote:
On Thu, Apr 28, 2022 at 05:33:44PM +0800, Jiaxin Yu wrote:
+/* clock source control */ +static const char * const mt8186_i2s_src_str[] = { + "Master", "Slave" +}; + +static const struct soc_enum mt8186_i2s_src_enum[] = { + SOC_ENUM_SINGLE_EXT(ARRAY_SIZE(mt8186_i2s_src_str), + mt8186_i2s_src_str), +};
Not clear why this is user visible? Shouldn't the machine driver be setting this. Also please use more modern provider/consumer terminology for the clocking.
Sorry, here are the controls left by debug, I should remove them. Yes, I should use provider/consumer terminology for the clocking, just link the SND_SOC_DAIFMT_CBP_CFC.
+static int mt8186_i2s_hd_set(struct snd_kcontrol *kcontrol, + struct snd_ctl_elem_value *ucontrol) +{ + return -EINVAL; + } + + i2s_priv->low_jitter_en = hd_en; + + return 0; +}
Same issue as on the other patch with the events - like I said there mixer-test will find a bunch of these issues for you. Yes, I remember it. Next I will continue to use mixer-test to check all of the mixer controls.