[S] Change in coreboot[master]: security/intel/txt: Helper function to disable TXT

[Arthur Heymans (Code Review)]

Attention is currently required from: Tarun Tuli, Subrata Banik, Angel Pons, Lean Sheng Tan, Werner Zeh.

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/71574 )

Change subject: security/intel/txt: Helper function to disable TXT
......................................................................

Patch Set 1: Code-Review+2

(2 comments)

Patchset:

PS1: 
Looks good. What do you think of my suggestion for a different function name? I don't feel strongly about it. I just reviewed the follow-up first and got confused, hence the suggestion.

File src/security/intel/txt/txtlib.c:

https://review.coreboot.org/c/coreboot/+/71574/comment/d1b191b5_07c0cf3e 
PS1, Line 57: /* Don't disable if INTEL_TXT config is selected */
            : 	if (CONFIG(INTEL_TXT))
            : 		return;
Having a disable function which does not disable anything depending on a Kconfig value, is a bit confusing? Would it make sense to call it something like 'early_intel_txt_setup' ?