[coreboot-gerrit] Change in coreboot[master]: arch/x86: Reserve some CAR memory for FSP single stack requirement

[Subrata Banik (Code Review)]

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35165 )

Change subject: arch/x86: Reserve some CAR memory for FSP single stack requirement
......................................................................

Patch Set 9:
> Patch Set 9:
> > Patch Set 4:
> >
> > (10 comments)
> > > Patch Set 4:
> > >
> > > Just curious, why would we want to make it explicit entry in car.ld when using .bss should achieve just the same?
> >
> > Isn't that more meaningful now to know why we are reserving that block in car.ld?
>
> I think Kyosti was suggesting one could use an array of uint8_t of a specified size (and appropriate alignment) in a C compilation unit to achieve the same allocation.

yes, this will work eventually but i'm afraid that someone might not able judge why we are declaring static variable with such big size :) where car.ld reserve might be worthy to explain with diagram.