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