Change in ...coreboot[master]: nb/intel/sandybridge/acpi: Add RMRR entry for iGPU

[Nico Huber (Code Review)]

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

Change subject: nb/intel/sandybridge/acpi: Add RMRR entry for iGPU
......................................................................

Patch Set 7:

(1 comment)

https://review.coreboot.org/#/c/29668/7/src/northbridge/intel/sandybridge/ac... 
File src/northbridge/intel/sandybridge/acpi.c:

https://review.coreboot.org/#/c/29668/7/src/northbridge/intel/sandybridge/ac... 
PS7, Line 101: 		current += acpi_create_igfx_rmrr(current);
> what happens on error if acpi_create_igfx_rmrr returns 0 ? […]

Hmmm, no, it wouldn't (and shouldn't, too complex). If any of the errors
in acpi_create_igfx_rmrr() happen, we are screwed anyway, but we should
handle it more gracefully.