Change in coreboot[master]: [WIP] drivers/intel/fsp2_0: Add FSP 2.3 support

[Subrata Banik (Code Review)]

Attention is currently required from: Anil Kumar K, Paul Menzel, Subrata Banik, Andrey Petrov, Patrick Rudolph.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59324 )

Change subject: [WIP] drivers/intel/fsp2_0: Add FSP 2.3 support
......................................................................

Patch Set 13:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/59324/comment/2fdbf055_4cd0e407 
PS3, Line 10: ext_revision.val = 0;
> the value is expected to be zero when we print this info for FSP 2.2 and earlier. […]

its local variable so you don't need the assignment

#include <stdio.h>

union extended_fsp_revision {
    unsigned short val;
    struct {
    	char  bld_num;
    	char revision;
    } rev;
};

int main()
{
    union extended_fsp_revision ext_revision;
    ext_revision.val = 0;

union extended_fsp_revision ext_revision_1;

printf("With Assigning the Zero ext_revision.val = %d\n", ext_revision.val);
    printf("Without Assigning the Zero ext_revision.val = %d\n", ext_revision_1.val);

return 0;
}

output:

With Assigning the Zero ext_revision.val = 0
Without Assigning the Zero ext_revision.val = 0