-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Update graphics-utilities.adoc #2123
Conversation
Related PR in userland repo - man page update - raspberrypi/userland#703 |
@@ -138,7 +138,7 @@ Returns the throttled state of the system. This is a bit pattern - a bit being s | |||
|
|||
==== measure_temp | |||
|
|||
Returns the temperature of the SoC as measured by the on-board temperature sensor. | |||
Returns the temperature of the SoC as measured by its internal temperature sensor. On Raspberry Pi 4 the `pmic` subcommand can be used to instead return the temperature of the PMIC. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not entirely clear if this means vcgencmd pmic
or vcgencmd measure_temp pmic
?
Also, I may just be reading it wrong but the "instead" implies that the former option isn't available and only the latter option is available? Maybe it's worth rewording to use "also" to clarify that both options are available, IYSWIM?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've changed the wording slightly.
(A quick test on a Pi 3B+ and Pi 1 reveals that on pre-Pi4 boards, vcgencmd measure_temp pmic
will silently ignore the pmic
parameter and just return the temperature of the SoC).
Ready to merge? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Document
pmic
subcommand tovcgencmd measure_temp