Re: [PATCH v4] dt-bindings: soundwire: qcom: Add SoundWire v2.2.0 compatible

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

On 11/5/25 12:19 PM, Prasad Kumpatla via B4 Relay wrote:
> From: Prasad Kumpatla <prasad.kumpatla@oss.qualcomm.com>
> 
> Add qcom,soundwire-v2.2.0 to the list of supported Qualcomm
> SoundWire controller versions. This version falls back to
> qcom,soundwire-v2.0.0 if not explicitly handled by the driver.
> 
> Signed-off-by: Jingyi Wang <jingyi.wang@oss.qualcomm.com>
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Signed-off-by: Prasad Kumpatla <prasad.kumpatla@oss.qualcomm.com>
> ---
> Add audio support for Kaanapali MTP boards. Introduces supporting
> dependencies required to enable audio functionality on MTP platforms.
> These changes have been validated on Kaanapali MTP hardware.
> 
> Changes in [v4]:
> 	- Dropped merged patches.
> 	- Link to v3: https://lore.kernel.org/linux-arm-msm/20251015-knp-audio-v2-v3-0-e0e3e4167d87@oss.qualcomm.com/
> 
> Changes in [v3]:
> 	- Correct SoB chain, comments from Krzysztof.
> 	- Link to v2: https://lore.kernel.org/linux-arm-msm/20251009143644.3296208-1-prasad.kumpatla@oss.qualcomm.com/
> 
> Changes in [v2]:
> 	- Addressed compilation issue for lpass version check patch.
> 	- Sorted compatible string in machine driver.
> 	- Link to v1: https://lore.kernel.org/linux-arm-msm/20250924-knp-audio-v1-0-5afa926b567c@oss.qualcomm.com/
> 
> Konrad Dybcio (1):
>   ASoC: codecs: va-macro: Rework version checking
> 
> Prasad Kumpatla (4):
>   ASoC: dt-bindings: qcom,sm8250: Add kaanapali sound card
>   ASoC: qcom: sc8280xp: Add support for Kaanapali
>   dt-bindings: soundwire: qcom: Add SoundWire v2.2.0 compatible
>   ASoC: dt-bindings: qcom: Add Kaanapali LPASS macro codecs
> 
>  .../bindings/sound/qcom,lpass-rx-macro.yaml   |  1 +
>  .../bindings/sound/qcom,lpass-tx-macro.yaml   |  1 +
>  .../bindings/sound/qcom,lpass-va-macro.yaml   |  1 +
>  .../bindings/sound/qcom,lpass-wsa-macro.yaml  |  1 +
>  .../bindings/sound/qcom,sm8250.yaml           |  1 +
>  .../bindings/soundwire/qcom,soundwire.yaml    |  1 +
>  sound/soc/codecs/lpass-va-macro.c             | 90 +++++++++++++------
>  sound/soc/qcom/sc8280xp.c                     |  1 +
>  8 files changed, 70 insertions(+), 27 deletions(-)
> 
> --
> 2.34.1
> ---
>  Documentation/devicetree/bindings/soundwire/qcom,soundwire.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/soundwire/qcom,soundwire.yaml b/Documentation/devicetree/bindings/soundwire/qcom,soundwire.yaml
> index 95d947fda6a7..1c4b0bdbb044 100644
> --- a/Documentation/devicetree/bindings/soundwire/qcom,soundwire.yaml
> +++ b/Documentation/devicetree/bindings/soundwire/qcom,soundwire.yaml
> @@ -26,6 +26,7 @@ properties:
>        - items:
>            - enum:
>                - qcom,soundwire-v2.1.0
> +              - qcom,soundwire-v2.2.0
>            - const: qcom,soundwire-v2.0.0
>  
>    reg:

LGTM,

Reviewed-by: Srinivas Kandagatla <srinivas.kandagatla@oss.qualcomm.com>

--srini>
> ---
> base-commit: 2b763d4652393c90eaa771a5164502ec9dd965ae
> change-id: 20251014-knp-audio-v2-9a37752cdc15
> 
> Best regards,