Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1363632 > unrolled thread

[PATCH 11/12] ARM: dts: dragonboard-600c: add i2c support

Started bySrinivas Kandagatla <srinivas.kandagatla@linaro.org>
First post2016-03-23 20:50 +0100
Last post2016-03-29 17:10 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [PATCH 11/12] ARM: dts: dragonboard-600c: add i2c support Srinivas Kandagatla <srinivas.kandagatla@linaro.org> - 2016-03-23 20:50 +0100
    Re: [PATCH 11/12] ARM: dts: dragonboard-600c: add i2c support Bjorn Andersson <bjorn.andersson@linaro.org> - 2016-03-29 16:50 +0200
      Re: [PATCH 11/12] ARM: dts: dragonboard-600c: add i2c support Srinivas Kandagatla <srinivas.kandagatla@linaro.org> - 2016-03-29 17:10 +0200

#1363632 — [PATCH 11/12] ARM: dts: dragonboard-600c: add i2c support

FromSrinivas Kandagatla <srinivas.kandagatla@linaro.org>
Date2016-03-23 20:50 +0100
Subject[PATCH 11/12] ARM: dts: dragonboard-600c: add i2c support
Message-ID<rfWWZ-8bL-1@gated-at.bofh.it>
This patch adds nodes required to enable 4 i2c buses on the board which
are connected to various sensors and eeprom.

Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
---
 .../arm/boot/dts/qcom-apq8064-dragonboard-600c.dts | 46 ++++++++++++++++++++++
 1 file changed, 46 insertions(+)

diff --git a/arch/arm/boot/dts/qcom-apq8064-dragonboard-600c.dts b/arch/arm/boot/dts/qcom-apq8064-dragonboard-600c.dts
index 4fecd94..942de04 100644
--- a/arch/arm/boot/dts/qcom-apq8064-dragonboard-600c.dts
+++ b/arch/arm/boot/dts/qcom-apq8064-dragonboard-600c.dts
@@ -8,6 +8,10 @@
 	aliases {
 		serial0 = &gsbi7_serial;
 		serial1 = &gsbi1_serial;
+		i2c0 = &gsbi2_i2c;
+		i2c1 = &gsbi3_i2c;
+		i2c2 = &gsbi4_i2c;
+		i2c3 = &gsbi7_i2c;
 	};
 
 	regulators {
@@ -180,6 +184,42 @@
 			};
 		};
 
+		gsbi@12480000 {
+			status = "okay";
+			qcom,mode = <GSBI_PROT_I2C>;
+			i2c@124a0000 {
+				/* On Low speed expansion and Sensors */
+				label = "LS-I2C0";
+				status = "okay";
+			};
+		};
+
+		gsbi@16200000 {
+			status = "okay";
+			qcom,mode = <GSBI_PROT_I2C>;
+			i2c@16280000 {
+			/* On Low speed expansion */
+				status = "okay";
+				label = "LS-I2C1";
+				clock-frequency = <200000>;
+				eeprom@52 {
+					compatible = "atmel,24c128";
+					reg = <0x52>;
+					pagesize = <64>;
+				};
+			};
+		};
+
+		gsbi@16300000 {
+			status = "okay";
+			qcom,mode = <GSBI_PROT_I2C>;
+			i2c@16380000 {
+				/* On High speed expansion */
+				label = "HS-CAM-I2C3";
+				status = "okay";
+			};
+		};
+
 		/* DEBUG UART  */
 		gsbi@16600000 {
 			status = "okay";
@@ -190,6 +230,12 @@
 				pinctrl-names = "default";
 				pinctrl-0 = <&gsbi7_uart_2pins>;
 			};
+
+			i2c@16680000 {
+				/* On High speed expansion */
+				status = "okay";
+				label = "HS-CAM-I2C2";
+			};
 		};
 
 		leds {
-- 
2.5.0

[toc] | [next] | [standalone]


#1366351

FromBjorn Andersson <bjorn.andersson@linaro.org>
Date2016-03-29 16:50 +0200
Message-ID<ri37X-7Fn-3@gated-at.bofh.it>
In reply to#1363632
On Wed 23 Mar 12:48 PDT 2016, Srinivas Kandagatla wrote:

> This patch adds nodes required to enable 4 i2c buses on the board which
> are connected to various sensors and eeprom.
> 

I would like for the pinconf for these blocks to be in this patch (and
file) to make it clear how they are configured and where to change them.

Regards,
Bjorn

> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
> ---
>  .../arm/boot/dts/qcom-apq8064-dragonboard-600c.dts | 46 ++++++++++++++++++++++
>  1 file changed, 46 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/qcom-apq8064-dragonboard-600c.dts b/arch/arm/boot/dts/qcom-apq8064-dragonboard-600c.dts
> index 4fecd94..942de04 100644
> --- a/arch/arm/boot/dts/qcom-apq8064-dragonboard-600c.dts
> +++ b/arch/arm/boot/dts/qcom-apq8064-dragonboard-600c.dts
> @@ -8,6 +8,10 @@
>  	aliases {
>  		serial0 = &gsbi7_serial;
>  		serial1 = &gsbi1_serial;
> +		i2c0 = &gsbi2_i2c;
> +		i2c1 = &gsbi3_i2c;
> +		i2c2 = &gsbi4_i2c;
> +		i2c3 = &gsbi7_i2c;
>  	};
>  
>  	regulators {
> @@ -180,6 +184,42 @@
>  			};
>  		};
>  
> +		gsbi@12480000 {
> +			status = "okay";
> +			qcom,mode = <GSBI_PROT_I2C>;
> +			i2c@124a0000 {
> +				/* On Low speed expansion and Sensors */
> +				label = "LS-I2C0";
> +				status = "okay";
> +			};
> +		};
> +
> +		gsbi@16200000 {
> +			status = "okay";
> +			qcom,mode = <GSBI_PROT_I2C>;
> +			i2c@16280000 {
> +			/* On Low speed expansion */
> +				status = "okay";
> +				label = "LS-I2C1";
> +				clock-frequency = <200000>;
> +				eeprom@52 {
> +					compatible = "atmel,24c128";
> +					reg = <0x52>;
> +					pagesize = <64>;
> +				};
> +			};
> +		};
> +
> +		gsbi@16300000 {
> +			status = "okay";
> +			qcom,mode = <GSBI_PROT_I2C>;
> +			i2c@16380000 {
> +				/* On High speed expansion */
> +				label = "HS-CAM-I2C3";
> +				status = "okay";
> +			};
> +		};
> +
>  		/* DEBUG UART  */
>  		gsbi@16600000 {
>  			status = "okay";
> @@ -190,6 +230,12 @@
>  				pinctrl-names = "default";
>  				pinctrl-0 = <&gsbi7_uart_2pins>;
>  			};
> +
> +			i2c@16680000 {
> +				/* On High speed expansion */
> +				status = "okay";
> +				label = "HS-CAM-I2C2";
> +			};
>  		};
>  
>  		leds {
> -- 
> 2.5.0
> 

[toc] | [prev] | [next] | [standalone]


#1366370

FromSrinivas Kandagatla <srinivas.kandagatla@linaro.org>
Date2016-03-29 17:10 +0200
Message-ID<ri3rl-82W-25@gated-at.bofh.it>
In reply to#1366351

On 29/03/16 15:44, Bjorn Andersson wrote:
> On Wed 23 Mar 12:48 PDT 2016, Srinivas Kandagatla wrote:
>
>> This patch adds nodes required to enable 4 i2c buses on the board which
>> are connected to various sensors and eeprom.
>>
>
> I would like for the pinconf for these blocks to be in this patch (and
> file) to make it clear how they are configured and where to change them.
>
When I started adding this I had same opinion, I will do that in next 
release.

thanks,
srini
> Regards,
> Bjorn
>
>> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
>> ---
>>   .../arm/boot/dts/qcom-apq8064-dragonboard-600c.dts | 46 ++++++++++++++++++++++
>>   1 file changed, 46 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/qcom-apq8064-dragonboard-600c.dts b/arch/arm/boot/dts/qcom-apq8064-dragonboard-600c.dts
>> index 4fecd94..942de04 100644
>> --- a/arch/arm/boot/dts/qcom-apq8064-dragonboard-600c.dts
>> +++ b/arch/arm/boot/dts/qcom-apq8064-dragonboard-600c.dts
>> @@ -8,6 +8,10 @@
>>   	aliases {
>>   		serial0 = &gsbi7_serial;
>>   		serial1 = &gsbi1_serial;
>> +		i2c0 = &gsbi2_i2c;
>> +		i2c1 = &gsbi3_i2c;
>> +		i2c2 = &gsbi4_i2c;
>> +		i2c3 = &gsbi7_i2c;
>>   	};
>>
>>   	regulators {
>> @@ -180,6 +184,42 @@
>>   			};
>>   		};
>>
>> +		gsbi@12480000 {
>> +			status = "okay";
>> +			qcom,mode = <GSBI_PROT_I2C>;
>> +			i2c@124a0000 {
>> +				/* On Low speed expansion and Sensors */
>> +				label = "LS-I2C0";
>> +				status = "okay";
>> +			};
>> +		};
>> +
>> +		gsbi@16200000 {
>> +			status = "okay";
>> +			qcom,mode = <GSBI_PROT_I2C>;
>> +			i2c@16280000 {
>> +			/* On Low speed expansion */
>> +				status = "okay";
>> +				label = "LS-I2C1";
>> +				clock-frequency = <200000>;
>> +				eeprom@52 {
>> +					compatible = "atmel,24c128";
>> +					reg = <0x52>;
>> +					pagesize = <64>;
>> +				};
>> +			};
>> +		};
>> +
>> +		gsbi@16300000 {
>> +			status = "okay";
>> +			qcom,mode = <GSBI_PROT_I2C>;
>> +			i2c@16380000 {
>> +				/* On High speed expansion */
>> +				label = "HS-CAM-I2C3";
>> +				status = "okay";
>> +			};
>> +		};
>> +
>>   		/* DEBUG UART  */
>>   		gsbi@16600000 {
>>   			status = "okay";
>> @@ -190,6 +230,12 @@
>>   				pinctrl-names = "default";
>>   				pinctrl-0 = <&gsbi7_uart_2pins>;
>>   			};
>> +
>> +			i2c@16680000 {
>> +				/* On High speed expansion */
>> +				status = "okay";
>> +				label = "HS-CAM-I2C2";
>> +			};
>>   		};
>>
>>   		leds {
>> --
>> 2.5.0
>>

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web