[coreboot-gerrit] Change in coreboot[master]: soc/intel/apollolake: Display apollolake platform information

[Werner Zeh (Code Review)]

Werner Zeh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38824 )

Change subject: soc/intel/apollolake: Display apollolake platform information
......................................................................

Patch Set 3:
> Patch Set 3:
> > Patch Set 3:
> >
> > Is there a reason why this needs to be part of bootblock?
> > We have size constrains in bootblock anyway and on the other hand have a plenty of space in the ramstage? Wouldn't it be as useful in ramstage?
>
> good point, right now its more over parity with other soc design we have where bootblock does report platform early.
>
> But fare point to move those in romstage may be as these are sorts of information to have, nothing as such mandatory to have inside BB itself. Agree ?

Sure, romstage is OK as well OFC.
I just stumbled across this issue as the bootblock size was increased and I do not see a good reason for it in this context.