Re: [PATCH 1/3] dt-bindings: interconnect: add clocks property to enable QoS on sa8775p

[Krzysztof Kozlowski <krzk@kernel.org>]

On 13/08/2025 07:55, Odelu Kukatla wrote:
> 
> 
> On 8/12/2025 3:47 PM, Krzysztof Kozlowski wrote:
>> On 08/08/2025 16:02, Odelu Kukatla wrote:
>>> Add reg and clocks properties to enable the clocks required
>>> for accessing QoS configuration.
>>
>>
>> Nothing here explains why EXISTING hardware is being changed. I also
>> remember big discussions and big confusing patches regarding sa8775p
>> (its rename, dropping/changing all providers), and this patch feels like
>> pieces of it without proper justification.
>>
> Thanks for the review.
> I have added description in cover letter, i will add here as well in next revision.> And this is hidden ABI break, no justification, no mentioning either.
>> Again we are discussing basics of ABI breaking patches?
>>
> If you are talking ABI break if we load old DT which may lead to crash, we have .qos_requires_clocks flag which takes care of skipping QoS if required clocks are not enabled.we have addressed this issue through https://lore.kernel.org/all/20240704125515.22194-1-quic_okukatla@quicinc.com/ 

Format your emails correctly, it's difficult to read.

Your binding did not require reg and clocks. Now it requires reg and
clocks. This is called ABI break.

Please follow Qualcomm extensive upstreaming guide, it explains this,
doesn't it? Or follow writing bindings...

Best regards,
Krzysztof