[coreboot-gerrit] Change in ...coreboot[master]: drivers/intel/fsp1_1: Configure UART after memory init

[Frans Hendriks (Code Review)]

Frans Hendriks has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/28464 )

Change subject: drivers/intel/fsp1_1: Configure UART after memory init
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/#/c/28464/3/src/drivers/intel/fsp1_1/raminit.c 
File src/drivers/intel/fsp1_1/raminit.c:

https://review.coreboot.org/#/c/28464/3/src/drivers/intel/fsp1_1/raminit.c@1... 
PS3, Line 128: mainboard_after_memory_init
> so you call the function and add a weak stub below. […]

This function is used with update of Facebook FBG-1701 ( https://review.coreboot.org/c/coreboot/+/30414)

Note: At the moment I created this patch, I was not aware that you can rebase a patch. The 30414 patch requires the function prototype of this patch. To have the 30414 patch build, I removed the call of mainboard_after_memory_init(). 
My plan is to wait till this patch in merged before updating the other patch.

Does this make sense?