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

Check if /dev/fb1 exists and select /dev/fb0 when not. #1580

Closed
wants to merge 1 commit into from

Conversation

holgern
Copy link
Contributor

@holgern holgern commented Sep 22, 2020

When using the hdmi, /dev/fb0 is available and not /dev/fb1. The change check if /dev/fb1 exists and when not , switch to /dev/fb0

When using the hdmi, /dev/fb0 is available and not /dev/fb1. The change check if /dev/fb1 exists and when not , switch to /dev/fb0
@rootzoll
Copy link
Collaborator

Interesting find. Does "using the hdmi" means that a HDMI display is "connected"? Because this could also then be used to "auto-detect" what is at the moment a "manual switch": https://github.com/rootzoll/raspiblitz/blob/v1.6/FAQ.md#can-i-run-the-raspiblitz-without-a-displaylcd

@holgern
Copy link
Contributor Author

holgern commented Sep 22, 2020

I have connected the hdmi to a 7zoll LCD.

The hdmi is always /dev/fb0 and the spi display is always /dev/fb1,

I think it is possible to use fb1 and fb0 for autodetect.

@rootzoll
Copy link
Collaborator

@holgern did you tested this also with the LCD hat on and Touchscreen on? In my tests with the PR it does not show the images/qr anymore and instead shows the pi-terminal. Will push this PR to v1.7 milestone.

@rootzoll rootzoll added this to the 1.7 Release milestone Sep 23, 2020
@rootzoll rootzoll changed the base branch from v1.6 to dev March 16, 2021 18:19
@rootzoll rootzoll changed the base branch from dev to v1.6 March 16, 2021 18:20
rootzoll added a commit that referenced this pull request Mar 16, 2021
@rootzoll
Copy link
Collaborator

@holgern I moved your code into a new PR because it needed to be rebased on dev branch --> #2064

So this fix got merged by the PR #2064 and will be part of the v1.7 release. Closing this one.

@rootzoll rootzoll closed this Mar 16, 2021
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

2 participants