[Freedreno] [PATCH 2/2] dt-bindings: phy: List supplies for qcom, edp-phy
Sankeerth Billakanti (QUIC)
quic_sbillaka at quicinc.com
Thu May 5 18:47:43 UTC 2022
>We're supposed to list the supplies in the dt bindings but there are none in
>the eDP PHY bindings.
>
>Looking at the driver in Linux, I can see that there seem to be two relevant
>supplies: "vdda-phy" and "vdda-pll". Let's add those to the bindings.
>
>NOTE: from looking at the Qualcomm datasheet for sc7280, it's not
>immediately clear how to figure out how to fill in these supplies. The only two
>eDP related supplies are simply described as "power for eDP 0.9V circuits" and
>"power for eDP 1.2V circuits". From guessing and from comparing how a
>similar PHY is hooked up on other similar Qualcomm boards, I'll make the
>educated guess that the 1.2V supply goes to "vdda-phy" and the 0.9V supply
>goes to "vdda-pll" and I'll use that in the example here.
>
>Signed-off-by: Douglas Anderson <dianders at chromium.org>
>---
>
> Documentation/devicetree/bindings/phy/qcom,edp-phy.yaml | 6 ++++++
> 1 file changed, 6 insertions(+)
>
>diff --git a/Documentation/devicetree/bindings/phy/qcom,edp-phy.yaml
>b/Documentation/devicetree/bindings/phy/qcom,edp-phy.yaml
>index a5850ff529f8..cf9e9b8011cb 100644
>--- a/Documentation/devicetree/bindings/phy/qcom,edp-phy.yaml
>+++ b/Documentation/devicetree/bindings/phy/qcom,edp-phy.yaml
>@@ -41,6 +41,9 @@ properties:
> "#phy-cells":
> const: 0
>
>+ vdda-phy-supply: true
>+ vdda-pll-supply: true
>+
> required:
> - compatible
> - reg
>@@ -65,5 +68,8 @@ examples:
>
> #clock-cells = <1>;
> #phy-cells = <0>;
>+
>+ vdda-phy-supply = <&vdd_a_edp_0_1p2>;
>+ vdda-pll-supply = <&vdd_a_edp_0_0p9>;
> };
> ...
>--
>2.36.0.rc2.479.g8af0fa9b8e-goog
Reviewed-by: Sankeerth Billakanti <quic_sbillaka at quicinc.com>
More information about the Freedreno
mailing list