[coreboot-gerrit] Change in coreboot[master]: soc/intel/jasperlake: Allow SataEnable to be filled from devicetree

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41029 )

Change subject: soc/intel/jasperlake: Allow SataEnable to be filled from devicetree
......................................................................

Patch Set 4:

(3 comments)

https://review.coreboot.org/c/coreboot/+/41029/4//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/41029/4//COMMIT_MSG@9 
PS4, Line 9: it
it to 1

https://review.coreboot.org/c/coreboot/+/41029/4//COMMIT_MSG@10 
PS4, Line 10: controller
Dot/period after *controller*.

https://review.coreboot.org/c/coreboot/+/41029/4//COMMIT_MSG@11 
PS4, Line 11: it can be set as per each board's requirement.
1.  Why can’t this be detected at run-time?
2.  What is the the disadvantage of always enabling it?