[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/braswell: Add C_ENVIRONMENT_BOOTBLOCK support

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29662 )

Change subject: soc/intel/braswell: Add C_ENVIRONMENT_BOOTBLOCK support
......................................................................

Patch Set 7:
> Patch Set 7:
> > Patch Set 7: Code-Review-2
> >
> > (1 comment)
> >
> > select C_ENVIRONMENT_BOOTBLOCK is the braswell Kconfig and drop the romcc bootblock option and add console init in the bootblock.
> > Also drop things that get unused in drivers/intel/fsp1_1
> > This patch is not even build tested...
>
> This patch is to support C_ENVIRONMENT_BOOTBLOCK for Braswell. To be backward compatible this support has made optional.

Please don't make it optional and remove the 'backwards compatibility', there is really no reason to keep that around.
> This patch has been build and tested. I will check if I made mistake by uploading.

No gerrit has not build-tested it.
> Can you clarify your comment?