Skip to content

mvebu: cortexa72: enable ARM_ARMADA_8K_CPUFREQ #9259

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

mvebu: cortexa72: enable ARM_ARMADA_8K_CPUFREQ #9259

wants to merge 1 commit into from

Conversation

clayface
Copy link
Contributor

Enable the AP806's cpufreq driver. This driver is compatible with the
Armada 7K and 8K platforms.

Tested on Mikrotik RB5009.

Signed-off-by: Matthew Hagan mnhagan88@gmail.com

@clayface
mvebu: cortexa72: enable ARM_ARMADA_8K_CPUFREQ
a8037d4 
Enable the AP806's cpufreq driver. This driver is compatible with the
Armada 7K and 8K platforms.

Signed-off-by: Matthew Hagan <mnhagan88@gmail.com>
@clayface clayface marked this pull request as draft February 17, 2022 08:27
@clayface clayface marked this pull request as ready for review February 17, 2022 09:42
@mans0n mans0n added the target/mvebu pull request/issue for mvebu target label Feb 18, 2022
@Borromini
Copy link

@clayface Any way to test this? E.g. any files in /sys I should check for frequency values?

@clayface
Copy link
Contributor Author

clayface commented Feb 18, 2022 •
edited
Loading

Yes, thanks for testing. /sys/devices/system/cpu/cpu(0-3)/cpufreq/ should be populated, eg
cat /sys/devices/system/cpu/cpu*/cpufreq/scaling_cur_freq should output all current frequencies. collectd/rrdtool should be funtional with the cpu frequency plugin as well.

@Borromini
Copy link

Thanks. Without your patch, there's no /sys/devices/system/cpu/cpu[0-3]/cpufreq/ nor is there anything in /sys/devices/system/cpu/cpufreq/. With your patch there is indeed:

# cat /sys/devices/system/cpu/cpu*/cpufreq/scaling_cur_freq
700000
700000
350000
350000

Tested on MikroTik RB5009UG+S+IN.

Tested-by: Stijn Segers foss@volatilesystems.org

@robimarko
Copy link
Contributor

LGTM, most likely an oversight so far.

Reviewed-by: Robert Marko robimarko@gmail.com

@chunkeey
Copy link
Member

Merged. Thanks!

@chunkeey chunkeey closed this Feb 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
target/mvebu pull request/issue for mvebu target
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@clayface @Borromini @robimarko @chunkeey @mans0n