* [PATCH 0/2] ARM: dts: Add PMU support for OMAP2+
@ 2012-12-14 21:18 Jon Hunter
2012-12-14 21:18 ` [PATCH 1/2] ARM: OMAP2+: Prepare for device-tree PMU support Jon Hunter
2012-12-14 21:18 ` [PATCH 2/2] ARM: dts: OMAP2+: Add PMU nodes Jon Hunter
0 siblings, 2 replies; 7+ messages in thread
From: Jon Hunter @ 2012-12-14 21:18 UTC (permalink / raw)
To: Tony Lindgren, Benoit Cousson
Cc: linux-omap, linux-arm, device-tree, Jon Hunter
Add device-tree PMU support for OMAP2, OMAP3 and OMAP4460.
Tested with PERF on OMAP3430 Beagle Board and OMAP4460 Panda Board.
Boot tested only on OMAP2420 H4.
Jon Hunter (2):
ARM: OMAP2+: Prepare for device-tree PMU support
ARM: dts: OMAP2+: Add PMU nodes
arch/arm/boot/dts/omap2.dtsi | 5 +++++
arch/arm/boot/dts/omap3.dtsi | 6 ++++++
arch/arm/boot/dts/omap4-panda-es.dts | 2 ++
arch/arm/boot/dts/omap4460.dtsi | 18 ++++++++++++++++++
arch/arm/mach-omap2/pmu.c | 14 +++++++++++---
5 files changed, 42 insertions(+), 3 deletions(-)
create mode 100644 arch/arm/boot/dts/omap4460.dtsi
--
1.7.10.4
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/2] ARM: OMAP2+: Prepare for device-tree PMU support
2012-12-14 21:18 [PATCH 0/2] ARM: dts: Add PMU support for OMAP2+ Jon Hunter
@ 2012-12-14 21:18 ` Jon Hunter
2012-12-14 21:18 ` [PATCH 2/2] ARM: dts: OMAP2+: Add PMU nodes Jon Hunter
1 sibling, 0 replies; 7+ messages in thread
From: Jon Hunter @ 2012-12-14 21:18 UTC (permalink / raw)
To: Tony Lindgren, Benoit Cousson
Cc: linux-omap, linux-arm, device-tree, Jon Hunter
If device-tree is present, then do not create the PMU device from within
the OMAP specific PMU code. This is required to allow device-tree to
create the PMU device from the PMU device-tree node.
PMU is not currently supported for OMAP4430 (due to a dependency on
having a cross-trigger interface driver) and so ensure that this
indicated on boot with or without device-tree.
Signed-off-by: Jon Hunter <jon-hunter@ti.com>
---
arch/arm/mach-omap2/pmu.c | 12 +++++++++---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/arch/arm/mach-omap2/pmu.c b/arch/arm/mach-omap2/pmu.c
index eb78ae7..6e620eb 100644
--- a/arch/arm/mach-omap2/pmu.c
+++ b/arch/arm/mach-omap2/pmu.c
@@ -64,6 +64,15 @@ static int __init omap_init_pmu(void)
unsigned oh_num;
char **oh_names;
+ /* XXX Remove this check when the CTI driver is available */
+ if (cpu_is_omap443x()) {
+ pr_info("ARM PMU: not yet supported on OMAP4430 due to missing CTI driver\n");
+ return 0;
+ }
+
+ if (of_have_populated_dt())
+ return 0;
+
/*
* To create an ARM-PMU device the following HWMODs
* are required for the various OMAP2+ devices.
@@ -76,9 +85,6 @@ static int __init omap_init_pmu(void)
if (cpu_is_omap443x()) {
oh_num = ARRAY_SIZE(omap4430_pmu_oh_names);
oh_names = omap4430_pmu_oh_names;
- /* XXX Remove the next two lines when CTI driver available */
- pr_info("ARM PMU: not yet supported on OMAP4430 due to missing CTI driver\n");
- return 0;
} else if (cpu_is_omap34xx() || cpu_is_omap44xx()) {
oh_num = ARRAY_SIZE(omap3_pmu_oh_names);
oh_names = omap3_pmu_oh_names;
--
1.7.10.4
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/2] ARM: dts: OMAP2+: Add PMU nodes
2012-12-14 21:18 [PATCH 0/2] ARM: dts: Add PMU support for OMAP2+ Jon Hunter
2012-12-14 21:18 ` [PATCH 1/2] ARM: OMAP2+: Prepare for device-tree PMU support Jon Hunter
@ 2012-12-14 21:18 ` Jon Hunter
2012-12-14 21:24 ` Jon Hunter
2012-12-17 8:16 ` Benoit Cousson
1 sibling, 2 replies; 7+ messages in thread
From: Jon Hunter @ 2012-12-14 21:18 UTC (permalink / raw)
To: Tony Lindgren, Benoit Cousson
Cc: linux-omap, linux-arm, device-tree, Jon Hunter
Add PMU nodes for OMAP2, OMAP3 and OMAP4460 devices.
Please note that the node for OMAP4460 has been placed in a separate
header file for OMAP4460, because the node is not compatible with
OMAP4430.
Signed-off-by: Jon Hunter <jon-hunter@ti.com>
---
arch/arm/boot/dts/omap2.dtsi | 5 +++++
arch/arm/boot/dts/omap3.dtsi | 6 ++++++
arch/arm/boot/dts/omap4-panda-es.dts | 2 ++
arch/arm/boot/dts/omap4460.dtsi | 18 ++++++++++++++++++
arch/arm/mach-omap2/pmu.c | 2 ++
5 files changed, 33 insertions(+)
create mode 100644 arch/arm/boot/dts/omap4460.dtsi
diff --git a/arch/arm/boot/dts/omap2.dtsi b/arch/arm/boot/dts/omap2.dtsi
index 761c4b6..27f5ea1 100644
--- a/arch/arm/boot/dts/omap2.dtsi
+++ b/arch/arm/boot/dts/omap2.dtsi
@@ -26,6 +26,11 @@
};
};
+ pmu {
+ compatible = "arm,arm1136-pmu";
+ interrupts = <3>;
+ };
+
soc {
compatible = "ti,omap-infra";
mpu {
diff --git a/arch/arm/boot/dts/omap3.dtsi b/arch/arm/boot/dts/omap3.dtsi
index 1acc261..6c63118 100644
--- a/arch/arm/boot/dts/omap3.dtsi
+++ b/arch/arm/boot/dts/omap3.dtsi
@@ -26,6 +26,12 @@
};
};
+ pmu {
+ compatible = "arm,cortex-a8-pmu";
+ interrupts = <3>;
+ ti,hwmods = "debugss";
+ };
+
/*
* The soc node represents the soc top level view. It is uses for IPs
* that are not memory mapped in the MPU view or for the MPU itself.
diff --git a/arch/arm/boot/dts/omap4-panda-es.dts b/arch/arm/boot/dts/omap4-panda-es.dts
index 73bc1a6..2a6e344 100644
--- a/arch/arm/boot/dts/omap4-panda-es.dts
+++ b/arch/arm/boot/dts/omap4-panda-es.dts
@@ -5,7 +5,9 @@
* it under the terms of the GNU General Public License version 2 as
* published by the Free Software Foundation.
*/
+
/include/ "omap4-panda.dts"
+/include/ "omap4460.dtsi"
/* Audio routing is differnet between PandaBoard4430 and PandaBoardES */
&sound {
diff --git a/arch/arm/boot/dts/omap4460.dtsi b/arch/arm/boot/dts/omap4460.dtsi
new file mode 100644
index 0000000..1270890
--- /dev/null
+++ b/arch/arm/boot/dts/omap4460.dtsi
@@ -0,0 +1,18 @@
+/*
+ * Device Tree Source for OMAP4460 SoC
+ *
+ * Copyright (C) 2012 Texas Instruments Incorporated - http://www.ti.com/
+ *
+ * This file is licensed under the terms of the GNU General Public License
+ * version 2. This program is licensed "as is" without any warranty of any
+ * kind, whether express or implied.
+ */
+
+/ {
+ pmu {
+ compatible = "arm,cortex-a9-pmu";
+ interrupts = <0 54 0x4
+ 0 55 0x4>;
+ ti,hwmods = "debugss";
+ };
+};
diff --git a/arch/arm/mach-omap2/pmu.c b/arch/arm/mach-omap2/pmu.c
index 6e620eb..1a0799c 100644
--- a/arch/arm/mach-omap2/pmu.c
+++ b/arch/arm/mach-omap2/pmu.c
@@ -11,6 +11,8 @@
* the Free Software Foundation; either version 2 of the License, or
* (at your option) any later version.
*/
+#include <linux/of.h>
+
#include <asm/pmu.h>
#include "soc.h"
--
1.7.10.4
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] ARM: dts: OMAP2+: Add PMU nodes
2012-12-14 21:18 ` [PATCH 2/2] ARM: dts: OMAP2+: Add PMU nodes Jon Hunter
@ 2012-12-14 21:24 ` Jon Hunter
2012-12-17 8:16 ` Benoit Cousson
1 sibling, 0 replies; 7+ messages in thread
From: Jon Hunter @ 2012-12-14 21:24 UTC (permalink / raw)
To: Jon Hunter
Cc: Tony Lindgren, Benoit Cousson, linux-omap, linux-arm, device-tree
On 12/14/2012 03:18 PM, Jon Hunter wrote:
> Add PMU nodes for OMAP2, OMAP3 and OMAP4460 devices.
>
> Please note that the node for OMAP4460 has been placed in a separate
> header file for OMAP4460, because the node is not compatible with
> OMAP4430.
>
> Signed-off-by: Jon Hunter <jon-hunter@ti.com>
> ---
> arch/arm/boot/dts/omap2.dtsi | 5 +++++
> arch/arm/boot/dts/omap3.dtsi | 6 ++++++
> arch/arm/boot/dts/omap4-panda-es.dts | 2 ++
> arch/arm/boot/dts/omap4460.dtsi | 18 ++++++++++++++++++
> arch/arm/mach-omap2/pmu.c | 2 ++
> 5 files changed, 33 insertions(+)
> create mode 100644 arch/arm/boot/dts/omap4460.dtsi
>
> diff --git a/arch/arm/boot/dts/omap2.dtsi b/arch/arm/boot/dts/omap2.dtsi
> index 761c4b6..27f5ea1 100644
> --- a/arch/arm/boot/dts/omap2.dtsi
> +++ b/arch/arm/boot/dts/omap2.dtsi
> @@ -26,6 +26,11 @@
> };
> };
>
> + pmu {
> + compatible = "arm,arm1136-pmu";
> + interrupts = <3>;
> + };
> +
> soc {
> compatible = "ti,omap-infra";
> mpu {
> diff --git a/arch/arm/boot/dts/omap3.dtsi b/arch/arm/boot/dts/omap3.dtsi
> index 1acc261..6c63118 100644
> --- a/arch/arm/boot/dts/omap3.dtsi
> +++ b/arch/arm/boot/dts/omap3.dtsi
> @@ -26,6 +26,12 @@
> };
> };
>
> + pmu {
> + compatible = "arm,cortex-a8-pmu";
> + interrupts = <3>;
> + ti,hwmods = "debugss";
> + };
> +
> /*
> * The soc node represents the soc top level view. It is uses for IPs
> * that are not memory mapped in the MPU view or for the MPU itself.
> diff --git a/arch/arm/boot/dts/omap4-panda-es.dts b/arch/arm/boot/dts/omap4-panda-es.dts
> index 73bc1a6..2a6e344 100644
> --- a/arch/arm/boot/dts/omap4-panda-es.dts
> +++ b/arch/arm/boot/dts/omap4-panda-es.dts
> @@ -5,7 +5,9 @@
> * it under the terms of the GNU General Public License version 2 as
> * published by the Free Software Foundation.
> */
> +
> /include/ "omap4-panda.dts"
> +/include/ "omap4460.dtsi"
>
> /* Audio routing is differnet between PandaBoard4430 and PandaBoardES */
> &sound {
> diff --git a/arch/arm/boot/dts/omap4460.dtsi b/arch/arm/boot/dts/omap4460.dtsi
> new file mode 100644
> index 0000000..1270890
> --- /dev/null
> +++ b/arch/arm/boot/dts/omap4460.dtsi
> @@ -0,0 +1,18 @@
> +/*
> + * Device Tree Source for OMAP4460 SoC
> + *
> + * Copyright (C) 2012 Texas Instruments Incorporated - http://www.ti.com/
> + *
> + * This file is licensed under the terms of the GNU General Public License
> + * version 2. This program is licensed "as is" without any warranty of any
> + * kind, whether express or implied.
> + */
> +
> +/ {
> + pmu {
> + compatible = "arm,cortex-a9-pmu";
> + interrupts = <0 54 0x4
> + 0 55 0x4>;
> + ti,hwmods = "debugss";
> + };
> +};
> diff --git a/arch/arm/mach-omap2/pmu.c b/arch/arm/mach-omap2/pmu.c
> index 6e620eb..1a0799c 100644
> --- a/arch/arm/mach-omap2/pmu.c
> +++ b/arch/arm/mach-omap2/pmu.c
> @@ -11,6 +11,8 @@
> * the Free Software Foundation; either version 2 of the License, or
> * (at your option) any later version.
> */
> +#include <linux/of.h>
> +
Oops! I screwed something up here when rebasing. Resending this shortly ...
Jon
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] ARM: dts: OMAP2+: Add PMU nodes
2012-12-14 21:18 ` [PATCH 2/2] ARM: dts: OMAP2+: Add PMU nodes Jon Hunter
2012-12-14 21:24 ` Jon Hunter
@ 2012-12-17 8:16 ` Benoit Cousson
2012-12-17 15:58 ` Jon Hunter
1 sibling, 1 reply; 7+ messages in thread
From: Benoit Cousson @ 2012-12-17 8:16 UTC (permalink / raw)
To: Jon Hunter; +Cc: Tony Lindgren, linux-omap, linux-arm, device-tree
Hi Jon,
On 12/14/2012 10:18 PM, Jon Hunter wrote:
> Add PMU nodes for OMAP2, OMAP3 and OMAP4460 devices.
>
> Please note that the node for OMAP4460 has been placed in a separate
> header file for OMAP4460, because the node is not compatible with
> OMAP4430.
But where is the omap4430 node then?
Regards,
Benoit
>
> Signed-off-by: Jon Hunter <jon-hunter@ti.com>
> ---
> arch/arm/boot/dts/omap2.dtsi | 5 +++++
> arch/arm/boot/dts/omap3.dtsi | 6 ++++++
> arch/arm/boot/dts/omap4-panda-es.dts | 2 ++
> arch/arm/boot/dts/omap4460.dtsi | 18 ++++++++++++++++++
> arch/arm/mach-omap2/pmu.c | 2 ++
> 5 files changed, 33 insertions(+)
> create mode 100644 arch/arm/boot/dts/omap4460.dtsi
>
> diff --git a/arch/arm/boot/dts/omap2.dtsi b/arch/arm/boot/dts/omap2.dtsi
> index 761c4b6..27f5ea1 100644
> --- a/arch/arm/boot/dts/omap2.dtsi
> +++ b/arch/arm/boot/dts/omap2.dtsi
> @@ -26,6 +26,11 @@
> };
> };
>
> + pmu {
> + compatible = "arm,arm1136-pmu";
> + interrupts = <3>;
> + };
> +
> soc {
> compatible = "ti,omap-infra";
> mpu {
> diff --git a/arch/arm/boot/dts/omap3.dtsi b/arch/arm/boot/dts/omap3.dtsi
> index 1acc261..6c63118 100644
> --- a/arch/arm/boot/dts/omap3.dtsi
> +++ b/arch/arm/boot/dts/omap3.dtsi
> @@ -26,6 +26,12 @@
> };
> };
>
> + pmu {
> + compatible = "arm,cortex-a8-pmu";
> + interrupts = <3>;
> + ti,hwmods = "debugss";
> + };
> +
> /*
> * The soc node represents the soc top level view. It is uses for IPs
> * that are not memory mapped in the MPU view or for the MPU itself.
> diff --git a/arch/arm/boot/dts/omap4-panda-es.dts b/arch/arm/boot/dts/omap4-panda-es.dts
> index 73bc1a6..2a6e344 100644
> --- a/arch/arm/boot/dts/omap4-panda-es.dts
> +++ b/arch/arm/boot/dts/omap4-panda-es.dts
> @@ -5,7 +5,9 @@
> * it under the terms of the GNU General Public License version 2 as
> * published by the Free Software Foundation.
> */
> +
> /include/ "omap4-panda.dts"
> +/include/ "omap4460.dtsi"
>
> /* Audio routing is differnet between PandaBoard4430 and PandaBoardES */
> &sound {
> diff --git a/arch/arm/boot/dts/omap4460.dtsi b/arch/arm/boot/dts/omap4460.dtsi
> new file mode 100644
> index 0000000..1270890
> --- /dev/null
> +++ b/arch/arm/boot/dts/omap4460.dtsi
> @@ -0,0 +1,18 @@
> +/*
> + * Device Tree Source for OMAP4460 SoC
> + *
> + * Copyright (C) 2012 Texas Instruments Incorporated - http://www.ti.com/
> + *
> + * This file is licensed under the terms of the GNU General Public License
> + * version 2. This program is licensed "as is" without any warranty of any
> + * kind, whether express or implied.
> + */
> +
> +/ {
> + pmu {
> + compatible = "arm,cortex-a9-pmu";
> + interrupts = <0 54 0x4
> + 0 55 0x4>;
> + ti,hwmods = "debugss";
> + };
> +};
> diff --git a/arch/arm/mach-omap2/pmu.c b/arch/arm/mach-omap2/pmu.c
> index 6e620eb..1a0799c 100644
> --- a/arch/arm/mach-omap2/pmu.c
> +++ b/arch/arm/mach-omap2/pmu.c
> @@ -11,6 +11,8 @@
> * the Free Software Foundation; either version 2 of the License, or
> * (at your option) any later version.
> */
> +#include <linux/of.h>
> +
> #include <asm/pmu.h>
>
> #include "soc.h"
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] ARM: dts: OMAP2+: Add PMU nodes
2012-12-17 8:16 ` Benoit Cousson
@ 2012-12-17 15:58 ` Jon Hunter
2012-12-17 16:04 ` Benoit Cousson
0 siblings, 1 reply; 7+ messages in thread
From: Jon Hunter @ 2012-12-17 15:58 UTC (permalink / raw)
To: Benoit Cousson; +Cc: Tony Lindgren, linux-omap, linux-arm, device-tree
On 12/17/2012 02:16 AM, Benoit Cousson wrote:
> Hi Jon,
>
> On 12/14/2012 10:18 PM, Jon Hunter wrote:
>> Add PMU nodes for OMAP2, OMAP3 and OMAP4460 devices.
>>
>> Please note that the node for OMAP4460 has been placed in a separate
>> header file for OMAP4460, because the node is not compatible with
>> OMAP4430.
>
> But where is the omap4430 node then?
I have left it out deliberately because OMAP4430 is not yet supported by
PMU as it is dependent on having a driver for the cross-trigger interface.
If you prefer to stick the node in the omap4.dtsi for now then that is
ok, but I wanted to make it clear that this is for omap4460.
Cheers
Jon
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] ARM: dts: OMAP2+: Add PMU nodes
2012-12-17 15:58 ` Jon Hunter
@ 2012-12-17 16:04 ` Benoit Cousson
0 siblings, 0 replies; 7+ messages in thread
From: Benoit Cousson @ 2012-12-17 16:04 UTC (permalink / raw)
To: Jon Hunter; +Cc: Tony Lindgren, linux-omap, linux-arm, device-tree
Hi Jon,
On 12/17/2012 04:58 PM, Jon Hunter wrote:
>
> On 12/17/2012 02:16 AM, Benoit Cousson wrote:
>> Hi Jon,
>>
>> On 12/14/2012 10:18 PM, Jon Hunter wrote:
>>> Add PMU nodes for OMAP2, OMAP3 and OMAP4460 devices.
>>>
>>> Please note that the node for OMAP4460 has been placed in a separate
>>> header file for OMAP4460, because the node is not compatible with
>>> OMAP4430.
>>
>> But where is the omap4430 node then?
>
> I have left it out deliberately because OMAP4430 is not yet supported by
> PMU as it is dependent on having a driver for the cross-trigger interface.
>
> If you prefer to stick the node in the omap4.dtsi for now then that is
> ok, but I wanted to make it clear that this is for omap4460.
No, that's fine, I was just wondering. You should just add that comment
in the cover letter to make it explicit.
Thanks,
Benoit
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2012-12-17 16:04 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-12-14 21:18 [PATCH 0/2] ARM: dts: Add PMU support for OMAP2+ Jon Hunter
2012-12-14 21:18 ` [PATCH 1/2] ARM: OMAP2+: Prepare for device-tree PMU support Jon Hunter
2012-12-14 21:18 ` [PATCH 2/2] ARM: dts: OMAP2+: Add PMU nodes Jon Hunter
2012-12-14 21:24 ` Jon Hunter
2012-12-17 8:16 ` Benoit Cousson
2012-12-17 15:58 ` Jon Hunter
2012-12-17 16:04 ` Benoit Cousson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).