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

[Michael Niewöhner (Code Review)]

Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43843 )

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

Patch Set 5: Code-Review+2

(1 comment)

https://review.coreboot.org/c/coreboot/+/43843/4/src/soc/intel/skylake/chip.... 
File src/soc/intel/skylake/chip.c:

https://review.coreboot.org/c/coreboot/+/43843/4/src/soc/intel/skylake/chip.... 
PS4, Line 170: dev ? dev->enabled : 0
> I would leave it like that to keep the consistency, since the existing code is handled like that.

ack, fix the others in that follow-up, too, then :)