[coreboot-gerrit] Change in coreboot[master]: lib/coreboot_table: Add Intel FSP version for cbmem to display

[Johnny Lin (Code Review)]

Johnny Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41809 )

Change subject: lib/coreboot_table: Add Intel FSP version for cbmem to display
......................................................................

Patch Set 5:
> Patch Set 3:
>
> (1 comment)
> > Patch Set 2:
> >
> > (1 comment)
> >
> > A solution without #if would be much preferred. All you need
> > is an always available fsp_get_version() forward declaration.
> > Then the call to lb_strings_fsp_version() can be guarded by
> > a C `if`:
> > if (CONFIG(FSP_ADD_VERSION_TO_CBMEM))
> >         lb_strings_fsp_version(head);
>
> Done, thanks.

I rollback to use #if to avoid re-defining FSP_VER_LEN here again.