TheThingsIndustries / generic-node-se

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

Feature/basic SE #59

Closed elsalahy closed 4 years ago

elsalahy commented 4 years ago

Summary:

Adds basic support for the secure element. Closes #30 Closes #37 Relates to #55

Changes:

Notes for Reviewers:

Still not happy with the LW app performance (the SE integration), so I will continue the improvements and open another PR

Release Notes: (optional)

...

elsalahy commented 4 years ago

I didn't merge this PR as there is a problem where the device would halt when we introduce the secure element to the basic app

Will open a descriptive issue once I found a concrete scenario

elsalahy commented 4 years ago

The source of the issue is still being investigated in #60 Since this application works if the UART printing is disabled and works with minimal printing, I will merge this and keep the investigating the issue and open a fix when ready.