Attention is currently required from: Anil Kumar K, Cliff Huang, Paul Menzel, Subrata Banik.
Jérémy Compostella has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/76762?usp=email )
Change subject: cbfstool: Add relocation support for EFI binaries
......................................................................
Patch Set 4:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/76762/comment/64b9f78f_daa9f171 :
PS3, Line 14: Platform specific
Maybe lower case? platform specific
Done
File src/commonlib/fsp_relocate.c:
https://review.coreboot.org/c/coreboot/+/76762/comment/c718c511_1881d901 :
PS3, Line 284: return 0;
Why does the return value need to be changed?
Excellent question: I could not find any reason why this function would return -1 on success but it was not an issue so far as no code was checking its return value.
Oher similar functions such as `te_relocate` do end on a `return 0`.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/76762?usp=email
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I5e98ffd1d864cf9ac675421471d888334d8ab28b
Gerrit-Change-Number: 76762
Gerrit-PatchSet: 4
Gerrit-Owner: Jérémy Compostella
jeremy.compostella@intel.com
Gerrit-Reviewer: Anil Kumar K
anil.kumar.k@intel.com
Gerrit-Reviewer: Cliff Huang
cliff.huang@intel.com
Gerrit-Reviewer: Subrata Banik
subratabanik@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@mailbox.org
Gerrit-Attention: Anil Kumar K
anil.kumar.k@intel.com
Gerrit-Attention: Cliff Huang
cliff.huang@intel.com
Gerrit-Attention: Subrata Banik
subratabanik@google.com
Gerrit-Attention: Paul Menzel
paulepanter@mailbox.org
Gerrit-Comment-Date: Tue, 01 Aug 2023 00:35:18 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Paul Menzel
paulepanter@mailbox.org
Gerrit-MessageType: comment