From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michael Turquette Subject: Re: [PATCH 3/5] clk: mediatek: Add MT2701 clock support Date: Wed, 30 Dec 2015 15:34:06 -0800 Message-ID: <20151230233406.19557.35006@quark.deferred.io> References: <1451456865-44139-1-git-send-email-jamesjj.liao@mediatek.com> <1451456865-44139-4-git-send-email-jamesjj.liao@mediatek.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8BIT Return-path: In-Reply-To: <1451456865-44139-4-git-send-email-jamesjj.liao@mediatek.com> Sender: linux-clk-owner@vger.kernel.org To: Matthias Brugger , Stephen Boyd Cc: Sascha Hauer , Daniel Kurtz , srv_heupstream@mediatek.com, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-clk@vger.kernel.org, Shunli Wang , James Liao List-Id: devicetree@vger.kernel.org Hi James, 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? Thanks, Mike