-----Original Message----- From: Pierre-Louis Bossart pierre-louis.bossart@linux.intel.com Sent: Thursday, September 9, 2021 11:26 AM To: Jack Yu jack.yu@realtek.com; broonie@kernel.org; lgirdwood@gmail.com Cc: Oder Chiou oder_chiou@realtek.com; alsa-devel@alsa-project.org; lars@metafoo.de; kent_chen@realtek.com [陳建宏] kent_chen@realtek.com; Derek [方德義] derek.fang@realtek.com; Shuming [范書銘] shumingf@realtek.com; Flove(HsinFu) flove@realtek.com Subject: Re: [PATCH] ASoC: rt1011: add i2s reference control for rt1011
+static const char * const rt1011_i2s_ref[] = {
- "None", "Left Channel", "Right Channel"
+};
+enum {
- RT1011_I2S_REF_NONE,
- RT1011_I2S_REF_LEFT_CH,
- RT1011_I2S_REF_RIGHT_CH,
+};
is it possible to get Left-Right-mix (mixed on rt1011) and Left-Right (two channels, with the application selecting what they want)?
It's a mono amp, so there's no Left-Right-mix setting. And for the choice of Left-Right channel, it's already available with this
patch.
So it's just to place a mono reference in either left or right channel of the I2S link? Why not dual-mono as well then?
The patch is a specific control for two amps to send AEC data on single I2S
data input pin by standard I2S protocol.
That's really useful information and worthy of a comment.
What you are doing here is similar to selecting the slot for each amp with set_tdm_slot(), but leaving the choice to userspace, isn't it?
Yes.
If there are two I2S input from SoC, we can add dual-mono mode for it, but
so far there is no requirement regarding to this mode.
ok.
------Please consider the environment before printing this e-mail.