答复: [PATCH v3 2/5] dt-bindings: scsi: ufs: add document for hisi-ufs
From: liwei CM <hidden>
Date: 2017-09-02 03:35:41
Also in:
linux-devicetree, linux-scsi
Hi, Rob Thanks for your advises, I will fix them in the next patch. Thank you. -----????----- ???: Rob Herring [mailto:robh at kernel.org] ????: 2017?9?1? 23:53 ???: liwei (CM) ??: mark.rutland at arm.com; xuwei (O); catalin.marinas at arm.com; will.deacon at arm.com; vinholikatti at gmail.com; jejb at linux.vnet.ibm.com; martin.petersen at oracle.com; khilman at baylibre.com; arnd at arndb.de; gregory.clement at free-electrons.com; thomas.petazzoni at free-electrons.com; yamada.masahiro at socionext.com; riku.voipio at linaro.org; treding at nvidia.com; krzk at kernel.org; eric at anholt.net; devicetree at vger.kernel.org; linux-kernel at vger.kernel.org; linux-arm-kernel at lists.infradead.org; linux-scsi at vger.kernel.org; guodong.xu at linaro.org; Fengbaopeng (kevin, Kirin Solution Dept) ??: Re: [PATCH v3 2/5] dt-bindings: scsi: ufs: add document for hisi-ufs On Tue, Aug 29, 2017 at 04:41:13PM +0800, Li Wei wrote:
quoted hunk ↗ jump to hunk
add ufs node document for Hisilicon Signed-off-by: Li Wei <redacted> --- Documentation/devicetree/bindings/ufs/ufs-hisi.txt | 35 ++++++++++++++++++++++ 1 file changed, 35 insertions(+) create mode 100644 Documentation/devicetree/bindings/ufs/ufs-hisi.txtdiff --git a/Documentation/devicetree/bindings/ufs/ufs-hisi.txtb/Documentation/devicetree/bindings/ufs/ufs-hisi.txt new file mode 100644 index 000000000000..cfc84c821d50--- /dev/null +++ b/Documentation/devicetree/bindings/ufs/ufs-hisi.txt@@ -0,0 +1,35 @@ +* Hisilicon Universal Flash Storage (UFS) Host Controller + +UFS nodes are defined to describe on-chip UFS hardware macro. +Each UFS Host Controller should have its own node. + +Required properties: +- compatible : compatible list, contains one of the following - + "hisilicon,hi3660-ufs" for hisi ufs host controller + present on Hi3660 chipset. +- reg : should contain UFS register address space & UFS SYS CTRL register address, +- interrupt-parent : interrupt device +- interrupts : interrupt number +- clocks : List of phandle and clock specifier pairs +- clock-names : List of clock input name strings sorted in the same + order as the clocks property. "clk_ref", "clk_phy" is +optional + +Optional properties for board device: +- reset-gpio : specifies to reset devices
reset-gpios Really, this is should be in a sub node representing the device. It's fine if this is it, but if we start adding power supplies and other properties it should be a separate node.
+
+Example:
+
+ ufs: ufs at ff3b0000 {
+ compatible = "jedec,ufs-1.1", "hisilicon,hi3660-ufs";Need to reverse the order here. Most specific first.
+ /* 0: HCI standard */ + /* 1: UFS SYS CTRL */ + reg = <0x0 0xff3b0000 0x0 0x1000>, + <0x0 0xff3b1000 0x0 0x1000>; + interrupt-parent = <&gic>; + interrupts = <GIC_SPI 278 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&crg_ctrl HI3660_CLK_GATE_UFSIO_REF>, + <&crg_ctrl HI3660_CLK_GATE_UFSPHY_CFG>; + clock-names = "clk_ref", "clk_phy"; + freq-table-hz = <0 0>, <0 0>; + reset-gpio = <&gpio18 1 0>; + } -- 2.11.0