Skip to content

Fix over_voltage description #1698

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

Merged
merged 1 commit into from
Jun 8, 2021
Merged

Fix over_voltage description #1698

merged 1 commit into from
Jun 8, 2021

Conversation

MichaIng
Copy link
Contributor

@MichaIng MichaIng commented Oct 14, 2020

The default over_voltage value equals 0 on all RPi models.
This equates to a maximum voltage of 1.35V (1.2V on RPi 1).
Changing over_voltage only affects the upper voltage limit by 25mA with each integer step.

over_voltage_min affects the minimum voltage which defaults to 0.
This equates to a minimum voltage of 1.2V.
Changing over_voltage_min affects the lower voltage limit by 25mA with each integer step.

Additional minor clarification with this commit:

  1. The default CPU scaling governor on RPi is powersave. A SysV service, shipped with raspi-config, changes it to ondemand during boot.
    The link to CPUFreq documentation is now pointing to an up-to-date official Linux documentation, since the previous reference is outdated and includes lots of information which does not apply on RPi.

  2. The ondemand governor has an effect even without overclocking settings, scaling the frequency between arm_freq and arm_freq_min.
    Only on RPi 1, these two settings are equal by default, practically disabling dynamic frequency scaling.

  3. The safest option to disable dynamic clocking is to apply a static scaling governor like powersave or performance, since this does not involve the risk to void warranty.

@lurch
Copy link
Contributor

lurch commented Oct 14, 2020

More questions for @popcornmix ? 😉

@MichaIng
Copy link
Contributor Author

Jep, sorry for being a bid noisy today, one test lead to the next and one question to the next 😄.

@MichaIng MichaIng marked this pull request as draft October 14, 2020 22:18
@MichaIng MichaIng changed the title With over_voltage=0 the max voltage is 1.35V Fix over_voltage description Oct 15, 2020
@MichaIng MichaIng marked this pull request as ready for review October 15, 2020 11:43
@MichaIng
Copy link
Contributor Author

MichaIng commented Oct 15, 2020

Okay, I think I got it now right. over_voltage and over_voltage_min basically behave like arm_freq and arm_freq_min, only without the intermediate steps and max voltage being applied as fast as ARM frequency does not equal arm_freq_min. The AVS description gives a different impression, but lets keep it like that.

The voltages ranges in the description are true for RPi 1 (upper limit = 1.2V => static voltage with default over_voltage=0) but 1.35V on all other models.

There is still one thing incorrect: vcgencmd measure_volts sdram_p reports 1.225V on all tested RPi models (Zero, 1, 2) and over_voltage_sdram_p setting has no effect on this, at least not on my RPi 2 within a tested range of -2 and 1. But probably this is a separate issue to investigate.

@JamesH65
Copy link
Contributor

@popcornmix When you have a few seconds free, could you double check this for accuracy? Ta.

@ghost
Copy link

ghost commented Oct 30, 2020

Only on RPi 1, these two settings are equal by default, practically disabling dynamic frequency scaling.

I'm not sure if Pi 1 can do dynamic frequency scaling - I had always assumed it can't.

@MichaIng
Copy link
Contributor Author

MichaIng commented Nov 6, 2020

I'm not sure if Pi 1 can do dynamic frequency scaling - I had always assumed it can't.

It can, but as mentioned does not do with defaults: arm_freq_min=arm_freq=700
I think this is the reason for the docs statement prior to this PR:

It has no effect if you have no overclock settings, but if you overclock, the CPU frequency will vary with processor load.

While it is true for RPi1, for all other models it is wrong and should hence be clarified 🙂.

@aallan
Copy link
Contributor

aallan commented Jun 7, 2021

I'd encourage you to wrap this PR up in the next week or so as we're in the process of transitioning the documentation from the current Markdown-based source format to Asciidoc. At some point soon — probably around the end of June, beginning of July — we will freeze the current documentation repo. After that time contributions and PRs based on the Markdown source will not be accepted, and any PRs that are still open will be closed.

See #1911 for full details.

The default over_voltage value equals 0 on all RPi models.
This equates to a maximum voltage of 1.35V (1.2V on RPi 1).
Changing over_voltage only affects the upper voltage limit by 25mA with each integer step.

over_voltage_min affects the minimum voltage which defaults to 0.
This equates to a minimum voltage of 1.2V.
Changing over_voltage_min affects the lower voltage limit by 25mA with each integer step.

Additional minor clarification with this commit:

1. The default CPU scaling governor on RPi is "powersave". A SysV service, shipped with raspi-config, changes it to "ondemand" during boot.
The link to CPUFreq documentation is now pointing to an up-to-date official Linux documentation, since the previous reference is outdated and includes lots of information which does not apply on RPi.

2. The "ondemand" governor has an effect even without overclocking settings, scaling the frequency between "arm_freq" and "arm_freq_min".
Only on RPi 1, these two settings are equal by default, practically disabling dynamic frequency scaling.

3. The safest option to disable dynamic clocking is to apply a static scaling governor like "powersave" or "performance", since this does not involve the risk to void warranty.

Signed-off-by: MichaIng <[email protected]>
@MichaIng
Copy link
Contributor Author

MichaIng commented Jun 8, 2021

I rebased this commit onto the current master branch, no changes required. Ready from my side.

@aallan aallan added the ready to merge The OP says this PR is ready to merge? Anyone object? label Jun 8, 2021
@JamesH65 JamesH65 merged commit 974995f into raspberrypi:master Jun 8, 2021
@MichaIng MichaIng deleted the patch-1 branch June 8, 2021 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge The OP says this PR is ready to merge? Anyone object? waiting for comment Awaiting a response
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants