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

Turn off HDMI on startup #71

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

htruong
Copy link
Contributor

@htruong htruong commented Dec 12, 2020

Turning of HDMI saves 25mA of power

Turning of HDMI saves 25mA of power
@htruong
Copy link
Contributor Author

htruong commented Dec 12, 2020

Source of claim

Also verified by me very unscientifically... Comparison

Before

IMG_4940

After

IMG_4941

@@ -22,6 +22,7 @@ BR2_LINUX_KERNEL_NEEDS_HOST_OPENSSL=y
BR2_LINUX_KERNEL_LZ4=y
BR2_PACKAGE_RPI_FIRMWARE=y
BR2_PACKAGE_RPI_FIRMWARE_X=y
BR2_PACKAGE_RPI_USERLAND=y
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're installing all of RPi userland (e.g. raspivid, raspistill, dtoverlay etc.) here. Is that what we want?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I kinda want to keep raspistill and raspivid to do debugging for end-users anyways. Those are nice.

All the others can go but since we can't patch the makefile it'd be more pain than it's worth to not install them. What do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe TODO in another PR #75

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's just that some userland tools break, as necessary libraries are not found.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, I will clean it up later in issue #75

@htruong
Copy link
Contributor Author

htruong commented Dec 14, 2020

I think this is probably safe to merge. Should we do it @tuyenld and @dgsiegel ?

@tuyenld
Copy link
Contributor

tuyenld commented Dec 14, 2020

@htruong
I concern about "Preview Mode" on Pi camera. For example, raspistill displays image on HDMI even we are running Pi Lite version.
This mode virtually executes on GPU and it has a built-in "Overlay" feature.
https://picamera.readthedocs.io/en/release-1.13/recipes1.html?highlight=overlay#overlaying-images-on-the-preview
If you turn it off, we may have problems to take advantage of the overlay feature.

@dgsiegel
Copy link
Contributor

I think this is probably safe to merge. Should we do it @tuyenld and @dgsiegel ?

Not until we understood all implications of somewhat broken apps and libs with userland?

On another note, could you try if you can get the same result with executing vcgencmd display_power 0? That might help as well :)

@htruong htruong marked this pull request as draft December 22, 2020 11:10
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.

None yet

3 participants