Re: [PATCH v3] arm64: dts: add all hi6220 i2c nodes
On 2015/12/2 15:34, Shawn Guo wrote: > On Wed, Dec 02, 2015 at 02:29:09PM +0800, Xinwei Kong wrote: >> This patch adds all I2C nodes for the Hi6220 SoC. This hi6220 Soc >> use this I2C IP of Synopsys Designware for HiKey board. >> >> Signed-off-by: Xinwei Kong>> --- >> arch/arm64/boot/dts/hisilicon/hi6220.dtsi | 31 >> +++ >> 1 file changed, 31 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi >> b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi >> index 82d2488..6eae673 100644 >> --- a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi >> +++ b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi >> @@ -208,5 +208,36 @@ >> clock-names = "uartclk", "apb_pclk"; >> status = "disabled"; >> }; >> + >> +i2c0: i2c@f710 { >> +compatible = "snps,designware-i2c"; >> +reg = <0x0 0xf710 0x0 0x1000>; >> +interrupts = <0 44 4>; >> +i2c-sda-hold-time-ns = <300>; >> +pinctrl-names = "default"; >> +pinctrl-0 = <_pmx_func _cfg_func>; >> +status = "disabled"; > > You got me wrong. I was asking you to drop property 'clock-names' only, > not 'clocks' together. I do not think it works if you drop property > 'clocks', because I see that dw_i2c_plat_probe() will fail if > devm_clk_get() fails. Did you test the patch before posting it out? I am sorry that my Hikey board don't work well. I will fix my board and test it. Thank you very much. xinwei > > Shawn > >> +}; >> + >> +i2c1: i2c@f7101000 { >> +compatible = "snps,designware-i2c"; >> +reg = <0x0 0xf7101000 0x0 0x1000>; >> +interrupts = <0 45 4>; >> +i2c-sda-hold-time-ns = <300>; >> +pinctrl-names = "default"; >> +pinctrl-0 = <_pmx_func _cfg_func>; >> +status = "disabled"; >> +}; >> + >> +i2c2: i2c@f7102000 { >> +compatible = "snps,designware-i2c"; >> +reg = <0x0 0xf7102000 0x0 0x1000>; >> +interrupts = <0 46 4>; >> +i2c-sda-hold-time-ns = <300>; >> +pinctrl-names = "default"; >> +pinctrl-0 = <_pmx_func _cfg_func>; >> +status = "disabled"; >> +}; >> + >> }; >> }; >> -- >> 1.9.1 >> >> >> > -- > To unsubscribe from this list: send the line "unsubscribe linux-i2c" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > > . > -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH v3] arm64: dts: add all hi6220 i2c nodes
This patch adds all I2C nodes for the Hi6220 SoC. This hi6220 Soc use this I2C IP of Synopsys Designware for HiKey board. Signed-off-by: Xinwei Kong--- arch/arm64/boot/dts/hisilicon/hi6220.dtsi | 31 +++ 1 file changed, 31 insertions(+) diff --git a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi index 82d2488..6eae673 100644 --- a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi +++ b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi @@ -208,5 +208,36 @@ clock-names = "uartclk", "apb_pclk"; status = "disabled"; }; + + i2c0: i2c@f710 { + compatible = "snps,designware-i2c"; + reg = <0x0 0xf710 0x0 0x1000>; + interrupts = <0 44 4>; + i2c-sda-hold-time-ns = <300>; + pinctrl-names = "default"; + pinctrl-0 = <_pmx_func _cfg_func>; + status = "disabled"; + }; + + i2c1: i2c@f7101000 { + compatible = "snps,designware-i2c"; + reg = <0x0 0xf7101000 0x0 0x1000>; + interrupts = <0 45 4>; + i2c-sda-hold-time-ns = <300>; + pinctrl-names = "default"; + pinctrl-0 = <_pmx_func _cfg_func>; + status = "disabled"; + }; + + i2c2: i2c@f7102000 { + compatible = "snps,designware-i2c"; + reg = <0x0 0xf7102000 0x0 0x1000>; + interrupts = <0 46 4>; + i2c-sda-hold-time-ns = <300>; + pinctrl-names = "default"; + pinctrl-0 = <_pmx_func _cfg_func>; + status = "disabled"; + }; + }; }; -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v3] arm64: dts: add all hi6220 i2c nodes
On Wed, Dec 02, 2015 at 02:29:09PM +0800, Xinwei Kong wrote: > This patch adds all I2C nodes for the Hi6220 SoC. This hi6220 Soc > use this I2C IP of Synopsys Designware for HiKey board. > > Signed-off-by: Xinwei Kong> --- > arch/arm64/boot/dts/hisilicon/hi6220.dtsi | 31 > +++ > 1 file changed, 31 insertions(+) > > diff --git a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi > b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi > index 82d2488..6eae673 100644 > --- a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi > +++ b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi > @@ -208,5 +208,36 @@ > clock-names = "uartclk", "apb_pclk"; > status = "disabled"; > }; > + > + i2c0: i2c@f710 { > + compatible = "snps,designware-i2c"; > + reg = <0x0 0xf710 0x0 0x1000>; > + interrupts = <0 44 4>; > + i2c-sda-hold-time-ns = <300>; > + pinctrl-names = "default"; > + pinctrl-0 = <_pmx_func _cfg_func>; > + status = "disabled"; You got me wrong. I was asking you to drop property 'clock-names' only, not 'clocks' together. I do not think it works if you drop property 'clocks', because I see that dw_i2c_plat_probe() will fail if devm_clk_get() fails. Did you test the patch before posting it out? Shawn > + }; > + > + i2c1: i2c@f7101000 { > + compatible = "snps,designware-i2c"; > + reg = <0x0 0xf7101000 0x0 0x1000>; > + interrupts = <0 45 4>; > + i2c-sda-hold-time-ns = <300>; > + pinctrl-names = "default"; > + pinctrl-0 = <_pmx_func _cfg_func>; > + status = "disabled"; > + }; > + > + i2c2: i2c@f7102000 { > + compatible = "snps,designware-i2c"; > + reg = <0x0 0xf7102000 0x0 0x1000>; > + interrupts = <0 46 4>; > + i2c-sda-hold-time-ns = <300>; > + pinctrl-names = "default"; > + pinctrl-0 = <_pmx_func _cfg_func>; > + status = "disabled"; > + }; > + > }; > }; > -- > 1.9.1 > > > -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html