Attention is currently required from: Tarun Tuli, Subrata Banik, Kapil Porwal.
Dinesh Gehlot 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 2:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/69681/comment/97954cfd_aac8341e PS1, Line 11:
Possible unwrapped commit description (prefer a maximum 72 chars per line) […]
Ack
https://review.coreboot.org/c/coreboot/+/69681/comment/aa5865d7_bcb4fd5d 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 […]
Ack
https://review.coreboot.org/c/coreboot/+/69681/comment/a9c28f13_1c0262b4 PS1, Line 16:
signoff is missing
Ack