[coreboot-gerrit] Change in coreboot[master]: cpu/x86/mtrrlib: Use `need restore mtrr()` from set var mtrr function

[Arthur Heymans (Code Review)]

Attention is currently required from: Subrata Banik, Tim Wawrzynczak, Nick Vaccaro, Eric Lai.
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63492 )

Change subject: cpu/x86/mtrrlib: Use `need_restore_mtrr()` from  set_var_mtrr  function
......................................................................

Patch Set 1: Code-Review-1

(1 comment)

File src/cpu/x86/mtrr/mtrrlib.c:

https://review.coreboot.org/c/coreboot/+/63492/comment/3537b706_64a8cafe 
PS1, Line 52: /* Need to restore mtrr later using remove_temp_solution. */
            : 	if (ENV_RAMSTAGE)
            : 		need_restore_mtrr();
I don't like this. set_var_mtrr should do just one thing: set a variable mtrr...
I if you want a temp mtrr with this function, it should be the caller setting this up.