[coreboot-gerrit] Change in coreboot[master]: mb/google/dedede: Enable Intel Speed Shift Technology

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39477 )

Change subject: mb/google/dedede: Enable Intel Speed Shift Technology
......................................................................

Patch Set 5:
> Patch Set 5:
>
> (2 comments)
> > Patch Set 5:
> >
> > Not to be too much of a stickler, but this is the whole reason for mainboard_silicon_init_params(FSP_S_CONFIG *params). It gives you a chance to update FSP_S UPDs before FSP-S runs. And if 'dee and 'doo are all the variants you have right now, you could just leave the check and setting of ISST in mainboard.c for now.
>
> I think what Karthik identified was yyou

Karthik and I chatted about this, it was a misunderstanding on my part.