lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 8 Nov 2022 22:12:02 +0100 From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org> To: Bryan O'Donoghue <bryan.odonoghue@...aro.org>, robdclark@...il.com, quic_abhinavk@...cinc.com, dmitry.baryshkov@...aro.org, krzysztof.kozlowski+dt@...aro.org, robh+dt@...nel.org, quic_mkrishn@...cinc.com, linux-arm-msm@...r.kernel.org Cc: Sean Paul <sean@...rly.run>, David Airlie <airlied@...il.com>, Daniel Vetter <daniel@...ll.ch>, dri-devel@...ts.freedesktop.org, freedreno@...ts.freedesktop.org, devicetree@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH v2 03/18] dt-bindings: msm: dsi-controller-main: Add vdd* descriptions back in On 08/11/2022 00:56, Bryan O'Donoghue wrote: > When converting from .txt to .yaml we didn't include descriptions for the > existing regulator supplies. > > - vdd > - vdda > - vddio > > Add those descriptions into the yaml now as they were prior to the > conversion. Mark the supplies as required as was previously the case in the > .txt implementation. > > Warnings about missing regulators can be resolved by updating the relevant > dtsi files to point to fixed always-on regulators where appropriate. > > Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings") > Cc: Rob Clark <robdclark@...il.com> > Cc: Abhinav Kumar <quic_abhinavk@...cinc.com> > Cc: Dmitry Baryshkov <dmitry.baryshkov@...aro.org> > Cc: Sean Paul <sean@...rly.run> > Cc: David Airlie <airlied@...il.com> > Cc: Daniel Vetter <daniel@...ll.ch> > Cc: Rob Herring <robh+dt@...nel.org> > Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org> > Cc: linux-arm-msm@...r.kernel.org > Cc: dri-devel@...ts.freedesktop.org > Cc: freedreno@...ts.freedesktop.org > Cc: devicetree@...r.kernel.org > Cc: linux-kernel@...r.kernel.org > Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@...aro.org> > --- > .../bindings/display/msm/dsi-controller-main.yaml | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > > diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml > index cf782c5f5bdb0..0f7747e55b9be 100644 > --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml > +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml > @@ -124,6 +124,18 @@ properties: > - port@0 > - port@1 > > + vdd-supply: > + description: > + Phandle to vdd regulator device node Drop "Phandle to" and "device node", so just "VDD regulator". You describe hardware rather (when applicable), not Devicetree syntax. The same in other places and with that: Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org> Best regards, Krzysztof
Powered by blists - more mailing lists