[S] Change in coreboot[main]: mb/google/brox/var/lotso: Fix goodix touchscreen power off sequence

[Karthik Ramasubramanian (Code Review)]

Attention is currently required from: Kun Liu, Shelley Chen.

Karthik Ramasubramanian has posted comments on this change by Kun Liu. ( https://review.coreboot.org/c/coreboot/+/84412?usp=email )

Change subject: mb/google/brox/var/lotso: Fix goodix touchscreen power off sequence
......................................................................

Patch Set 2:

(1 comment)

Commit Message:

https://review.coreboot.org/c/coreboot/+/84412/comment/d3714ad1_d66ca7fb?usp... :
PS2, Line 9: Power off does not seem to use the ACPI _OFF function
Did you try to use "has_power_resource" = "1" in the concerned touchscreen device in the override tree. That should expose the power resources in the ACPI table.

That is what I would recommend instead of smihandler approach. Fingerprint module is on a similar SPI bus and does not use SMI handler to manage the power.