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

firmware: Adding DEBUG mode to each module #346

Merged
merged 1 commit into from
Sep 15, 2022

Conversation

CW-75
Copy link
Collaborator

@CW-75 CW-75 commented Sep 13, 2022

Contribution description

This PR adds debug mode in each module, This will implement the capacity to test each module and print all data processed sent or saved. Was added debug mode in kconfig. you should load in any module used by a test or an example application.

Testing procedure

Load the menuconfig

make menuconfig flash term

Then enter in debug mode menu, you should options, those options are the sub-categories from firmware modules. seek and select the wished module to run the debug mode inside it.

Issues/PRs references

Copy link
Contributor

@luisan00 luisan00 left a comment

Choose a reason for hiding this comment

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

almost done, review suggested change and do a squash

firmware/sys/uniqueid/uniqueid.c Outdated Show resolved Hide resolved
@CW-75 CW-75 force-pushed the feature/debug_include branch from 1731601 to 88655df Compare September 15, 2022 13:17
@CW-75 CW-75 requested a review from luisan00 September 15, 2022 13:22
Copy link
Contributor

@luisan00 luisan00 left a comment

Choose a reason for hiding this comment

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

looks good, going ahead!

@CW-75 CW-75 merged commit e93377f into Mesh4all:main Sep 15, 2022
@CW-75 CW-75 deleted the feature/debug_include branch September 15, 2022 13:49
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.

2 participants