28 Apr
2022
28 Apr
'22
2:13 p.m.
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.