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 some debugging doc, after i found out how to print debug messages #536

Merged
merged 2 commits into from
Jul 22, 2016

Conversation

pvinis
Copy link
Contributor

@pvinis pvinis commented Jul 18, 2016

No description provided.

@@ -152,7 +152,7 @@ This allows you to use the system and audio control key codes.

`CONSOLE_ENABLE`

TODO
This allows you to print messages that can be read using `hid_listen`. Add this to your `Makefile`, and set it to `yes`. Then put `println`, `printf`, etc. in your keymap or anywhere in the `qmk` source. Finally, open `hid_listen` and enjoy looking at your printed messages.
Copy link
Contributor

Choose a reason for hiding this comment

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

Might want to add a link with where to get the hid_listen tool.

@jackhumbert
Copy link
Member

Nice - thanks!

@jackhumbert jackhumbert merged commit d5d2eb8 into qmk:master Jul 22, 2016
@fredizzimo
Copy link
Contributor

I was going to comment on this before it was merged, but never had time to do it.

I think that the

dprint
dprintf
dprintln
dmsg

Functions should be documented too.

The output of those are are controlled by the debug enable magic and debug_config, and it can further be refined to include/exclude matrix, keyboard and/or mouse debug output.

These could be used for keymaps as well. Theoretically the unused bits of the debug config could be used too, and you could even bind keys to modify that, and it could be saved in the eeprom. But there's no guarantee that it will stay so in the future, so it's probably not a very good idea.

I'm not sure how and where it should be documented and how much detail would be needed though.

@pvinis
Copy link
Contributor Author

pvinis commented Jul 23, 2016

that sounds useful. well, i didnt know what they can do, so i only used 'print'. all the more reason to document them in some way..

ryaninvents pushed a commit to ryaninvents/qmk_firmware that referenced this pull request Aug 12, 2016
add some debugging doc, after i found out how to print debug messages
@pvinis pvinis deleted the debug-doc branch September 21, 2016 09:18
BlueTufa pushed a commit to BlueTufa/qmk_firmware that referenced this pull request Aug 6, 2021
* add json lint

* fix

* fix

* crash test

* final fix
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.

4 participants