Attention is currently required from: Angel Pons, Light. Anastasia Klimchuk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/62764 )
Change subject: ich_descriptors.c: Ensure unsigned types >=0 on to prevent underflow error ......................................................................
Patch Set 14:
(2 comments)
Commit Message:
https://review.coreboot.org/c/flashrom/+/62764/comment/7a02235b_7a29ee85 PS12, Line 7: ich_descriptors.c: Assert unsigned types >=0 on to prevent underflow error
Done
The second part is left: to fit commit description into 72 chars, and it is 74 at the moment. I think you can just drop the last word (error), so that will be "to prevent underflow".
Patchset:
PS14: One more thing I wanted to say: at the moment this patch depends on the previous one, because they are in the chain and modifying the same file. But technically the diffs are independent of each other. This one seems almost ready, the previous one has an open question. So, if you want, you can either swap this patch with previous in the chain, or move it out of the chain entirely. Then it can be merged independently when it's ready. Currently, one comment left :)