Re: [PATCH v4 4/7] soundwire: qcom: deprecate qcom,din/out-ports

["Luca Weiss" <luca.weiss@fairphone.com>]

On Sun Feb 22, 2026 at 10:09 AM CET, Srinivas Kandagatla wrote:
> On 2/20/26 1:34 PM, Luca Weiss wrote:
>> Hi Srinivas,
>> 
>> On Fri Sep 12, 2025 at 10:32 AM CEST, Srinivas Kandagatla wrote:
>>> Number of input and output ports can be dynamically read from the
>>> controller registers, getting this value from Device Tree is redundant
>>> and potentially lead to bugs.
>>>
>>> Remove the code parsing this property along with marking this as
>>> deprecated in device tree bindings.
>> 
>> Could you help suggest how to correct kodiak.dtsi?
>> 
>
> This needs fixing in kodiak.dtsi, for somereason the dtsi is trying to
> add ports that does not clearly reflect the actual hardware.

How should the qcom,ports-* properties be updated to be correct?

I'm seeing the same on Milos, where the hardware registers are +1 from
what downstream describes, in all 4 soundwire controllers there.

swr_rx (reg=0x03210000):
    SWRM_COMP_PARAMS din: 1, dout: 6 (=7)
    (downstream: rx_swr_master - qcom,swr-num-ports = <6>)

swr_tx (reg=0x033b0000):
    SWRM_COMP_PARAMS din: 4, dout: 0 (=4)
    (downstream va_swr_master - qcom,swr-num-ports = <3>)

swr_wsa (reg=0x03250000):
    SWRM_COMP_PARAMS din: 3, dout: 6 (=9)
    downstream: wsa_swr_master - qcom,swr-num-ports = <8>

bt_swr (reg=0x031e0000):
    SWRM_COMP_PARAMS din: 5, dout: 4 (=9)
    downstream: bt_swr_mstr - qcom,swr-num-ports = <8>

Regards
Luca

>
> --srini
>
>> I'm seeing these warnings on QCM6490:
>> 
>> [   36.952517] qcom-soundwire 3210000.soundwire: din-ports (0) mismatch with controller (1)
>> [   36.952524] qcom-soundwire 3210000.soundwire: dout-ports (5) mismatch with controller (6)
>> [   36.954285] qcom-soundwire 3230000.soundwire: din-ports (3) mismatch with controller (4)