[coreboot-gerrit] Change in coreboot[master]: util/mb/google/hatch: update CRC calculation for correctness

[Paul Fagerburg (Code Review)]

Paul Fagerburg has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35604 )

Change subject: util/mb/google/hatch: update CRC calculation for correctness
......................................................................

Patch Set 2:

(1 comment)
> Patch Set 2:
> > Patch Set 2:
> >
> > (1 comment)
>
> Ah, I see the discussion reference to https://docs.python.org/3/library/zlib.html
> Please put this link in for context.

Link and minimal context added.

https://review.coreboot.org/c/coreboot/+/35604/2/util/mainboard/google/hatch... 
File util/mainboard/google/hatch/kconfig.py:

https://review.coreboot.org/c/coreboot/+/35604/2/util/mainboard/google/hatch... 
PS2, Line 68: .
> I am not sure what the original intended rules are or how the 'correct' value is determined. […]

The second option (treat as unsigned) is what we've been doing in the manual process for v2 HWIDs, so that's what the code will automatically do now.