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

[Furquan Shaikh (Code Review)]

Furquan Shaikh 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:
> > 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.

Woops.. I had typed that and removed it after you posted your latest comment, but somehow it didn't get removed completely.