Re: [PATCH] dt-bindings: msm/dsi: Don't require vdds-supply on 7nm PHY

[Dmitry Baryshkov]

On Mon, 16 Jan 2023 at 13:51, Konrad Dybcio <konrad.dybcio@xxxxxxxxxx> wrote:
>
> On some SoCs (hello SM6375) vdds-supply is not wired to any smd-rpm
> or rpmh regulator, but instead powered by the VDD_MX/mx.lvl line,
> which is voted for in the DSI ctrl node.

I think we should have an explicit `if compatible then required:
vdds-supply' clause. WDYT?

>
> Signed-off-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxx>
> ---
>  Documentation/devicetree/bindings/display/msm/dsi-phy-7nm.yaml | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-phy-7nm.yaml b/Documentation/devicetree/bindings/display/msm/dsi-phy-7nm.yaml
> index 9c9184f94c44..8e9031bbde73 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-phy-7nm.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-phy-7nm.yaml
> @@ -48,7 +48,6 @@ required:
>    - compatible
>    - reg
>    - reg-names
> -  - vdds-supply
>
>  unevaluatedProperties: false
>
> --
> 2.39.0
>


-- 
With best wishes
Dmitry