Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consider making the CRC size configurable #15

Open
skliper opened this issue Apr 22, 2022 · 0 comments
Open

Consider making the CRC size configurable #15

skliper opened this issue Apr 22, 2022 · 0 comments

Comments

@skliper
Copy link
Contributor

skliper commented Apr 22, 2022

There are two CRC algorithm types specified in the platform config. Are both of these using 32-bit CRC? What if we add another CRC algo later that isn't 32-bit? It might make sense to define the type that holds a CRC result in platform config as well.

Comment is on the CS_ComputeEepromMemory function regarding the uint32 ComputedCSValue parameter.

Imported from GSFCCFS-1326

skliper added a commit to skliper/CS that referenced this issue Sep 9, 2022
skliper pushed a commit to skliper/CS that referenced this issue Sep 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant