[coreboot-gerrit] [M] Change in coreboot[main]: acpi: Add support for DHRD size reporting

[Shuo Liu (Code Review)]

Attention is currently required from: Angel Pons, Arthur Heymans, Chen, Gang C, Christian Walter, Cliff Huang, Dinesh Gehlot, Eran Mitrani, Jakub Czapiga, Jeff Daly, Jincheng Li, Johnny Lin, Jonathan Zhang, Kapil Porwal, Lance Zhao, Lean Sheng Tan, Nick Vaccaro, Nico Huber, Patrick Rudolph, Paul Menzel, Sean Rhodes, Subrata Banik, Tarun, Tim Chu, Tim Wawrzynczak, Vanessa Eusebio, Werner Zeh.

Shuo Liu has posted comments on this change by Shuo Liu. ( https://review.coreboot.org/c/coreboot/+/82429?usp=email )

Change subject: acpi: Add support for DHRD size reporting
......................................................................

Patch Set 15:

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/82429/comment/c139148d_373fbf58?usp... :
PS15, Line 7: DHRD
> typo: DRHD

Done

File src/include/acpi/acpi.h:

https://review.coreboot.org/c/coreboot/+/82429/comment/aad287d1_41f52174?usp... :
PS15, Line 1852: unsigned long acpi_create_dmar_drhd(unsigned long current, u8 flags,
               : 				    u16 segment, u64 bar, u32 size);
> To avoid having to touch all platforms, would it make sense to have a specialised `acpi_create_dmar_ […]

I made some attempts between using npages or size, and didn't get clear benefit of  using npages because even using npages, we still need to do log() and we need to transform size to npages in the caller as well. Due to this reason I keep the size as of now.