From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jun Nie Subject: Re: [PATCH v6 1/5] mmc: dt-bindings: add ZTE ZX296718 MMC bindings Date: Thu, 24 Nov 2016 10:17:48 +0800 Message-ID: References: <1479450555-19047-1-git-send-email-jun.nie@linaro.org> <1479450555-19047-2-git-send-email-jun.nie@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <1479450555-19047-2-git-send-email-jun.nie@linaro.org> Sender: linux-mmc-owner@vger.kernel.org To: Shawn Guo , "xie.baoyou" , Rob Herring , mark.rutland@arm.com Cc: Ulf Hansson , Jaehoon Chung , Jason Liu , chen.chaokai@zte.com.cn, lai.binz@zte.com.cn, linux-mmc , Jun Nie , devicetree@vger.kernel.org List-Id: devicetree@vger.kernel.org 2016-11-18 14:29 GMT+08:00 Jun Nie : > Document the device-tree binding of ZTE MMC host on > ZX296718 SoC. > > Signed-off-by: Jun Nie > --- > .../devicetree/bindings/mmc/zx-dw-mshc.txt | 35 ++++++++++++++++++++++ > 1 file changed, 35 insertions(+) > create mode 100644 Documentation/devicetree/bindings/mmc/zx-dw-mshc.txt > > diff --git a/Documentation/devicetree/bindings/mmc/zx-dw-mshc.txt b/Documentation/devicetree/bindings/mmc/zx-dw-mshc.txt > new file mode 100644 > index 0000000..c175c4b > --- /dev/null > +++ b/Documentation/devicetree/bindings/mmc/zx-dw-mshc.txt > @@ -0,0 +1,35 @@ > +* ZTE specific extensions to the Synopsys Designware Mobile Storage > + Host Controller > + > +The Synopsys designware mobile storage host controller is used to interface > +a SoC with storage medium such as eMMC or SD/MMC cards. This file documents > +differences between the core Synopsys dw mshc controller properties described > +by synopsys-dw-mshc.txt and the properties used by the ZTE specific > +extensions to the Synopsys Designware Mobile Storage Host Controller. > + > +Required Properties: > + > +* compatible: should be > + - "zte,zx296718-dw-mshc": for ZX SoCs > + > +Example: > + > + mmc1: mmc@1110000 { > + compatible = "zte,zx296718-dw-mshc"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x01110000 0x1000>; > + interrupts = ; > + fifo-depth = <32>; > + data-addr = <0x200>; > + fifo-watermark-aligned; > + bus-width = <4>; > + clock-frequency = <50000000>; > + clocks = <&topcrm SD0_AHB>, <&topcrm SD0_WCLK>; > + clock-names = "biu", "ciu"; > + num-slots = <1>; > + max-frequency = <50000000>; > + cap-sdio-irq; > + cap-sd-highspeed; > + status = "disabled"; > + }; > -- > 1.9.1 > Hi Rob & Mark, Could you help review and act this patch if you think it is OK? Thank you! Jun