Re: [PATCH v2 09/14] ASoC: dt-bindings: qcom,q6dsp-lpass-ports: Add Senary MI2S port

[Krzysztof Kozlowski]

On Mon, Feb 23, 2026 at 06:07:35PM +0000, Srinivas Kandagatla wrote:
> From: Mohammad Rafi Shaik <mohammad.rafi.shaik@xxxxxxxxxxxxxxxx>
> 
> Qualcomm platforms support the Senary MI2S interface for audio playback
> and capture. Add a new definitions for the Senary MI2S RX and TX ports,
> which are required for boards that utilize the Senary MI2S interface
> for audio routing.
> 
> Signed-off-by: Mohammad Rafi Shaik <mohammad.rafi.shaik@xxxxxxxxxxxxxxxx>
> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@xxxxxxxxxxxxxxxx>
> ---
>  include/dt-bindings/sound/qcom,q6dsp-lpass-ports.h | 2 ++
>  sound/soc/qcom/common.h                            | 2 +-
>  2 files changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/include/dt-bindings/sound/qcom,q6dsp-lpass-ports.h b/include/dt-bindings/sound/qcom,q6dsp-lpass-ports.h
> index e27a31ece1f2..45850f2d4342 100644
> --- a/include/dt-bindings/sound/qcom,q6dsp-lpass-ports.h
> +++ b/include/dt-bindings/sound/qcom,q6dsp-lpass-ports.h
> @@ -150,6 +150,8 @@
>  #define LPI_MI2S_TX_3		144
>  #define LPI_MI2S_RX_4		145
>  #define LPI_MI2S_TX_4		146
> +#define SENARY_MI2S_RX		147
> +#define SENARY_MI2S_TX		148
>  
>  #define LPASS_CLK_ID_PRI_MI2S_IBIT	1
>  #define LPASS_CLK_ID_PRI_MI2S_EBIT	2
> diff --git a/sound/soc/qcom/common.h b/sound/soc/qcom/common.h
> index 85ca73408151..ee6662885593 100644
> --- a/sound/soc/qcom/common.h
> +++ b/sound/soc/qcom/common.h
> @@ -7,7 +7,7 @@
>  #include <dt-bindings/sound/qcom,q6afe.h>
>  #include <sound/soc.h>
>  
> -#define LPASS_MAX_PORT			(LPI_MI2S_TX_4 + 1)
> +#define LPASS_MAX_PORT			(SENARY_MI2S_TX + 1)

Same problem as v1. You already changed this. Don't change the same line
multiple times.

Anyway, this belongs to the code patch which uses the new LPASS_MAX_PORT
(e.g. ues new SENARY bindings).

Best regards,
Krzysztof