Shaunak Saha has posted comments on this change. ( https://review.coreboot.org/28424 )
Change subject: soc/intel/skylake: Add support for CmdTriStateDis UPD in devicetree ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/28424/1/src/soc/intel/skylake/chip.h File src/soc/intel/skylake/chip.h:
https://review.coreboot.org/#/c/28424/1/src/soc/intel/skylake/chip.h@157 PS1, Line 157: Enable/disable
Just say "Disable" since this config is used to basically disable command tristate?
Ack