[coreboot-gerrit] Change in ...coreboot[master]: drivers/intel/fsp1_0: Use new memory clearing API

[Patrick Rudolph (Code Review)]

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31551 )

Change subject: drivers/intel/fsp1_0: Use new memory clearing API
......................................................................

Patch Set 6:

(2 comments)

https://review.coreboot.org/#/c/31551/5/src/soc/intel/fsp_broadwell_de/romst... 
File src/soc/intel/fsp_broadwell_de/romstage/romstage.c:

https://review.coreboot.org/#/c/31551/5/src/soc/intel/fsp_broadwell_de/romst... 
PS5, Line 112: if (GetUsableHighMemTop(hob_list_ptr) > 4ULL * GiB)
             : 		highmem_size = GetUsableHighMemTop(hob_list_ptr) - 4ULL * GiB;
             : 	tolm         = GetPhysicalLowMemTop(hob_list_ptr);
> totally wacky formatting

Done

https://review.coreboot.org/#/c/31551/5/src/soc/intel/fsp_broadwell_de/romst... 
PS5, Line 158: 	if (security_clear_dram_request())
             : 		romstage_clear_mem(hob_list_ptr);
> isn't there a more generic piece of code where this could be put in?

Done