Please use a better subject line. On 03/07/2019 18:48:16+0200, Frank Wunderlich wrote: > From: Josef Friedl <josef.friedl@xxxxxxxx> > > add documentation for pmic, rtc and power/reset devicetree bindings > > Suggested-by: Frank Wunderlich <frank-w@xxxxxxxxxxxxxxx> > Signed-off-by: Josef Friedl <josef.friedl@xxxxxxxx> > Signed-off-by: Frank Wunderlich <frank-w@xxxxxxxxxxxxxxx> > --- > .../devicetree/bindings/mfd/mt6397.txt | 10 ++++++- > .../bindings/power/reset/mt6323-poweroff.txt | 20 +++++++++++++ > .../devicetree/bindings/rtc/rtc-mt6397.txt | 29 +++++++++++++++++++ This file is unrelated to the patch, it should be separated. > 3 files changed, 58 insertions(+), 1 deletion(-) > create mode 100644 Documentation/devicetree/bindings/power/reset/mt6323-poweroff.txt > create mode 100644 Documentation/devicetree/bindings/rtc/rtc-mt6397.txt > > diff --git a/Documentation/devicetree/bindings/mfd/mt6397.txt b/Documentation/devicetree/bindings/mfd/mt6397.txt > index 0ebd08af777d..44acb9827716 100644 > --- a/Documentation/devicetree/bindings/mfd/mt6397.txt > +++ b/Documentation/devicetree/bindings/mfd/mt6397.txt > @@ -8,6 +8,7 @@ MT6397/MT6323 is a multifunction device with the following sub modules: > - Clock > - LED > - Keys > +- Power controller > > It is interfaced to host controller using SPI interface by a proprietary hardware > called PMIC wrapper or pwrap. MT6397/MT6323 MFD is a child device of pwrap. > @@ -22,8 +23,10 @@ compatible: "mediatek,mt6397" or "mediatek,mt6323" > Optional subnodes: > > - rtc > - Required properties: > + Required properties: Should be one of follows > + - compatible: "mediatek,mt6323-rtc" > - compatible: "mediatek,mt6397-rtc" > + For details, see Documentation/devicetree/bindings/rtc/rtc-mt6397.txt > - regulators > Required properties: > - compatible: "mediatek,mt6397-regulator" > @@ -46,6 +49,11 @@ Optional subnodes: > - compatible: "mediatek,mt6397-keys" or "mediatek,mt6323-keys" > see Documentation/devicetree/bindings/input/mtk-pmic-keys.txt > > +- power-controller > + Required properties: > + - compatible: "mediatek,mt6323-pwrc" > + For details, see Documentation/devicetree/bindings/power/reset/mt6323-poweroff.txt > + > Example: > pwrap: pwrap@1000f000 { > compatible = "mediatek,mt8135-pwrap"; > diff --git a/Documentation/devicetree/bindings/power/reset/mt6323-poweroff.txt b/Documentation/devicetree/bindings/power/reset/mt6323-poweroff.txt > new file mode 100644 > index 000000000000..933f0c48e887 > --- /dev/null > +++ b/Documentation/devicetree/bindings/power/reset/mt6323-poweroff.txt > @@ -0,0 +1,20 @@ > +Device Tree Bindings for Power Controller on MediaTek PMIC > + > +The power controller which could be found on PMIC is responsible for externally > +powering off or on the remote MediaTek SoC through the circuit BBPU. > + > +Required properties: > +- compatible: Should be one of follows > + "mediatek,mt6323-pwrc": for MT6323 PMIC > + > +Example: > + > + pmic { > + compatible = "mediatek,mt6323"; > + > + ... > + > + power-controller { > + compatible = "mediatek,mt6323-pwrc"; > + }; > + } > diff --git a/Documentation/devicetree/bindings/rtc/rtc-mt6397.txt b/Documentation/devicetree/bindings/rtc/rtc-mt6397.txt > new file mode 100644 > index 000000000000..ebd1cf80dcc8 > --- /dev/null > +++ b/Documentation/devicetree/bindings/rtc/rtc-mt6397.txt > @@ -0,0 +1,29 @@ > +Device-Tree bindings for MediaTek PMIC based RTC > + > +MediaTek PMIC based RTC is an independent function of MediaTek PMIC that works > +as a type of multi-function device (MFD). The RTC can be configured and set up > +with PMIC wrapper bus which is a common resource shared with the other > +functions found on the same PMIC. > + > +For MediaTek PMIC MFD bindings, see: > +Documentation/devicetree/bindings/mfd/mt6397.txt > + > +For MediaTek PMIC wrapper bus bindings, see: > +Documentation/devicetree/bindings/soc/mediatek/pwrap.txt > + > +Required properties: > +- compatible: Should be one of follows > + "mediatek,mt6323-rtc": for MT6323 PMIC > + "mediatek,mt6397-rtc": for MT6397 PMIC > + > +Example: > + > + pmic { > + compatible = "mediatek,mt6323"; > + > + ... > + > + rtc { > + compatible = "mediatek,mt6323-rtc"; > + }; > + }; > -- > 2.17.1 > -- Alexandre Belloni, Bootlin Embedded Linux and Kernel engineering https://bootlin.com