From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Liao Subject: Re: [PATCH 3/5] clk: mediatek: Add MT2701 clock support Date: Thu, 31 Dec 2015 17:59:02 +0800 Message-ID: <1451555942.10610.18.camel@mtksdaap41> References: <1451456865-44139-1-git-send-email-jamesjj.liao@mediatek.com> <1451456865-44139-4-git-send-email-jamesjj.liao@mediatek.com> <20151230233406.19557.35006@quark.deferred.io> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20151230233406.19557.35006-/Ffx6e7uQFNsG52AEeRyZ2GXanvQGlWp@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: Michael Turquette Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, srv_heupstream-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org, Stephen Boyd , linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Shunli Wang , Sascha Hauer , Matthias Brugger , linux-clk-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: devicetree@vger.kernel.org Hi Mike On Wed, 2015-12-30 at 15:34 -0800, Michael Turquette wrote: > Quoting James Liao (2015-12-29 22:27:43) > > +CLK_OF_DECLARE(mtk_topckgen, "mediatek,mt2701-topckgen", mtk_topckgen_init); > > +CLK_OF_DECLARE(mtk_infrasys, "mediatek,mt2701-infracfg", mtk_infrasys_init); > > +CLK_OF_DECLARE(mtk_pericfg, "mediatek,mt2701-pericfg", mtk_pericfg_init); > > +CLK_OF_DECLARE(mtk_mmsys, "mediatek,mt2701-mmsys", mtk_mmsys_init); > > +CLK_OF_DECLARE(mtk_imgsys, "mediatek,mt2701-imgsys", mtk_imgsys_init); > > +CLK_OF_DECLARE(mtk_vdecsys, "mediatek,mt2701-vdecsys", mtk_vdecsys_init); > > +CLK_OF_DECLARE(mtk_hifsys, "mediatek,mt2701-hifsys", mtk_hifsys_init); > > +CLK_OF_DECLARE(mtk_ethsys, "mediatek,mt2701-ethsys", mtk_ethsys_init); > > +CLK_OF_DECLARE(mtk_bdpsys, "mediatek,mt2701-bdpsys", mtk_bdpsys_init); > > +CLK_OF_DECLARE(mtk_apmixedsys, "mediatek,mt2701-apmixedsys", > > None of these compatible strings are documented with a binding > description in this patch series. In fact looking through > Documentation/devicetree/bindings/clock/ on the latest linux-next tag I > see that the only existing binding is mt8173-cpu-dvfs.txt. > > Am I looking in the wrong place or do we have a case of missing binding > descriptions? The binding documents are located in Documentation/devicetree/bindings/arm/mediatek/ . And I missed to add mt2701 compatible strings into it. I'll add them in next patchset. Best regards, James