Re: [PATCH 1/3] dt-bindings: sound: add bindings for pm4125 audio codec

[Krzysztof Kozlowski <krzk@kernel.org>]

On 02/07/2025 01:30, Alexey Klimov wrote:
>>>>> +  It has RX and TX Soundwire slave devices. This bindings is for the
>>>>> +  slave devices.
>>>>
>>>> Last sentence is redundant and makes no sense. Codec has only slave
>>>> devices, so how this can be anything else than for slave devices?
>>>
>>> This came from other similar files that describe bindings for child codec nodes
>>> of soundwire nodes. For example from qcom,wcd937x-sdw.yaml.
>>> Should this be rephrased to "This bindings is for the soundwire slave devices." ?
>>
>> You just pasted the same, so I don't get how you want to rephrase into
>> the same sentence.
> 
> Not really.
> Original sentence: "This bindings is for the slave devices."
> Sentence from my email: "This bindings is for the soundwire slave devices."
> 
> The difference is 1 word.
> If it doesn't work, then maybe any suggestions?
> 
> Maybe "This bindings is for audio codec node that must be a child node of the
> associated soundwire master node."?
No, drop, it's not the pattern in the bindings. We don't explain that
I2C device should be in I2C bus, because that's obvious. Saying this is
a Soundwire device should be enough.

Best regards,
Krzysztof