Change in coreboot[master]: drivers/intel/fsp2_0/include/fsp: fix fsp_header

[Felix Held (Code Review)]

Attention is currently required from: Jason Glenesk, Raul Rangel, Subrata Banik, Marshall Dawson, Tim Wawrzynczak, Paul Menzel, Name of user not set #1003801, Andrey Petrov, Julian Schroeder, Patrick Rudolph.
Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58869 )

Change subject: drivers/intel/fsp2_0/include/fsp: fix fsp_header
......................................................................

Patch Set 9: Code-Review+1

(2 comments)

Patchset:

PS9: 
this patch looks correct to me, but it would be good if Subrata or Tim can also review and test it on some intel soc that uses the fsp. this is the reason why i only give a +1 for now

File src/drivers/intel/fsp2_0/header_display.c:

https://review.coreboot.org/c/coreboot/+/58869/comment/a420c916_47a49a99 
PS9, Line 11: 	int i;
> I like to stick to K&R style where int is the default data type for things like 'for loops'.

i try to only used signed when i need a signed data type tomake it clearer that that number will never be negative. don't have a too strong opinion on this, so if you want to keep this a signed int then keep it the way it is right now