From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH 1/3] spis: mediatek: add bindings for Mediatek MT2712 soc platform Date: Tue, 4 Sep 2018 08:18:38 -0500 Message-ID: <5b8e8a7f.1c69fb81.dab09.e985@mx.google.com> References: <1535437685-30230-1-git-send-email-leilk.liu@mediatek.com> <1535437685-30230-2-git-send-email-leilk.liu@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: <1535437685-30230-2-git-send-email-leilk.liu@mediatek.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Leilk Liu Cc: Mark Rutland , devicetree@vger.kernel.org, Sascha Hauer , linux-kernel@vger.kernel.org, linux-spi@vger.kernel.org, yt.shen@mediatek.com, Mark Brown , linux-mediatek@lists.infradead.org, Matthias Brugger , mengqi.zhang@mediatek.com, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org On Tue, Aug 28, 2018 at 02:28:03PM +0800, Leilk Liu wrote: > This patch adds a DT binding documentation for the MT2712 soc. > > Signed-off-by: Leilk Liu > --- > .../devicetree/bindings/spi/spi-slave-mt27xx.txt | 39 ++++++++++++++++++++ > 1 file changed, 39 insertions(+) > create mode 100644 Documentation/devicetree/bindings/spi/spi-slave-mt27xx.txt > > diff --git a/Documentation/devicetree/bindings/spi/spi-slave-mt27xx.txt b/Documentation/devicetree/bindings/spi/spi-slave-mt27xx.txt > new file mode 100644 > index 0000000..dcb8934 > --- /dev/null > +++ b/Documentation/devicetree/bindings/spi/spi-slave-mt27xx.txt > @@ -0,0 +1,39 @@ > +Binding for MTK SPI Slave controller Only does slave mode? If not, then the file name and doc should be just for the SPI controller (master and slave). The "spi-slave" property selects the mode. If it is only slave mode, then you don't need the spi-slave property. > + > +Required properties: > +- compatible: should be one of the following. > + - mediatek,mt2712-spi: for mt2712 platforms > + > +- reg: Address and length of the register set for the device > + > +- interrupts: Should contain spi interrupt > + > +- clocks: phandles to input clocks. > + The first should be one of the following. It's PLL. > + - <&topckgen CLK_TOP_UNIVPLL1_D2>: specify parent clock 312MHZ. > + It's the default one. > + - <&topckgen CLK_TOP_UNIVPLL1_D4>: specify parent clock 156MHZ. > + - <&topckgen CLK_TOP_UNIVPLL2_D4>: specify parent clock 104MHZ. > + - <&topckgen CLK_TOP_UNIVPLL1_D8>: specify parent clock 78MHZ. > + The second should be <&topckgen CLK_TOP_SPISLV_SEL>. It's clock mux. > + The third is <&infracfg CLK_INFRA_AO_SPI1>. It's clock gate. > + > +- clock-names: shall be "parent-clk" for the parent clock, "sel-clk" for the > + muxes clock, and "spi-clk" for the clock gate. "-clk" is redundant. Is the parent clock actually connected to the block? The assigned-clocks properties are used for constraints on parent clocks. > + > +- spi-slave: Empty property indicating the SPI controller is used in slave mode. > + > +Example: > + > +- SoC Specific Portion: > +spis: spi@10013000 { > + compatible = "mediatek,mt2712-spi-slave"; > + reg = <0 0x10013000 0 0x100>; > + interrupts = ; > + clocks = <&topckgen CLK_TOP_UNIVPLL1_D2>, > + <&topckgen CLK_TOP_SPISLV_SEL>, > + <&infracfg CLK_INFRA_AO_SPI1>; > + clock-names = "parent-clk", "sel-clk", "spi-clk"; > + spi-slave; > + status = "disabled"; Don't should status in examples. > +}; > -- > 1.7.9.5 > From mboxrd@z Thu Jan 1 00:00:00 1970 From: robh@kernel.org (Rob Herring) Date: Tue, 4 Sep 2018 08:18:38 -0500 Subject: [PATCH 1/3] spis: mediatek: add bindings for Mediatek MT2712 soc platform In-Reply-To: <1535437685-30230-2-git-send-email-leilk.liu@mediatek.com> References: <1535437685-30230-1-git-send-email-leilk.liu@mediatek.com> <1535437685-30230-2-git-send-email-leilk.liu@mediatek.com> Message-ID: <5b8e8a7f.1c69fb81.dab09.e985@mx.google.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Aug 28, 2018 at 02:28:03PM +0800, Leilk Liu wrote: > This patch adds a DT binding documentation for the MT2712 soc. > > Signed-off-by: Leilk Liu > --- > .../devicetree/bindings/spi/spi-slave-mt27xx.txt | 39 ++++++++++++++++++++ > 1 file changed, 39 insertions(+) > create mode 100644 Documentation/devicetree/bindings/spi/spi-slave-mt27xx.txt > > diff --git a/Documentation/devicetree/bindings/spi/spi-slave-mt27xx.txt b/Documentation/devicetree/bindings/spi/spi-slave-mt27xx.txt > new file mode 100644 > index 0000000..dcb8934 > --- /dev/null > +++ b/Documentation/devicetree/bindings/spi/spi-slave-mt27xx.txt > @@ -0,0 +1,39 @@ > +Binding for MTK SPI Slave controller Only does slave mode? If not, then the file name and doc should be just for the SPI controller (master and slave). The "spi-slave" property selects the mode. If it is only slave mode, then you don't need the spi-slave property. > + > +Required properties: > +- compatible: should be one of the following. > + - mediatek,mt2712-spi: for mt2712 platforms > + > +- reg: Address and length of the register set for the device > + > +- interrupts: Should contain spi interrupt > + > +- clocks: phandles to input clocks. > + The first should be one of the following. It's PLL. > + - <&topckgen CLK_TOP_UNIVPLL1_D2>: specify parent clock 312MHZ. > + It's the default one. > + - <&topckgen CLK_TOP_UNIVPLL1_D4>: specify parent clock 156MHZ. > + - <&topckgen CLK_TOP_UNIVPLL2_D4>: specify parent clock 104MHZ. > + - <&topckgen CLK_TOP_UNIVPLL1_D8>: specify parent clock 78MHZ. > + The second should be <&topckgen CLK_TOP_SPISLV_SEL>. It's clock mux. > + The third is <&infracfg CLK_INFRA_AO_SPI1>. It's clock gate. > + > +- clock-names: shall be "parent-clk" for the parent clock, "sel-clk" for the > + muxes clock, and "spi-clk" for the clock gate. "-clk" is redundant. Is the parent clock actually connected to the block? The assigned-clocks properties are used for constraints on parent clocks. > + > +- spi-slave: Empty property indicating the SPI controller is used in slave mode. > + > +Example: > + > +- SoC Specific Portion: > +spis: spi at 10013000 { > + compatible = "mediatek,mt2712-spi-slave"; > + reg = <0 0x10013000 0 0x100>; > + interrupts = ; > + clocks = <&topckgen CLK_TOP_UNIVPLL1_D2>, > + <&topckgen CLK_TOP_SPISLV_SEL>, > + <&infracfg CLK_INFRA_AO_SPI1>; > + clock-names = "parent-clk", "sel-clk", "spi-clk"; > + spi-slave; > + status = "disabled"; Don't should status in examples. > +}; > -- > 1.7.9.5 >