[coreboot-gerrit] Change in coreboot[master]: lib/crc_byte: Add CRC32 implementation

[Patrick Rudolph (Code Review)]

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37753 )

Change subject: lib/crc_byte: Add CRC32 implementation
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/37753/1/src/include/crc_byte.h 
File src/include/crc_byte.h:

https://review.coreboot.org/c/coreboot/+/37753/1/src/include/crc_byte.h@52 
PS1, Line 52: ~0;
> Why the negation here and for the result? Does it not work when you just start with 0? If so, do we  […]

That's described in  ITU I.363.5 /ITU V.42.
Starting with 0 is also possible, but would give different results.