diff mbox series

[1/5] dt-bindings: phy: qcom,qmp: Document SM8250 PCIe PHY bindings

Message ID 20200916132000.1850-2-manivannan.sadhasivam@linaro.org
State Superseded
Headers show
Series Add PCIe support for SM8250 SoC | expand

Commit Message

Manivannan Sadhasivam Sept. 16, 2020, 1:19 p.m. UTC
Document the DT bindings of below PCIe PHY versions used on SM8250:

QMP GEN3x1 PHY - 1 lane
QMP GEN3x2 PHY - 2 lanes
QMP Modem PHY - 2 lanes

Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
---
 Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Bjorn Andersson Sept. 16, 2020, 10:45 p.m. UTC | #1
On Wed 16 Sep 08:19 CDT 2020, Manivannan Sadhasivam wrote:

> Document the DT bindings of below PCIe PHY versions used on SM8250:
> 
> QMP GEN3x1 PHY - 1 lane
> QMP GEN3x2 PHY - 2 lanes
> QMP Modem PHY - 2 lanes

How about something like "Add the three PCIe PHYs found in SM8250 to the
QMP binding"?

> 
> Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> ---
>  Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml
> index 185cdea9cf81..69b67f79075c 100644
> --- a/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml
> +++ b/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml
> @@ -31,6 +31,9 @@ properties:
>        - qcom,sdm845-qmp-usb3-uni-phy
>        - qcom,sm8150-qmp-ufs-phy
>        - qcom,sm8250-qmp-ufs-phy
> +      - qcom,qcom,sm8250-qmp-gen3x1-pcie-phy
> +      - qcom,qcom,sm8250-qmp-gen3x2-pcie-phy
> +      - qcom,qcom,sm8250-qmp-modem-pcie-phy

One "qcom," should be enough.

>  
>    reg:
>      items:
> @@ -259,6 +262,8 @@ allOf:
>              enum:
>                - qcom,sdm845-qhp-pcie-phy
>                - qcom,sdm845-qmp-pcie-phy
> +              - qcom,sm8250-qhp-pcie-phy
> +              - qcom,sm8250-qmp-pcie-phy

Adjust these.

Regards,
Bjorn

>      then:
>        properties:
>          clocks:
> -- 
> 2.17.1
>
Vinod Koul Sept. 17, 2020, 4:32 a.m. UTC | #2
On 16-09-20, 17:45, Bjorn Andersson wrote:
> On Wed 16 Sep 08:19 CDT 2020, Manivannan Sadhasivam wrote:
> 
> > Document the DT bindings of below PCIe PHY versions used on SM8250:
> > 
> > QMP GEN3x1 PHY - 1 lane
> > QMP GEN3x2 PHY - 2 lanes
> > QMP Modem PHY - 2 lanes
> 
> How about something like "Add the three PCIe PHYs found in SM8250 to the
> QMP binding"?

Or add just one compatible sm8250-qmp-pcie and then use number of lanes
as dt property?

> 
> > 
> > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> > ---
> >  Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml | 5 +++++
> >  1 file changed, 5 insertions(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml
> > index 185cdea9cf81..69b67f79075c 100644
> > --- a/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml
> > +++ b/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml
> > @@ -31,6 +31,9 @@ properties:
> >        - qcom,sdm845-qmp-usb3-uni-phy
> >        - qcom,sm8150-qmp-ufs-phy
> >        - qcom,sm8250-qmp-ufs-phy
> > +      - qcom,qcom,sm8250-qmp-gen3x1-pcie-phy
> > +      - qcom,qcom,sm8250-qmp-gen3x2-pcie-phy
> > +      - qcom,qcom,sm8250-qmp-modem-pcie-phy
> 
> One "qcom," should be enough.
> 
> >  
> >    reg:
> >      items:
> > @@ -259,6 +262,8 @@ allOf:
> >              enum:
> >                - qcom,sdm845-qhp-pcie-phy
> >                - qcom,sdm845-qmp-pcie-phy
> > +              - qcom,sm8250-qhp-pcie-phy
> > +              - qcom,sm8250-qmp-pcie-phy
> 
> Adjust these.
> 
> Regards,
> Bjorn
> 
> >      then:
> >        properties:
> >          clocks:
> > -- 
> > 2.17.1
> >
Bjorn Andersson Sept. 17, 2020, 5:10 a.m. UTC | #3
On Wed 16 Sep 23:32 CDT 2020, Vinod Koul wrote:

> On 16-09-20, 17:45, Bjorn Andersson wrote:
> > On Wed 16 Sep 08:19 CDT 2020, Manivannan Sadhasivam wrote:
> > 
> > > Document the DT bindings of below PCIe PHY versions used on SM8250:
> > > 
> > > QMP GEN3x1 PHY - 1 lane
> > > QMP GEN3x2 PHY - 2 lanes
> > > QMP Modem PHY - 2 lanes
> > 
> > How about something like "Add the three PCIe PHYs found in SM8250 to the
> > QMP binding"?
> 
> Or add just one compatible sm8250-qmp-pcie and then use number of lanes
> as dt property?
> 

If we have the same initialization sequence then that sounds reasonable.
Perhaps we can derive the number of lanes from the child node?

A bigger question is how we deal with this going forward, if there are
more crazy setups like on sc8180x where the lanes might be grouped
differently based on board...

Regards,
Bjorn

> > 
> > > 
> > > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> > > ---
> > >  Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml | 5 +++++
> > >  1 file changed, 5 insertions(+)
> > > 
> > > diff --git a/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml
> > > index 185cdea9cf81..69b67f79075c 100644
> > > --- a/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml
> > > +++ b/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml
> > > @@ -31,6 +31,9 @@ properties:
> > >        - qcom,sdm845-qmp-usb3-uni-phy
> > >        - qcom,sm8150-qmp-ufs-phy
> > >        - qcom,sm8250-qmp-ufs-phy
> > > +      - qcom,qcom,sm8250-qmp-gen3x1-pcie-phy
> > > +      - qcom,qcom,sm8250-qmp-gen3x2-pcie-phy
> > > +      - qcom,qcom,sm8250-qmp-modem-pcie-phy
> > 
> > One "qcom," should be enough.
> > 
> > >  
> > >    reg:
> > >      items:
> > > @@ -259,6 +262,8 @@ allOf:
> > >              enum:
> > >                - qcom,sdm845-qhp-pcie-phy
> > >                - qcom,sdm845-qmp-pcie-phy
> > > +              - qcom,sm8250-qhp-pcie-phy
> > > +              - qcom,sm8250-qmp-pcie-phy
> > 
> > Adjust these.
> > 
> > Regards,
> > Bjorn
> > 
> > >      then:
> > >        properties:
> > >          clocks:
> > > -- 
> > > 2.17.1
> > > 
> 
> -- 
> ~Vinod
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml
index 185cdea9cf81..69b67f79075c 100644
--- a/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml
+++ b/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml
@@ -31,6 +31,9 @@  properties:
       - qcom,sdm845-qmp-usb3-uni-phy
       - qcom,sm8150-qmp-ufs-phy
       - qcom,sm8250-qmp-ufs-phy
+      - qcom,qcom,sm8250-qmp-gen3x1-pcie-phy
+      - qcom,qcom,sm8250-qmp-gen3x2-pcie-phy
+      - qcom,qcom,sm8250-qmp-modem-pcie-phy
 
   reg:
     items:
@@ -259,6 +262,8 @@  allOf:
             enum:
               - qcom,sdm845-qhp-pcie-phy
               - qcom,sdm845-qmp-pcie-phy
+              - qcom,sm8250-qhp-pcie-phy
+              - qcom,sm8250-qmp-pcie-phy
     then:
       properties:
         clocks: