[coreboot-gerrit] [L] Change in coreboot[main]: acpi: Add acpigen_write_OSC_pci_domain

[Paul Menzel (Code Review)]

Attention is currently required from: Arthur Heymans, Christian Walter, Cliff Huang, Johnny Lin, Jonathan Zhang, Lance Zhao, Shuo Liu, Tim Chu, Tim Wawrzynczak.

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81375?usp=email )

Change subject: acpi: Add acpigen_write_OSC_pci_domain
......................................................................

Patch Set 12:

(3 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/81375/comment/40761d52_3e5e6d02 :
PS12, Line 11: 
Tested how?

File src/acpi/acpigen_pci.c:

https://review.coreboot.org/c/coreboot/+/81375/comment/205da230_cf3044bc :
PS12, Line 98: With
Width

https://review.coreboot.org/c/coreboot/+/81375/comment/54ef3f72_12947749 :
PS12, Line 98: Name - With,  Source, Offset, Description
             :  * --------------------------------
             :  * QSUP - DWord, Local7, 0x00,   Query support
             :  * RETE - DWord, Arg3,   0x00,   Returned errors
             :  * SUPP - Dword, Arg3,   0x04,   PCIe Features that OS supported
             :  * CTRL - Dword, Arg3,   0x08,   PCIe Features that firmware grant control to OS
             :  * OTRL - Dword, Local7, 0x08,   PCIe Features that OS requests for control
             :  * SUPC - Dword, Arg3,   0x0C,   CXL Features that OS supported
             :  * CTRC - Dword, Arg3,   0x10,   CXL Features that firmware grant control to OS
             :  * OTRC - Dword, Local7, 0x10,   CXL Features that OS requests for control
If it’s a table, I wouldn’t separate the columns by a comma.