Skip to content
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.

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

Further increase minimum CPU voltage to 1.2V #330

Closed
wants to merge 5 commits into from
Closed

Further increase minimum CPU voltage to 1.2V #330

wants to merge 5 commits into from

Conversation

tb59427
Copy link
Contributor

@tb59427 tb59427 commented Sep 19, 2022

This change, when applied, changes the OPP voltages for both 144MHz and 312MHz from 1V to 1.2V when creating Openvario images. This fixes freezes previously encountered. It is a "follow up" change to PR #324. It turned out that even at 1.1V OV occasionally crashed.

For more information on the why and how of this change please see comment in PR #324

This change, when applied, changes the OPP voltages for both 144MHz and 312MHz
from 1V to 1.2V when creating Openvario images. This fixes freezes previously
encountered. It is a "follow up" change to PR #324. It turned out that even
at 1.1V OV occasionally crashed.

For more information on the why and how of this change please see comment in
PR #324
@MaxKellermann
Copy link
Collaborator

I think commit messages should refer to commit ids, not to PR numbers, because those are proprietary GitHub things out of git's scope

@mihu-ov
Copy link
Member

mihu-ov commented Sep 19, 2022

I think further increasing voltage for the low frequencies is good but avoiding them by additionally setting a minimum frequency of >400MHz like in the old 3.4 kernel images would be even better.

@MaxKellermann
Copy link
Collaborator

You don't need to recompile or change the kernel to change minimum/maximum frequencies; just set /sys/devices/system/cpu/cpu*/cpufreq/scaling_min_freq

@tb59427
Copy link
Contributor Author

tb59427 commented Sep 19, 2022

@mihu - my idea (and I am currently working on it) is to add a separate "fix cpu freq" service to systemd (simple service - one time run) running a shell script with a set frequency (just like @MaxKellermann just commented). Currently trying to understand how I include that into the bitbake logic of OV....

This change, when applied, changes the OPP voltages for both 144MHz and 312MHz
from 1V to 1.2V when creating Openvario images. This fixes freezes previously
encountered. It is a "follow up" change to PR #324. It turned out that even
at 1.1V OV occasionally crashed.

For more information on the why and how of this change please see comment in
commit #0426cb7023a415d2c4645d19ec54f9462579bd5c
…nd 312MHz from 1V to 1.2V when

creating Openvario images. This fixes freezes previously encountered. It is a "follow up" change to
commit 0426cb7. It turned out that even at 1.1V OV occasionally crashed.

For more information on the why and how of this change please see comment in PR
…nd 312MHz from 1V to 1.2V when

creating Openvario images. This fixes freezes previously encountered. It is a "follow up" change to
commit 0426cb7. It turned out that even at 1.1V OV occasionally crashed.

For more information on the why and how of this change please see comment in PR

Also corrected a missing \ in the linux bb file I had accidentally deleted.
This change, when applied, changes the OPP voltages for both 144MHz and 312MHz
from 1V to 1.2V when creating Openvario images. This fixes freezes previously
encountered. It is a "follow up" change to PR #324. It turned out that even
at 1.1V OV occasionally crashed.

For more information on the why and how of this change please see comment in #0426cb7023a415d2c4645d19ec54f9462579bd5c
@bomilkar bomilkar mentioned this pull request Oct 12, 2022
@Blaubart
Copy link
Contributor

Can someone merge this and this PR, please?

@linuxianer99
Copy link
Member

@tb59427 : There are conficts in the PR. Can you please fix this ?

@tb59427
Copy link
Contributor Author

tb59427 commented Oct 30, 2022

Same situation as with the other commit: github says "Only those with write access to this repository can merge pull requests." And "resolve conflicts" is greyed out for me.

@tb59427 tb59427 closed this by deleting the head repository Oct 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants