Re: [RFC PATCH 20/23] ASoC: qcom: x1e80100: carry WSA VI feedback on the WSA Playback BE link

[Srinivas Kandagatla <srinivas.kandagatla@oss.qualcomm.com>]

On 11/15/26 5:36 PM, Mark Brown wrote:
>> Speaker protection needs the WSA V/I sense capture running whenever the
>> speakers play.
>>
>> Set multi_direction_codecs on the WSA Playback link so it can carry the
>> VI codec DAIs, with a params fixup for their 8 kHz, 32-bit format. The
>> core's capture stream events for such links power the VI widgets.
>> Nothing changes until the DT adds the VI DAIs to the link.
> 
>> +/* VI codec DAIs on the WSA playback link run at 8 kHz, S32_LE */
>> +static int x1e80100_wsa_vi_params_fixup(struct snd_soc_pcm_runtime *rtd,
>> +					struct snd_pcm_hw_params *params,
>> +					int stream)
>> +{
>> +	struct snd_interval *rate =
>> +		hw_param_interval(params, SNDRV_PCM_HW_PARAM_RATE);
>> +	struct snd_mask *fmt =
>> +		hw_param_mask(params, SNDRV_PCM_HW_PARAM_FORMAT);
>> +
>> +	if (stream != SNDRV_PCM_STREAM_CAPTURE)
>> +		return 0;
>> +
>> +	rate->min = rate->max = 8000;
>> +	snd_mask_none(fmt);
>> +	snd_mask_set_format(fmt, SNDRV_PCM_FORMAT_S32_LE);
>> +	return 0;
>> +}
> 
> Shouldn't this also limit channels so we don't get something wider from
> playback?
vi has min max of 2 channels, so its fixed in v2.

--srini
>