diff mbox

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

Message ID 1458762490-9814-1-git-send-email-srinivas.kandagatla@linaro.org (mailing list archive)
State Superseded, archived
Delegated to: Andy Gross
Headers show

Commit Message

Srinivas Kandagatla March 23, 2016, 7:48 p.m. UTC 
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(+)

Comments

Bjorn Andersson March 29, 2016, 2:44 p.m. UTC | #1 
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
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Srinivas Kandagatla March 29, 2016, 3:02 p.m. UTC | #2 
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
>>
--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

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 {