Skip to content
This repository has been archived by the owner on Apr 5, 2024. It is now read-only.

scheduler: add HDMI switching #45

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

escoand
Copy link

@escoand escoand commented Sep 13, 2019

Add ability to switch HDMI connected displays with the scheduler as well.
Maybe we should rename the variables from BACKLIGHT to DISPLAY.

My RPi display is not usable currently. Is somebody willing to test it with this change?

@escoand
Copy link
Author

escoand commented Sep 14, 2019

I've to realize that's not gonna work as the cec-client on alpine has no RPi support compiled.

@escoand
Copy link
Author

escoand commented Sep 14, 2019

Tested Debian as well. cec-client and tvservice are hanging on my RPi2.

@escoand escoand force-pushed the hdmi_scheduler branch 4 times, most recently from 4ecb93a to 47ad7b8 Compare September 16, 2019 12:43
@escoand
Copy link
Author

escoand commented Sep 16, 2019

finally got a working solution, we have to build libcec by hand to get RPi support

@escoand
Copy link
Author

escoand commented Sep 29, 2019

@dansku @chrisys @shaunmulligan are u interested in upstreaming this?

@escoand
Copy link
Author

escoand commented Sep 29, 2019

Just for reference the alpine bug for this https://gitlab.alpinelinux.org/alpine/aports/issues/7754

@chrisys
Copy link
Member

chrisys commented Sep 30, 2019

@escoand yes, I'm absolutely interested in getting this merged! Thanks so much for sharing your work here.

I'll give it a test and review here, but in the meantime could you take a look here and update your commit message accordingly: https://github.com/balena-io-projects/balena-dash/blob/master/CONTRIBUTING.md

@escoand
Copy link
Author

escoand commented Sep 30, 2019

@chrisys yes, sure. So it has to be only a single commit? And what kind of commit is it? Minor?

@chrisys
Copy link
Member

chrisys commented Oct 1, 2019

@escoand yep just one of the commits, and minor is good for this as it's a (great) new feature!

@escoand
Copy link
Author

escoand commented Oct 6, 2019

@chrisys the code should be ready for merge.

@escoand
Copy link
Author

escoand commented Oct 23, 2019

Are there open problems? Or why is the PR still pending?

@chrisys
Copy link
Member

chrisys commented Jan 2, 2020

@escoand I tested this today finally - I couldn't get it to work on my monitor. I would prefer it if the changes leave the default behavior unchanged and add functionality; right now it looks like there's a risk of it being a breaking change as it defaults to using cec-client if the output from tvservice -s doesn't match. What are your thoughts on this? Is there a way to default to the existing method but change to cec-client if a compatible display is detected?

@escoand
Copy link
Author

escoand commented Jan 2, 2020

Sure, but there seem to be a lot interesting states https://www.raspberrypi.org/forums/viewtopic.php?t=52309

@escoand
Copy link
Author

escoand commented Jan 2, 2020

@chrisys what does tvservice -s show on your installation? And what is happening when you call the script or echo scan | cec-client -s -d 1 manually?

@ReubenBTalbott
Copy link

Is there any update on this? I would like to be able to sleep my HDMI monitor at night and all the documentation seems to say that only the RPi screen is able to do that at the moment.

@escoand
Copy link
Author

escoand commented Dec 22, 2020

With this PR it should work but switched to Screenly and don't use this anymore.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants