From 2cf52d80a6ecca97d44d2142521a984df45398f2 Mon Sep 17 00:00:00 2001 From: Matt DeVillier Date: Thu, 1 Sep 2022 15:09:24 -0500 Subject: mb/*/{device,override}tree: Set touchpads to use detect (vs probed) flag Historically, ChromeOS devices have worked around the problem of OEMs using several different parts for touchpads/touchscreens by using a ChromeOS kernel-specific 'probed' flag (rejected by the upstream kernel) to indicate that the device may or may not be present, and that the driver should probe to confirm device presence. Since c636142b, coreboot now supports detection for i2c devices at runtime when creating the device entries for the ACPI/SSDT tables, rendering the 'probed' flag obsolete for touchpads. Switch all touchpads in the tree from using the 'probed' flag to the 'detect' flag. Touchscreens require more involved power sequencing, which will be done at some future time, after which they will switch over as well. TEST: build/boot at least one variant for each baseboard in the tree. Verify touchpad works under Linux and Windows. Verify only a single touchpad device is present in the ACPI tables. Change-Id: I47c6eed37eb34c044e27963532e544d3940a7c15 Signed-off-by: Matt DeVillier Reviewed-on: https://review.coreboot.org/c/coreboot/+/67305 Reviewed-by: Sean Rhodes Tested-by: build bot (Jenkins) Reviewed-by: Tim Wawrzynczak --- src/mainboard/google/octopus/variants/ampton/overridetree.cb | 2 +- src/mainboard/google/octopus/variants/bloog/overridetree.cb | 2 +- src/mainboard/google/octopus/variants/bobba/overridetree.cb | 4 ++-- src/mainboard/google/octopus/variants/casta/overridetree.cb | 4 ++-- src/mainboard/google/octopus/variants/dood/overridetree.cb | 4 ++-- src/mainboard/google/octopus/variants/fleex/overridetree.cb | 4 ++-- src/mainboard/google/octopus/variants/foob/overridetree.cb | 4 ++-- src/mainboard/google/octopus/variants/garg/overridetree.cb | 4 ++-- src/mainboard/google/octopus/variants/lick/overridetree.cb | 4 ++-- src/mainboard/google/octopus/variants/meep/overridetree.cb | 2 +- src/mainboard/google/octopus/variants/phaser/overridetree.cb | 4 ++-- src/mainboard/google/octopus/variants/yorp/overridetree.cb | 2 +- 12 files changed, 20 insertions(+), 20 deletions(-) (limited to 'src/mainboard/google/octopus') diff --git a/src/mainboard/google/octopus/variants/ampton/overridetree.cb b/src/mainboard/google/octopus/variants/ampton/overridetree.cb index cfc5e8ac78..1d1748a81d 100644 --- a/src/mainboard/google/octopus/variants/ampton/overridetree.cb +++ b/src/mainboard/google/octopus/variants/ampton/overridetree.cb @@ -104,7 +104,7 @@ chip soc/intel/apollolake register "desc" = ""ELAN Touchpad"" register "irq" = "ACPI_IRQ_WAKE_EDGE_LOW(GPIO_135_IRQ)" register "wake" = "GPE0_DW3_27" - register "probed" = "1" + register "detect" = "1" device i2c 15 on end end end # - I2C 6 diff --git a/src/mainboard/google/octopus/variants/bloog/overridetree.cb b/src/mainboard/google/octopus/variants/bloog/overridetree.cb index b585c853e3..a950be8a54 100644 --- a/src/mainboard/google/octopus/variants/bloog/overridetree.cb +++ b/src/mainboard/google/octopus/variants/bloog/overridetree.cb @@ -120,7 +120,7 @@ chip soc/intel/apollolake register "desc" = ""ELAN Touchpad"" register "irq" = "ACPI_IRQ_WAKE_EDGE_LOW(GPIO_135_IRQ)" register "wake" = "GPE0_DW3_27" - register "probed" = "1" + register "detect" = "1" device i2c 15 on end end end # - I2C 6 diff --git a/src/mainboard/google/octopus/variants/bobba/overridetree.cb b/src/mainboard/google/octopus/variants/bobba/overridetree.cb index c43153ac33..94291b8d53 100644 --- a/src/mainboard/google/octopus/variants/bobba/overridetree.cb +++ b/src/mainboard/google/octopus/variants/bobba/overridetree.cb @@ -170,7 +170,7 @@ chip soc/intel/apollolake register "desc" = ""ELAN Touchpad"" register "irq" = "ACPI_IRQ_WAKE_EDGE_LOW(GPIO_135_IRQ)" register "wake" = "GPE0_DW3_27" - register "probed" = "1" + register "detect" = "1" device i2c 15 on end end chip drivers/i2c/hid @@ -178,7 +178,7 @@ chip soc/intel/apollolake register "generic.desc" = ""Synaptics Touchpad"" register "generic.irq" = "ACPI_IRQ_WAKE_LEVEL_LOW(GPIO_135_IRQ)" register "generic.wake" = "GPE0_DW3_27" - register "generic.probed" = "1" + register "generic.detect" = "1" register "hid_desc_reg_offset" = "0x20" device i2c 0x2c on end end diff --git a/src/mainboard/google/octopus/variants/casta/overridetree.cb b/src/mainboard/google/octopus/variants/casta/overridetree.cb index a09bcbb9e1..3dc2f092ab 100644 --- a/src/mainboard/google/octopus/variants/casta/overridetree.cb +++ b/src/mainboard/google/octopus/variants/casta/overridetree.cb @@ -134,7 +134,7 @@ chip soc/intel/apollolake register "desc" = ""ELAN Touchpad"" register "irq" = "ACPI_IRQ_WAKE_EDGE_LOW(GPIO_135_IRQ)" register "wake" = "GPE0_DW3_27" - register "probed" = "1" + register "detect" = "1" device i2c 15 on end end chip drivers/i2c/hid @@ -142,7 +142,7 @@ chip soc/intel/apollolake register "generic.desc" = ""Zinitix Touchpad"" register "generic.irq" = "ACPI_IRQ_WAKE_LEVEL_LOW(GPIO_135_IRQ)" register "generic.wake" = "GPE0_DW3_27" - register "generic.probed" = "1" + register "generic.detect" = "1" register "hid_desc_reg_offset" = "0xE" device i2c 40 on end end diff --git a/src/mainboard/google/octopus/variants/dood/overridetree.cb b/src/mainboard/google/octopus/variants/dood/overridetree.cb index 465078f8c3..168c83f5ca 100644 --- a/src/mainboard/google/octopus/variants/dood/overridetree.cb +++ b/src/mainboard/google/octopus/variants/dood/overridetree.cb @@ -106,7 +106,7 @@ chip soc/intel/apollolake register "desc" = ""ELAN Touchpad"" register "irq" = "ACPI_IRQ_WAKE_EDGE_LOW(GPIO_135_IRQ)" register "wake" = "GPE0_DW3_27" - register "probed" = "1" + register "detect" = "1" device i2c 15 on end end chip drivers/i2c/hid @@ -114,7 +114,7 @@ chip soc/intel/apollolake register "generic.desc" = ""Synaptics Touchpad"" register "generic.irq" = "ACPI_IRQ_WAKE_LEVEL_LOW(GPIO_135_IRQ)" register "generic.wake" = "GPE0_DW3_27" - register "generic.probed" = "1" + register "generic.detect" = "1" register "hid_desc_reg_offset" = "0x20" device i2c 0x2c on end end diff --git a/src/mainboard/google/octopus/variants/fleex/overridetree.cb b/src/mainboard/google/octopus/variants/fleex/overridetree.cb index 5007719002..6a058e325a 100644 --- a/src/mainboard/google/octopus/variants/fleex/overridetree.cb +++ b/src/mainboard/google/octopus/variants/fleex/overridetree.cb @@ -158,7 +158,7 @@ chip soc/intel/apollolake register "desc" = ""ELAN Touchpad"" register "irq" = "ACPI_IRQ_WAKE_EDGE_LOW(GPIO_135_IRQ)" register "wake" = "GPE0_DW3_27" - register "probed" = "1" + register "detect" = "1" device i2c 15 on end end chip drivers/i2c/hid @@ -166,7 +166,7 @@ chip soc/intel/apollolake register "generic.desc" = ""Goodix Touchpad"" register "generic.irq" = "ACPI_IRQ_LEVEL_LOW(GPIO_135_IRQ)" register "generic.wake" = "GPE0_DW3_27" - register "generic.probed" = "1" + register "generic.detect" = "1" register "hid_desc_reg_offset" = "0x20" device i2c 2c on end end diff --git a/src/mainboard/google/octopus/variants/foob/overridetree.cb b/src/mainboard/google/octopus/variants/foob/overridetree.cb index 4161090aa6..16ea379e41 100644 --- a/src/mainboard/google/octopus/variants/foob/overridetree.cb +++ b/src/mainboard/google/octopus/variants/foob/overridetree.cb @@ -120,7 +120,7 @@ chip soc/intel/apollolake register "desc" = ""ELAN Touchpad"" register "irq" = "ACPI_IRQ_WAKE_EDGE_LOW(GPIO_135_IRQ)" register "wake" = "GPE0_DW3_27" - register "probed" = "1" + register "detect" = "1" device i2c 15 on end end chip drivers/i2c/hid @@ -128,7 +128,7 @@ chip soc/intel/apollolake register "generic.desc" = ""Synaptics Touchpad"" register "generic.irq" = "ACPI_IRQ_WAKE_LEVEL_LOW(GPIO_135_IRQ)" register "generic.wake" = "GPE0_DW3_27" - register "generic.probed" = "1" + register "generic.detect" = "1" register "hid_desc_reg_offset" = "0x20" device i2c 0x2c on end end diff --git a/src/mainboard/google/octopus/variants/garg/overridetree.cb b/src/mainboard/google/octopus/variants/garg/overridetree.cb index c2a6997479..586983c158 100644 --- a/src/mainboard/google/octopus/variants/garg/overridetree.cb +++ b/src/mainboard/google/octopus/variants/garg/overridetree.cb @@ -148,7 +148,7 @@ chip soc/intel/apollolake register "desc" = ""ELAN Touchpad"" register "irq" = "ACPI_IRQ_WAKE_EDGE_LOW(GPIO_135_IRQ)" register "wake" = "GPE0_DW3_27" - register "probed" = "1" + register "detect" = "1" device i2c 15 on end end chip drivers/i2c/hid @@ -156,7 +156,7 @@ chip soc/intel/apollolake register "generic.desc" = ""Synaptics Touchpad"" register "generic.irq" = "ACPI_IRQ_WAKE_LEVEL_LOW(GPIO_135_IRQ)" register "generic.wake" = "GPE0_DW3_27" - register "generic.probed" = "1" + register "generic.detect" = "1" register "hid_desc_reg_offset" = "0x20" device i2c 0x2c on end end diff --git a/src/mainboard/google/octopus/variants/lick/overridetree.cb b/src/mainboard/google/octopus/variants/lick/overridetree.cb index e2b35cb598..96c2afd7a4 100644 --- a/src/mainboard/google/octopus/variants/lick/overridetree.cb +++ b/src/mainboard/google/octopus/variants/lick/overridetree.cb @@ -98,7 +98,7 @@ chip soc/intel/apollolake register "desc" = ""ELAN Touchpad"" register "irq" = "ACPI_IRQ_WAKE_EDGE_LOW(GPIO_135_IRQ)" register "wake" = "GPE0_DW3_27" - register "probed" = "1" + register "detect" = "1" device i2c 15 on end end chip drivers/i2c/hid @@ -106,7 +106,7 @@ chip soc/intel/apollolake register "generic.desc" = ""Synaptics Touchpad"" register "generic.irq" = "ACPI_IRQ_WAKE_LEVEL_LOW(GPIO_135_IRQ)" register "generic.wake" = "GPE0_DW3_27" - register "generic.probed" = "1" + register "generic.detect" = "1" register "hid_desc_reg_offset" = "0x20" device i2c 0x2c on end end diff --git a/src/mainboard/google/octopus/variants/meep/overridetree.cb b/src/mainboard/google/octopus/variants/meep/overridetree.cb index 87521413d1..e98df71aab 100644 --- a/src/mainboard/google/octopus/variants/meep/overridetree.cb +++ b/src/mainboard/google/octopus/variants/meep/overridetree.cb @@ -201,7 +201,7 @@ chip soc/intel/apollolake register "desc" = ""ELAN Touchpad"" register "irq" = "ACPI_IRQ_WAKE_EDGE_LOW(GPIO_135_IRQ)" register "wake" = "GPE0_DW3_27" - register "probed" = "1" + register "detect" = "1" device i2c 15 on end end end # - I2C 6 diff --git a/src/mainboard/google/octopus/variants/phaser/overridetree.cb b/src/mainboard/google/octopus/variants/phaser/overridetree.cb index ae52c3dc57..108d482239 100644 --- a/src/mainboard/google/octopus/variants/phaser/overridetree.cb +++ b/src/mainboard/google/octopus/variants/phaser/overridetree.cb @@ -133,7 +133,7 @@ chip soc/intel/apollolake register "desc" = ""ELAN Touchpad"" register "irq" = "ACPI_IRQ_WAKE_EDGE_LOW(GPIO_135_IRQ)" register "wake" = "GPE0_DW3_27" - register "probed" = "1" + register "detect" = "1" device i2c 15 on end end chip drivers/i2c/hid @@ -141,7 +141,7 @@ chip soc/intel/apollolake register "generic.desc" = ""Synaptics Touchpad"" register "generic.irq" = "ACPI_IRQ_WAKE_LEVEL_LOW(GPIO_135_IRQ)" register "generic.wake" = "GPE0_DW3_27" - register "generic.probed" = "1" + register "generic.detect" = "1" register "hid_desc_reg_offset" = "0x20" device i2c 0x2c on end end diff --git a/src/mainboard/google/octopus/variants/yorp/overridetree.cb b/src/mainboard/google/octopus/variants/yorp/overridetree.cb index a7fec75066..bde3ea4ea3 100644 --- a/src/mainboard/google/octopus/variants/yorp/overridetree.cb +++ b/src/mainboard/google/octopus/variants/yorp/overridetree.cb @@ -79,7 +79,7 @@ chip soc/intel/apollolake register "desc" = ""ELAN Touchpad"" register "irq" = "ACPI_IRQ_WAKE_EDGE_LOW(GPIO_135_IRQ)" register "wake" = "GPE0_DW3_27" - register "probed" = "1" + register "detect" = "1" device i2c 15 on end end end # - I2C 6 -- cgit v1.2.3