linux-kernel - Re: [PATCH 4/4] arm64: dts: qcom: sa8775p-ride: Add QCS9100 compatible

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: <72b2d710-a7cb-45cf-9dad-e9fbd876697b@quicinc.com>
Date: Mon, 12 Aug 2024 15:07:22 +0800
From: Tingwei Zhang <quic_tingweiz@...cinc.com>
To: Krzysztof Kozlowski <krzk@...nel.org>,
        Tengfei Fan
	<quic_tengfan@...cinc.com>,
        Bjorn Andersson <andersson@...nel.org>,
        "Konrad
 Dybcio" <konrad.dybcio@...aro.org>,
        Rob Herring <robh@...nel.org>,
        "Krzysztof
 Kozlowski" <krzk+dt@...nel.org>,
        Conor Dooley <conor+dt@...nel.org>
CC: <kernel@...cinc.com>, <linux-arm-msm@...r.kernel.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 4/4] arm64: dts: qcom: sa8775p-ride: Add QCS9100
 compatible

在 8/12/2024 2:15 PM, Krzysztof Kozlowski 写道:
> On 12/08/2024 04:16, Tingwei Zhang wrote:
>> On 8/8/2024 7:05 PM, Krzysztof Kozlowski wrote:
>>> On 07/08/2024 13:04, Tingwei Zhang wrote:
>>>> On 8/7/2024 5:35 PM, Krzysztof Kozlowski wrote:
>>>>> On 07/08/2024 11:17, Tengfei Fan wrote:
>>>>>>
>>>>>>
>>>>>> On 8/7/2024 3:28 PM, Krzysztof Kozlowski wrote:
>>>>>>> On 06/08/2024 06:19, Tengfei Fan wrote:
>>>>>>>> Add QCS9100 compatible in sa8775p ride and sa8775p ride r3 board DTS.
>>>>>>>> QCS9100 references SA8775p, they share the same SoC DTSI and board DTS.
>>>>>>>>
>>>>>>>
>>>>>>> I don't understand this. You claim here that QCS9100 references SA8775p
>>>>>>> but your diff says other way: SA8775p references QCS9100.
>>>>>>>
>>>>>>> Sorry, that's confusing.
>>>>>>>
>>>>>>> Best regards,
>>>>>>> Krzysztof
>>>>>>>
>>>>>>
>>>>>> I will update the compatible as follows to indicate that QCS9100
>>>>>> references SA8775p.
>>>>>>
>>>>>> compatible = "qcom,sa8775p-ride", "qcom,qcs9100", "qcom,sa8775p";
>>>>>
>>>>> Is this still correct, though? sa8775p won't come with qcs9100 SoC.
>>>> We have a new board. Hardware is same as sa877p-ride except sa8775p is
>>>> replaced with qcs9100. We add qcs9100 SoC compatible to sa8775p-ride
>>>
>>> Does "new board" mean that "old board" disappears? No users to care
>>> about it? Or just the existing board is being changed (like new revision)?
>>
>> We will support both boards. Sa8775p-ride board with sa8775p chipset and
>> sa8775p-ride board with qcs9100 chipset. Both of them can be used for
>> development.
> 
> Patch does something else then - changes compatibles for the existing
> (old) board.

Can you educate us the right way to add the qcs9100 SoC support in 
sa8775p-ride board? We don't want to duplicate whole device tree file 
since all the hardwares are same except the SoC, so we add qcs9100 SoC 
compatible to sa8775p-ride board and still keep sa8775p SoC compatible.
> 
> Best regards,
> Krzysztof
> 


-- 
Thanks,
Tingwei