[coreboot-gerrit] Change in ...coreboot[master]: cpu/intel/car/core2: Update microcode in CAR setup

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30775 )

Change subject: cpu/intel/car/core2: Update microcode in CAR setup
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/#/c/30775/4//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/30775/4//COMMIT_MSG@9 
PS4, Line 9: This updates the BSP microcode during CAR setup.
> Updating microcode during CAR hangs the system. […]

I just wondered if it's worth to have a conditional
path here. If we know that updating ucode in ramstage
is good enough, why not do it there? We'd not only
save some lines but also the additional CBFS walk.