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: <2229cc1a-0fa0-4291-874e-43b31f82ef50@quicinc.com>
Date: Tue, 13 Aug 2024 16:59:34 +0800
From: "Aiqun Yu (Maria)" <quic_aiquny@...cinc.com>
To: Krzysztof Kozlowski <krzk@...nel.org>,
        Tingwei Zhang
	<quic_tingweiz@...cinc.com>,
        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



On 8/13/2024 4:34 PM, Krzysztof Kozlowski wrote:
> On 12/08/2024 09:07, Tingwei Zhang wrote:
>> 在 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.
> 
> Split board DTS into shared DTSI (just don't forget about proper
> -M/-C/-B arguments for format-patch) and include it in relevant boards.
> You also need new SoC DTSI. This will be unusual code, but it matches
> what you want to achieve.

If we create two additional DTSs, a total of four DTBs will be generated.
Should we update the current board DTSs (sa8775p-ride-r3.dts and
sa8775p-ride.dts) to support the pin-to-pin compatible QCS9100 and
SA8775p SoCs?

Considering the higher usage of QCS9100 boards in IoT compared to
SA8775p in automotive for these DTBs, perhaps we should prioritize the
'qcom,qcs9100' compatibility before 'qcom,sa8775p'.
> 
> Best regards,
> Krzysztof
> 

-- 
Thx and BRs,
Aiqun(Maria) Yu