diff mbox series

[v1,1/9] ARM: tegra: Add ACTMON support on Tegra114

Message ID 20250225143501.68966-2-clamor95@gmail.com (mailing list archive)
State Not Applicable, archived
Headers show
Series Tegra114: implement EMC support | expand

Commit Message

Svyatoslav Ryhel Feb. 25, 2025, 2:34 p.m. UTC
Add support for ACTMON on Tegra114. This is used to monitor activity from
different components. Based on the collected statistics, the rate at which
the external memory needs to be clocked can be derived.

Actmon driver has T30 and T124 compatibles, T124 fits for T114 as well.

Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
---
 arch/arm/boot/dts/nvidia/tegra114.dtsi | 11 +++++++++++
 1 file changed, 11 insertions(+)

Comments

Krzysztof Kozlowski Feb. 25, 2025, 5:33 p.m. UTC | #1
On 25/02/2025 15:34, Svyatoslav Ryhel wrote:
> Add support for ACTMON on Tegra114. This is used to monitor activity from
> different components. Based on the collected statistics, the rate at which
> the external memory needs to be clocked can be derived.
> 
> Actmon driver has T30 and T124 compatibles, T124 fits for T114 as well.
> 
> Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
> ---

DTS at beginning of patchset suggests you have dependency and nothing
can depend on DTS. Please organize it for standard way, so DTS is at the
end or, *better* separate patchset.

Nothing in cover letter explained dependecies, so I assume this can be
split into separate patchsets, because you target two different subsystems.

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/nvidia/tegra114.dtsi b/arch/arm/boot/dts/nvidia/tegra114.dtsi
index 86f14e2fd29f..a309999e7988 100644
--- a/arch/arm/boot/dts/nvidia/tegra114.dtsi
+++ b/arch/arm/boot/dts/nvidia/tegra114.dtsi
@@ -246,6 +246,17 @@  ahb: ahb@6000c000 {
 		reg = <0x6000c000 0x150>;
 	};
 
+	actmon: actmon@6000c800 {
+		compatible = "nvidia,tegra114-actmon", "nvidia,tegra124-actmon";
+		reg = <0x6000c800 0x400>;
+		interrupts = <GIC_SPI 45 IRQ_TYPE_LEVEL_HIGH>;
+		clocks = <&tegra_car TEGRA114_CLK_ACTMON>,
+			 <&tegra_car TEGRA114_CLK_EMC>;
+		clock-names = "actmon", "emc";
+		resets = <&tegra_car TEGRA114_CLK_ACTMON>;
+		reset-names = "actmon";
+	};
+
 	gpio: gpio@6000d000 {
 		compatible = "nvidia,tegra114-gpio", "nvidia,tegra30-gpio";
 		reg = <0x6000d000 0x1000>;