[coreboot-gerrit] Change in coreboot[master]: soc/skylake: Enable LAN depending on devicetree configuration

[Felix Singer (Code Review)]

Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43844 )

Change subject: soc/skylake: Enable LAN depending on devicetree configuration
......................................................................

Patch Set 6:

(3 comments)

https://review.coreboot.org/c/coreboot/+/43844/5//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/43844/5//COMMIT_MSG@9 
PS5, Line 9: EnableLAN
> EnableLan

Done

https://review.coreboot.org/c/coreboot/+/43844/5//COMMIT_MSG@14 
PS5, Line 14: EnableLAN
> EnableLan

Done

https://review.coreboot.org/c/coreboot/+/43844/5/src/soc/intel/skylake/chip.... 
File src/soc/intel/skylake/chip.c:

https://review.coreboot.org/c/coreboot/+/43844/5/src/soc/intel/skylake/chip.... 
PS5, Line 246: dev ? dev->enabled : 0
> `dev && dev->enabled`

I would leave it like that to keep the consistency, since the existing code is handled like that.