[coreboot-gerrit] Change in coreboot[master]: soc/intel/alderlake: Skip setting D0I3 bit for HECI devices

[Angel Pons (Code Review)]

Attention is currently required from: Maulik V Vaghela, Tim Wawrzynczak, Subrata Banik, Patrick Rudolph.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57815 )

Change subject: soc/intel/alderlake: Skip setting D0I3 bit for HECI devices
......................................................................

Patch Set 4:

(1 comment)

Commit Message:

https://review.coreboot.org/c/coreboot/+/57815/comment/b3660d29_86e7cc3f 
PS4, Line 10: if coreboot decides to send EOP msg at post.
If possible, I'd like to know the reasoning for this change.

I would guess it's because HECI devices can't be used when the D0I3 bit is set? If so, why can't coreboot simply unset the D0I3 bit before sending EOP and set it again afterwards?

And if FSP no longer sets the D0I3 bit, doesn't coreboot need to set D0I3?