[coreboot-gerrit] Change in coreboot[master]: soc/intel/broadwell: Revise SA lockdown sequence

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46324 )

Change subject: soc/intel/broadwell: Revise SA lockdown sequence
......................................................................

Patch Set 4:
> Patch Set 4:
>
> (1 comment)
>
> This would be a lot easier to review if you wouldn't change
> semantics and introduce macros in the same patch. Macro
> names that I also can't review because I can't find them in
> docs (and the limited code in coreboot provides no hint if
> the names make sense).

Good point. I'll split up the actual "revise" part from the "use macros" part