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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





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@xxxxxxxxxxx>
---
  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




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux