[coreboot-gerrit] Change in coreboot[master]: ec/google/chromeec: Add dptc interface support

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44265 )

Change subject: ec/google/chromeec: Add dptc interface support
......................................................................

Patch Set 11:

(1 comment)

https://review.coreboot.org/c/coreboot/+/44265/11/src/ec/google/chromeec/acp... 
File src/ec/google/chromeec/acpi/ec.asl:

https://review.coreboot.org/c/coreboot/+/44265/11/src/ec/google/chromeec/acp... 
PS11, Line 386: _SB.DPTC()
I think it would be safer to add a check to ensure that DPTC is really provided:

If (CondRefOf (_SB.DPTC)) {
    _SB.DPTC()
}

Thus, if some board accidentally selects DPTC_SUPPORT_ENABLE, but doesn't set dptc_enable = 1, then this would not result in ACPI errors.