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

Add 8-bit bound algorithms covered by e.g. https://docs.rs/crc_all/latest/crc_all/ or https://reveng.sourceforge.io/crc-catalogue not yet covered by InvertedTomato.Crc #13

Open
maettu-this opened this issue Jul 24, 2023 · 2 comments

Comments

@maettu-this
Copy link
Contributor

maettu-this commented Jul 24, 2023

8-bit bound algorithms covered by Rust crc_all not yet covered by InvertedTomato.Crc:

  • CRC-8/AUTOSAR
  • CRC-8/BLUETOOTH
  • CRC-8/GSM-A
  • CRC-8/GSM-B
  • CRC-8/LTE
  • CRC-8/NRSC-5
  • CRC-8/OPENSAFETY
  • CRC-8/SAE-J1850
  • CRC-8/SMBUS
  • CRC-8/TECH-3250

https://reveng.sourceforge.io/crc-catalogue/1-15.htm lists them as well.

  • CRC-16/CMS
  • CRC-16/GSM
  • CRC-16/LJ1200
  • CRC-16/NRSC-5
  • CRC-16/OPENSAFETY-A
  • CRC-16/OPENSAFETY-B
  • CRC-16/PROFIBUS

https://reveng.sourceforge.io/crc-catalogue/16.htm lists them as well. Interestingly, this link also reveals "CRC-A" as "CRC-16/ISO-IEC-14443-3-A".

  • CRC-24/BLE

  • CRC-24/INTERLAKEN

  • CRC-24/LTE-A

  • CRC-24/LTE-B

  • CRC-24/OS-9

  • CRC-32/AUTOSAR

https://reveng.sourceforge.io/crc-catalogue/17plus.htm lists them as well. Even more:

  • CRC-64/GO-ISO
  • CRC-64/MS
  • CRC-64/REDIS

Maybe better to extract this into a separate issue.

Originally posted by @maettu-this in #9 (comment)

@maettu-this maettu-this changed the title Add 8-bit bound algorithms covered by e.g. [Rust crc_all](https://docs.rs/crc_all/latest/crc_all/) not yet covered by InvertedTomato.Crc Add 8-bit bound algorithms covered by e.g. https://docs.rs/crc_all/latest/crc_all/ or https://reveng.sourceforge.io/crc-catalogue not yet covered by InvertedTomato.Crc Jul 24, 2023
@invertedtomato
Copy link
Owner

@maettu-this , sorry for the slow reply on this one. If you made a PR on this I'd be happy to accept it.

@maettu-this
Copy link
Contributor Author

@invertedtomato, isn't it just adding the e.g....

width=8 poly=0x2f init=0xff refin=false refout=false xorout=0xff check=0xdf residue=0x42 name="CRC-8/AUTOSAR"

...of the mentioned links in the InvertedTomato.Crc required format? Doesn't sound like that much of effort. But when it comes to naming the CRCs in the InvertedTomato.Crc preferred name I think you are the expert much more than me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants