Change in coreboot[master]: cpu/x86/smm_module_loader.c: Rewrite setup more purely function

[build bot (Jenkins) (Code Review)]

Attention is currently required from: Edward O'Callaghan.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63475 )

Change subject: cpu/x86/smm_module_loader.c: Rewrite setup more purely function
......................................................................

Patch Set 2:

(8 comments)

File src/cpu/x86/smm/smm_module_loader.c:

Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-145801):
https://review.coreboot.org/c/coreboot/+/63475/comment/c1b54aeb_f955ad4d 
PS2, Line 468: 	const uintptr_t handler_base = ALIGN_DOWN(fx_save_area_base - handler_size, handler_alignment);
line over 96 characters

Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-145801):
https://review.coreboot.org/c/coreboot/+/63475/comment/cbc95d3b_658dcd95 
PS2, Line 469: 	printk(BIOS_DEBUG, "smihandler    [0x%lx-0x%lx[\n", handler_base, handler_base + handler_size);
line over 96 characters

Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-145801):
https://review.coreboot.org/c/coreboot/+/63475/comment/35ed96bb_28369913 
PS2, Line 515: 	return smm_module_setup_stub(stub_segment_base, smram_size, params, (void *)fx_save_area_base);
line over 96 characters

Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-145801):
https://review.coreboot.org/c/coreboot/+/63475/comment/ddaeeabe_4808a516 
PS2, Line 561: 	const uintptr_t save_state_bottom = cpus[params->num_concurrent_save_states - 1].ss_start;
line over 96 characters

Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-145801):
https://review.coreboot.org/c/coreboot/+/63475/comment/76f2f255_b8403c96 
PS2, Line 569: 		printk(BIOS_ERR, "fxsave wont fit in smram\n");
'wont' may be misspelled - perhaps 'won't'?

Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-145801):
https://review.coreboot.org/c/coreboot/+/63475/comment/614ac745_d26b8148 
PS2, Line 574: 	const size_t bottom_space = cpus[params->num_concurrent_save_states - 1].code_start - smm_stack_top;
line over 96 characters

Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-145801):
https://review.coreboot.org/c/coreboot/+/63475/comment/34ceeb69_edee25f8 
PS2, Line 583: 			printk(BIOS_ERR, "handler wont fit in top of smram\n");
'wont' may be misspelled - perhaps 'won't'?

Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-145801):
https://review.coreboot.org/c/coreboot/+/63475/comment/e024e14b_9236253c 
PS2, Line 590: 			printk(BIOS_ERR, "handler wont fit in bottom of smram\n");
'wont' may be misspelled - perhaps 'won't'?