linux-kernel - Re: [PATCH] dt-bindings: pmic: mt6366: add binding document

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Thu, 12 May 2022 10:15:48 +0800
From:   Rex-BC Chen <rex-bc.chen@...iatek.com>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
        Zhiyong Tao <zhiyong.tao@...iatek.com>, <lee.jones@...aro.org>,
        <robh+dt@...nel.org>, <matthias.bgg@...il.com>,
        <lgirdwood@...il.com>, <broonie@...nel.org>,
        <eddie.huang@...iatek.com>, <a.zummo@...ertech.it>,
        <alexandre.belloni@...tlin.com>, <fshao@...omium.org>
CC:     <srv_heupstream@...iatek.com>, <hui.liu@...iatek.com>,
        <tinghan.shen@...iatek.com>, <hsin-hsiung.wang@...iatek.com>,
        <sean.wang@...iatek.com>, <macpaul.lin@...iatek.com>,
        <wen.su@...iatek.com>, <devicetree@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>, <linux-rtc@...r.kernel.org>,
        <Project_Global_Chrome_Upstream_Group@...iatek.com>,
        <linux-arm-kernel@...ts.infradead.org>,
        <linux-mediatek@...ts.infradead.org>
Subject: Re: [PATCH] dt-bindings: pmic: mt6366: add binding document

On Wed, 2022-05-11 at 17:36 +0200, Krzysztof Kozlowski wrote:
> On 10/05/2022 09:56, Rex-BC Chen wrote:
> > > +examples:
> > > +  - |
> > > +    pmic {
> > > +      regulators {
> > > +        mt6366_vdram1_reg: buck_vdram1 {
> > > +            regulator-name = "vdram1";
> > > +            regulator-min-microvolt = <500000>;
> > > +            regulator-max-microvolt = <2087500>;
> > > +            regulator-ramp-delay = <12500>;
> > > +            regulator-enable-ramp-delay = <0>;
> > > +            regulator-always-on;
> > > +            regulator-allowed-modes = <0 1>;
> > 
> > should be two space instead of 4 space?
> > mt6366_vdram1_reg: buck_vdram1 {
> >   regulator-name = "vdram1";
> >   ...
> > };
> > 
> > There is the same issue for this example.
> 
> Indentation is four spaces.
> 
https://urldefense.com/v3/__https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git/tree/Documentation/devicetree/bindings/writing-schema.rst?h=for-next&id=7a150b0d850e37e7bdfc87459edd0ff302f67478*n111__;Iw!!CTRNKA9wMg0ARbw!yGLurJNfCjmllj-0EbUwL_HZef0f6MlfC8U968TLmzDG13EL7BErcZjtKu9cf6MMcz-E$
>  
> 
> 
> Best regards,
> Krzysztof

Hello Krzysztof,

Thanks for your correction.

BRs,
Rex