[coreboot-gerrit] Change in coreboot[master]: src/soc/intel: Enable PCH M.2 RTD3 flow to allow system to enter S0i3.2

[Sukumar Ghorai (Code Review)]

Sukumar Ghorai has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43600 )

Change subject: src/soc/intel: Enable PCH M.2 RTD3 flow to allow system to enter S0i3.2
......................................................................

Patch Set 10:
> Patch Set 10:
> > Patch Set 10:
> > > Patch Set 10:
> > > > Patch Set 10:
> > > >
> > > > Hi Duncan & Shreesh, any update this week?
> > >
> > > Patches have been at https://review.coreboot.org/c/coreboot/+/46262/9 but they need tested on real hardware.
> >
> > Hi Duncan, your patch CL:46262 is not the complete implementation of D3Cold requirement. would you pls review this patch here, we need this patch in mainline soon
>
> Could you indicate what is missing?  My patch is implementing what is done here in the SSDT.  We aren't going to merge static ASL code into the SOC which has dependencies on a mainboard-specific implementation.

RP09 _ON/_OFF method required for D3Cold implementaiton. 
Pls review the changes.