On 16.01.2023 13:52, Dmitry Baryshkov wrote: > 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? Well, the driver will still probe and function without it, so I'm not sure if it's strictly required.. Konrad > >> >> 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 >> > >
