TheThingsIndustries / generic-node-se

Generic Node Sensor Edition
https://www.genericnode.com
Other
108 stars 31 forks source link

Feature/system tracer #159

Closed elsalahy closed 3 years ago

elsalahy commented 3 years ago

Summary:

This update adds a system wide advanced and tiny tracer that can be used according to application need. Apps like basic_bootloader and freertos(basic, LoRaWAN) can use tiny tracer to reduce overhead. Apps like basic and basic_lorawan can use Advanced tracer

Closes #90

Also added a minor fix to freertos_lorawan app when testing the new tracer, relates to #88 and #99

Changes:

Notes for Reviewers:

I ran my tests, but I could have overlooked something. Please give it a run and test random apps and report if you faced any issue compiling or observing a trace.

Will update the cmake_dev (faster cmakes) when the PR is approved, to avoid multiple change with reviews.

Release Notes: (optional)

...

elsalahy commented 3 years ago

@marnixcro Thanks for the review, I will not update tools/cmake_dev now and I think we need to come up with a script that automates the generation of these files