Hello,
something happened during the last cycle and an old version of the devfreq
driver was merged.
This thread contains patches that bring it up to date to the last submitted
version and also incorporates the feedback that that version received, plus
some other small fixes and improvements that came up during rebase and
testing.
These patches implement support for setting the rate of the EMC clock based on
stats collected from the ACTMON, a piece of hw in the Tegra124 that counts
memory accesses (among others).
It depends on the following in-flight patches:
* EMC driver: http://thread.gmane.org/gmane.linux.kernel/1907035
* CPUFreq driver: http://thread.gmane.org/gmane.linux.kernel/1897078
I have pushed a branch here for testing:
http://cgit.collabora.com/git/user/tomeu/linux.git/log/?h=actmon-v6
Regards,
Tomeu
Tomeu Vizoso (8):
of: Add binding for NVIDIA Tegra ACTMON node
PM / devfreq: tegra: Update to v5 of the submitted patches
clk: tegra: Have EMC clock implement determine_rate()
PM / devfreq: tegra: Use clock rate constraints
PM / devfreq: tegra: remove operating-points
PM / devfreq: tegra: Set drvdata before enabling the irq
PM / devfreq: tegra: Enable interrupts after resuming the devfreq
monitor
ARM: tegra: Add Tegra124 ACTMON support
Acked-by: MyungJoo Ham <myungjoo.ham@samsung.com>
for all PM / devfreq patches (2, 4, 5, 6, 7)
And merged in for-rc tree with a little modification.
In the patch 2/8, I would like to add "const" in Line 748.
Would it be fine with you?
(You may look at: https://git.kernel.org/cgit/linux/kernel/git/mzx/devfreq.git/log/?h=for-rc )
The diff after applying all 2/8 to 7/8 will be:
diff --git a/drivers/devfreq/tegra-devfreq.c b/drivers/devfreq/tegra-devfreq.c
index 0d1edd5..8e633a6 100644
--- a/drivers/devfreq/tegra-devfreq.c
+++ b/drivers/devfreq/tegra-devfreq.c
@@ -740,7 +740,7 @@ static int tegra_devfreq_remove(struct platform_device *pdev)
return 0;
}
-static struct of_device_id tegra_devfreq_of_match[] = {
+static const struct of_device_id tegra_devfreq_of_match[] = {
{ .compatible = "nvidia,tegra124-actmon" },
{ },
};
.../devicetree/bindings/arm/tegra/actmon.txt | 28 ++
arch/arm/boot/dts/tegra124.dtsi | 11 +
drivers/clk/tegra/clk-emc.c | 19 +-
drivers/devfreq/tegra-devfreq.c | 480 +++++++++++----------
4 files changed, 316 insertions(+), 222 deletions(-)
create mode 100644 Documentation/devicetree/bindings/arm/tegra/actmon.txt
--
2.1.0
On 18 March 2015 at 06:10, MyungJoo Ham [off-list ref] wrote:
quoted
Hello,
something happened during the last cycle and an old version of the devfreq
driver was merged.
This thread contains patches that bring it up to date to the last submitted
version and also incorporates the feedback that that version received, plus
some other small fixes and improvements that came up during rebase and
testing.
These patches implement support for setting the rate of the EMC clock based on
stats collected from the ACTMON, a piece of hw in the Tegra124 that counts
memory accesses (among others).
It depends on the following in-flight patches:
* EMC driver: http://thread.gmane.org/gmane.linux.kernel/1907035
* CPUFreq driver: http://thread.gmane.org/gmane.linux.kernel/1897078
I have pushed a branch here for testing:
http://cgit.collabora.com/git/user/tomeu/linux.git/log/?h=actmon-v6
Regards,
Tomeu
Tomeu Vizoso (8):
of: Add binding for NVIDIA Tegra ACTMON node
PM / devfreq: tegra: Update to v5 of the submitted patches
clk: tegra: Have EMC clock implement determine_rate()
PM / devfreq: tegra: Use clock rate constraints
PM / devfreq: tegra: remove operating-points
PM / devfreq: tegra: Set drvdata before enabling the irq
PM / devfreq: tegra: Enable interrupts after resuming the devfreq
monitor
ARM: tegra: Add Tegra124 ACTMON support
Acked-by: MyungJoo Ham <redacted>
for all PM / devfreq patches (2, 4, 5, 6, 7)
And merged in for-rc tree with a little modification.
Thanks. Though that's fine with me, I was wondering if Mikko or
Alexandre would have any comments on the changes, even if they are
small regarding what they already reviewed.
In the patch 2/8, I would like to add "const" in Line 748.
Would it be fine with you?
Yes, thanks, that's fine.
Regards,
Tomeu
quoted hunk
(You may look at: https://git.kernel.org/cgit/linux/kernel/git/mzx/devfreq.git/log/?h=for-rc )
The diff after applying all 2/8 to 7/8 will be:
diff --git a/drivers/devfreq/tegra-devfreq.c b/drivers/devfreq/tegra-devfreq.c
index 0d1edd5..8e633a6 100644
--- a/drivers/devfreq/tegra-devfreq.c
+++ b/drivers/devfreq/tegra-devfreq.c
@@ -740,7 +740,7 @@ static int tegra_devfreq_remove(struct platform_device *pdev)
return 0;
}
-static struct of_device_id tegra_devfreq_of_match[] = {
+static const struct of_device_id tegra_devfreq_of_match[] = {
{ .compatible = "nvidia,tegra124-actmon" },
{ },
};
quoted
.../devicetree/bindings/arm/tegra/actmon.txt | 28 ++
arch/arm/boot/dts/tegra124.dtsi | 11 +
drivers/clk/tegra/clk-emc.c | 19 +-
drivers/devfreq/tegra-devfreq.c | 480 +++++++++++----------
4 files changed, 316 insertions(+), 222 deletions(-)
create mode 100644 Documentation/devicetree/bindings/arm/tegra/actmon.txt
--
2.1.0
On Wed, Mar 18, 2015 at 3:27 PM, Tomeu Vizoso
[off-list ref] wrote:
On 18 March 2015 at 06:10, MyungJoo Ham [off-list ref] wrote:
quoted
quoted
Hello,
something happened during the last cycle and an old version of the devfreq
driver was merged.
This thread contains patches that bring it up to date to the last submitted
version and also incorporates the feedback that that version received, plus
some other small fixes and improvements that came up during rebase and
testing.
These patches implement support for setting the rate of the EMC clock based on
stats collected from the ACTMON, a piece of hw in the Tegra124 that counts
memory accesses (among others).
It depends on the following in-flight patches:
* EMC driver: http://thread.gmane.org/gmane.linux.kernel/1907035
* CPUFreq driver: http://thread.gmane.org/gmane.linux.kernel/1897078
I have pushed a branch here for testing:
http://cgit.collabora.com/git/user/tomeu/linux.git/log/?h=actmon-v6
Regards,
Tomeu
Tomeu Vizoso (8):
of: Add binding for NVIDIA Tegra ACTMON node
PM / devfreq: tegra: Update to v5 of the submitted patches
clk: tegra: Have EMC clock implement determine_rate()
PM / devfreq: tegra: Use clock rate constraints
PM / devfreq: tegra: remove operating-points
PM / devfreq: tegra: Set drvdata before enabling the irq
PM / devfreq: tegra: Enable interrupts after resuming the devfreq
monitor
ARM: tegra: Add Tegra124 ACTMON support
Acked-by: MyungJoo Ham <myungjoo.ham@samsung.com>
for all PM / devfreq patches (2, 4, 5, 6, 7)
And merged in for-rc tree with a little modification.
Thanks. Though that's fine with me, I was wondering if Mikko or
Alexandre would have any comments on the changes, even if they are
small regarding what they already reviewed.
After a quick look I think I'm good with it. I will try to look in
further detail. If there is anything wrong, we can fix it with fixup
patches, since what has been merged is good stuff already!
I guess it would now be interesting to look at the watermark support
for devfreq. I am not sure if anything is happening to it? Arto?
https://lkml.org/lkml/2014/12/5/262