Re: [PATCH v1 1/2] dt-bindings: soundwire: qcom: add qcom,swr-master-ee-val property

From: Krzysztof Kozlowski

Date: Mon Jun 08 2026 - 15:46:32 EST


On 08/06/2026 19:53, Mohammad Rafi Shaik wrote:
> Add documentation for the qcom,swr-master-ee-val Device Tree property
> used by Qualcomm SoundWire masters to describe the execution-environment
> value for interrupt routing.
>
> This property allows platform DTs to specify the EE value used to direct

Please describe more what this EE value is for.

> SoundWire master interrupts to the appropriate CPU target.

Interrupt affinity defines where the interrupts are routed. Not custom
properties.

>
> Signed-off-by: Mohammad Rafi Shaik <mohammad.rafi.shaik@xxxxxxxxxxxxxxxx>
> ---
> .../devicetree/bindings/soundwire/qcom,soundwire.yaml | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/soundwire/qcom,soundwire.yaml b/Documentation/devicetree/bindings/soundwire/qcom,soundwire.yaml
> index 9447a2f37..5b06cc1a5 100644
> --- a/Documentation/devicetree/bindings/soundwire/qcom,soundwire.yaml
> +++ b/Documentation/devicetree/bindings/soundwire/qcom,soundwire.yaml
> @@ -215,6 +215,12 @@ properties:
> maximum: 4
> - const: 0xff
>
> + qcom,swr-master-ee-val:

Aren't there existing properties for execution context? I think Qualcomm
has something, so you should not come with a new property. Assuming this
stays in the first place...


> + $ref: /schemas/types.yaml#/definitions/uint8-array

Missing constraints.

> + description:
> + Execution-environment value used to route SoundWire master
> + interrupts to CPU0 or CPU1.
> +
> label:
> maxItems: 1
>


Best regards,
Krzysztof