search

bristlemouth / bm_protocol

Primary Bristlemouth firmware repository
https://www.bristlemouth.org/
Apache License 2.0
12 stars 8 forks source link

Adds Linked List To Sensor Sampler #155

Open matt001k opened 3 months ago

matt001k commented 3 months ago

Expands sensor list array to utilize linked list Adds static allocation based linked list module in lib/common Updates sensorSamplerAdd calls to utilize update API

This needs to be tested on actual hardware (I don't have any yet 😄, idk if a dev kit can be sent before the 26th). Wanted to get my hands a bit dirty by digging in the code a bit. Thought taking on a TODO would be a good first step. Utilized a LL library I wrote a while back that does not utilize any heap.

I was able to run the unit tests and might add unit tests for both of these lib items.

towynlin commented 2 months ago

Thanks @matt001k! 👏 I approved CI workflows to run, and it caught a failure, so check that out.

towynlin commented 2 months ago

@matt001k Do you have all the rights to contribute the linked list library to Bristlemouth under the Apache 2 license?

matt001k commented 2 months ago

@towynlin yep! This is something I developed on my own time a while back when challenging myself to use data structures with static allocation.