apache / apache/nuttx

[13.0] CRC cleanup.

Open
#16,174 1 comment 1 reaction 1 assignee Claimed by @anchao View on GitHub
Type: Enhancement
Dominant language
C
Stars
4k
Forks
1.7k
Avg merge
1d 17h
Merged PRs (30d)
237

Description

### Is your feature request related to a problem? Please describe.

* Current CRC implementation in NuttX in some areas is not compatible with other platforms (i.e. Linux).
* There various CRC16 and CRC32 algorithms out there, while there is no one common standard.
* CRC changes impacts various storage, transmission, etc, and may impact self-compatibility.

### Describe the solution you'd like

* We need to provide implementations for other commonly used CRC.
* We need explicit implementation of various CRC with clearly distinguishable function name or parameters.
* In order to maintain backward-compatibility any change in defaults may happen in next major release (13.0).
* Core crc defaults should stay untouched.
* Where necessary and possible some components crc implementation may change in order to improve compatibility with other platforms.
* User code may choose what CRC implementation to use, but core components should not allow KConfig like CRC algo selection as this will impact NuttX self-compatibility.

### Describe alternatives you've considered

_No response_

### Verification

- [x] I have verified before submitting the report.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.