Attention is currently required from: Alok Agarwal, Intel coreboot Reviewers, Jayvik Desai, Kapil Porwal, Pranava Y N, Ronak Kanabar, srinivas.kulkarni@intel.com.
Subrata Banik has posted comments on this change by Alok Agarwal. ( https://review.coreboot.org/c/coreboot/+/86949?usp=email )
Change subject: src/soc/intel/pantherlake: Adopt the MEMORY_INFO_DATA_HOB. ......................................................................
Patch Set 4:
(5 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/86949/comment/9c1ce02e_fbc73185?usp... : PS4, Line 7: src remove `src`
https://review.coreboot.org/c/coreboot/+/86949/comment/23bb1d28_dc9cc9cd?usp... : PS4, Line 7: . no `.` in title
https://review.coreboot.org/c/coreboot/+/86949/comment/90b671f7_85fb6912?usp... : PS4, Line 8: : Update MEMORY_INFO_DATA_HOB, child structures and their data for SMBIOS type 16/17 design change. : wrap up text
https://review.coreboot.org/c/coreboot/+/86949/comment/00dfd783_85ed04eb?usp... : PS4, Line 11: 406375765 please file a bug to understand this failure better. IMO, this is due to change in mem info data hob which is part of the new FSP release. There is no value to change the code unless we first land the FSP headers.
https://review.coreboot.org/c/coreboot/+/86949/comment/588b918e_c4e482aa?usp... : PS4, Line 12: Able to build google/fatcat. Use a meaningful TEST message to explain what issue this CL resolves.