Make the CRC Platform Abstraction Layer multi-context-safe#90
Conversation
22074e5 to
13a4548
Compare
|
I just realized that this PR targets the |
71273fe to
eda5991
Compare
|
Tx to solve conflicts.
|
13a4548 to
a709937
Compare
|
Fix the conflicts by rebasing my branch ( |
71161d0 to
a64d510
Compare
9a062c3 to
3c51576
Compare
|
@TofMassilia13320 I noticed you force-pushed these 2 commits a couple of days ago onto the upstream
This rewrote the commit history for the last 3 weeks. Just a suggestion: as a general rule, I only force-push on my own forks where I'm the sole person working, and I avoid it on upstream branches shared by multiple developers, since it rewrites shared history. Anyone who has already pulled or branched off the old commits ends up with diverging history and has to manually reconcile or reset to match the new one. Force-pushing on an upstream branch usually makes sense when you need to remove something sensitive from history, like leaked credentials, secrets, or accidentally committed files, where rewriting history is actually necessary. For regular fixes or updates, though. Pushing new commits on top, rather than rewriting old ones, would prevent this issue for anyone else working off this branch. |
|
@parmi93, |
stse_crc16_context_tstruct context.stse_platform_Crc16_ContextInit()andstse_platform_Crc16_ContextRelease()PAL APIs.stse_platform_Crc16_Accumulate()PAL API.stse_platform_Crc16_Calculate()PAL API.These changes resolve issue #65 by allowing users to adapt the CRC16 PAL implementation to scenarios where communication with multiple STSAFE devices takes place concurrently.
Additionally, they make it possible to implement thread-safe CRC16 PAL APIs when CRC computation is performed by a shared hardware peripheral. This ensures that concurrent access from multiple threads can be managed safely, even when those threads are not necessarily communicating with different STSAFE devices.