From: Rex-BC Chen <rex-bc.chen@mediatek.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
Zhiyong Tao <zhiyong.tao@mediatek.com>, <lee.jones@linaro.org>,
<robh+dt@kernel.org>, <matthias.bgg@gmail.com>,
<lgirdwood@gmail.com>, <broonie@kernel.org>,
<eddie.huang@mediatek.com>, <a.zummo@towertech.it>,
<alexandre.belloni@bootlin.com>, <fshao@chromium.org>
Cc: <srv_heupstream@mediatek.com>, <hui.liu@mediatek.com>,
<tinghan.shen@mediatek.com>, <hsin-hsiung.wang@mediatek.com>,
<sean.wang@mediatek.com>, <macpaul.lin@mediatek.com>,
<wen.su@mediatek.com>, <devicetree@vger.kernel.org>,
<linux-kernel@vger.kernel.org>, <linux-rtc@vger.kernel.org>,
<Project_Global_Chrome_Upstream_Group@mediatek.com>,
<linux-arm-kernel@lists.infradead.org>,
<linux-mediatek@lists.infradead.org>
Subject: Re: [PATCH] dt-bindings: pmic: mt6366: add binding document
Date: Thu, 12 May 2022 10:15:48 +0800 [thread overview]
Message-ID: <cf30fef1db3f42f7c188f1488f5289517ff04844.camel@mediatek.com> (raw)
In-Reply-To: <3e5ddde1-9d0c-a5e8-1828-aef1d1165a3e@linaro.org>
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
next prev parent reply other threads:[~2022-05-12 2:17 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-05-10 6:46 [PATCH v1 0/1] Mediatek MT6366 PMIC patch Zhiyong Tao
2022-05-10 6:46 ` [PATCH] dt-bindings: pmic: mt6366: add binding document Zhiyong Tao
2022-05-10 7:56 ` Rex-BC Chen
2022-05-11 15:36 ` Krzysztof Kozlowski
2022-05-12 2:15 ` Rex-BC Chen [this message]
2022-05-10 10:14 ` Krzysztof Kozlowski
2022-05-10 11:33 ` Mark Brown
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cf30fef1db3f42f7c188f1488f5289517ff04844.camel@mediatek.com \
--to=rex-bc.chen@mediatek.com \
--cc=Project_Global_Chrome_Upstream_Group@mediatek.com \
--cc=a.zummo@towertech.it \
--cc=alexandre.belloni@bootlin.com \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=eddie.huang@mediatek.com \
--cc=fshao@chromium.org \
--cc=hsin-hsiung.wang@mediatek.com \
--cc=hui.liu@mediatek.com \
--cc=krzysztof.kozlowski@linaro.org \
--cc=lee.jones@linaro.org \
--cc=lgirdwood@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-rtc@vger.kernel.org \
--cc=macpaul.lin@mediatek.com \
--cc=matthias.bgg@gmail.com \
--cc=robh+dt@kernel.org \
--cc=sean.wang@mediatek.com \
--cc=srv_heupstream@mediatek.com \
--cc=tinghan.shen@mediatek.com \
--cc=wen.su@mediatek.com \
--cc=zhiyong.tao@mediatek.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).