[coreboot-gerrit] Change in ...coreboot[master]: mb/google/sarien: Fix s5 touchscreen power leakage

[Lijian Zhao (Code Review)]

Lijian Zhao has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32367 )

Change subject: mb/google/sarien: Fix s5 touchscreen power leakage
......................................................................

Patch Set 5:

(2 comments)

https://review.coreboot.org/#/c/32367/4/src/mainboard/google/sarien/variants... 
File src/mainboard/google/sarien/variants/arcada/include/variant/acpi/mainboard.asl:

https://review.coreboot.org/#/c/32367/4/src/mainboard/google/sarien/variants... 
PS4, Line 21: 5
> S3 is not a POR on both sarien and arcada.

S3 still need to be working as fail safe option?

https://review.coreboot.org/#/c/32367/4/src/mainboard/google/sarien/variants... 
PS4, Line 23: CTXS (TS_PD)
> Won't you be violating the power down specification by not asserting reset before disabling power?

@Roy, would you like to answer here?