[coreboot-gerrit] Change in ...coreboot[master]: Untangle CBFS microcode updates

[Subrata Banik (Code Review)]

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29934 )

Change subject: Untangle CBFS microcode updates
......................................................................

Patch Set 9:
> > Patch Set 9:
> > > Patch Set 9:
> > > > This seems to have broken mb/googl/{sarien,arcada}. I'm not
>
> sure what's the root cause, but I just wanted to note this in here.
> We might need to flip a local config value in chromium.
> > > Hi Aaron,
> > >
> > > is it failing to build or to boot?
> >
> > boot
> > > Is that Cannon Lake or Coffee Lake? In the former case, you
> > > still need an external microcode binary. The Kconfig symbol
> > > for that case was renamed from CPU_MICROCODE_CBFS_GENERATE
> > > to CPU_MICROCODE_CBFS_EXTERNAL_BINS.
> > >
> > > Might also still be needed for some Coffee Lake steppings.
> >
> > It's whiskey lake which is using one of those two. I can't
>
> remember which honestly. I suspect it's an issue w/ an
> uncoordinated flag setting change. Once hear exact root cause I'll
> update.
>
> At least one report was an unsynchronized setting change. So we're
> good so far there.

this CL is required

https://chromium-review.googlesource.com/c/chromiumos/overlays/chromiumos-ov...