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

Fix Ctrl-Command-Power reboot key combo #27

Merged
merged 3 commits into from
Feb 14, 2024
Merged

Conversation

morio
Copy link
Collaborator

@morio morio commented Feb 14, 2024

After the reboot key combo is held down the Mac OS does a talk reg 2 continuously until the modifier keys are lifted. The talk reg 2 wasn't being updated during that loop so system would start.

Now reg 2 is checked right before it is sent over the ADB bus so the modifier keys are always current.

Another fix was done with the power button. The power key up state was malformed so the power button was stuck down after pressing it. This has been fixed.

Removed code related to double queuing the power button on the USB side. The double queuing is done now on the ADB side. The power button takes up both key press packets so it need to be queued twice and sent on the same talk register 0.

After the reboot key combo is held down the Mac OS does a talk reg 2
continuously until the modifier keys are lifted. The talk reg 2 wasn't
being updated during that loop so system would start.

Now reg 2 is checked right before it is sent over the ADB bus so
the modifier keys are always current.

Another fix was done with the power button. The power key up state
was malformed so the power button was stuck down after pressing it.
This has been fixed.

Removed code related to double queuing the power button on the USB side.
The double queuing is done now on the ADB side. The power button
takes up both key press packets so it need to be queued twice and
sent on the same talk register 0.
@morio morio marked this pull request as ready for review February 14, 2024 21:06
@aperezbios aperezbios merged commit 72e43c3 into master Feb 14, 2024
1 check passed
@aperezbios aperezbios deleted the fix/reset-key-combo branch February 14, 2024 21:47
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.

2 participants