Change in ...coreboot[master]: drivers/intel/fsp1_1/raminit.c: Always check FSP HOBs

[Frans Hendriks (Code Review)]

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

Change subject: drivers/intel/fsp1_1/raminit.c: Always check FSP HOBs
......................................................................

Patch Set 9:

(4 comments)

Will update with new patchset

https://review.coreboot.org/#/c/29371/9/src/drivers/intel/fsp1_1/raminit.c 
File src/drivers/intel/fsp1_1/raminit.c:

https://review.coreboot.org/#/c/29371/9/src/drivers/intel/fsp1_1/raminit.c@a... 
PS9, Line 209:
> Done

Done

https://review.coreboot.org/#/c/29371/9/src/drivers/intel/fsp1_1/raminit.c@2... 
PS9, Line 205: 	if (hob_ptr.Raw == NULL)
> merge with next if

Will merge the 2 if lines

https://review.coreboot.org/#/c/29371/9/src/drivers/intel/fsp1_1/raminit.c@2... 
PS9, Line 217: 	if (hob_list_ptr != NULL) {
> merge with next if

This check is already performed. Will remove it.

https://review.coreboot.org/#/c/29371/9/src/drivers/intel/fsp1_1/raminit.c@2... 
PS9, Line 220: 	} else {
> unreachable as you die above

Done