[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/cse: Helper function to set D0I3 for all HECI devices

[Subrata Banik (Code Review)]

Attention is currently required from: Arthur Heymans,  Felix Singer, Tim Wawrzynczak, Angel Pons, Nick Vaccaro, Patrick Rudolph, EricR Lai.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/60404 )

Change subject: soc/intel/common/cse: Helper function to set D0I3 for all HECI devices
......................................................................

Patch Set 2:

(1 comment)

File src/soc/intel/common/block/cse/cse_eop.c:

https://review.coreboot.org/c/coreboot/+/60404/comment/1a932f3e_5f12263a 
PS2, Line 17: /* SoC callback to set D0I3 for all HECI devices */
            : __weak void soc_heci_set_d0i3(void) { /* no-op */ }
> > > Is the weak empty function meaningful on most SoC's or should it be implemented by all and this  […]

Yes, In my mind still that __weak declaration was running hence wrongly commented about guarding the declaration. Abandon this CL now as I have move the declaration into CB:60405