[coreboot-gerrit] Change in coreboot[master]: soc/intel/skylake: Factor out unnecessary if-else-block

[Felix Singer (Code Review)]

Hello Patrick Rudolph,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/44369

to look at the new patch set (#2).

Change subject: soc/intel/skylake: Factor out unnecessary if-else-block
......................................................................

soc/intel/skylake: Factor out unnecessary if-else-block

Move InternalGfx config option out of the if-else-block and replace the
left over config option IgdDvmt50PreAlloc by a ternary expression. Also,
adjust related code comments to fit the new logic of this code.

This changes the logic of the code, since InternalGfx is configured
first and IgdDvmt50PreAlloc depends on its value. The negation in the
ternary expression is removed to improve the readability.

Change-Id: I89ff17f4574a7ade228c1791f17ea072fb731775
Signed-off-by: Felix Singer 
---
M src/soc/intel/skylake/romstage/romstage.c
1 file changed, 14 insertions(+), 19 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/44369/2