[S] Change in coreboot[master]: mb/google/skyrim: Use detect vs probed flag for touchscreens

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/67779 )

Change subject: mb/google/skyrim: Use detect vs probed flag for touchscreens
......................................................................

mb/google/skyrim: Use detect vs probed flag for touchscreens

Now that coreboot performs the necessary power sequencing, switch
from using the 'probed' flag to 'detect' for all I2C touchscreens.
This alleviates ChromeOS from having to probe to see which
touchscreen model is actually present, prevents breaking ACPI spec
by generating device entries with status 'enabled and present'
which aren't actually present, and improves compatibility with
upstream Linux and Windows.

BUG=b:121309055
TEST=build/boot ChromeOS and Linux on skyrim, ensure touchscreen is
functional, and ACPI device entry generated for correct touchscreen
model.

Change-Id: Id9e3089decf0f94a1358929684ce248e52cbe41f
Signed-off-by: Matt DeVillier 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/67779
Tested-by: build bot (Jenkins) 
Reviewed-by: Raul Rangel 
Reviewed-by: Frank Wu 
---
M src/mainboard/google/skyrim/variants/frostflow/overridetree.cb
M src/mainboard/google/skyrim/variants/skyrim/overridetree.cb
M src/mainboard/google/skyrim/variants/winterhold/overridetree.cb
3 files changed, 32 insertions(+), 5 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Frank Wu: Looks good to me, approved
  Raul Rangel: Looks good to me, approved

diff --git a/src/mainboard/google/skyrim/variants/frostflow/overridetree.cb b/src/mainboard/google/skyrim/variants/frostflow/overridetree.cb
index 1b49a10..a3858c8 100644
--- a/src/mainboard/google/skyrim/variants/frostflow/overridetree.cb
+++ b/src/mainboard/google/skyrim/variants/frostflow/overridetree.cb
@@ -49,7 +49,7 @@
    		register "generic.hid" = ""ELAN9008""
    		register "generic.desc" = ""ELAN Touchscreen""
    		register "generic.irq_gpio" = "ACPI_GPIO_IRQ_LEVEL_LOW(GPIO_29)"
-			register "generic.probed" = "1"
+			register "generic.detect" = "1"
    		register "generic.enable_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPIO_131)"
    		register "generic.enable_delay_ms" = "1"
    		register "generic.reset_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPIO_136)"
diff --git a/src/mainboard/google/skyrim/variants/skyrim/overridetree.cb b/src/mainboard/google/skyrim/variants/skyrim/overridetree.cb
index c36a8d4..65b4413 100644
--- a/src/mainboard/google/skyrim/variants/skyrim/overridetree.cb
+++ b/src/mainboard/google/skyrim/variants/skyrim/overridetree.cb
@@ -120,7 +120,7 @@
    		register "generic.hid" = ""GDIX0000""
    		register "generic.desc" = ""Goodix Touchscreen""
    		register "generic.irq_gpio" = "ACPI_GPIO_IRQ_LEVEL_LOW(GPIO_29)"
-			register "generic.probed" = "1"
+			register "generic.detect" = "1"
    		register "generic.enable_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPIO_131)"
    		register "generic.enable_delay_ms" = "10"
    		register "generic.reset_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPIO_136)"
@@ -136,7 +136,7 @@
    	chip drivers/i2c/generic
    		register "hid" = ""ELAN0001""
    		register "desc" = ""ELAN Touchscreen""
-			register "probed" = "1"
+			register "detect" = "1"
    		register "irq_gpio" = "ACPI_GPIO_IRQ_EDGE_LOW(GPIO_29)"
    		register "enable_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPIO_131)"
    		register "enable_delay_ms" = "1"
diff --git a/src/mainboard/google/skyrim/variants/winterhold/overridetree.cb b/src/mainboard/google/skyrim/variants/winterhold/overridetree.cb
index ab0bda2..78a8091 100644
--- a/src/mainboard/google/skyrim/variants/winterhold/overridetree.cb
+++ b/src/mainboard/google/skyrim/variants/winterhold/overridetree.cb
@@ -89,7 +89,7 @@
    		register "generic.hid" = ""ELAN900C""
    		register "generic.desc" = ""ELAN Touchscreen""
    		register "generic.irq_gpio" = "ACPI_GPIO_IRQ_LEVEL_LOW(GPIO_29)"
-			register "generic.probed" = "1"
+			register "generic.detect" = "1"
    		register "generic.enable_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPIO_131)"
    		register "generic.enable_delay_ms" = "10"
    		register "generic.reset_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPIO_136)"
@@ -105,7 +105,7 @@
    	chip drivers/i2c/generic
    		register "hid" = ""MELF0410""
    		register "desc" = ""Melfas Touchscreen""
-			register "probed" = "1"
+			register "detect" = "1"
    		register "irq_gpio" = "ACPI_GPIO_IRQ_EDGE_LOW(GPIO_29)"
    		register "enable_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPIO_131)"
    		register "enable_delay_ms" = "1"