| Message ID | 20230830-topic-refgenphy-v1-1-892db196a1c0@linaro.org |
|---|---|
| State | Changes Requested |
| Headers | show |
| Series | FEMTOPHY REFGEN regulator | expand |
On Wed, Aug 30, 2023 at 11:13:51PM +0200, Konrad Dybcio wrote: > The HSPHY is (or at least can be) one of the users of the reference > voltage generating regulator. Ensure that dependency is described > properly. > > Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org> > --- > Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml b/Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml > index 0f200e3f97a9..e895b6c4ee49 100644 > --- a/Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml > +++ b/Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml > @@ -53,6 +53,9 @@ properties: > items: > - description: PHY core reset > > + refgen-supply: > + description: phandle to the REFGEN regulator node > + > vdda-pll-supply: > description: phandle to the regulator VDD supply node. My OCD isn't keen on the inconsistent full stop usage but that's a nitpick of the context, not of the patch itself. Acked-by: Conor Dooley <conor.dooley@microchip.com> Thanks, Conor, > > > -- > 2.42.0 >
On Wed, Aug 30, 2023 at 11:13:51PM +0200, Konrad Dybcio wrote: > The HSPHY is (or at least can be) one of the users of the reference > voltage generating regulator. Ensure that dependency is described > properly. > > Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org> > --- > Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml b/Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml > index 0f200e3f97a9..e895b6c4ee49 100644 > --- a/Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml > +++ b/Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml > @@ -53,6 +53,9 @@ properties: > items: > - description: PHY core reset > > + refgen-supply: > + description: phandle to the REFGEN regulator node Supply names are local to the device (sink name), but this seems to be named based on the supply source. Maybe hard to do if all internal stuff. What is this supplying and what's the source if not REFGEN? > + > vdda-pll-supply: > description: phandle to the regulator VDD supply node. > > > -- > 2.42.0 >
On 31.08.2023 20:25, Rob Herring wrote: > On Wed, Aug 30, 2023 at 11:13:51PM +0200, Konrad Dybcio wrote: >> The HSPHY is (or at least can be) one of the users of the reference >> voltage generating regulator. Ensure that dependency is described >> properly. >> >> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org> >> --- >> Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml b/Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml >> index 0f200e3f97a9..e895b6c4ee49 100644 >> --- a/Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml >> +++ b/Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml >> @@ -53,6 +53,9 @@ properties: >> items: >> - description: PHY core reset >> >> + refgen-supply: >> + description: phandle to the REFGEN regulator node > > Supply names are local to the device (sink name), but this seems to be > named based on the supply source. Maybe hard to do if all internal > stuff. What is this supplying and what's the source if not REFGEN? I don't think I have anything that would answer this question.. Though I would not at all be surprised if Qualcomm also called the input "refgen".. Maybe Bjorn or Dmitry would know. Konrad
diff --git a/Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml b/Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml index 0f200e3f97a9..e895b6c4ee49 100644 --- a/Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml +++ b/Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml @@ -53,6 +53,9 @@ properties: items: - description: PHY core reset + refgen-supply: + description: phandle to the REFGEN regulator node + vdda-pll-supply: description: phandle to the regulator VDD supply node.
The HSPHY is (or at least can be) one of the users of the reference voltage generating regulator. Ensure that dependency is described properly. Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org> --- Documentation/devicetree/bindings/phy/qcom,usb-snps-femto-v2.yaml | 3 +++ 1 file changed, 3 insertions(+)