From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gregory CLEMENT Subject: Re: [PATCH v7 3/4] arm64: dts: marvell: Add I2C definitions for the Armada 3700 Date: Tue, 03 Jan 2017 16:20:34 +0100 Message-ID: <8760lwgofx.fsf@free-electrons.com> References: <20161201110440.27530-1-romain.perier@free-electrons.com> <20161201110440.27530-4-romain.perier@free-electrons.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20161201110440.27530-4-romain.perier@free-electrons.com> (Romain Perier's message of "Thu, 1 Dec 2016 12:04:39 +0100") 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: Romain Perier Cc: Mark Rutland , devicetree@vger.kernel.org, Yahuda Yitschak , Omri Itach , Jason Cooper , Pawel Moll , Ian Campbell , Igal Liberman , Hanna Hawa , Wolfram Sang , Neta Zur Hershkovits , Nadav Haklai , Rob Herring , Andrew Lunn , linux-i2c@vger.kernel.org, Kumar Gala , Shadi Ammouri , Marcin Wojtas , Thomas Petazzoni , linux-arm-kernel@lists.infradead.org, Sebastian Hesselbarth List-Id: linux-i2c@vger.kernel.org SGkgUm9tYWluLAogCiBPbiBqZXUuLCBkw6ljLiAwMSAyMDE2LCBSb21haW4gUGVyaWVyIDxyb21h aW4ucGVyaWVyQGZyZWUtZWxlY3Ryb25zLmNvbT4gd3JvdGU6Cgo+IFRoZSBBcm1hZGEgMzcwMCBo YXMgdHdvIGkyYyBidXMgaW50ZXJmYWNlIHVuaXRzLCB0aGlzIGNvbW1pdCBhZGRzIHRoZQo+IGRl ZmluaXRpb25zIG9mIHRoZSBjb3JyZXNwb25kaW5nIGRldmljZSBub2Rlcy4gSXQgYWxzbyBlbmFi bGVzIHRoZSBub2RlCj4gb24gdGhlIGRldmVsb3BtZW50IGJvYXJkIGZvciB0aGlzIFNvQy4KPgo+ IFNpZ25lZC1vZmYtYnk6IFJvbWFpbiBQZXJpZXIgPHJvbWFpbi5wZXJpZXJAZnJlZS1lbGVjdHJv bnMuY29tPgo+IEFja2VkLWJ5OiBHcmVnb3J5IENMRU1FTlQgPGdyZWdvcnkuY2xlbWVudEBmcmVl LWVsZWN0cm9ucy5jb20+CgpBcHBsaWVkIG9uIG12ZWJ1L2R0NjQKClRoYW5rcywKCkdyZWdvcnkK Cj4gLS0tCj4gIGFyY2gvYXJtNjQvYm9vdC9kdHMvbWFydmVsbC9hcm1hZGEtMzcyMC1kYi5kdHMg fCAgNCArKysrCj4gIGFyY2gvYXJtNjQvYm9vdC9kdHMvbWFydmVsbC9hcm1hZGEtMzd4eC5kdHNp ICAgfCAxOCArKysrKysrKysrKysrKysrKysKPiAgMiBmaWxlcyBjaGFuZ2VkLCAyMiBpbnNlcnRp b25zKCspCj4KPiBkaWZmIC0tZ2l0IGEvYXJjaC9hcm02NC9ib290L2R0cy9tYXJ2ZWxsL2FybWFk YS0zNzIwLWRiLmR0cyBiL2FyY2gvYXJtNjQvYm9vdC9kdHMvbWFydmVsbC9hcm1hZGEtMzcyMC1k Yi5kdHMKPiBpbmRleCAxMzcyZTlhNi4uMTZkODRhZiAxMDA2NDQKPiAtLS0gYS9hcmNoL2FybTY0 L2Jvb3QvZHRzL21hcnZlbGwvYXJtYWRhLTM3MjAtZGIuZHRzCj4gKysrIGIvYXJjaC9hcm02NC9i b290L2R0cy9tYXJ2ZWxsL2FybWFkYS0zNzIwLWRiLmR0cwo+IEBAIC02Miw2ICs2MiwxMCBAQAo+ ICAJfTsKPiAgfTsKPiAgCj4gKyZpMmMwIHsKPiArCXN0YXR1cyA9ICJva2F5IjsKPiArfTsKPiAr Cj4gIC8qIENPTjMgKi8KPiAgJnNhdGEgewo+ICAJc3RhdHVzID0gIm9rYXkiOwo+IGRpZmYgLS1n aXQgYS9hcmNoL2FybTY0L2Jvb3QvZHRzL21hcnZlbGwvYXJtYWRhLTM3eHguZHRzaSBiL2FyY2gv YXJtNjQvYm9vdC9kdHMvbWFydmVsbC9hcm1hZGEtMzd4eC5kdHNpCj4gaW5kZXggZTliZDU4Ny4u MWIwZmQyMSAxMDA2NDQKPiAtLS0gYS9hcmNoL2FybTY0L2Jvb3QvZHRzL21hcnZlbGwvYXJtYWRh LTM3eHguZHRzaQo+ICsrKyBiL2FyY2gvYXJtNjQvYm9vdC9kdHMvbWFydmVsbC9hcm1hZGEtMzd4 eC5kdHNpCj4gQEAgLTk4LDYgKzk4LDI0IEBACj4gIAkJCS8qIDMyTSBpbnRlcm5hbCByZWdpc3Rl ciBAIDB4ZDAwMF8wMDAwICovCj4gIAkJCXJhbmdlcyA9IDwweDAgMHgwIDB4ZDAwMDAwMDAgMHgy MDAwMDAwPjsKPiAgCj4gKwkJCWkyYzA6IGkyY0AxMTAwMCB7Cj4gKwkJCQljb21wYXRpYmxlID0g Im1hcnZlbGwsYXJtYWRhLTM3MDAtaTJjIjsKPiArCQkJCXJlZyA9IDwweDExMDAwIDB4MjQ+Owo+ ICsJCQkJY2xvY2tzID0gPCZuYl9wZXJpcGhfY2xrIDEwPjsKPiArCQkJCWludGVycnVwdHMgPSA8 R0lDX1NQSSAxIElSUV9UWVBFX0xFVkVMX0hJR0g+Owo+ICsJCQkJbXJ2bCxpMmMtZmFzdC1tb2Rl Owo+ICsJCQkJc3RhdHVzID0gImRpc2FibGVkIjsKPiArCQkJfTsKPiArCj4gKwkJCWkyYzE6IGky Y0AxMTA4MCB7Cj4gKwkJCQljb21wYXRpYmxlID0gIm1hcnZlbGwsYXJtYWRhLTM3MDAtaTJjIjsK PiArCQkJCXJlZyA9IDwweDExMDgwIDB4MjQ+Owo+ICsJCQkJY2xvY2tzID0gPCZuYl9wZXJpcGhf Y2xrIDk+Owo+ICsJCQkJaW50ZXJydXB0cyA9IDxHSUNfU1BJIDIgSVJRX1RZUEVfTEVWRUxfSElH SD47Cj4gKwkJCQltcnZsLGkyYy1mYXN0LW1vZGU7Cj4gKwkJCQlzdGF0dXMgPSAiZGlzYWJsZWQi Owo+ICsJCQl9Owo+ICsKPiAgCQkJdWFydDA6IHNlcmlhbEAxMjAwMCB7Cj4gIAkJCQljb21wYXRp YmxlID0gIm1hcnZlbGwsYXJtYWRhLTM3MDAtdWFydCI7Cj4gIAkJCQlyZWcgPSA8MHgxMjAwMCAw eDQwMD47Cj4gLS0gCj4gMi45LjMKPgoKLS0gCkdyZWdvcnkgQ2xlbWVudCwgRnJlZSBFbGVjdHJv bnMKS2VybmVsLCBkcml2ZXJzLCByZWFsLXRpbWUgYW5kIGVtYmVkZGVkIExpbnV4CmRldmVsb3Bt ZW50LCBjb25zdWx0aW5nLCB0cmFpbmluZyBhbmQgc3VwcG9ydC4KaHR0cDovL2ZyZWUtZWxlY3Ry b25zLmNvbQoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18K bGludXgtYXJtLWtlcm5lbCBtYWlsaW5nIGxpc3QKbGludXgtYXJtLWtlcm5lbEBsaXN0cy5pbmZy YWRlYWQub3JnCmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8vbGlu dXgtYXJtLWtlcm5lbAo= From mboxrd@z Thu Jan 1 00:00:00 1970 From: gregory.clement@free-electrons.com (Gregory CLEMENT) Date: Tue, 03 Jan 2017 16:20:34 +0100 Subject: [PATCH v7 3/4] arm64: dts: marvell: Add I2C definitions for the Armada 3700 In-Reply-To: <20161201110440.27530-4-romain.perier@free-electrons.com> (Romain Perier's message of "Thu, 1 Dec 2016 12:04:39 +0100") References: <20161201110440.27530-1-romain.perier@free-electrons.com> <20161201110440.27530-4-romain.perier@free-electrons.com> Message-ID: <8760lwgofx.fsf@free-electrons.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Romain, On jeu., d?c. 01 2016, Romain Perier wrote: > The Armada 3700 has two i2c bus interface units, this commit adds the > definitions of the corresponding device nodes. It also enables the node > on the development board for this SoC. > > Signed-off-by: Romain Perier > Acked-by: Gregory CLEMENT Applied on mvebu/dt64 Thanks, Gregory > --- > arch/arm64/boot/dts/marvell/armada-3720-db.dts | 4 ++++ > arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 18 ++++++++++++++++++ > 2 files changed, 22 insertions(+) > > diff --git a/arch/arm64/boot/dts/marvell/armada-3720-db.dts b/arch/arm64/boot/dts/marvell/armada-3720-db.dts > index 1372e9a6..16d84af 100644 > --- a/arch/arm64/boot/dts/marvell/armada-3720-db.dts > +++ b/arch/arm64/boot/dts/marvell/armada-3720-db.dts > @@ -62,6 +62,10 @@ > }; > }; > > +&i2c0 { > + status = "okay"; > +}; > + > /* CON3 */ > &sata { > status = "okay"; > diff --git a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi > index e9bd587..1b0fd21 100644 > --- a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi > +++ b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi > @@ -98,6 +98,24 @@ > /* 32M internal register @ 0xd000_0000 */ > ranges = <0x0 0x0 0xd0000000 0x2000000>; > > + i2c0: i2c at 11000 { > + compatible = "marvell,armada-3700-i2c"; > + reg = <0x11000 0x24>; > + clocks = <&nb_periph_clk 10>; > + interrupts = ; > + mrvl,i2c-fast-mode; > + status = "disabled"; > + }; > + > + i2c1: i2c at 11080 { > + compatible = "marvell,armada-3700-i2c"; > + reg = <0x11080 0x24>; > + clocks = <&nb_periph_clk 9>; > + interrupts = ; > + mrvl,i2c-fast-mode; > + status = "disabled"; > + }; > + > uart0: serial at 12000 { > compatible = "marvell,armada-3700-uart"; > reg = <0x12000 0x400>; > -- > 2.9.3 > -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com