Change in coreboot[master]: soc/intel/common/acpi: Correct IPC sub command for reading LPM requir...

[Ethan Tsao (Code Review)]

Attention is currently required from: Ethan Tsao, Paul Menzel, Tim Wawrzynczak, Subrata Banik, Raj Astekar, Patrick Rudolph, Felix Held.
Ethan Tsao has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58317 )

Change subject: soc/intel/common/acpi: Correct IPC sub command for reading LPM requirement
......................................................................

Patch Set 2:

(4 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/58317/comment/1340c51d_3ba2bddb 
PS1, Line 9: Modify IPC sub command to 2 from 0 for reading LPM requirement from PMC.
> Please add that URL.

Hi Paul,
Can you please clarify what URL you are looking for here?

Patchset:

PS1:
> the indentation of the new code looks a bit off to me. […]

Thanks reminding. Tab size of 4 is accidently used. Has update patch.

File src/soc/intel/common/block/acpi/pep.c:

https://review.coreboot.org/c/coreboot/+/58317/comment/2c404ff1_becb296f 
PS1, Line 51:
> line up with `PMC_IPC_CMD_RD_PMC_REG` above?

Thanks reminding. Has update patch.

File src/soc/intel/common/block/include/intelblocks/pmc_ipc.h:

https://review.coreboot.org/c/coreboot/+/58317/comment/9b2e8435_9955fd2c 
PS1, Line 42:
> remove the extra tab here

Thanks reminding. Has update patch.