Attention is currently required from: Tarun Tuli, Dinesh Gehlot, Kapil Porwal.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69681 )
Change subject: soc/intel/meteorlake: Implement report_cache_info() function ......................................................................
Patch Set 1:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/69681/comment/5303a42e_71752874 PS1, Line 11:
Possible unwrapped commit description (prefer a maximum 72 chars per line)
Please fix.
https://review.coreboot.org/c/coreboot/+/69681/comment/730a0d06_f66e4df1 PS1, Line 12: Cache: Level 3: Associativity = 12 Partitions = 1 Line Size = 64 Sets = 32768 Please follow the comment style from https://review.coreboot.org/c/coreboot/+/69680/1//COMMIT_MSG
like, BUG, TEST and mention the CL from where it's backported.
https://review.coreboot.org/c/coreboot/+/69681/comment/a0f82ba2_11f6edb4 PS1, Line 16: signoff is missing