linux-kernel - Re: [PATCH v2 1/2] dt-bindings: PCI: qcom-ep: Add support for QCS9100 SoC

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] [day] [month] [year] [list]

Message-ID: <38503bfb-1b85-49a3-8b87-72318261b5e7@quicinc.com>
Date: Mon, 29 Jul 2024 18:08:37 +0800
From: Tengfei Fan <quic_tengfan@...cinc.com>
To: "Aiqun Yu (Maria)" <quic_aiquny@...cinc.com>,
        Bjorn Helgaas
	<helgaas@...nel.org>
CC: Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>,
        "Lorenzo
 Pieralisi" <lpieralisi@...nel.org>,
        Krzysztof Wilczyński
	<kw@...ux.com>,
        Rob Herring <robh@...nel.org>, Bjorn Helgaas
	<bhelgaas@...gle.com>,
        Krzysztof Kozlowski <krzk+dt@...nel.org>,
        Conor Dooley
	<conor+dt@...nel.org>, <kernel@...cinc.com>,
        <linux-pci@...r.kernel.org>, <linux-arm-msm@...r.kernel.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 1/2] dt-bindings: PCI: qcom-ep: Add support for QCS9100
 SoC



On 7/10/2024 6:31 PM, Aiqun Yu (Maria) wrote:
> 
> 
> On 7/10/2024 12:28 AM, Bjorn Helgaas wrote:
>> On Tue, Jul 09, 2024 at 10:53:43PM +0800, Tengfei Fan wrote:
>>> Add devicetree bindings support for QCS9100 SoC. It has DMA register
>>> space and dma interrupt to support HDMA.
>>
>> s/dma/DMA/ above for consistency.
>>
>> Add blank line here if this is a paragraph break.
>>
>>> QCS9100 is drived from SA8775p. Currently, both the QCS9100 and SA8775p
>>> platform use non-SCMI resource. In the future, the SA8775p platform will
>>> move to use SCMI resources and it will have new sa8775p-related device
>>> tree. Consequently, introduce "qcom,qcs9100-pcie-ep" to describe non-SCMI
>>> based PCIe.
>>
>> s/drived/derived/
>>
>> This patch doesn't add anything related to SCMI, so this paragraph
>> seems like a distraction.
> 
> The paragraph can be removed from the commit message in next patchsets.
> Let me know if others looks good to you or not.
>>
>> The first paragraph seems a bit of a distraction too, since the patch
>> doesn't say anything about DMA register space or interrupt.
> 
> agree. Suggest to remove the first paragraph as well.

After considering the feedback provided on the subject, We have decided
to keep current SA8775p compatible and ABI compatibility in drivers.
Let's close this session and ignore all the current patches here.
Thank you for your input.

>>
>>> Signed-off-by: Tengfei Fan <quic_tengfan@...cinc.com>
>>> ---
>>>   Documentation/devicetree/bindings/pci/qcom,pcie-ep.yaml | 2 ++
>>>   1 file changed, 2 insertions(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/pci/qcom,pcie-ep.yaml b/Documentation/devicetree/bindings/pci/qcom,pcie-ep.yaml
>>> index 46802f7d9482..8012663e7efc 100644
>>> --- a/Documentation/devicetree/bindings/pci/qcom,pcie-ep.yaml
>>> +++ b/Documentation/devicetree/bindings/pci/qcom,pcie-ep.yaml
>>> @@ -13,6 +13,7 @@ properties:
>>>     compatible:
>>>       oneOf:
>>>         - enum:
>>> +          - qcom,qcs9100-pcie-ep
>>>             - qcom,sa8775p-pcie-ep
>>>             - qcom,sdx55-pcie-ep
>>>             - qcom,sm8450-pcie-ep
>>> @@ -203,6 +204,7 @@ allOf:
>>>           compatible:
>>>             contains:
>>>               enum:
>>> +              - qcom,qcs9100-pcie-ep
>>>                 - qcom,sa8775p-pcie-ep
>>>       then:
>>>         properties:
>>>
>>> -- 
>>> 2.25.1
>>>
> 

-- 
Thx and BRs,
Tengfei Fan