-
Notifications
You must be signed in to change notification settings - Fork 215
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
the sequoia t8103/t8112 display brightness workaround results in black screen on some devices #406
Comments
Some more details on my installation:
The bug does not seem to be entirely deterministically. When I first rebooted after installing the new version it worked fine. On the next day when I booted (with almost entirely empty battery) the screen stayed dark. Trying again now after a few days the backlight stays off during boot but comes back once Linux is up. I now have m1n1 built with #399 and try to collect logs next time it happens. EDIT: here is a first log from a run where the backlight stayed off during boot but came back later |
I think there's a set brightness for the dcp iboot protocol. Trying to set brightness during display configure might worth a try. Annoying (for debugging) that this doesn't reproduce. |
Some dmesg output from previous boots below. Broken:
Working:
One obvious difference is |
Both reports are on j413 and I just confirmed that that model does not need the workaround, so let's change the logic. How about I'll also add a delay to the poweroff/on sequence since it happening too quickly is the most obvious explanation for why this breaks (DCP is full of stupid races...). |
It seems this model does not need it *and* it somehow causes nondeterministic problems. Also add a delay to the poweroff/on sequence to be more conservative. Fixes: #406 Signed-off-by: Hector Martin <[email protected]>
There are two reports m1n1 with cdb6d41 not working as expected. Display configuration in m1n1 results in a black screen.
The text was updated successfully, but these errors were encountered: