Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752563AbcLLLei (ORCPT ); Mon, 12 Dec 2016 06:34:38 -0500 Received: from szxga03-in.huawei.com ([119.145.14.66]:51575 "EHLO szxga03-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750899AbcLLLeg (ORCPT ); Mon, 12 Dec 2016 06:34:36 -0500 Subject: Re: [PATCH 3/3] dts: hisi: add dts files for Hi3516CV300 demo board To: Pan Wen , , , , , "Arnd Bergmann" References: <20161017120705.3726-1-wenpan@hisilicon.com> <20161017120705.3726-4-wenpan@hisilicon.com> CC: , , , , , , , , From: Jiancheng Xue Message-ID: <4b7476e4-b4ef-965a-d5d4-2e16c83ea21a@hisilicon.com> Date: Mon, 12 Dec 2016 19:34:17 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0 MIME-Version: 1.0 In-Reply-To: <20161017120705.3726-4-wenpan@hisilicon.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.67.245.243] X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 15447 Lines: 586 On 2016/10/17 20:07, Pan Wen wrote: > Add dts files for Hi3516CV300 demo board. > > Signed-off-by: Pan Wen > --- Could you help to review this patch, please? > arch/arm/boot/dts/Makefile | 1 + > arch/arm/boot/dts/hi3516cv300-demb.dts | 148 ++++++++++++ > arch/arm/boot/dts/hi3516cv300.dtsi | 397 +++++++++++++++++++++++++++++++++ > 3 files changed, 546 insertions(+) > create mode 100644 arch/arm/boot/dts/hi3516cv300-demb.dts > create mode 100644 arch/arm/boot/dts/hi3516cv300.dtsi > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > index befcd26..1f25530 100644 > --- a/arch/arm/boot/dts/Makefile > +++ b/arch/arm/boot/dts/Makefile > @@ -171,6 +171,7 @@ dtb-$(CONFIG_ARCH_HIP01) += \ > dtb-$(CONFIG_ARCH_HIP04) += \ > hip04-d01.dtb > dtb-$(CONFIG_ARCH_HISI) += \ > + hi3516cv300-demb.dtb \ > hi3519-demb.dtb > dtb-$(CONFIG_ARCH_HIX5HD2) += \ > hisi-x5hd2-dkb.dtb > diff --git a/arch/arm/boot/dts/hi3516cv300-demb.dts b/arch/arm/boot/dts/hi3516cv300-demb.dts > new file mode 100644 > index 0000000..6a75cd6 > --- /dev/null > +++ b/arch/arm/boot/dts/hi3516cv300-demb.dts > @@ -0,0 +1,148 @@ > +/* > + * Copyright (c) 2016 HiSilicon Technologies Co., Ltd. > + * > + * This program is free software; you can redistribute it and/or modify it > + * under the terms of the GNU General Public License as published by the > + * Free Software Foundation; either version 2 of the License, or (at your > + * option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License > + * along with this program. If not, see . > + * > + */ > + > + > +/dts-v1/; > +#include "hi3516cv300.dtsi" > + > +/ { > + model = "Hisilicon Hi3516CV300 DEMO Board"; > + compatible = "hisilicon,hi3516cv300"; > + > + aliases { > + serial0 = &uart0; > + serial1 = &uart1; > + serial2 = &uart2; > + i2c0 = &i2c_bus0; > + i2c1 = &i2c_bus1; > + spi0 = &spi_bus0; > + spi1 = &spi_bus1; > + }; > + > + chosen { > + stdout-path = "serial0:115200n8"; > + }; > + > + memory { > + device_type = "memory"; > + reg = <0x80000000 0x10000000>; > + }; > +}; > + > +&dual_timer0 { > + status = "okay"; > +}; > + > +&watchdog { > + status = "okay"; > +}; > + > +&pwm { > + status = "okay"; > +}; > + > +&uart0 { > + status = "okay"; > +}; > + > +&i2c_bus0 { > + status = "okay"; > + clock-frequency = <100000>; > + pinctrl-names = "default"; > + pinctrl-0 = <&i2c0_pmux>; > +}; > + > +&i2c_bus1 { > + status = "okay"; > + clock-frequency = <100000>; > + pinctrl-names = "default"; > + pinctrl-0 = <&i2c1_pmux>; > +}; > + > +&spi_bus0{ > + status = "disabled"; > + num-cs = <1>; > + cs-gpios = <&gpio_chip0 6 0>; > + pinctrl-names = "default"; > + pinctrl-0 = <&spi0_pmux>; > +}; > + > +&spi_bus1{ > + status = "okay"; > + num-cs = <2>; > + cs-gpios = <&gpio_chip5 3 0>, <&gpio_chip5 4 0>; > + pinctrl-names = "default"; > + pinctrl-0 = <&spi1_pmux>; > +}; > + > +&fmc { > + spi-nor@0 { > + compatible = "jedec,spi-nor"; > + reg = <0>; > + spi-max-frequency = <160000000>; > + m25p,fast-read; > + }; > +}; > + > +&mdio { > + phy0: phy@1 { > + reg = <1>; > + }; > +}; > + > +&hisi_femac { > + mac-address = [00 00 00 00 00 00]; > + phy-mode = "rmii"; > + phy-handle = <&phy0>; > + hisilicon,phy-reset-delays-us = <10000 10000 150000>; > +}; > + > +&dmac { > + status = "okay"; > +}; > + > +&pmux { > + i2c0_pmux: i2c0_pmux { > + pinctrl-single,pins = < > + 0x2c 0x3 > + 0x30 0x3>; > + }; > + > + i2c1_pmux: i2c1_pmux { > + pinctrl-single,pins = < > + 0x20 0x1 > + 0x24 0x1>; > + }; > + > + spi0_pmux: spi0_pmux { > + pinctrl-single,pins = < > + 0x28 0x1 > + 0x2c 0x1 > + 0x30 0x1 > + 0x34 0x1>; > + }; > + > + spi1_pmux: spi1_pmux { > + pinctrl-single,pins = < > + 0xc4 0x1 > + 0xc8 0x1 > + 0xcc 0x1 > + 0xd0 0x1 > + 0xd4 0x1>; > + }; > +}; > diff --git a/arch/arm/boot/dts/hi3516cv300.dtsi b/arch/arm/boot/dts/hi3516cv300.dtsi > new file mode 100644 > index 0000000..1da41ab > --- /dev/null > +++ b/arch/arm/boot/dts/hi3516cv300.dtsi > @@ -0,0 +1,397 @@ > +/* > + * Copyright (c) 2016 HiSilicon Technologies Co., Ltd. > + * > + * This program is free software; you can redistribute it and/or modify it > + * under the terms of the GNU General Public License as published by the > + * Free Software Foundation; either version 2 of the License, or (at your > + * option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License > + * along with this program. If not, see . > + * > + */ > + > +#include "skeleton.dtsi" > +#include > +/ { > + #address-cells = <1>; > + #size-cells = <1>; > + > + cpus { > + #address-cells = <1>; > + #size-cells = <0>; > + > + cpu@0 { > + device_type = "cpu"; > + compatible = "arm,arm926ej-s"; > + reg = <0>; > + }; > + }; > + > + vic: interrupt-controller@10040000 { > + compatible = "arm,pl190-vic"; > + interrupt-controller; > + #interrupt-cells = <1>; > + reg = <0x10040000 0x1000>; > + }; > + > + soc { > + #address-cells = <1>; > + #size-cells = <1>; > + compatible = "simple-bus"; > + interrupt-parent = <&vic>; > + ranges; > + > + clk_3m: clk_3m { > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + clock-frequency = <3000000>; > + }; > + > + clk_apb: clk_apb { > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + clock-frequency = <50000000>; > + }; > + > + crg: clock-reset-controller@12010000 { > + compatible = "hisilicon,hi3516cv300-crg"; > + reg = <0x12010000 0x1000>; > + #clock-cells = <1>; > + #reset-cells = <2>; > + }; > + > + sysctrl: system-controller@12020000 { > + compatible = "hisilicon,hi3516cv300-sysctrl", "syscon"; > + reg = <0x12020000 0x1000>; > + #clock-cells = <1>; > + }; > + > + reboot { > + compatible = "syscon-reboot"; > + regmap = <&sysctrl>; > + offset = <0x4>; > + mask = <0xdeadbeef>; > + }; > + > + dual_timer0: dual_timer@12000000 { > + compatible = "arm,sp804", "arm,primecell"; > + reg = <0x12000000 0x1000>; > + interrupts = <3>; > + clocks = <&clk_3m>, <&clk_3m>, <&clk_apb>; > + clock-names = "timer0", "timer1", "apb_pclk"; > + status = "disabled"; > + }; > + > + dual_timer1: dual_timer@12001000 { > + compatible = "arm,sp804", "arm,primecell"; > + reg = <0x12001000 0x1000>; > + interrupts = <4>; > + clocks = <&clk_3m>, <&clk_3m>, <&clk_apb>; > + clock-names = "timer0", "timer1", "apb_pclk"; > + status = "disabled"; > + }; > + > + watchdog: watchdog@12080000 { > + compatible = "arm,sp805-wdt", "arm,primecell"; > + arm,primecell-periphid = <0x00141805>; > + reg = <0x12080000 0x1000>; > + clocks = <&sysctrl HI3516CV300_WDT_CLK>, > + <&crg HI3516CV300_APB_CLK>; > + clock-names = "wdog_clk", "apb_pclk"; > + status = "disabled"; > + }; > + > + pwm: pwm@12130000 { > + compatible = "hisilicon,hi3516cv300-pwm", > + "hisilicon,hibvt-pwm"; > + reg = <0x12130000 0x10000>; > + clocks = <&crg HI3516CV300_PWM_CLK>; > + resets = <&crg 0x38 0>; > + #pwm-cells = <2>; > + status = "disabled"; > + }; > + > + uart0: uart@12100000 { > + compatible = "arm,pl011", "arm,primecell"; > + reg = <0x12100000 0x1000>; > + interrupts = <5>; > + clocks = <&crg HI3516CV300_UART0_CLK>; > + clock-names = "apb_pclk"; > + status = "disabled"; > + }; > + > + uart1: uart@12101000 { > + compatible = "arm,pl011", "arm,primecell"; > + reg = <0x12101000 0x1000>; > + interrupts = <30>; > + clocks = <&crg HI3516CV300_UART1_CLK>; > + clock-names = "apb_pclk"; > + status = "disabled"; > + }; > + > + uart2: uart@12102000 { > + compatible = "arm,pl011", "arm,primecell"; > + reg = <0x12102000 0x1000>; > + interrupts = <25>; > + clocks = <&crg HI3516CV300_UART2_CLK>; > + clock-names = "apb_pclk"; > + status = "disabled"; > + }; > + > + i2c_bus0: i2c@12110000 { > + compatible = "hisilicon,hi3516cv300-i2c", > + "hisilicon,hibvt-i2c"; > + reg = <0x12110000 0x1000>; > + clocks = <&crg HI3516CV300_APB_CLK>; > + status = "disabled"; > + }; > + > + i2c_bus1: i2c@12112000 { > + compatible = "hisilicon,hi3516cv300-i2c", > + "hisilicon,hibvt-i2c"; > + reg = <0x12112000 0x1000>; > + clocks = <&crg HI3516CV300_APB_CLK>; > + status = "disabled"; > + }; > + > + spi_bus0: spi@12120000 { > + compatible = "arm,pl022", "arm,primecell"; > + reg = <0x12120000 0x1000>; > + interrupts = <6>; > + clocks = <&crg HI3516CV300_SPI0_CLK>; > + clock-names = "apb_pclk"; > + dmas = <&dmac 12 1>, <&dmac 13 2>; > + dma-names = "rx", "tx"; > + #address-cells = <1>; > + #size-cells = <0>; > + status = "disabled"; > + }; > + > + spi_bus1: spi@12121000 { > + compatible = "arm,pl022", "arm,primecell"; > + reg = <0x12121000 0x1000>, <0x12030000 0x4>; > + interrupts = <7>; > + clocks = <&crg HI3516CV300_SPI1_CLK>; > + clock-names = "apb_pclk"; > + dmas = <&dmac 14 1>, <&dmac 15 2>; > + dma-names = "rx", "tx"; > + #address-cells = <1>; > + #size-cells = <0>; > + status = "disabled"; > + }; > + > + fmc: spi-nor-controller@10000000 { > + compatible = "hisilicon,fmc-spi-nor"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x10000000 0x1000>, <0x14000000 0x1000000>; > + reg-names = "control", "memory"; > + clocks = <&crg HI3516CV300_FMC_CLK>; > + assigned-clocks = <&crg HI3516CV300_FMC_CLK>; > + assigned-clock-rates = <24000000>; > + }; > + > + mdio: mdio@10051100 { > + compatible = "hisilicon,hisi-femac-mdio"; > + reg = <0x10051100 0x10>; > + clocks = <&crg HI3516CV300_ETH_CLK>; > + #address-cells = <1>; > + #size-cells = <0>; > + }; > + > + hisi_femac: ethernet@10090000 { > + compatible = "hisilicon,hi3516cv300-femac", > + "hisilicon,hisi-femac-v2"; > + reg = <0x10050000 0x1000>,<0x10051300 0x200>; > + interrupts = <12>; > + clocks = <&crg HI3516CV300_ETH_CLK>; > + resets = <&crg 0xec 0>, <&crg 0xec 3>; > + reset-names = "mac","phy"; > + }; > + > + dmac: dma-controller@10030000 { > + compatible = "arm,pl080", "arm,primecell"; > + reg = <0x10030000 0x1000>; > + interrupts = <14>; > + clocks = <&crg HI3516CV300_DMAC_CLK>; > + clock-names = "apb_pclk"; > + lli-bus-interface-ahb1; > + lli-bus-interface-ahb2; > + mem-bus-interface-ahb1; > + mem-bus-interface-ahb2; > + memcpy-burst-size = <256>; > + memcpy-bus-width = <32>; > + #dma-cells = <2>; > + status = "disabled"; > + }; > + > + gpio_chip0: gpio@12140000 { > + compatible = "arm,pl061", "arm,primecell"; > + reg = <0x12140000 0x1000>; > + interrupts = <31>; > + clocks = <&crg HI3516CV300_APB_CLK>; > + clock-names = "apb_pclk"; > + #gpio-cells = <2>; > + gpio-ranges = <&pmux 0 61 2>, > + <&pmux 4 11 1>, > + <&pmux 5 10 1>, > + <&pmux 6 13 2>; > + > + status = "disabled"; > + }; > + > + gpio_chip1: gpio@12141000 { > + compatible = "arm,pl061", "arm,primecell"; > + reg = <0x12141000 0x1000>; > + interrupts = <31>; > + clocks = <&crg HI3516CV300_APB_CLK>; > + clock-names = "apb_pclk"; > + #gpio-cells = <2>; > + gpio-ranges = <&pmux 0 16 7>, > + <&pmux 7 0 1>; > + status = "disabled"; > + }; > + > + gpio_chip2: gpio@12142000 { > + compatible = "arm,pl061", "arm,primecell"; > + reg = <0x12142000 0x1000>; > + interrupts = <31>; > + clocks = <&crg HI3516CV300_APB_CLK>; > + clock-names = "apb_pclk"; > + #gpio-cells = <2>; > + gpio-ranges = <&pmux 0 46 1>, > + <&pmux 1 45 1>, > + <&pmux 2 44 1>, > + <&pmux 3 43 1>, > + <&pmux 4 39 1>, > + <&pmux 5 38 1>, > + <&pmux 6 40 1>, > + <&pmux 7 48 1>; > + status = "disabled"; > + }; > + > + gpio_chip3: gpio@12143000 { > + compatible = "arm,pl061", "arm,primecell"; > + reg = <0x12143000 0x1000>; > + interrupts = <31>; > + clocks = <&crg HI3516CV300_APB_CLK>; > + clock-names = "apb_pclk"; > + #gpio-cells = <2>; > + gpio-ranges = <&pmux 0 37 1>, > + <&pmux 1 36 1>, > + <&pmux 2 35 1>, > + <&pmux 3 34 1>, > + <&pmux 4 23 2>, > + <&pmux 6 8 2>; > + status = "disabled"; > + }; > + > + gpio_chip4: gpio@12144000 { > + compatible = "arm,pl061", "arm,primecell"; > + reg = <0x12144000 0x1000>; > + interrupts = <31>; > + clocks = <&crg HI3516CV300_APB_CLK>; > + clock-names = "apb_pclk"; > + #gpio-cells = <2>; > + gpio-ranges = <&pmux 0 27 1>, > + <&pmux 1 26 1>, > + <&pmux 2 31 1>, > + <&pmux 3 30 1>, > + <&pmux 4 28 2>, > + <&pmux 6 33 1>, > + <&pmux 7 32 1>; > + status = "disabled"; > + }; > + > + gpio_chip5: gpio@12145000 { > + compatible = "arm,pl061", "arm,primecell"; > + reg = <0x12145000 0x1000>; > + interrupts = <31>; > + clocks = <&crg HI3516CV300_APB_CLK>; > + clock-names = "apb_pclk"; > + #gpio-cells = <2>; > + gpio-ranges = <&pmux 0 53 1>, > + <&pmux 1 51 2>, > + <&pmux 3 50 1>, > + <&pmux 4 49 1>, > + <&pmux 5 47 1>, > + <&pmux 6 40 2>; > + status = "disabled"; > + }; > + > + gpio_chip6: gpio@12146000 { > + compatible = "arm,pl061", "arm,primecell"; > + reg = <0x12146000 0x1000>; > + interrupts = <31>; > + clocks = <&crg HI3516CV300_APB_CLK>; > + clock-names = "apb_pclk"; > + #gpio-cells = <2>; > + gpio-ranges = <&pmux 0 7 1>, > + <&pmux 1 6 1>, > + <&pmux 2 4 1>, > + <&pmux 3 5 1>, > + <&pmux 4 15 1>, > + <&pmux 5 1 3>; > + status = "disabled"; > + }; > + > + gpio_chip7: gpio@12147000 { > + compatible = "arm,pl061", "arm,primecell"; > + reg = <0x12147000 0x1000>; > + interrupts = <31>; > + clocks = <&crg HI3516CV300_APB_CLK>; > + clock-names = "apb_pclk"; > + #gpio-cells = <2>; > + gpio-ranges = <&pmux 1 55 6>, > + <&pmux 7 25 1>; > + status = "disabled"; > + }; > + > + gpio_chip8: gpio@12148000 { > + compatible = "arm,pl061", "arm,primecell"; > + reg = <0x12148000 0x1000>; > + interrupts = <31>; > + clocks = <&crg HI3516CV300_APB_CLK>; > + clock-names = "apb_pclk"; > + #gpio-cells = <2>; > + gpio-ranges = <&pmux 0 63 3>, > + <&pmux 3 12 1>; > + status = "disabled"; > + }; > + > + pmux: pinmux@12040000 { > + compatible = "pinctrl-single"; > + reg = <0x12040000 0x108>; > + #address-cells = <1>; > + #size-cells = <1>; > + #gpio-range-cells = <3>; > + ranges; > + > + pinctrl-single,register-width = <32>; > + pinctrl-single,function-mask = <7>; > + /* pin base, nr pins & gpio function */ > + pinctrl-single,gpio-range = <&range 0 54 0 > + &range 55 6 1 &range 61 5 0>; > + > + range: gpio-range { > + #pinctrl-single,gpio-range-cells = <3>; > + }; > + }; > + > + pconf: pinconf@12040800 { > + compatible = "pinconf-single"; > + reg = <0x12040800 0x130>; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges; > + > + pinctrl-single,register-width = <32>; > + }; > + }; > +}; >