From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ryder Lee Subject: Re: [PATCH 1/3] add doc and MAINTAINERS for poweroff Date: Tue, 2 Jul 2019 18:00:15 +0800 Message-ID: <1562061615.29303.2.camel@mtkswgap22> References: <20190702094045.3652-1-frank-w@public-files.de> <20190702094045.3652-2-frank-w@public-files.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20190702094045.3652-2-frank-w@public-files.de> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Frank Wunderlich , Sean Wang Cc: Mark Rutland , devicetree@vger.kernel.org, Josef Friedl , linux-kernel@vger.kernel.org, Rob Herring , linux-mediatek@lists.infradead.org, Matthias Brugger , linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org +Sean On Tue, 2019-07-02 at 11:40 +0200, Frank Wunderlich wrote: > From: Josef Friedl > > poweroff for BPI-R2 > Suggested-by: Frank Wunderlich > > Signed-off-by: Josef Friedl > --- > .../devicetree/bindings/mfd/mt6397.txt | 10 ++++++- > .../bindings/power/reset/mt6323-poweroff.txt | 20 +++++++++++++ > .../devicetree/bindings/rtc/rtc-mt6397.txt | 29 +++++++++++++++++++ > MAINTAINERS | 6 ++++ > 4 files changed, 64 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"; > + }; > + }; > diff --git a/MAINTAINERS b/MAINTAINERS > index 01a52fc964da..ec6ff342aa3c 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -9920,6 +9920,12 @@ S: Maintained > F: drivers/net/dsa/mt7530.* > F: net/dsa/tag_mtk.c > > +MEDIATEK BOARD LEVEL SHUTDOWN DRIVERS > +M: Sean Wang > +L: linux-pm@vger.kernel.org > +S: Maintained > +F: Documentation/devicetree/bindings/power/reset/mt6323-poweroff.txt It's odd that there's no Sean's SOB tag in the series but add him in the entry. > MEDIATEK JPEG DRIVER > M: Rick Chang > M: Bin Liu > -- > 2.17.1 > > > _______________________________________________ > Linux-mediatek mailing list > Linux-mediatek@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-mediatek