mbox series

[v12,0/5] Add DT bindings and DT nodes for PCIe and PHY in SC7280

Message ID 1633628923-25047-1-git-send-email-pmaliset@codeaurora.org (mailing list archive)
Headers show
Series Add DT bindings and DT nodes for PCIe and PHY in SC7280 | expand

Message

Prasad Malisetty Oct. 7, 2021, 5:48 p.m. UTC
Changes added in v12:

	* Sorted pipe_clk muxing changes in patch 4 & 5 as per the commit log.
	  -Suggested by Bjorn.

Changes added in v11:

	* Modified nvme_pwren name as nvme_pwren.
	* Removed bias-pullup option in nvme_pwren entry [v11 Patch 3/5].
	* Changed pcie1_default_state name to pcie1_clkreq_n.
	* Added NULL pointer check for pcie_cfg.

Changes added in v10:

	* v9 [Patch 4/4/] has been split into two separate patches
	* Addressed all comments in IDP [Patch 3/4] file.
 	
Changes added in v9:
    * Added fixed regulator entry for nvme.suggested by Stephen Boyd
    * Added NULL pointer check before accessing ops in pcie probe
      Suggested by Stephen Boyd

Changes added in v8:

    * Added seperate pinctrl state for NVMe LDO enable pin [v8 P3/4]
    * Removed pointer initialization for pcie_cfg [v8 P4/4]
    * Replaced bool pcie_pipe_clk_src with unsigned int:1 [v8 P4/4]
    * Changed gcc_pcie_1_pipe_clk_src to pipe_clk_src

Changes added in v7:

        * Removed two fallbacks qcom,pcie-sm8250 and snps,dw-pcie.
        * Replaced compatible method in get_resources_2_7_0 with
            flag approach suggested by Bjorn Helgaas .
        * Setting gcc_pcie_1_clk_src as XO in init_2_7_0 for
          gdsc enable.
        * Added specific NVMe GPIO entries for SKU1 and SKU2 support
          in idp.dts and idp2.dts respectively.
        * Moved pcie_1 and pcie_1_phy board specific entries into common
          board file sc7280-idp.dtsi file.

Changes in v6:

    * Removed platform check while setting gcc_pcie_1_pipe_clk_src
          as clk_set_parent will return 0 with nop if platform doesn't
          need to switch pipe clk source.
        * Moved wake-n gpio to board specific file sc7280-idp.dtsi
        * Sorted gpio.h header entry in sc7280.dtsi file

Changes in v5:

        * Re ordered PCIe, PHY nodes in Soc and board specific dtsi files.
        * Removed ref_clk entry in current patch [PATCH v4 P4/4].
        * I will add ref clk entry in suspend/ resume commits.
        * Added boolean flag in Soc specific dtsi file to differentiate
          SM8250 and SC7280 platforms. based on boolean flag, platforms will handle
          the pipe clk handling.

Changes in v4 as suggested by Bjorn:

        * Changed pipe clk mux name as gcc_pcie_1_pipe_clk_src.
        * Changed pipe_ext_src as phy_pipe_clk.
        * Updated commit message for [PATCH v4 4/4].

Changes in v3:
        * Changed pipe clock names in dt bindings as pipe_mux and phy_pipe.
        * Moved reset and NVMe GPIO pin configs into board specific file.
        * Updated pipe clk mux commit message.

Changes in v2:
        * Moved pcie pin control settings into IDP file.
        * Replaced pipe_clk_src with pipe_clk_mux in pcie driver
        * Included pipe clk mux setting change set in this series

Prasad Malisetty (5):
  dt-bindings: pci: qcom: Document PCIe bindings for SC7280
  arm64: dts: qcom: sc7280: Add PCIe and PHY related nodes
  arm64: dts: qcom: sc7280: Add PCIe nodes for IDP board
  PCI: qcom: Add a flag in match data along with ops
  PCI: qcom: Switch pcie_1_pipe_clk_src after PHY init in SC7280

 .../devicetree/bindings/pci/qcom,pcie.txt          |  17 +++
 arch/arm64/boot/dts/qcom/sc7280-idp.dts            |   8 ++
 arch/arm64/boot/dts/qcom/sc7280-idp.dtsi           |  50 +++++++++
 arch/arm64/boot/dts/qcom/sc7280-idp2.dts           |   8 ++
 arch/arm64/boot/dts/qcom/sc7280.dtsi               | 118 +++++++++++++++++++++
 drivers/pci/controller/dwc/pcie-qcom.c             |  95 +++++++++++++++--
 6 files changed, 285 insertions(+), 11 deletions(-)

Comments

Lorenzo Pieralisi Oct. 13, 2021, 10 a.m. UTC | #1
On Thu, Oct 07, 2021 at 11:18:38PM +0530, Prasad Malisetty wrote:
> Changes added in v12:
> 
> 	* Sorted pipe_clk muxing changes in patch 4 & 5 as per the commit log.
> 	  -Suggested by Bjorn.
> 
> Changes added in v11:
> 
> 	* Modified nvme_pwren name as nvme_pwren.
> 	* Removed bias-pullup option in nvme_pwren entry [v11 Patch 3/5].
> 	* Changed pcie1_default_state name to pcie1_clkreq_n.
> 	* Added NULL pointer check for pcie_cfg.
> 
> Changes added in v10:
> 
> 	* v9 [Patch 4/4/] has been split into two separate patches
> 	* Addressed all comments in IDP [Patch 3/4] file.
>  	
> Changes added in v9:
>     * Added fixed regulator entry for nvme.suggested by Stephen Boyd
>     * Added NULL pointer check before accessing ops in pcie probe
>       Suggested by Stephen Boyd
> 
> Changes added in v8:
> 
>     * Added seperate pinctrl state for NVMe LDO enable pin [v8 P3/4]
>     * Removed pointer initialization for pcie_cfg [v8 P4/4]
>     * Replaced bool pcie_pipe_clk_src with unsigned int:1 [v8 P4/4]
>     * Changed gcc_pcie_1_pipe_clk_src to pipe_clk_src
> 
> Changes added in v7:
> 
>         * Removed two fallbacks qcom,pcie-sm8250 and snps,dw-pcie.
>         * Replaced compatible method in get_resources_2_7_0 with
>             flag approach suggested by Bjorn Helgaas .
>         * Setting gcc_pcie_1_clk_src as XO in init_2_7_0 for
>           gdsc enable.
>         * Added specific NVMe GPIO entries for SKU1 and SKU2 support
>           in idp.dts and idp2.dts respectively.
>         * Moved pcie_1 and pcie_1_phy board specific entries into common
>           board file sc7280-idp.dtsi file.
> 
> Changes in v6:
> 
>     * Removed platform check while setting gcc_pcie_1_pipe_clk_src
>           as clk_set_parent will return 0 with nop if platform doesn't
>           need to switch pipe clk source.
>         * Moved wake-n gpio to board specific file sc7280-idp.dtsi
>         * Sorted gpio.h header entry in sc7280.dtsi file
> 
> Changes in v5:
> 
>         * Re ordered PCIe, PHY nodes in Soc and board specific dtsi files.
>         * Removed ref_clk entry in current patch [PATCH v4 P4/4].
>         * I will add ref clk entry in suspend/ resume commits.
>         * Added boolean flag in Soc specific dtsi file to differentiate
>           SM8250 and SC7280 platforms. based on boolean flag, platforms will handle
>           the pipe clk handling.
> 
> Changes in v4 as suggested by Bjorn:
> 
>         * Changed pipe clk mux name as gcc_pcie_1_pipe_clk_src.
>         * Changed pipe_ext_src as phy_pipe_clk.
>         * Updated commit message for [PATCH v4 4/4].
> 
> Changes in v3:
>         * Changed pipe clock names in dt bindings as pipe_mux and phy_pipe.
>         * Moved reset and NVMe GPIO pin configs into board specific file.
>         * Updated pipe clk mux commit message.
> 
> Changes in v2:
>         * Moved pcie pin control settings into IDP file.
>         * Replaced pipe_clk_src with pipe_clk_mux in pcie driver
>         * Included pipe clk mux setting change set in this series
> 
> Prasad Malisetty (5):
>   dt-bindings: pci: qcom: Document PCIe bindings for SC7280
>   arm64: dts: qcom: sc7280: Add PCIe and PHY related nodes
>   arm64: dts: qcom: sc7280: Add PCIe nodes for IDP board
>   PCI: qcom: Add a flag in match data along with ops
>   PCI: qcom: Switch pcie_1_pipe_clk_src after PHY init in SC7280
> 
>  .../devicetree/bindings/pci/qcom,pcie.txt          |  17 +++
>  arch/arm64/boot/dts/qcom/sc7280-idp.dts            |   8 ++
>  arch/arm64/boot/dts/qcom/sc7280-idp.dtsi           |  50 +++++++++
>  arch/arm64/boot/dts/qcom/sc7280-idp2.dts           |   8 ++
>  arch/arm64/boot/dts/qcom/sc7280.dtsi               | 118 +++++++++++++++++++++
>  drivers/pci/controller/dwc/pcie-qcom.c             |  95 +++++++++++++++--
>  6 files changed, 285 insertions(+), 11 deletions(-)

I applied patches [4-5] to pci/qcom for v5.16, thanks I expect other
patches to go via the relevant trees.

Lorenzo
Prasad Malisetty Oct. 13, 2021, 5:27 p.m. UTC | #2
On 2021-10-13 15:30, Lorenzo Pieralisi wrote:
> On Thu, Oct 07, 2021 at 11:18:38PM +0530, Prasad Malisetty wrote:
>> Changes added in v12:
>> 
>> 	* Sorted pipe_clk muxing changes in patch 4 & 5 as per the commit 
>> log.
>> 	  -Suggested by Bjorn.
>> 
>> Changes added in v11:
>> 
>> 	* Modified nvme_pwren name as nvme_pwren.
>> 	* Removed bias-pullup option in nvme_pwren entry [v11 Patch 3/5].
>> 	* Changed pcie1_default_state name to pcie1_clkreq_n.
>> 	* Added NULL pointer check for pcie_cfg.
>> 
>> Changes added in v10:
>> 
>> 	* v9 [Patch 4/4/] has been split into two separate patches
>> 	* Addressed all comments in IDP [Patch 3/4] file.
>> 
>> Changes added in v9:
>>     * Added fixed regulator entry for nvme.suggested by Stephen Boyd
>>     * Added NULL pointer check before accessing ops in pcie probe
>>       Suggested by Stephen Boyd
>> 
>> Changes added in v8:
>> 
>>     * Added seperate pinctrl state for NVMe LDO enable pin [v8 P3/4]
>>     * Removed pointer initialization for pcie_cfg [v8 P4/4]
>>     * Replaced bool pcie_pipe_clk_src with unsigned int:1 [v8 P4/4]
>>     * Changed gcc_pcie_1_pipe_clk_src to pipe_clk_src
>> 
>> Changes added in v7:
>> 
>>         * Removed two fallbacks qcom,pcie-sm8250 and snps,dw-pcie.
>>         * Replaced compatible method in get_resources_2_7_0 with
>>             flag approach suggested by Bjorn Helgaas .
>>         * Setting gcc_pcie_1_clk_src as XO in init_2_7_0 for
>>           gdsc enable.
>>         * Added specific NVMe GPIO entries for SKU1 and SKU2 support
>>           in idp.dts and idp2.dts respectively.
>>         * Moved pcie_1 and pcie_1_phy board specific entries into 
>> common
>>           board file sc7280-idp.dtsi file.
>> 
>> Changes in v6:
>> 
>>     * Removed platform check while setting gcc_pcie_1_pipe_clk_src
>>           as clk_set_parent will return 0 with nop if platform doesn't
>>           need to switch pipe clk source.
>>         * Moved wake-n gpio to board specific file sc7280-idp.dtsi
>>         * Sorted gpio.h header entry in sc7280.dtsi file
>> 
>> Changes in v5:
>> 
>>         * Re ordered PCIe, PHY nodes in Soc and board specific dtsi 
>> files.
>>         * Removed ref_clk entry in current patch [PATCH v4 P4/4].
>>         * I will add ref clk entry in suspend/ resume commits.
>>         * Added boolean flag in Soc specific dtsi file to 
>> differentiate
>>           SM8250 and SC7280 platforms. based on boolean flag, 
>> platforms will handle
>>           the pipe clk handling.
>> 
>> Changes in v4 as suggested by Bjorn:
>> 
>>         * Changed pipe clk mux name as gcc_pcie_1_pipe_clk_src.
>>         * Changed pipe_ext_src as phy_pipe_clk.
>>         * Updated commit message for [PATCH v4 4/4].
>> 
>> Changes in v3:
>>         * Changed pipe clock names in dt bindings as pipe_mux and 
>> phy_pipe.
>>         * Moved reset and NVMe GPIO pin configs into board specific 
>> file.
>>         * Updated pipe clk mux commit message.
>> 
>> Changes in v2:
>>         * Moved pcie pin control settings into IDP file.
>>         * Replaced pipe_clk_src with pipe_clk_mux in pcie driver
>>         * Included pipe clk mux setting change set in this series
>> 
>> Prasad Malisetty (5):
>>   dt-bindings: pci: qcom: Document PCIe bindings for SC7280
>>   arm64: dts: qcom: sc7280: Add PCIe and PHY related nodes
>>   arm64: dts: qcom: sc7280: Add PCIe nodes for IDP board
>>   PCI: qcom: Add a flag in match data along with ops
>>   PCI: qcom: Switch pcie_1_pipe_clk_src after PHY init in SC7280
>> 
>>  .../devicetree/bindings/pci/qcom,pcie.txt          |  17 +++
>>  arch/arm64/boot/dts/qcom/sc7280-idp.dts            |   8 ++
>>  arch/arm64/boot/dts/qcom/sc7280-idp.dtsi           |  50 +++++++++
>>  arch/arm64/boot/dts/qcom/sc7280-idp2.dts           |   8 ++
>>  arch/arm64/boot/dts/qcom/sc7280.dtsi               | 118 
>> +++++++++++++++++++++
>>  drivers/pci/controller/dwc/pcie-qcom.c             |  95 
>> +++++++++++++++--
>>  6 files changed, 285 insertions(+), 11 deletions(-)
> 
> I applied patches [4-5] to pci/qcom for v5.16, thanks I expect other
> patches to go via the relevant trees.
> 
> Lorenzo

Thanks a lot Lorenzo for the update.

-Prasad
Stephen Boyd Oct. 15, 2021, 7:43 p.m. UTC | #3
Quoting Lorenzo Pieralisi (2021-10-13 03:00:05)
> On Thu, Oct 07, 2021 at 11:18:38PM +0530, Prasad Malisetty wrote:
> > Prasad Malisetty (5):
> >   dt-bindings: pci: qcom: Document PCIe bindings for SC7280
> >   arm64: dts: qcom: sc7280: Add PCIe and PHY related nodes
> >   arm64: dts: qcom: sc7280: Add PCIe nodes for IDP board
> >   PCI: qcom: Add a flag in match data along with ops
> >   PCI: qcom: Switch pcie_1_pipe_clk_src after PHY init in SC7280
> >
> >  .../devicetree/bindings/pci/qcom,pcie.txt          |  17 +++
> >  arch/arm64/boot/dts/qcom/sc7280-idp.dts            |   8 ++
> >  arch/arm64/boot/dts/qcom/sc7280-idp.dtsi           |  50 +++++++++
> >  arch/arm64/boot/dts/qcom/sc7280-idp2.dts           |   8 ++
> >  arch/arm64/boot/dts/qcom/sc7280.dtsi               | 118 +++++++++++++++++++++
> >  drivers/pci/controller/dwc/pcie-qcom.c             |  95 +++++++++++++++--
> >  6 files changed, 285 insertions(+), 11 deletions(-)
>
> I applied patches [4-5] to pci/qcom for v5.16, thanks I expect other
> patches to go via the relevant trees.
>

Lorenzo, can you pick up patch 1 too? It's the binding update for the
compatible string used in patch 4-5.
Doug Anderson Oct. 18, 2021, 9:57 p.m. UTC | #4
Hi,

On Fri, Oct 15, 2021 at 12:43 PM Stephen Boyd <swboyd@chromium.org> wrote:
>
> Quoting Lorenzo Pieralisi (2021-10-13 03:00:05)
> > On Thu, Oct 07, 2021 at 11:18:38PM +0530, Prasad Malisetty wrote:
> > > Prasad Malisetty (5):
> > >   dt-bindings: pci: qcom: Document PCIe bindings for SC7280
> > >   arm64: dts: qcom: sc7280: Add PCIe and PHY related nodes
> > >   arm64: dts: qcom: sc7280: Add PCIe nodes for IDP board
> > >   PCI: qcom: Add a flag in match data along with ops
> > >   PCI: qcom: Switch pcie_1_pipe_clk_src after PHY init in SC7280
> > >
> > >  .../devicetree/bindings/pci/qcom,pcie.txt          |  17 +++
> > >  arch/arm64/boot/dts/qcom/sc7280-idp.dts            |   8 ++
> > >  arch/arm64/boot/dts/qcom/sc7280-idp.dtsi           |  50 +++++++++
> > >  arch/arm64/boot/dts/qcom/sc7280-idp2.dts           |   8 ++
> > >  arch/arm64/boot/dts/qcom/sc7280.dtsi               | 118 +++++++++++++++++++++
> > >  drivers/pci/controller/dwc/pcie-qcom.c             |  95 +++++++++++++++--
> > >  6 files changed, 285 insertions(+), 11 deletions(-)
> >
> > I applied patches [4-5] to pci/qcom for v5.16, thanks I expect other
> > patches to go via the relevant trees.
> >
>
> Lorenzo, can you pick up patch 1 too? It's the binding update for the
> compatible string used in patch 4-5.

I think that means that patches 2-3 are ready to land in the Qualcomm
tree assuming Bjorn Andersson is still accepting patches there for
5.16, right?

-Doug