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

Add Suspend functions #3112

Merged
merged 11 commits into from
Jun 3, 2018
Merged

Conversation

drashna
Copy link
Member

@drashna drashna commented Jun 2, 2018

Adds the suspend_power_down_kb and suspend_wakeup_init_kb functions referenced by the rgbmatrix docs.
(they don't exist currently, but are present in wilba's fork, IIRC).

@drashna drashna changed the title Add Suspend and Keyboard Init functions Add Suspend functions Jun 2, 2018
@jackhumbert jackhumbert merged commit cb91320 into qmk:master Jun 3, 2018
@jackhumbert
Copy link
Member

Awesome - thanks!

@drashna drashna deleted the fix_suspend_wakeup_code branch June 8, 2018 22:16
@jpetermans jpetermans mentioned this pull request Mar 29, 2019
11 tasks
yamad pushed a commit to yamad/qmk_firmware that referenced this pull request Apr 10, 2019
* Add suspend functions

* Disable RGB code if it's disabled

* Add suspend code to ChibiOS for future compatibility

* Add keyboard_init functions

* Change where references so it will compile

* Wrong command chained in wake up kb function

* Fix non-feature file changes

* Add documentation

* Re-add matrix init docs

* add rgblight code to example

* Remove keyboard init stuff for separate PR
### `keyboard_init_*` Function Documentation

* Keyboard/Revision: `void suspend_power_down_kb(void)` and `void suspend_wakeup_init_user(void)`
* Keymap: `void suspend_power_down_kb(void)` and `void suspend_wakeup_init_user(void)`
Copy link

Choose a reason for hiding this comment

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

Shouldn't this mention *_user for keymap, and *_kb for Keyboard/Revision?

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

Successfully merging this pull request may close these issues.

3 participants