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.
+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.