Re: [PATCH] ARM: dts: qcom: Update msm8960 device trees

2014-05-28 Thread Kumar Gala

On May 28, 2014, at 3:09 PM, Josh Cartwright  wrote:

> On Wed, May 28, 2014 at 01:27:23PM -0500, Kumar Gala wrote:
>> * Move SoC peripherals into an SoC container node
>> * Move serial enabling into board file (qcom-msm8960-cdp.dts)
>> * Cleanup cpu node to match binding spec, enable-method and compatible
>>  should be per cpu, not part of the container
>> * Drop interrupts property from l2-cache node as its not part of the
>>  binding spec
>> 
>> Signed-off-by: Kumar Gala 
>> ---
>> arch/arm/boot/dts/qcom-msm8960-cdp.dts |   6 ++
>> arch/arm/boot/dts/qcom-msm8960.dtsi| 165 
>> +
>> 2 files changed, 93 insertions(+), 78 deletions(-)
>> 
>> diff --git a/arch/arm/boot/dts/qcom-msm8960-cdp.dts 
>> b/arch/arm/boot/dts/qcom-msm8960-cdp.dts
>> index a58fb88..8e77ed7 100644
>> --- a/arch/arm/boot/dts/qcom-msm8960-cdp.dts
>> +++ b/arch/arm/boot/dts/qcom-msm8960-cdp.dts
>> @@ -3,4 +3,10 @@
>> / {
>>  model = "Qualcomm MSM8960 CDP";
>>  compatible = "qcom,msm8960-cdp", "qcom,msm8960";
>> +
>> +soc {
>> +serial@1644 {
>> +status = "ok";
>> +};
>> +};
>> };
> 
> Is now the time put these serial nodes under a GSBI parent node?

Yeah, I’ll make the change to the 8960 & 8660 dts

- k

-- 
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by 
The Linux Foundation

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] ARM: dts: qcom: Update msm8960 device trees

2014-05-28 Thread Josh Cartwright
On Wed, May 28, 2014 at 01:27:23PM -0500, Kumar Gala wrote:
> * Move SoC peripherals into an SoC container node
> * Move serial enabling into board file (qcom-msm8960-cdp.dts)
> * Cleanup cpu node to match binding spec, enable-method and compatible
>   should be per cpu, not part of the container
> * Drop interrupts property from l2-cache node as its not part of the
>   binding spec
> 
> Signed-off-by: Kumar Gala 
> ---
>  arch/arm/boot/dts/qcom-msm8960-cdp.dts |   6 ++
>  arch/arm/boot/dts/qcom-msm8960.dtsi| 165 
> +
>  2 files changed, 93 insertions(+), 78 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/qcom-msm8960-cdp.dts 
> b/arch/arm/boot/dts/qcom-msm8960-cdp.dts
> index a58fb88..8e77ed7 100644
> --- a/arch/arm/boot/dts/qcom-msm8960-cdp.dts
> +++ b/arch/arm/boot/dts/qcom-msm8960-cdp.dts
> @@ -3,4 +3,10 @@
>  / {
>   model = "Qualcomm MSM8960 CDP";
>   compatible = "qcom,msm8960-cdp", "qcom,msm8960";
> +
> + soc {
> + serial@1644 {
> + status = "ok";
> + };
> + };
>  };

Is now the time put these serial nodes under a GSBI parent node?

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
hosted by The Linux Foundation
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] ARM: dts: qcom: Update msm8960 device trees

2014-05-28 Thread Kumar Gala
* Move SoC peripherals into an SoC container node
* Move serial enabling into board file (qcom-msm8960-cdp.dts)
* Cleanup cpu node to match binding spec, enable-method and compatible
  should be per cpu, not part of the container
* Drop interrupts property from l2-cache node as its not part of the
  binding spec

Signed-off-by: Kumar Gala 
---
 arch/arm/boot/dts/qcom-msm8960-cdp.dts |   6 ++
 arch/arm/boot/dts/qcom-msm8960.dtsi| 165 +
 2 files changed, 93 insertions(+), 78 deletions(-)

diff --git a/arch/arm/boot/dts/qcom-msm8960-cdp.dts 
b/arch/arm/boot/dts/qcom-msm8960-cdp.dts
index a58fb88..8e77ed7 100644
--- a/arch/arm/boot/dts/qcom-msm8960-cdp.dts
+++ b/arch/arm/boot/dts/qcom-msm8960-cdp.dts
@@ -3,4 +3,10 @@
 / {
model = "Qualcomm MSM8960 CDP";
compatible = "qcom,msm8960-cdp", "qcom,msm8960";
+
+   soc {
+   serial@1644 {
+   status = "ok";
+   };
+   };
 };
diff --git a/arch/arm/boot/dts/qcom-msm8960.dtsi 
b/arch/arm/boot/dts/qcom-msm8960.dtsi
index 997b7b9..c38e54c 100644
--- a/arch/arm/boot/dts/qcom-msm8960.dtsi
+++ b/arch/arm/boot/dts/qcom-msm8960.dtsi
@@ -13,10 +13,10 @@
#address-cells = <1>;
#size-cells = <0>;
interrupts = <1 14 0x304>;
-   compatible = "qcom,krait";
-   enable-method = "qcom,kpss-acc-v1";
 
cpu@0 {
+   compatible = "qcom,krait";
+   enable-method = "qcom,kpss-acc-v1";
device_type = "cpu";
reg = <0>;
next-level-cache = <>;
@@ -25,6 +25,8 @@
};
 
cpu@1 {
+   compatible = "qcom,krait";
+   enable-method = "qcom,kpss-acc-v1";
device_type = "cpu";
reg = <1>;
next-level-cache = <>;
@@ -35,7 +37,6 @@
L2: l2-cache {
compatible = "cache";
cache-level = <2>;
-   interrupts = <0 2 0x4>;
};
};
 
@@ -45,91 +46,99 @@
qcom,no-pc-write;
};
 
-   intc: interrupt-controller@200 {
-   compatible = "qcom,msm-qgic2";
-   interrupt-controller;
-   #interrupt-cells = <3>;
-   reg = < 0x0200 0x1000 >,
- < 0x02002000 0x1000 >;
-   };
+   soc: soc {
+   #address-cells = <1>;
+   #size-cells = <1>;
+   ranges;
+   compatible = "simple-bus";
+
+   intc: interrupt-controller@200 {
+   compatible = "qcom,msm-qgic2";
+   interrupt-controller;
+   #interrupt-cells = <3>;
+   reg = <0x0200 0x1000>,
+ <0x02002000 0x1000>;
+   };
 
-   timer@200a000 {
-   compatible = "qcom,kpss-timer", "qcom,msm-timer";
-   interrupts = <1 1 0x301>,
-<1 2 0x301>,
-<1 3 0x301>;
-   reg = <0x0200a000 0x100>;
-   clock-frequency = <2700>,
- <32768>;
-   cpu-offset = <0x8>;
-   };
+   timer@200a000 {
+   compatible = "qcom,kpss-timer", "qcom,msm-timer";
+   interrupts = <1 1 0x301>,
+<1 2 0x301>,
+<1 3 0x301>;
+   reg = <0x0200a000 0x100>;
+   clock-frequency = <2700>,
+ <32768>;
+   cpu-offset = <0x8>;
+   };
 
-   msmgpio: gpio@80 {
-   compatible = "qcom,msm-gpio";
-   gpio-controller;
-   #gpio-cells = <2>;
-   ngpio = <150>;
-   interrupts = <0 16 0x4>;
-   interrupt-controller;
-   #interrupt-cells = <2>;
-   reg = <0x80 0x4000>;
-   };
+   msmgpio: gpio@80 {
+   compatible = "qcom,msm-gpio";
+   gpio-controller;
+   #gpio-cells = <2>;
+   ngpio = <150>;
+   interrupts = <0 16 0x4>;
+   interrupt-controller;
+   #interrupt-cells = <2>;
+   reg = <0x80 0x4000>;
+   };
 
-   gcc: clock-controller@90 {
-   compatible = "qcom,gcc-msm8960";
-   #clock-cells = <1>;
-   #reset-cells = <1>;
-   reg = <0x90 0x4000>;
-   };
+   gcc: clock-controller@90 {
+   compatible = "qcom,gcc-msm8960";
+

Re: [PATCH] ARM: dts: qcom: Update msm8960 device trees

2014-05-28 Thread Josh Cartwright
On Wed, May 28, 2014 at 01:27:23PM -0500, Kumar Gala wrote:
 * Move SoC peripherals into an SoC container node
 * Move serial enabling into board file (qcom-msm8960-cdp.dts)
 * Cleanup cpu node to match binding spec, enable-method and compatible
   should be per cpu, not part of the container
 * Drop interrupts property from l2-cache node as its not part of the
   binding spec
 
 Signed-off-by: Kumar Gala ga...@codeaurora.org
 ---
  arch/arm/boot/dts/qcom-msm8960-cdp.dts |   6 ++
  arch/arm/boot/dts/qcom-msm8960.dtsi| 165 
 +
  2 files changed, 93 insertions(+), 78 deletions(-)
 
 diff --git a/arch/arm/boot/dts/qcom-msm8960-cdp.dts 
 b/arch/arm/boot/dts/qcom-msm8960-cdp.dts
 index a58fb88..8e77ed7 100644
 --- a/arch/arm/boot/dts/qcom-msm8960-cdp.dts
 +++ b/arch/arm/boot/dts/qcom-msm8960-cdp.dts
 @@ -3,4 +3,10 @@
  / {
   model = Qualcomm MSM8960 CDP;
   compatible = qcom,msm8960-cdp, qcom,msm8960;
 +
 + soc {
 + serial@1644 {
 + status = ok;
 + };
 + };
  };

Is now the time put these serial nodes under a GSBI parent node?

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
hosted by The Linux Foundation
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] ARM: dts: qcom: Update msm8960 device trees

2014-05-28 Thread Kumar Gala

On May 28, 2014, at 3:09 PM, Josh Cartwright jo...@codeaurora.org wrote:

 On Wed, May 28, 2014 at 01:27:23PM -0500, Kumar Gala wrote:
 * Move SoC peripherals into an SoC container node
 * Move serial enabling into board file (qcom-msm8960-cdp.dts)
 * Cleanup cpu node to match binding spec, enable-method and compatible
  should be per cpu, not part of the container
 * Drop interrupts property from l2-cache node as its not part of the
  binding spec
 
 Signed-off-by: Kumar Gala ga...@codeaurora.org
 ---
 arch/arm/boot/dts/qcom-msm8960-cdp.dts |   6 ++
 arch/arm/boot/dts/qcom-msm8960.dtsi| 165 
 +
 2 files changed, 93 insertions(+), 78 deletions(-)
 
 diff --git a/arch/arm/boot/dts/qcom-msm8960-cdp.dts 
 b/arch/arm/boot/dts/qcom-msm8960-cdp.dts
 index a58fb88..8e77ed7 100644
 --- a/arch/arm/boot/dts/qcom-msm8960-cdp.dts
 +++ b/arch/arm/boot/dts/qcom-msm8960-cdp.dts
 @@ -3,4 +3,10 @@
 / {
  model = Qualcomm MSM8960 CDP;
  compatible = qcom,msm8960-cdp, qcom,msm8960;
 +
 +soc {
 +serial@1644 {
 +status = ok;
 +};
 +};
 };
 
 Is now the time put these serial nodes under a GSBI parent node?

Yeah, I’ll make the change to the 8960  8660 dts

- k

-- 
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by 
The Linux Foundation

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] ARM: dts: qcom: Update msm8960 device trees

2014-05-28 Thread Kumar Gala
* Move SoC peripherals into an SoC container node
* Move serial enabling into board file (qcom-msm8960-cdp.dts)
* Cleanup cpu node to match binding spec, enable-method and compatible
  should be per cpu, not part of the container
* Drop interrupts property from l2-cache node as its not part of the
  binding spec

Signed-off-by: Kumar Gala ga...@codeaurora.org
---
 arch/arm/boot/dts/qcom-msm8960-cdp.dts |   6 ++
 arch/arm/boot/dts/qcom-msm8960.dtsi| 165 +
 2 files changed, 93 insertions(+), 78 deletions(-)

diff --git a/arch/arm/boot/dts/qcom-msm8960-cdp.dts 
b/arch/arm/boot/dts/qcom-msm8960-cdp.dts
index a58fb88..8e77ed7 100644
--- a/arch/arm/boot/dts/qcom-msm8960-cdp.dts
+++ b/arch/arm/boot/dts/qcom-msm8960-cdp.dts
@@ -3,4 +3,10 @@
 / {
model = Qualcomm MSM8960 CDP;
compatible = qcom,msm8960-cdp, qcom,msm8960;
+
+   soc {
+   serial@1644 {
+   status = ok;
+   };
+   };
 };
diff --git a/arch/arm/boot/dts/qcom-msm8960.dtsi 
b/arch/arm/boot/dts/qcom-msm8960.dtsi
index 997b7b9..c38e54c 100644
--- a/arch/arm/boot/dts/qcom-msm8960.dtsi
+++ b/arch/arm/boot/dts/qcom-msm8960.dtsi
@@ -13,10 +13,10 @@
#address-cells = 1;
#size-cells = 0;
interrupts = 1 14 0x304;
-   compatible = qcom,krait;
-   enable-method = qcom,kpss-acc-v1;
 
cpu@0 {
+   compatible = qcom,krait;
+   enable-method = qcom,kpss-acc-v1;
device_type = cpu;
reg = 0;
next-level-cache = L2;
@@ -25,6 +25,8 @@
};
 
cpu@1 {
+   compatible = qcom,krait;
+   enable-method = qcom,kpss-acc-v1;
device_type = cpu;
reg = 1;
next-level-cache = L2;
@@ -35,7 +37,6 @@
L2: l2-cache {
compatible = cache;
cache-level = 2;
-   interrupts = 0 2 0x4;
};
};
 
@@ -45,91 +46,99 @@
qcom,no-pc-write;
};
 
-   intc: interrupt-controller@200 {
-   compatible = qcom,msm-qgic2;
-   interrupt-controller;
-   #interrupt-cells = 3;
-   reg =  0x0200 0x1000 ,
-  0x02002000 0x1000 ;
-   };
+   soc: soc {
+   #address-cells = 1;
+   #size-cells = 1;
+   ranges;
+   compatible = simple-bus;
+
+   intc: interrupt-controller@200 {
+   compatible = qcom,msm-qgic2;
+   interrupt-controller;
+   #interrupt-cells = 3;
+   reg = 0x0200 0x1000,
+ 0x02002000 0x1000;
+   };
 
-   timer@200a000 {
-   compatible = qcom,kpss-timer, qcom,msm-timer;
-   interrupts = 1 1 0x301,
-1 2 0x301,
-1 3 0x301;
-   reg = 0x0200a000 0x100;
-   clock-frequency = 2700,
- 32768;
-   cpu-offset = 0x8;
-   };
+   timer@200a000 {
+   compatible = qcom,kpss-timer, qcom,msm-timer;
+   interrupts = 1 1 0x301,
+1 2 0x301,
+1 3 0x301;
+   reg = 0x0200a000 0x100;
+   clock-frequency = 2700,
+ 32768;
+   cpu-offset = 0x8;
+   };
 
-   msmgpio: gpio@80 {
-   compatible = qcom,msm-gpio;
-   gpio-controller;
-   #gpio-cells = 2;
-   ngpio = 150;
-   interrupts = 0 16 0x4;
-   interrupt-controller;
-   #interrupt-cells = 2;
-   reg = 0x80 0x4000;
-   };
+   msmgpio: gpio@80 {
+   compatible = qcom,msm-gpio;
+   gpio-controller;
+   #gpio-cells = 2;
+   ngpio = 150;
+   interrupts = 0 16 0x4;
+   interrupt-controller;
+   #interrupt-cells = 2;
+   reg = 0x80 0x4000;
+   };
 
-   gcc: clock-controller@90 {
-   compatible = qcom,gcc-msm8960;
-   #clock-cells = 1;
-   #reset-cells = 1;
-   reg = 0x90 0x4000;
-   };
+   gcc: clock-controller@90 {
+   compatible = qcom,gcc-msm8960;
+   #clock-cells = 1;
+   #reset-cells = 1;
+   reg =