[coreboot-gerrit] Change in ...coreboot[master]: nb/intel/i945: Use parallel MP init

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/25600 )

Change subject: nb/intel/i945: Use parallel MP init
......................................................................

Patch Set 40: Code-Review+1

(1 comment)

https://review.coreboot.org/#/c/25600/40/src/cpu/intel/model_f4x/model_f4x_i... 
File src/cpu/intel/model_f4x/model_f4x_init.c:

https://review.coreboot.org/#/c/25600/40/src/cpu/intel/model_f4x/model_f4x_i... 
PS40, Line 27: if (!IS_ENABLED(CONFIG_PARALLEL_MP) && !intel_ht_sibling()) {
             : 		/* MTRRs are shared between threads */
             : 		x86_setup_mtrrs();
             : 		x86_mtrr_check();
             : 
             : 		/* Update the microcode */
             : 		intel_update_microcode_from_cbfs();
             : 	}
please, why don't you drop this and "intel_sibling_init(cpu)" ?