From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michael Turquette Subject: Re: [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation Date: Wed, 20 Dec 2017 11:26:55 -0800 Message-ID: <151379801537.37313.10157953819670256557@resonance> References: <20171201120708.30129-1-matthias.bgg@gmail.com> <20171219013222.GZ7997@codeaurora.org> <06b927e4-ed1b-9e1a-becf-e2818e7efe6d@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <06b927e4-ed1b-9e1a-becf-e2818e7efe6d-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Matthias Brugger , Stephen Boyd Cc: linux-I+IVW8TIWO2tmTQ+vhA3Yw@public.gmane.org, sean.wang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org, chen.zhong-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: devicetree@vger.kernel.org Quoting Matthias Brugger (2017-12-20 09:13:12) > = > = > On 12/19/2017 02:32 AM, Stephen Boyd wrote: > > On 12/14, Matthias Brugger wrote: > >> Hi Stephen, Michael, > >> > >> On 12/01/2017 01:07 PM, Matthias Brugger wrote: > >>> The ethsys registers a reset controller, so we need to specify a > >>> reset cell. This patch fixes the documentation. > >>> > >>> Signed-off-by: Matthias Brugger > >>> --- > >>> Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt |= 1 + > >>> 1 file changed, 1 insertion(+) > >>> > >>> diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,= ethsys.txt b/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys= .txt > >>> index 7aa3fa167668..6cc7840ff37a 100644 > >>> --- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.= txt > >>> +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.= txt > >>> @@ -20,4 +20,5 @@ ethsys: clock-controller@1b000000 { > >>> compatible =3D "mediatek,mt2701-ethsys", "syscon"; > >>> reg =3D <0 0x1b000000 0 0x1000>; > >>> #clock-cells =3D <1>; > >>> + #reset-cells =3D <1>; > >>> }; > >>> > >> > >> Will you take this patch through the clk tree, or shall I take it thro= ugh my SoC > >> tree? > >> > > = > > It's resets, we are clk maintainers. I'm clkfused. > > = > > You can take it, along with my > > = > > Acked-by: Stephen Boyd > > = > > if you like/expect conflicts. > > = > = > These are resets in the clock IP-block. I'll take it through my branch, I= don't > expect any conflicts. Sounds good to me. Best regards, Mike > = > Regards, > Matthias -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html