Subject: [PATCH] arm64: dts: mediatek: mt6795: Add CPUX system timer node

Add a node for mt6795-systimer: this is necessary to start the
System Timer(s) for all cores, finally making CNTVCT_EL0 usable.

Signed-off-by: AngeloGioacchino Del Regno <[email protected]>
---
arch/arm64/boot/dts/mediatek/mt6795.dtsi | 8 ++++++++
1 file changed, 8 insertions(+)

diff --git a/arch/arm64/boot/dts/mediatek/mt6795.dtsi b/arch/arm64/boot/dts/mediatek/mt6795.dtsi
index d4842b4a4eb7..46f0e54be766 100644
--- a/arch/arm64/boot/dts/mediatek/mt6795.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt6795.dtsi
@@ -230,6 +230,14 @@ sysirq: intpol-controller@10200620 {
reg = <0 0x10200620 0 0x20>;
};

+ systimer: timer@10200670 {
+ compatible = "mediatek,mt6795-systimer";
+ reg = <0 0x10200670 0 0x10>;
+ interrupts = <GIC_SPI 64 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&system_clk>;
+ clock-names = "clk13m";
+ };
+
gic: interrupt-controller@10221000 {
compatible = "arm,gic-400";
#interrupt-cells = <3>;
--
2.35.1


Subject: Re: [PATCH] arm64: dts: mediatek: mt6795: Add CPUX system timer node

Il 29/07/22 11:35, AngeloGioacchino Del Regno ha scritto:
> Add a node for mt6795-systimer: this is necessary to start the
> System Timer(s) for all cores, finally making CNTVCT_EL0 usable.
>
> Signed-off-by: AngeloGioacchino Del Regno <[email protected]>

Hello Matthias,

gentle ping for this one - this has no floating dependencies and it's
verified to work as expected.

Cheers,
Angelo

> ---
> arch/arm64/boot/dts/mediatek/mt6795.dtsi | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt6795.dtsi b/arch/arm64/boot/dts/mediatek/mt6795.dtsi
> index d4842b4a4eb7..46f0e54be766 100644
> --- a/arch/arm64/boot/dts/mediatek/mt6795.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt6795.dtsi
> @@ -230,6 +230,14 @@ sysirq: intpol-controller@10200620 {
> reg = <0 0x10200620 0 0x20>;
> };
>
> + systimer: timer@10200670 {
> + compatible = "mediatek,mt6795-systimer";
> + reg = <0 0x10200670 0 0x10>;
> + interrupts = <GIC_SPI 64 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&system_clk>;
> + clock-names = "clk13m";
> + };
> +
> gic: interrupt-controller@10221000 {
> compatible = "arm,gic-400";
> #interrupt-cells = <3>;


2022-09-15 17:31:22

by Matthias Brugger

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: mediatek: mt6795: Add CPUX system timer node



On 14/09/2022 13:01, AngeloGioacchino Del Regno wrote:
> Il 29/07/22 11:35, AngeloGioacchino Del Regno ha scritto:
>> Add a node for mt6795-systimer: this is necessary to start the
>> System Timer(s) for all cores, finally making CNTVCT_EL0 usable.
>>
>> Signed-off-by: AngeloGioacchino Del Regno
>> <[email protected]>
>
> Hello Matthias,
>
> gentle ping for this one - this has no floating dependencies and it's
> verified to work as expected.
>

Applied, thanks!

> Cheers,
> Angelo
>
>> ---
>>   arch/arm64/boot/dts/mediatek/mt6795.dtsi | 8 ++++++++
>>   1 file changed, 8 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/mediatek/mt6795.dtsi
>> b/arch/arm64/boot/dts/mediatek/mt6795.dtsi
>> index d4842b4a4eb7..46f0e54be766 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt6795.dtsi
>> +++ b/arch/arm64/boot/dts/mediatek/mt6795.dtsi
>> @@ -230,6 +230,14 @@ sysirq: intpol-controller@10200620 {
>>               reg = <0 0x10200620 0 0x20>;
>>           };
>> +        systimer: timer@10200670 {
>> +            compatible = "mediatek,mt6795-systimer";
>> +            reg = <0 0x10200670 0 0x10>;
>> +            interrupts = <GIC_SPI 64 IRQ_TYPE_LEVEL_HIGH>;
>> +            clocks = <&system_clk>;
>> +            clock-names = "clk13m";
>> +        };
>> +
>>           gic: interrupt-controller@10221000 {
>>               compatible = "arm,gic-400";
>>               #interrupt-cells = <3>;
>
>