From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH v6 2/5] dt-bindings: mediatek: Add smi dts binding Date: Tue, 8 Dec 2015 21:35:03 -0600 Message-ID: <20151209033503.GA14549@rob-hp-laptop> References: <1449568153-15643-1-git-send-email-yong.wu@mediatek.com> <1449568153-15643-3-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: <1449568153-15643-3-git-send-email-yong.wu-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org> 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 Cc: Mark Rutland , Catalin Marinas , Will Deacon , yingjoe.chen-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org, Thierry Reding , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Sasha Hauer , arnd-r2nGTMty4D4@public.gmane.org, Tomasz Figa , linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Matthias Brugger , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, pebolle-IWqWACnzNjzz+pZb47iToQ@public.gmane.org, srv_heupstream-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org, Daniel Kurtz , p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org, Lucas Stach List-Id: devicetree@vger.kernel.org On Tue, Dec 08, 2015 at 05:49:10PM +0800, Yong Wu wrote: > This patch add smi binding document. > > Signed-off-by: Yong Wu Acked-by: Rob Herring > --- > .../memory-controllers/mediatek,smi-common.txt | 24 +++++++++++++++++++++ > .../memory-controllers/mediatek,smi-larb.txt | 25 ++++++++++++++++++++++ > 2 files changed, 49 insertions(+) > create mode 100644 Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.txt > create mode 100644 Documentation/devicetree/bindings/memory-controllers/mediatek,smi-larb.txt > > diff --git a/Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.txt b/Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.txt > new file mode 100644 > index 0000000..06a83ce > --- /dev/null > +++ b/Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.txt > @@ -0,0 +1,24 @@ > +SMI (Smart Multimedia Interface) Common > + > +The hardware block diagram please check bindings/iommu/mediatek,iommu.txt > + > +Required properties: > +- compatible : must be "mediatek,mt8173-smi-common" > +- reg : the register and size of the SMI block. > +- power-domains : a phandle to the power domain of this local arbiter. > +- clocks : Must contain an entry for each entry in clock-names. > +- clock-names : must contain 2 entries, as follows: > + - "apb" : Advanced Peripheral Bus clock, It's the clock for setting > + the register. > + - "smi" : It's the clock for transfer data and command. > + They may be the same if both source clocks are the same. > + > +Example: > + smi_common: smi@14022000 { > + compatible = "mediatek,mt8173-smi-common"; > + reg = <0 0x14022000 0 0x1000>; > + power-domains = <&scpsys MT8173_POWER_DOMAIN_MM>; > + clocks = <&mmsys CLK_MM_SMI_COMMON>, > + <&mmsys CLK_MM_SMI_COMMON>; > + clock-names = "apb", "smi"; > + }; > diff --git a/Documentation/devicetree/bindings/memory-controllers/mediatek,smi-larb.txt b/Documentation/devicetree/bindings/memory-controllers/mediatek,smi-larb.txt > new file mode 100644 > index 0000000..55ff3b7 > --- /dev/null > +++ b/Documentation/devicetree/bindings/memory-controllers/mediatek,smi-larb.txt > @@ -0,0 +1,25 @@ > +SMI (Smart Multimedia Interface) Local Arbiter > + > +The hardware block diagram please check bindings/iommu/mediatek,iommu.txt > + > +Required properties: > +- compatible : must be "mediatek,mt8173-smi-larb" > +- reg : the register and size of this local arbiter. > +- mediatek,smi : a phandle to the smi_common node. > +- power-domains : a phandle to the power domain of this local arbiter. > +- clocks : Must contain an entry for each entry in clock-names. > +- clock-names: must contain 2 entries, as follows: > + - "apb" : Advanced Peripheral Bus clock, It's the clock for setting > + the register. > + - "smi" : It's the clock for transfer data and command. > + > +Example: > + larb1: larb@16010000 { > + compatible = "mediatek,mt8173-smi-larb"; > + reg = <0 0x16010000 0 0x1000>; > + mediatek,smi = <&smi_common>; > + power-domains = <&scpsys MT8173_POWER_DOMAIN_VDEC>; > + clocks = <&vdecsys CLK_VDEC_CKEN>, > + <&vdecsys CLK_VDEC_LARB_CKEN>; > + clock-names = "apb", "smi"; > + }; > -- > 1.8.1.1.dirty >