Attention is currently required from: Jonathan Zhang, Johnny Lin, Martin Roth.
Ed Sharma has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68637 )
Change subject: commonlib/fsp_relocate: Fix Coverity Issues ......................................................................
Patch Set 2:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/68637/comment/bee7d6a7_54fc0744 PS1, Line 7: commonlib/fsp_relocate: Fix Coverity Issues
Pls. refer to these existing coverity related commits: […]
I looked and reviewed all of these changes. They are not about the same Coverity errors as reported for my issue so I could not leverage help from them.
https://review.coreboot.org/c/coreboot/+/68637/comment/428f3b9c_6545d329 PS1, Line 9: Recently committed change 1df1cf9 resulted in some Coverity
Use [CB: gerrid ID] as the hyper-link.
There is no gerrit ID. The Coverity issues were reported as comments for the previous CR which was already committed.
https://review.coreboot.org/c/coreboot/+/68637/comment/0b5bff34_5eb88bc1 PS1, Line 14: This code is tested with FSP version 33A for DeltaLake boot
FSP --> Intel CooperLake-SP FSP […]
Done
-
Ed Sharma (Code Review)