diff mbox

[v3,5/6] dt-bindings: Document devicetree binding for ARM DSU PMU

Message ID 1501168218-26741-6-git-send-email-suzuki.poulose@arm.com (mailing list archive)
State New, archived
Headers show

Commit Message

Suzuki K Poulose July 27, 2017, 3:10 p.m. UTC
This patch documents the devicetree bindings for ARM DSU PMU.

Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Will Deacon <will.deacon@arm.com>
Cc: Rob Herring <robh@kernel.org>
Signed-off-by: Suzuki K Poulose <suzuki.poulose@arm.com>
---
 .../devicetree/bindings/arm/arm-dsu-pmu.txt        | 27 ++++++++++++++++++++++
 1 file changed, 27 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/arm/arm-dsu-pmu.txt

Comments

Rob Herring July 27, 2017, 3:52 p.m. UTC | #1
+DT list

On Thu, Jul 27, 2017 at 10:10 AM, Suzuki K Poulose
<suzuki.poulose@arm.com> wrote:
> This patch documents the devicetree bindings for ARM DSU PMU.
>
> Cc: Mark Rutland <mark.rutland@arm.com>
> Cc: Will Deacon <will.deacon@arm.com>
> Cc: Rob Herring <robh@kernel.org>
> Signed-off-by: Suzuki K Poulose <suzuki.poulose@arm.com>
> ---
>  .../devicetree/bindings/arm/arm-dsu-pmu.txt        | 27 ++++++++++++++++++++++
>  1 file changed, 27 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/arm/arm-dsu-pmu.txt
>
> diff --git a/Documentation/devicetree/bindings/arm/arm-dsu-pmu.txt b/Documentation/devicetree/bindings/arm/arm-dsu-pmu.txt
> new file mode 100644
> index 0000000..b9935ac
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/arm-dsu-pmu.txt
> @@ -0,0 +1,27 @@
> +* ARM DynamIQ Shared Unit (DSU) Performance Monitor Unit (PMU)
> +
> +ARM DyanmIQ Shared Unit (DSU) integrates one or more CPU cores
> +with a shared L3 memory system, control logic and external interfaces to
> +form a multicore cluster. The PMU enables to gather various statistics on
> +the operations of the DSU. The PMU provides independent 32bit counters that
> +can count any of the supported events, along with a 64bit cycle counter.
> +The PMU is accessed via CPU system registers and has no MMIO component.
> +
> +** DSU PMU required properties:
> +
> +- compatible   : should be one of :
> +
> +               "arm,dsu-pmu"

Seems kind of generic. There's only one flavor is DSU?

> +
> +- interrupts   : Exactly 1 SPI must be listed.
> +
> +- cpus         : List of phandles for the CPUs connected to this DSU instance.
> +
> +
> +** Example:
> +
> +dsu_pmu@0 {

Don't use "_" in node names and unit-address is not valid without a
reg property.

> +       compatible = "arm,dsu-pmu";
> +       interrupts = <GIC_SPI 02 IRQ_TYPE_LEVEL_HIGH>;
> +       cpus = <&cpu_0>, <&cpu_1>;
> +};
> --
> 2.7.5
>
Suzuki K Poulose July 27, 2017, 4:09 p.m. UTC | #2
On 27/07/17 16:52, Rob Herring wrote:
> +DT list
>
> On Thu, Jul 27, 2017 at 10:10 AM, Suzuki K Poulose
> <suzuki.poulose@arm.com> wrote:
>> This patch documents the devicetree bindings for ARM DSU PMU.
>>
>> Cc: Mark Rutland <mark.rutland@arm.com>
>> Cc: Will Deacon <will.deacon@arm.com>
>> Cc: Rob Herring <robh@kernel.org>
>> Signed-off-by: Suzuki K Poulose <suzuki.poulose@arm.com>
>> ---
>>  .../devicetree/bindings/arm/arm-dsu-pmu.txt        | 27 ++++++++++++++++++++++
>>  1 file changed, 27 insertions(+)
>>  create mode 100644 Documentation/devicetree/bindings/arm/arm-dsu-pmu.txt
>>
>> diff --git a/Documentation/devicetree/bindings/arm/arm-dsu-pmu.txt b/Documentation/devicetree/bindings/arm/arm-dsu-pmu.txt
>> new file mode 100644
>> index 0000000..b9935ac
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/arm/arm-dsu-pmu.txt
>> @@ -0,0 +1,27 @@
>> +* ARM DynamIQ Shared Unit (DSU) Performance Monitor Unit (PMU)
>> +
>> +ARM DyanmIQ Shared Unit (DSU) integrates one or more CPU cores
>> +with a shared L3 memory system, control logic and external interfaces to
>> +form a multicore cluster. The PMU enables to gather various statistics on
>> +the operations of the DSU. The PMU provides independent 32bit counters that
>> +can count any of the supported events, along with a 64bit cycle counter.
>> +The PMU is accessed via CPU system registers and has no MMIO component.
>> +
>> +** DSU PMU required properties:
>> +
>> +- compatible   : should be one of :
>> +
>> +               "arm,dsu-pmu"
>
> Seems kind of generic. There's only one flavor is DSU?
>

Yes, there is only flavor. It is not something others can modify.
There could be different revisions of the product (in the future), but that
can be identified from the ID registers and can be handled accordingly.

>> +
>> +- interrupts   : Exactly 1 SPI must be listed.
>> +
>> +- cpus         : List of phandles for the CPUs connected to this DSU instance.
>> +
>> +
>> +** Example:
>> +
>> +dsu_pmu@0 {
>
> Don't use "_" in node names and unit-address is not valid without a
> reg property.

Ok.

Thanks for the comments, I will fix them in the next version.

Suzuki
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/arm/arm-dsu-pmu.txt b/Documentation/devicetree/bindings/arm/arm-dsu-pmu.txt
new file mode 100644
index 0000000..b9935ac
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/arm-dsu-pmu.txt
@@ -0,0 +1,27 @@ 
+* ARM DynamIQ Shared Unit (DSU) Performance Monitor Unit (PMU)
+
+ARM DyanmIQ Shared Unit (DSU) integrates one or more CPU cores
+with a shared L3 memory system, control logic and external interfaces to
+form a multicore cluster. The PMU enables to gather various statistics on
+the operations of the DSU. The PMU provides independent 32bit counters that
+can count any of the supported events, along with a 64bit cycle counter.
+The PMU is accessed via CPU system registers and has no MMIO component.
+
+** DSU PMU required properties:
+
+- compatible	: should be one of :
+
+		"arm,dsu-pmu"
+
+- interrupts	: Exactly 1 SPI must be listed.
+
+- cpus		: List of phandles for the CPUs connected to this DSU instance.
+
+
+** Example:
+
+dsu_pmu@0 {
+	compatible = "arm,dsu-pmu";
+	interrupts = <GIC_SPI 02 IRQ_TYPE_LEVEL_HIGH>;
+	cpus = <&cpu_0>, <&cpu_1>;
+};