Re: [PATCH v5 08/13] ASoC: qcom: q6dsp: Add Senary MI2S audio interface support

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

On 3/19/26 5:17 AM, Val Packett wrote:
> Hi,
> 
> On 3/9/26 3:51 AM, Srinivas Kandagatla wrote:
>> From: Mohammad Rafi Shaik <mohammad.rafi.shaik@oss.qualcomm.com>
>>
>> Introduces support for the Senary MI2S audio interface in the Qualcomm
>> q6dsp. Add new AFE port IDs for Senary MI2S RX and TX and include the
>> necessary mappings in the port configuration  to allow audio routing
>> over the Senary MI2S interface.
>>
>> Signed-off-by: Mohammad Rafi Shaik <mohammad.rafi.shaik@oss.qualcomm.com>
>> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@oss.qualcomm.com>
>> ---
>>   sound/soc/qcom/qdsp6/q6afe-dai.c         |  6 +++++
>>   sound/soc/qcom/qdsp6/q6afe.c             |  6 +++++
>>   sound/soc/qcom/qdsp6/q6dsp-lpass-ports.c | 29 ++++++++++++++++++++++++
>>   3 files changed, 41 insertions(+)
>> [..]
>> diff --git a/sound/soc/qcom/qdsp6/q6afe.c b/sound/soc/qcom/qdsp6/q6afe.c
>> index 43d877322bae..7bd40e32f758 100644
>> --- a/sound/soc/qcom/qdsp6/q6afe.c
>> +++ b/sound/soc/qcom/qdsp6/q6afe.c
>> [..]
> 
> 
> One chunk is missing for this file, adding it to the big switch in
> q6afe_port_get_from_id:

Thanks Val, I will fold this into this patch.

--srini
> 
> diff --git a/sound/soc/qcom/qdsp6/q6afe.c b/sound/soc/qcom/qdsp6/q6afe.c
> index 7bd40e32f7..40237267fd 100644
> --- a/sound/soc/qcom/qdsp6/q6afe.c
> +++ b/sound/soc/qcom/qdsp6/q6afe.c
> @@ -1783,6 +1783,8 @@
>      case AFE_PORT_ID_QUATERNARY_MI2S_TX:
>      case AFE_PORT_ID_QUINARY_MI2S_RX:
>      case AFE_PORT_ID_QUINARY_MI2S_TX:
> +    case AFE_PORT_ID_SENARY_MI2S_RX:
> +    case AFE_PORT_ID_SENARY_MI2S_TX:
>          cfg_type = AFE_PARAM_ID_I2S_CONFIG;
>          break;
>      case AFE_PORT_ID_PRIMARY_TDM_RX ... AFE_PORT_ID_QUINARY_TDM_TX_7:
> 
> 
> With that + adding senary to q6routing and to SoC specific sndcard,
> 
> Tested-by: Val Packett <val@packett.cool> # sm7325-motorola-dubai
> 
> (WIP device, patches coming Soon™)
> 
> 
> Thanks,
> ~val
>