From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756309AbbCFLOO (ORCPT ); Fri, 6 Mar 2015 06:14:14 -0500 Received: from foss.arm.com ([217.140.101.70]:59503 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751884AbbCFLOI (ORCPT ); Fri, 6 Mar 2015 06:14:08 -0500 Date: Fri, 6 Mar 2015 11:13:38 +0000 From: Mark Rutland To: "yong.wu@mediatek.com" Cc: Rob Herring , Joerg Roedel , Matthias Brugger , Robin Murphy , Will Deacon , Daniel Kurtz , Tomasz Figa , Lucas Stach , Catalin Marinas , "linux-mediatek@lists.infradead.org" , Sasha Hauer , "srv_heupstream@mediatek.com" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "iommu@lists.linux-foundation.org" Subject: Re: [PATCH 3/5] dt-bindings: mediatek: Add smi dts binding Message-ID: <20150306111338.GD8700@leverpostej> References: <1425638900-24989-1-git-send-email-yong.wu@mediatek.com> <1425638900-24989-4-git-send-email-yong.wu@mediatek.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1425638900-24989-4-git-send-email-yong.wu@mediatek.com> Thread-Topic: [PATCH 3/5] dt-bindings: mediatek: Add smi dts binding Accept-Language: en-GB, en-US Content-Language: en-US User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Mar 06, 2015 at 10:48:18AM +0000, yong.wu@mediatek.com wrote: > From: Yong Wu > > This patch add smi binding document. Please move binding documents to the start of the series. It makes things far easier to review. > > Signed-off-by: Yong Wu > --- > .../devicetree/bindings/soc/mediatek/mediatek,smi.txt | 17 +++++++++++++++++ > 1 file changed, 17 insertions(+) > create mode 100644 Documentation/devicetree/bindings/soc/mediatek/mediatek,smi.txt > > diff --git a/Documentation/devicetree/bindings/soc/mediatek/mediatek,smi.txt b/Documentation/devicetree/bindings/soc/mediatek/mediatek,smi.txt > new file mode 100644 > index 0000000..0fc9d1a > --- /dev/null > +++ b/Documentation/devicetree/bindings/soc/mediatek/mediatek,smi.txt > @@ -0,0 +1,17 @@ > +SMI hardware block diagram please help check > + > +Required properties: > +- compatible : must be "mediatek,mediatek,mt8173-smi-larb" Double vendor prefix? What does "larb" mean? It would be nice for the intorductory paragraph in this file to explain. > +- reg : the register of each local arbiter > +- clocks : the clocks of each local arbiter > +- clock-name: larb_sub*(3 clockes at most) The names required _must_ be specified here, or clock-names is pointless. The clock names should be from the PoV of _this_ device (i.e. they should be the names of the inputs) not from the PoV of the provider (i.e. they should not be the names of the outputs from the provider). Mark. > + > +Example: > + larb1:larb@16010000 { > + compatible = "mediatek,mt8173-smi-larb"; > + reg = <0 0x16010000 0 0x1000>; > + clocks = <&mmsys MM_SMI_COMMON>, > + <&vdecsys VDEC_CKEN>, > + <&vdecsys VDEC_LARB_CKEN>; > + clock-names = "larb_sub0", "larb_sub1", "larb_sub2"; > + }; > -- > 1.8.1.1.dirty > > From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Rutland Subject: Re: [PATCH 3/5] dt-bindings: mediatek: Add smi dts binding Date: Fri, 6 Mar 2015 11:13:38 +0000 Message-ID: <20150306111338.GD8700@leverpostej> References: <1425638900-24989-1-git-send-email-yong.wu@mediatek.com> <1425638900-24989-4-git-send-email-yong.wu@mediatek.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1425638900-24989-4-git-send-email-yong.wu-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: "yong.wu-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org" Cc: "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "srv_heupstream-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org" , Catalin Marinas , Will Deacon , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Tomasz Figa , "iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org" , Rob Herring , Daniel Kurtz , Sasha Hauer , Matthias Brugger , "linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , Lucas Stach List-Id: devicetree@vger.kernel.org On Fri, Mar 06, 2015 at 10:48:18AM +0000, yong.wu-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org wrote: > From: Yong Wu > > This patch add smi binding document. Please move binding documents to the start of the series. It makes things far easier to review. > > Signed-off-by: Yong Wu > --- > .../devicetree/bindings/soc/mediatek/mediatek,smi.txt | 17 +++++++++++++++++ > 1 file changed, 17 insertions(+) > create mode 100644 Documentation/devicetree/bindings/soc/mediatek/mediatek,smi.txt > > diff --git a/Documentation/devicetree/bindings/soc/mediatek/mediatek,smi.txt b/Documentation/devicetree/bindings/soc/mediatek/mediatek,smi.txt > new file mode 100644 > index 0000000..0fc9d1a > --- /dev/null > +++ b/Documentation/devicetree/bindings/soc/mediatek/mediatek,smi.txt > @@ -0,0 +1,17 @@ > +SMI hardware block diagram please help check > + > +Required properties: > +- compatible : must be "mediatek,mediatek,mt8173-smi-larb" Double vendor prefix? What does "larb" mean? It would be nice for the intorductory paragraph in this file to explain. > +- reg : the register of each local arbiter > +- clocks : the clocks of each local arbiter > +- clock-name: larb_sub*(3 clockes at most) The names required _must_ be specified here, or clock-names is pointless. The clock names should be from the PoV of _this_ device (i.e. they should be the names of the inputs) not from the PoV of the provider (i.e. they should not be the names of the outputs from the provider). Mark. > + > +Example: > + larb1:larb@16010000 { > + compatible = "mediatek,mt8173-smi-larb"; > + reg = <0 0x16010000 0 0x1000>; > + clocks = <&mmsys MM_SMI_COMMON>, > + <&vdecsys VDEC_CKEN>, > + <&vdecsys VDEC_LARB_CKEN>; > + clock-names = "larb_sub0", "larb_sub1", "larb_sub2"; > + }; > -- > 1.8.1.1.dirty > > From mboxrd@z Thu Jan 1 00:00:00 1970 From: mark.rutland@arm.com (Mark Rutland) Date: Fri, 6 Mar 2015 11:13:38 +0000 Subject: [PATCH 3/5] dt-bindings: mediatek: Add smi dts binding In-Reply-To: <1425638900-24989-4-git-send-email-yong.wu@mediatek.com> References: <1425638900-24989-1-git-send-email-yong.wu@mediatek.com> <1425638900-24989-4-git-send-email-yong.wu@mediatek.com> Message-ID: <20150306111338.GD8700@leverpostej> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, Mar 06, 2015 at 10:48:18AM +0000, yong.wu at mediatek.com wrote: > From: Yong Wu > > This patch add smi binding document. Please move binding documents to the start of the series. It makes things far easier to review. > > Signed-off-by: Yong Wu > --- > .../devicetree/bindings/soc/mediatek/mediatek,smi.txt | 17 +++++++++++++++++ > 1 file changed, 17 insertions(+) > create mode 100644 Documentation/devicetree/bindings/soc/mediatek/mediatek,smi.txt > > diff --git a/Documentation/devicetree/bindings/soc/mediatek/mediatek,smi.txt b/Documentation/devicetree/bindings/soc/mediatek/mediatek,smi.txt > new file mode 100644 > index 0000000..0fc9d1a > --- /dev/null > +++ b/Documentation/devicetree/bindings/soc/mediatek/mediatek,smi.txt > @@ -0,0 +1,17 @@ > +SMI hardware block diagram please help check > + > +Required properties: > +- compatible : must be "mediatek,mediatek,mt8173-smi-larb" Double vendor prefix? What does "larb" mean? It would be nice for the intorductory paragraph in this file to explain. > +- reg : the register of each local arbiter > +- clocks : the clocks of each local arbiter > +- clock-name: larb_sub*(3 clockes at most) The names required _must_ be specified here, or clock-names is pointless. The clock names should be from the PoV of _this_ device (i.e. they should be the names of the inputs) not from the PoV of the provider (i.e. they should not be the names of the outputs from the provider). Mark. > + > +Example: > + larb1:larb at 16010000 { > + compatible = "mediatek,mt8173-smi-larb"; > + reg = <0 0x16010000 0 0x1000>; > + clocks = <&mmsys MM_SMI_COMMON>, > + <&vdecsys VDEC_CKEN>, > + <&vdecsys VDEC_LARB_CKEN>; > + clock-names = "larb_sub0", "larb_sub1", "larb_sub2"; > + }; > -- > 1.8.1.1.dirty > >