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

[Mohammad Rafi Shaik]

On 6/9/2026 1:16 AM, Krzysztof Kozlowski wrote:
> 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.

sure, will add more description what this EE value and why we need this.

> > SoundWire master interrupts to the appropriate CPU target.
>
> Interrupt affinity defines where the interrupts are routed. Not custom
> properties.

Yes, I noticed a recent change in the SoC Device Tree where a fourth 
cell (Interrupt Affinity / Target CPU Mask) has been added.

interrupts = <GIC_SPI 330 IRQ_TYPE_LEVEL_HIGH 0>;

I will try using the same configuration and verify the behavior.


> > 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...

Thanks for pointing out.

i see "qcom,ee" already exist in qcom,spmi-pmic-arb-common.yaml.

will use same.

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

Ack, will fix in next version.

Thanks & regards,
Rafi.

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