lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <335d6661-8719-46d7-89c6-3392fc13f396@quicinc.com>
Date: Tue, 2 Sep 2025 20:56:41 +0800
From: Luo Jie <quic_luoj@...cinc.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
CC: Bjorn Andersson <andersson@...nel.org>,
        Michael Turquette
	<mturquette@...libre.com>,
        Stephen Boyd <sboyd@...nel.org>,
        "Varadarajan
 Narayanan" <quic_varada@...cinc.com>,
        Georgi Djakov <djakov@...nel.org>, "Rob
 Herring" <robh@...nel.org>,
        Krzysztof Kozlowski <krzk+dt@...nel.org>,
        "Conor
 Dooley" <conor+dt@...nel.org>,
        Anusha Rao <quic_anusha@...cinc.com>,
        "Manikanta Mylavarapu" <quic_mmanikan@...cinc.com>,
        Devi Priya
	<quic_devipriy@...cinc.com>,
        Philipp Zabel <p.zabel@...gutronix.de>,
        "Richard
 Cochran" <richardcochran@...il.com>,
        Konrad Dybcio <konradybcio@...nel.org>,
        Catalin Marinas <catalin.marinas@....com>,
        Will Deacon <will@...nel.org>, <linux-arm-msm@...r.kernel.org>,
        <linux-clk@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <linux-pm@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <netdev@...r.kernel.org>, <linux-arm-kernel@...ts.infradead.org>,
        <quic_kkumarcs@...cinc.com>, <quic_linchen@...cinc.com>,
        <quic_leiwei@...cinc.com>, <quic_pavir@...cinc.com>,
        <quic_suruchia@...cinc.com>
Subject: Re: [PATCH v4 06/10] dt-bindings: clock: Add required
 "interconnect-cells" property



On 8/29/2025 3:37 PM, Krzysztof Kozlowski wrote:
> On Thu, Aug 28, 2025 at 06:32:19PM +0800, Luo Jie wrote:
>> ICC clocks are always provided by the NSS clock controller of IPQ9574,
>> so add interconnect-cells as required DT property.
> 
> Does not make sense. If clocks are always, you require different
> property?

The '#interconnect-cells' property is mandatory in the DTS so that the
client driver such as PPE driver can acquire the ICC clocks from the NSS
CC provider. This is already part of the IPQ9574 DTS which is merged. We
had missed adding it as 'Required' at that time and fixing it now.

I will update the commit message to describe this better in the next
version to avoid confusion.

> 
>>
>> Fixes: 28300ecedce4 ("dt-bindings: clock: Add ipq9574 NSSCC clock and reset definitions")
>> Signed-off-by: Luo Jie <quic_luoj@...cinc.com>
> 
> Fixes cannot be in the middle of patchset.
> 
> See submitting patches and your internal guideline explaining it with
> great details.
> 

I understand that fixes should not be placed in the middle of a
patchset. I will reorder the patches to ensure that all fixes are at the
beginning of the series.

>> ---
>>   Documentation/devicetree/bindings/clock/qcom,ipq9574-nsscc.yaml | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/clock/qcom,ipq9574-nsscc.yaml b/Documentation/devicetree/bindings/clock/qcom,ipq9574-nsscc.yaml
>> index 17252b6ea3be..fc604279114f 100644
>> --- a/Documentation/devicetree/bindings/clock/qcom,ipq9574-nsscc.yaml
>> +++ b/Documentation/devicetree/bindings/clock/qcom,ipq9574-nsscc.yaml
>> @@ -57,6 +57,7 @@ required:
>>     - compatible
>>     - clocks
>>     - clock-names
>> +  - '#interconnect-cells'
> 
> ABI break without explanation.
> 
> Best regards,
> Krzysztof
> 

This property is already present in the current IPQ9574 SoC DTS file
which is already merged, and hence no ABI break is introduced for
IPQ9574. I will update the commit message in the next version to clarify
this.


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ