[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/block/sa: Add compiletime error on undefined SA_PCIE...

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35887 )

Change subject: soc/intel/common/block/sa: Add compiletime error on undefined SA_PCIE_LENGTH
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35887/2/src/soc/intel/common/block/... 
File src/soc/intel/common/block/systemagent/systemagent_early.c:

https://review.coreboot.org/c/coreboot/+/35887/2/src/soc/intel/common/block/... 
PS2, Line 56: 	default:
> NB. This will result in a wrong reservation. And I know at least some Linux […]

Hah, dead_code() actually works. Just replace the next line
with `dead_code();` and we won't need the `#if`.