All of lore.kernel.org
 help / color / mirror / Atom feed
* Re: [PATCH] ARM: apq8064: Add pinmux and i2c pinctrl nodes
@ 2014-08-15 19:08 ` kiran.padwal
  0 siblings, 0 replies; 10+ messages in thread
From: kiran.padwal @ 2014-08-15 19:08 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	Russell King, David Brown, linux-arm-kernel, linux-arm-msm,
	devicetree, linux-kernel


On Thursday, August 14, 2014 6:37pm, "Bjorn Andersson" <bjorn@kryo.se> said:

> On Thu, Aug 14, 2014 at 12:20 AM, Kiran Padwal
> <kiran.padwal@smartplayin.com> wrote:
>> diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi
>> b/arch/arm/boot/dts/qcom-apq8064.dtsi
>> index 92bf793..fbebf5c 100644
>> --- a/arch/arm/boot/dts/qcom-apq8064.dtsi
>> +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
>> @@ -70,6 +70,17 @@
>>                 ranges;
>>                 compatible = "simple-bus";
>>
>> +               qcom_pinmux: pinmux@800000 {
> 
> There are (at least) three different pinmuxes in these platforms: TLMM, PMIC
> GPIO, PMIC MPP. Also this is the phandle that is used to reference the gpio
> chip throughout the board.
> 
> So I would like to suggest that we name it "tlmm" or like in the downstream
> kernel "msmgpio".

ok. How about "qcomgpio" or rename it to msmgpio?

> 
>> +                       compatible = "qcom,apq8064-pinctrl";
>> +                       reg = <0x800000 0x4000>;
>> +
>> +                       gpio-controller;
>> +                       #gpio-cells = <2>;
>> +                       interrupt-controller;
>> +                       #interrupt-cells = <2>;
>> +                       interrupts = <0 32 0x4>;
> 
> I must have gotten this wrong in the dt binding example, sorry about that.
> interrupts should be <0 16 0x4>.

Thanks for information. I will update that.
 
> 
>> +               };
> 
> Regards,
> Bjorn
> 

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] ARM: apq8064: Add pinmux and i2c pinctrl nodes
@ 2014-08-15 19:08 ` kiran.padwal
  0 siblings, 0 replies; 10+ messages in thread
From: kiran.padwal @ 2014-08-15 19:08 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	Russell King, David Brown, linux-arm-kernel, linux-arm-msm,
	devicetree, linux-kernel


On Thursday, August 14, 2014 6:37pm, "Bjorn Andersson" <bjorn@kryo.se> said:

> On Thu, Aug 14, 2014 at 12:20 AM, Kiran Padwal
> <kiran.padwal@smartplayin.com> wrote:
>> diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi
>> b/arch/arm/boot/dts/qcom-apq8064.dtsi
>> index 92bf793..fbebf5c 100644
>> --- a/arch/arm/boot/dts/qcom-apq8064.dtsi
>> +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
>> @@ -70,6 +70,17 @@
>>                 ranges;
>>                 compatible = "simple-bus";
>>
>> +               qcom_pinmux: pinmux@800000 {
> 
> There are (at least) three different pinmuxes in these platforms: TLMM, PMIC
> GPIO, PMIC MPP. Also this is the phandle that is used to reference the gpio
> chip throughout the board.
> 
> So I would like to suggest that we name it "tlmm" or like in the downstream
> kernel "msmgpio".

ok. How about "qcomgpio" or rename it to msmgpio?

> 
>> +                       compatible = "qcom,apq8064-pinctrl";
>> +                       reg = <0x800000 0x4000>;
>> +
>> +                       gpio-controller;
>> +                       #gpio-cells = <2>;
>> +                       interrupt-controller;
>> +                       #interrupt-cells = <2>;
>> +                       interrupts = <0 32 0x4>;
> 
> I must have gotten this wrong in the dt binding example, sorry about that.
> interrupts should be <0 16 0x4>.

Thanks for information. I will update that.
 
> 
>> +               };
> 
> Regards,
> Bjorn
> 




^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] ARM: apq8064: Add pinmux and i2c pinctrl nodes
  2014-08-15 19:08 ` kiran.padwal
  (?)
@ 2014-08-18 15:48   ` kiran.padwal
  -1 siblings, 0 replies; 10+ messages in thread
From: kiran.padwal @ 2014-08-18 15:48 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	Russell King, David Brown, linux-arm-kernel, linux-arm-msm,
	devicetree, linux-kernel


On Friday, August 15, 2014 3:08pm, kiran.padwal@smartplayin.com said:

> 
> On Thursday, August 14, 2014 6:37pm, "Bjorn Andersson" <bjorn@kryo.se> said:
> 
>> On Thu, Aug 14, 2014 at 12:20 AM, Kiran Padwal
>> <kiran.padwal@smartplayin.com> wrote:
>>> diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi
>>> b/arch/arm/boot/dts/qcom-apq8064.dtsi
>>> index 92bf793..fbebf5c 100644
>>> --- a/arch/arm/boot/dts/qcom-apq8064.dtsi
>>> +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
>>> @@ -70,6 +70,17 @@
>>>                 ranges;
>>>                 compatible = "simple-bus";
>>>
>>> +               qcom_pinmux: pinmux@800000 {
>>
>> There are (at least) three different pinmuxes in these platforms: TLMM, PMIC
>> GPIO, PMIC MPP. Also this is the phandle that is used to reference the gpio
>> chip throughout the board.
>>
>> So I would like to suggest that we name it "tlmm" or like in the downstream
>> kernel "msmgpio".
> 
> ok. How about "qcomgpio" or rename it to msmgpio?

Thanks for suggestion, I will rename it to "tlmm".

> 
>>
>>> +                       compatible = "qcom,apq8064-pinctrl";
>>> +                       reg = <0x800000 0x4000>;
>>> +
>>> +                       gpio-controller;
>>> +                       #gpio-cells = <2>;
>>> +                       interrupt-controller;
>>> +                       #interrupt-cells = <2>;
>>> +                       interrupts = <0 32 0x4>;
>>
>> I must have gotten this wrong in the dt binding example, sorry about that.
>> interrupts should be <0 16 0x4>.
> 
> Thanks for information. I will update that.
> 
>>
>>> +               };
>>
>> Regards,
>> Bjorn
>>
> 
> 
> 

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] ARM: apq8064: Add pinmux and i2c pinctrl nodes
@ 2014-08-18 15:48   ` kiran.padwal
  0 siblings, 0 replies; 10+ messages in thread
From: kiran.padwal @ 2014-08-18 15:48 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	Russell King, David Brown, linux-arm-kernel, linux-arm-msm,
	devicetree, linux-kernel


On Friday, August 15, 2014 3:08pm, kiran.padwal@smartplayin.com said:

> 
> On Thursday, August 14, 2014 6:37pm, "Bjorn Andersson" <bjorn@kryo.se> said:
> 
>> On Thu, Aug 14, 2014 at 12:20 AM, Kiran Padwal
>> <kiran.padwal@smartplayin.com> wrote:
>>> diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi
>>> b/arch/arm/boot/dts/qcom-apq8064.dtsi
>>> index 92bf793..fbebf5c 100644
>>> --- a/arch/arm/boot/dts/qcom-apq8064.dtsi
>>> +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
>>> @@ -70,6 +70,17 @@
>>>                 ranges;
>>>                 compatible = "simple-bus";
>>>
>>> +               qcom_pinmux: pinmux@800000 {
>>
>> There are (at least) three different pinmuxes in these platforms: TLMM, PMIC
>> GPIO, PMIC MPP. Also this is the phandle that is used to reference the gpio
>> chip throughout the board.
>>
>> So I would like to suggest that we name it "tlmm" or like in the downstream
>> kernel "msmgpio".
> 
> ok. How about "qcomgpio" or rename it to msmgpio?

Thanks for suggestion, I will rename it to "tlmm".

> 
>>
>>> +                       compatible = "qcom,apq8064-pinctrl";
>>> +                       reg = <0x800000 0x4000>;
>>> +
>>> +                       gpio-controller;
>>> +                       #gpio-cells = <2>;
>>> +                       interrupt-controller;
>>> +                       #interrupt-cells = <2>;
>>> +                       interrupts = <0 32 0x4>;
>>
>> I must have gotten this wrong in the dt binding example, sorry about that.
>> interrupts should be <0 16 0x4>.
> 
> Thanks for information. I will update that.
> 
>>
>>> +               };
>>
>> Regards,
>> Bjorn
>>
> 
> 
> 



^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH] ARM: apq8064: Add pinmux and i2c pinctrl nodes
@ 2014-08-18 15:48   ` kiran.padwal
  0 siblings, 0 replies; 10+ messages in thread
From: kiran.padwal at smartplayin.com @ 2014-08-18 15:48 UTC (permalink / raw)
  To: linux-arm-kernel


On Friday, August 15, 2014 3:08pm, kiran.padwal at smartplayin.com said:

> 
> On Thursday, August 14, 2014 6:37pm, "Bjorn Andersson" <bjorn@kryo.se> said:
> 
>> On Thu, Aug 14, 2014 at 12:20 AM, Kiran Padwal
>> <kiran.padwal@smartplayin.com> wrote:
>>> diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi
>>> b/arch/arm/boot/dts/qcom-apq8064.dtsi
>>> index 92bf793..fbebf5c 100644
>>> --- a/arch/arm/boot/dts/qcom-apq8064.dtsi
>>> +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
>>> @@ -70,6 +70,17 @@
>>>                 ranges;
>>>                 compatible = "simple-bus";
>>>
>>> +               qcom_pinmux: pinmux at 800000 {
>>
>> There are (at least) three different pinmuxes in these platforms: TLMM, PMIC
>> GPIO, PMIC MPP. Also this is the phandle that is used to reference the gpio
>> chip throughout the board.
>>
>> So I would like to suggest that we name it "tlmm" or like in the downstream
>> kernel "msmgpio".
> 
> ok. How about "qcomgpio" or rename it to msmgpio?

Thanks for suggestion, I will rename it to "tlmm".

> 
>>
>>> +                       compatible = "qcom,apq8064-pinctrl";
>>> +                       reg = <0x800000 0x4000>;
>>> +
>>> +                       gpio-controller;
>>> +                       #gpio-cells = <2>;
>>> +                       interrupt-controller;
>>> +                       #interrupt-cells = <2>;
>>> +                       interrupts = <0 32 0x4>;
>>
>> I must have gotten this wrong in the dt binding example, sorry about that.
>> interrupts should be <0 16 0x4>.
> 
> Thanks for information. I will update that.
> 
>>
>>> +               };
>>
>> Regards,
>> Bjorn
>>
> 
> 
> 

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] ARM: apq8064: Add pinmux and i2c pinctrl nodes
  2014-08-14  7:20 ` Kiran Padwal
  (?)
@ 2014-08-14 22:37   ` Bjorn Andersson
  -1 siblings, 0 replies; 10+ messages in thread
From: Bjorn Andersson @ 2014-08-14 22:37 UTC (permalink / raw)
  To: Kiran Padwal
  Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	Russell King, David Brown, linux-arm-kernel, linux-arm-msm,
	devicetree, linux-kernel

On Thu, Aug 14, 2014 at 12:20 AM, Kiran Padwal
<kiran.padwal@smartplayin.com> wrote:
> diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi
> index 92bf793..fbebf5c 100644
> --- a/arch/arm/boot/dts/qcom-apq8064.dtsi
> +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
> @@ -70,6 +70,17 @@
>                 ranges;
>                 compatible = "simple-bus";
>
> +               qcom_pinmux: pinmux@800000 {

There are (at least) three different pinmuxes in these platforms: TLMM, PMIC
GPIO, PMIC MPP. Also this is the phandle that is used to reference the gpio
chip throughout the board.

So I would like to suggest that we name it "tlmm" or like in the downstream
kernel "msmgpio".

> +                       compatible = "qcom,apq8064-pinctrl";
> +                       reg = <0x800000 0x4000>;
> +
> +                       gpio-controller;
> +                       #gpio-cells = <2>;
> +                       interrupt-controller;
> +                       #interrupt-cells = <2>;
> +                       interrupts = <0 32 0x4>;

I must have gotten this wrong in the dt binding example, sorry about that.
interrupts should be <0 16 0x4>.

> +               };

Regards,
Bjorn

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] ARM: apq8064: Add pinmux and i2c pinctrl nodes
@ 2014-08-14 22:37   ` Bjorn Andersson
  0 siblings, 0 replies; 10+ messages in thread
From: Bjorn Andersson @ 2014-08-14 22:37 UTC (permalink / raw)
  To: Kiran Padwal
  Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	Russell King, David Brown, linux-arm-kernel, linux-arm-msm,
	devicetree, linux-kernel

On Thu, Aug 14, 2014 at 12:20 AM, Kiran Padwal
<kiran.padwal@smartplayin.com> wrote:
> diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi
> index 92bf793..fbebf5c 100644
> --- a/arch/arm/boot/dts/qcom-apq8064.dtsi
> +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
> @@ -70,6 +70,17 @@
>                 ranges;
>                 compatible = "simple-bus";
>
> +               qcom_pinmux: pinmux@800000 {

There are (at least) three different pinmuxes in these platforms: TLMM, PMIC
GPIO, PMIC MPP. Also this is the phandle that is used to reference the gpio
chip throughout the board.

So I would like to suggest that we name it "tlmm" or like in the downstream
kernel "msmgpio".

> +                       compatible = "qcom,apq8064-pinctrl";
> +                       reg = <0x800000 0x4000>;
> +
> +                       gpio-controller;
> +                       #gpio-cells = <2>;
> +                       interrupt-controller;
> +                       #interrupt-cells = <2>;
> +                       interrupts = <0 32 0x4>;

I must have gotten this wrong in the dt binding example, sorry about that.
interrupts should be <0 16 0x4>.

> +               };

Regards,
Bjorn

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH] ARM: apq8064: Add pinmux and i2c pinctrl nodes
@ 2014-08-14 22:37   ` Bjorn Andersson
  0 siblings, 0 replies; 10+ messages in thread
From: Bjorn Andersson @ 2014-08-14 22:37 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, Aug 14, 2014 at 12:20 AM, Kiran Padwal
<kiran.padwal@smartplayin.com> wrote:
> diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi
> index 92bf793..fbebf5c 100644
> --- a/arch/arm/boot/dts/qcom-apq8064.dtsi
> +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
> @@ -70,6 +70,17 @@
>                 ranges;
>                 compatible = "simple-bus";
>
> +               qcom_pinmux: pinmux at 800000 {

There are (at least) three different pinmuxes in these platforms: TLMM, PMIC
GPIO, PMIC MPP. Also this is the phandle that is used to reference the gpio
chip throughout the board.

So I would like to suggest that we name it "tlmm" or like in the downstream
kernel "msmgpio".

> +                       compatible = "qcom,apq8064-pinctrl";
> +                       reg = <0x800000 0x4000>;
> +
> +                       gpio-controller;
> +                       #gpio-cells = <2>;
> +                       interrupt-controller;
> +                       #interrupt-cells = <2>;
> +                       interrupts = <0 32 0x4>;

I must have gotten this wrong in the dt binding example, sorry about that.
interrupts should be <0 16 0x4>.

> +               };

Regards,
Bjorn

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH] ARM: apq8064: Add pinmux and i2c pinctrl nodes
@ 2014-08-14  7:20 ` Kiran Padwal
  0 siblings, 0 replies; 10+ messages in thread
From: Kiran Padwal @ 2014-08-14  7:20 UTC (permalink / raw)
  To: robh+dt, pawel.moll, mark.rutland, ijc+devicetree, galak, linux,
	davidb, linux-arm-kernel, linux-arm-msm
  Cc: devicetree, linux-kernel, Kiran Padwal

This patch adds pinmux and i2c pinctrl DT node for IFC6410 board.
It also adds necessary DT support for i2c eeprom which is present on
IFC6410.

Tested on IFC6410 board.

Signed-off-by: Kiran Padwal <kiran.padwal@smartplayin.com>
---
 arch/arm/boot/dts/qcom-apq8064-ifc6410.dts |   29 ++++++++++++++
 arch/arm/boot/dts/qcom-apq8064.dtsi        |   59 ++++++++++++++++++++++++++++
 2 files changed, 88 insertions(+)

diff --git a/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts b/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
index 7c2441d..d52ac3c 100644
--- a/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
+++ b/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
@@ -5,6 +5,15 @@
 	compatible = "qcom,apq8064-ifc6410", "qcom,apq8064";
 
 	soc {
+		pinmux@800000 {
+			i2c1_pins: i2c1_pinmux {
+				mux {
+					pins = "gpio20", "gpio21";
+					function = "gsbi1";
+				};
+			};
+		};
+
 		gsbi@16600000 {
 			status = "ok";
 			qcom,mode = <GSBI_PROT_I2C_UART>;
@@ -12,5 +21,25 @@
 				status = "ok";
 			};
 		};
+
+		gsbi1: gsbi@12440000 {
+			qcom,mode = <GSBI_PROT_I2C>;
+			status = "ok";
+
+			i2c1: i2c@12460000 {
+				status = "ok";
+
+				clock-frequency = <200000>;
+
+				pinctrl-0 = <&i2c1_pins>;
+				pinctrl-names = "default";
+
+				eeprom: eeprom@52 {
+					compatible = "atmel,24c128";
+					reg = <0x52>;
+					pagesize = <32>;
+				};
+			};
+		};
 	};
 };
diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi
index 92bf793..fbebf5c 100644
--- a/arch/arm/boot/dts/qcom-apq8064.dtsi
+++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
@@ -70,6 +70,17 @@
 		ranges;
 		compatible = "simple-bus";
 
+		qcom_pinmux: pinmux@800000 {
+			compatible = "qcom,apq8064-pinctrl";
+			reg = <0x800000 0x4000>;
+
+			gpio-controller;
+			#gpio-cells = <2>;
+			interrupt-controller;
+			#interrupt-cells = <2>;
+			interrupts = <0 32 0x4>;
+		};
+
 		intc: interrupt-controller@2000000 {
 			compatible = "qcom,msm-qgic2";
 			interrupt-controller;
@@ -133,6 +144,54 @@
 			regulator;
 		};
 
+		gsbi1: gsbi@12440000 {
+			compatible = "qcom,gsbi-v1.0.0";
+			reg = <0x12440000 0x100>;
+			clocks = <&gcc GSBI1_H_CLK>;
+			clock-names = "iface";
+			#address-cells = <1>;
+			#size-cells = <1>;
+			ranges;
+			status = "disabled";
+
+			i2c@12460000 {
+				compatible = "qcom,i2c-qup-v1.1.1";
+				reg = <0x12460000 0x1000>;
+				interrupts = <0 194 0>;
+
+				clocks = <&gcc GSBI1_QUP_CLK>, <&gcc GSBI1_H_CLK>;
+				clock-names = "core", "iface";
+				status = "disabled";
+
+				#address-cells = <1>;
+				#size-cells = <0>;
+			};
+		};
+
+		gsbi2: gsbi@12480000 {
+			compatible = "qcom,gsbi-v1.0.0";
+			reg = <0x12480000 0x100>;
+			clocks = <&gcc GSBI2_H_CLK>;
+			clock-names = "iface";
+			#address-cells = <1>;
+			#size-cells = <1>;
+			ranges;
+			status = "disabled";
+
+			i2c@124a0000 {
+				compatible = "qcom,i2c-qup-v1.1.1";
+				reg = <0x124a0000 0x1000>;
+				interrupts = <0 196 0>;
+
+				clocks = <&gcc GSBI2_QUP_CLK>, <&gcc GSBI2_H_CLK>;
+				clock-names = "core", "iface";
+				status = "disabled";
+
+				#address-cells = <1>;
+				#size-cells = <0>;
+			};
+		};
+
 		gsbi7: gsbi@16600000 {
 			status = "disabled";
 			compatible = "qcom,gsbi-v1.0.0";
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [PATCH] ARM: apq8064: Add pinmux and i2c pinctrl nodes
@ 2014-08-14  7:20 ` Kiran Padwal
  0 siblings, 0 replies; 10+ messages in thread
From: Kiran Padwal @ 2014-08-14  7:20 UTC (permalink / raw)
  To: linux-arm-kernel

This patch adds pinmux and i2c pinctrl DT node for IFC6410 board.
It also adds necessary DT support for i2c eeprom which is present on
IFC6410.

Tested on IFC6410 board.

Signed-off-by: Kiran Padwal <kiran.padwal@smartplayin.com>
---
 arch/arm/boot/dts/qcom-apq8064-ifc6410.dts |   29 ++++++++++++++
 arch/arm/boot/dts/qcom-apq8064.dtsi        |   59 ++++++++++++++++++++++++++++
 2 files changed, 88 insertions(+)

diff --git a/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts b/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
index 7c2441d..d52ac3c 100644
--- a/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
+++ b/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
@@ -5,6 +5,15 @@
 	compatible = "qcom,apq8064-ifc6410", "qcom,apq8064";
 
 	soc {
+		pinmux at 800000 {
+			i2c1_pins: i2c1_pinmux {
+				mux {
+					pins = "gpio20", "gpio21";
+					function = "gsbi1";
+				};
+			};
+		};
+
 		gsbi at 16600000 {
 			status = "ok";
 			qcom,mode = <GSBI_PROT_I2C_UART>;
@@ -12,5 +21,25 @@
 				status = "ok";
 			};
 		};
+
+		gsbi1: gsbi at 12440000 {
+			qcom,mode = <GSBI_PROT_I2C>;
+			status = "ok";
+
+			i2c1: i2c at 12460000 {
+				status = "ok";
+
+				clock-frequency = <200000>;
+
+				pinctrl-0 = <&i2c1_pins>;
+				pinctrl-names = "default";
+
+				eeprom: eeprom at 52 {
+					compatible = "atmel,24c128";
+					reg = <0x52>;
+					pagesize = <32>;
+				};
+			};
+		};
 	};
 };
diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi
index 92bf793..fbebf5c 100644
--- a/arch/arm/boot/dts/qcom-apq8064.dtsi
+++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
@@ -70,6 +70,17 @@
 		ranges;
 		compatible = "simple-bus";
 
+		qcom_pinmux: pinmux at 800000 {
+			compatible = "qcom,apq8064-pinctrl";
+			reg = <0x800000 0x4000>;
+
+			gpio-controller;
+			#gpio-cells = <2>;
+			interrupt-controller;
+			#interrupt-cells = <2>;
+			interrupts = <0 32 0x4>;
+		};
+
 		intc: interrupt-controller at 2000000 {
 			compatible = "qcom,msm-qgic2";
 			interrupt-controller;
@@ -133,6 +144,54 @@
 			regulator;
 		};
 
+		gsbi1: gsbi at 12440000 {
+			compatible = "qcom,gsbi-v1.0.0";
+			reg = <0x12440000 0x100>;
+			clocks = <&gcc GSBI1_H_CLK>;
+			clock-names = "iface";
+			#address-cells = <1>;
+			#size-cells = <1>;
+			ranges;
+			status = "disabled";
+
+			i2c at 12460000 {
+				compatible = "qcom,i2c-qup-v1.1.1";
+				reg = <0x12460000 0x1000>;
+				interrupts = <0 194 0>;
+
+				clocks = <&gcc GSBI1_QUP_CLK>, <&gcc GSBI1_H_CLK>;
+				clock-names = "core", "iface";
+				status = "disabled";
+
+				#address-cells = <1>;
+				#size-cells = <0>;
+			};
+		};
+
+		gsbi2: gsbi at 12480000 {
+			compatible = "qcom,gsbi-v1.0.0";
+			reg = <0x12480000 0x100>;
+			clocks = <&gcc GSBI2_H_CLK>;
+			clock-names = "iface";
+			#address-cells = <1>;
+			#size-cells = <1>;
+			ranges;
+			status = "disabled";
+
+			i2c at 124a0000 {
+				compatible = "qcom,i2c-qup-v1.1.1";
+				reg = <0x124a0000 0x1000>;
+				interrupts = <0 196 0>;
+
+				clocks = <&gcc GSBI2_QUP_CLK>, <&gcc GSBI2_H_CLK>;
+				clock-names = "core", "iface";
+				status = "disabled";
+
+				#address-cells = <1>;
+				#size-cells = <0>;
+			};
+		};
+
 		gsbi7: gsbi at 16600000 {
 			status = "disabled";
 			compatible = "qcom,gsbi-v1.0.0";
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2014-08-18 15:48 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-15 19:08 [PATCH] ARM: apq8064: Add pinmux and i2c pinctrl nodes kiran.padwal
2014-08-15 19:08 ` kiran.padwal
2014-08-18 15:48 ` kiran.padwal
2014-08-18 15:48   ` kiran.padwal at smartplayin.com
2014-08-18 15:48   ` kiran.padwal
  -- strict thread matches above, loose matches on Subject: below --
2014-08-14  7:20 Kiran Padwal
2014-08-14  7:20 ` Kiran Padwal
2014-08-14 22:37 ` Bjorn Andersson
2014-08-14 22:37   ` Bjorn Andersson
2014-08-14 22:37   ` Bjorn Andersson

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.