mbox series

[v9,0/5] arm64: qcom: sa8775p: add support for EP PCIe

Message ID 1701432377-16899-1-git-send-email-quic_msarkar@quicinc.com
Headers show
Series arm64: qcom: sa8775p: add support for EP PCIe | expand

Message

Mrinmay Sarkar Dec. 1, 2023, 12:06 p.m. UTC
This series adds the relavent DT bindings, new compatible string,
add support to EPF driver and add EP PCIe node in dtsi file for
ep pcie0 controller.

v8 -> v9:
- update author in "Add pci_epf_mhi_ prefix to the function" patch.
- add ack by and reviewed by tag in commit message.

v7 -> v8:
- Add new patch PCI: epf-mhi: Add "pci_epf_mhi_" prefix to the function
  names
- Update PCI: epf-mhi: Add support for SA8775P patch on top of the new
  patch and update commit message.

v6 -> v7:
- add reviewed by tag in commit message in all patches.
- update commit message in patch 2 as per comment.
- update reason for reusing PID in commit message.

v5 -> v6:
- update cover letter.

v4 -> v5:
- add maxItems to the respective field to constrain io space and
  interrupt in all variants.

v3 -> v4:
- add maxItems field in dt bindings
- update comment in patch2
- dropped PHY driver patch as it is already applied [1]
- update comment in EPF driver patch
- update commect in dtsi and add iommus instead of iommu-map

[1] https://lore.kernel.org/all/169804254205.383714.18423881810869732517.b4-ty@kernel.org/

v2 -> v3:
- removed if/then schemas, added minItems for reg,
  reg-bnames, interrupt and interrupt-names instead.
- adding qcom,sa8775p-pcie-ep compitable for sa8775p
  as we have some specific change to add.
- reusing sm8450's pcs_misc num table as it is same as sa8775p.
  used appropriate namespace for pcs.
- remove const from sa8775p_header as kernel test robot
  throwing some warnings due to this.
- remove fallback compatiable as we are adding compatiable for sa8775p.

v1 -> v2:
- update description for dma
- Reusing qcom,sdx55-pcie-ep compatibe so remove compaitable
  for sa8775p
- sort the defines in phy header file and remove extra defines
- add const in return type pci_epf_header and remove MHI_EPF_USE_DMA
  flag as hdma patch is not ready
- add fallback compatiable as qcom,sdx55-pcie-ep, add iommu property


Manivannan Sadhasivam (1):
  PCI: epf-mhi: Add "pci_epf_mhi_" prefix to the function names

Mrinmay Sarkar (4):
  dt-bindings: PCI: qcom-ep: Add support for SA8775P SoC
  PCI: qcom-ep: Add support for SA8775P SOC
  PCI: epf-mhi: Add support for SA8775P
  arm64: dts: qcom: sa8775p: Add ep pcie0 controller node

 .../devicetree/bindings/pci/qcom,pcie-ep.yaml      | 64 +++++++++++++++++++++-
 arch/arm64/boot/dts/qcom/sa8775p.dtsi              | 46 ++++++++++++++++
 drivers/pci/controller/dwc/pcie-qcom-ep.c          |  1 +
 drivers/pci/endpoint/functions/pci-epf-mhi.c       | 21 ++++++-
 4 files changed, 128 insertions(+), 4 deletions(-)

Comments

Manivannan Sadhasivam Jan. 29, 2024, 12:39 p.m. UTC | #1
On Sat, Jan 27, 2024 at 08:58:17PM -0600, Bjorn Andersson wrote:
> On Fri, Dec 01, 2023 at 05:36:11PM +0530, Mrinmay Sarkar wrote:
> > This series adds the relavent DT bindings, new compatible string,
> > add support to EPF driver and add EP PCIe node in dtsi file for
> > ep pcie0 controller.
> > 
> 
> Waiting for the driver changes to be picked up, so that I can merge
> patch 5 through the qcom tree. Are there any unresolved issues that I'm
> failing to spot?
> 

No, there are no issues. But they fail to catch the eye of maintainers during
the last merge window. I really hope that these will get picked for 6.9.

- Mani

> Regards,
> Bjorn
> 
> > v8 -> v9:
> > - update author in "Add pci_epf_mhi_ prefix to the function" patch.
> > - add ack by and reviewed by tag in commit message.
> > 
> > v7 -> v8:
> > - Add new patch PCI: epf-mhi: Add "pci_epf_mhi_" prefix to the function
> >   names
> > - Update PCI: epf-mhi: Add support for SA8775P patch on top of the new
> >   patch and update commit message.
> > 
> > v6 -> v7:
> > - add reviewed by tag in commit message in all patches.
> > - update commit message in patch 2 as per comment.
> > - update reason for reusing PID in commit message.
> > 
> > v5 -> v6:
> > - update cover letter.
> > 
> > v4 -> v5:
> > - add maxItems to the respective field to constrain io space and
> >   interrupt in all variants.
> > 
> > v3 -> v4:
> > - add maxItems field in dt bindings
> > - update comment in patch2
> > - dropped PHY driver patch as it is already applied [1]
> > - update comment in EPF driver patch
> > - update commect in dtsi and add iommus instead of iommu-map
> > 
> > [1] https://lore.kernel.org/all/169804254205.383714.18423881810869732517.b4-ty@kernel.org/
> > 
> > v2 -> v3:
> > - removed if/then schemas, added minItems for reg,
> >   reg-bnames, interrupt and interrupt-names instead.
> > - adding qcom,sa8775p-pcie-ep compitable for sa8775p
> >   as we have some specific change to add.
> > - reusing sm8450's pcs_misc num table as it is same as sa8775p.
> >   used appropriate namespace for pcs.
> > - remove const from sa8775p_header as kernel test robot
> >   throwing some warnings due to this.
> > - remove fallback compatiable as we are adding compatiable for sa8775p.
> > 
> > v1 -> v2:
> > - update description for dma
> > - Reusing qcom,sdx55-pcie-ep compatibe so remove compaitable
> >   for sa8775p
> > - sort the defines in phy header file and remove extra defines
> > - add const in return type pci_epf_header and remove MHI_EPF_USE_DMA
> >   flag as hdma patch is not ready
> > - add fallback compatiable as qcom,sdx55-pcie-ep, add iommu property
> > 
> > 
> > Manivannan Sadhasivam (1):
> >   PCI: epf-mhi: Add "pci_epf_mhi_" prefix to the function names
> > 
> > Mrinmay Sarkar (4):
> >   dt-bindings: PCI: qcom-ep: Add support for SA8775P SoC
> >   PCI: qcom-ep: Add support for SA8775P SOC
> >   PCI: epf-mhi: Add support for SA8775P
> >   arm64: dts: qcom: sa8775p: Add ep pcie0 controller node
> > 
> >  .../devicetree/bindings/pci/qcom,pcie-ep.yaml      | 64 +++++++++++++++++++++-
> >  arch/arm64/boot/dts/qcom/sa8775p.dtsi              | 46 ++++++++++++++++
> >  drivers/pci/controller/dwc/pcie-qcom-ep.c          |  1 +
> >  drivers/pci/endpoint/functions/pci-epf-mhi.c       | 21 ++++++-
> >  4 files changed, 128 insertions(+), 4 deletions(-)
> > 
> > -- 
> > 2.7.4
> >
Mrinmay Sarkar March 6, 2024, 1:35 p.m. UTC | #2
On 2/16/2024 4:22 PM, Manivannan Sadhasivam wrote:
> On Fri, Dec 01, 2023 at 05:36:11PM +0530, Mrinmay Sarkar wrote:
>> This series adds the relavent DT bindings, new compatible string,
>> add support to EPF driver and add EP PCIe node in dtsi file for
>> ep pcie0 controller.
>>
> Applied patches 3 and 4 to pci/endpoint!
>
> - Mani
>
>> v8 -> v9:
>> - update author in "Add pci_epf_mhi_ prefix to the function" patch.
>> - add ack by and reviewed by tag in commit message.
>>
>> v7 -> v8:
>> - Add new patch PCI: epf-mhi: Add "pci_epf_mhi_" prefix to the function
>>    names
>> - Update PCI: epf-mhi: Add support for SA8775P patch on top of the new
>>    patch and update commit message.
>>
>> v6 -> v7:
>> - add reviewed by tag in commit message in all patches.
>> - update commit message in patch 2 as per comment.
>> - update reason for reusing PID in commit message.
>>
>> v5 -> v6:
>> - update cover letter.
>>
>> v4 -> v5:
>> - add maxItems to the respective field to constrain io space and
>>    interrupt in all variants.
>>
>> v3 -> v4:
>> - add maxItems field in dt bindings
>> - update comment in patch2
>> - dropped PHY driver patch as it is already applied [1]
>> - update comment in EPF driver patch
>> - update commect in dtsi and add iommus instead of iommu-map
>>
>> [1] https://lore.kernel.org/all/169804254205.383714.18423881810869732517.b4-ty@kernel.org/
>>
>> v2 -> v3:
>> - removed if/then schemas, added minItems for reg,
>>    reg-bnames, interrupt and interrupt-names instead.
>> - adding qcom,sa8775p-pcie-ep compitable for sa8775p
>>    as we have some specific change to add.
>> - reusing sm8450's pcs_misc num table as it is same as sa8775p.
>>    used appropriate namespace for pcs.
>> - remove const from sa8775p_header as kernel test robot
>>    throwing some warnings due to this.
>> - remove fallback compatiable as we are adding compatiable for sa8775p.
>>
>> v1 -> v2:
>> - update description for dma
>> - Reusing qcom,sdx55-pcie-ep compatibe so remove compaitable
>>    for sa8775p
>> - sort the defines in phy header file and remove extra defines
>> - add const in return type pci_epf_header and remove MHI_EPF_USE_DMA
>>    flag as hdma patch is not ready
>> - add fallback compatiable as qcom,sdx55-pcie-ep, add iommu property
>>
>>
>> Manivannan Sadhasivam (1):
>>    PCI: epf-mhi: Add "pci_epf_mhi_" prefix to the function names
>>
>> Mrinmay Sarkar (4):
>>    dt-bindings: PCI: qcom-ep: Add support for SA8775P SoC
>>    PCI: qcom-ep: Add support for SA8775P SOC
>>    PCI: epf-mhi: Add support for SA8775P
>>    arm64: dts: qcom: sa8775p: Add ep pcie0 controller node
>>
>>   .../devicetree/bindings/pci/qcom,pcie-ep.yaml      | 64 +++++++++++++++++++++-
>>   arch/arm64/boot/dts/qcom/sa8775p.dtsi              | 46 ++++++++++++++++
>>   drivers/pci/controller/dwc/pcie-qcom-ep.c          |  1 +
>>   drivers/pci/endpoint/functions/pci-epf-mhi.c       | 21 ++++++-
>>   4 files changed, 128 insertions(+), 4 deletions(-)
>>
>> -- 
>> 2.7.4
Thanks for applying patches 3 and 4.

Seems like patches 1,2 and 5 are yet to apply on linux-next.
Please help me by applying those patches.

Thanks,
Mrinmay
>>
>>