[coreboot-gerrit] Change in ...coreboot[master]: sb/intel/common/firmware: Add an option to use ifdtool

[Mario Scheithauer (Code Review)]

Mario Scheithauer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31639 )

Change subject: sb/intel/common/firmware: Add an option to use ifdtool
......................................................................

Patch Set 3:
> Patch Set 3:
>
> I would pretty much prefer the opposite, i.e. only modify
> the descriptor if asked to.
>
> In any case, as this implementation only affects the
> descriptor lock, a Kconfig choice would be better. Would
> make both Kconfig and Makefile simpler, I guess.

Just so I get it right, you mean that else branch of CONFIG_LOCK_MANAGEMENT_ENGINE in the Makefile doesn’t use ifdtool, right?