Change in coreboot[master]: soc/intel/broadwell/pch: Replace ACPI device NVS

[Angel Pons (Code Review)]

Attention is currently required from: Nico Huber, Martin Roth, Tim Wawrzynczak, Arthur Heymans, Kyösti Mälkki, Patrick Rudolph.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52520 )

Change subject: soc/intel/broadwell/pch: Replace ACPI device NVS
......................................................................

Patch Set 3:

(2 comments)

File src/soc/intel/broadwell/pch/acpi.c:

https://review.coreboot.org/c/coreboot/+/52520/comment/9b925b00_8a08e5cd 
PS2, Line 23: const struct opregion opreg = {
            : 			.name         = "SPRT",
            : 			.regionspace  = SYSTEMMEMORY,
            : 			.regionoffset = bar1 + 0x84,
            : 			.regionlen    = spcs.bits / 8,
            : 		};
> You don't like the macros? 😋 […]

I'm not a fan of macros, but I can switch to using them if it's customary in acpigen.

File src/soc/intel/broadwell/pch/serialio.c:

https://review.coreboot.org/c/coreboot/+/52520/comment/446ad9ea_0eab460b 
PS1, Line 19: #include <version.h>
> unrelated?

Gone. I originally wanted to add the acpigen code in serialio.c, but decided against it later.