From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yingjoe Chen Subject: Re: [PATCH 1/2] ASoC: dt-bingdings: mediatek: mt8183: add a property "medaitek, toprgu" Date: Fri, 13 Sep 2019 00:52:17 +0800 Message-ID: <1568307137.22948.5.camel@mtksdaap41> References: <1568282096-13821-1-git-send-email-jiaxin.yu@mediatek.com> <1568282096-13821-2-git-send-email-jiaxin.yu@mediatek.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1568282096-13821-2-git-send-email-jiaxin.yu-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+glpam-linux-mediatek=m.gmane.org-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org To: Jiaxin Yu Cc: alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw@public.gmane.org, lgirdwood-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, perex-/Fr2/VpizcU@public.gmane.org, tzungbi-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org, broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, eason.yen-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org List-Id: linux-mediatek@lists.infradead.org On Thu, 2019-09-12 at 17:54 +0800, Jiaxin Yu wrote: > This patch adds a property "mediatek,toprgu" in example so that we could > use reset controller(usually we call it toprgu or watchdog) to reset audio > domain regs. > > Signed-off-by: Jiaxin Yu > --- > Documentation/devicetree/bindings/sound/mt8183-afe-pcm.txt | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/Documentation/devicetree/bindings/sound/mt8183-afe-pcm.txt b/Documentation/devicetree/bindings/sound/mt8183-afe-pcm.txt > index 396ba38619f6..45ca182a4ecc 100644 > --- a/Documentation/devicetree/bindings/sound/mt8183-afe-pcm.txt > +++ b/Documentation/devicetree/bindings/sound/mt8183-afe-pcm.txt > @@ -4,6 +4,7 @@ Required properties: > - compatible = "mediatek,mt68183-audio"; > - reg: register location and size > - interrupts: should contain AFE interrupt > +- mediatek,toprgu: A phandle to the TOPRGU which for reset controller unit > - power-domains: should define the power domain > - clocks: Must contain an entry for each entry in clock-names > - clock-names: should have these clock names: > @@ -20,6 +21,7 @@ Example: > compatible = "mediatek,mt8183-audio"; > reg = <0 0x11220000 0 0x1000>; > interrupts = ; > + mediatek,toprgu = <&watchdog>; As mentioned in the other patch, since this is reset controller, you should use Documentation/devicetree/bindings/reset/reset.txt instead. Joe.C