mb/google/brya/var/*: Add disable_gpio_export_in_crs to all devicetrees
None of the touchscreens used in the brya program (any brya board) should require exporting of GPIOs in the ACPI _CRS method for any i2c device. This can cause i2c devices to malfunction or cause timing sequence violations if: 1) ACPI exports a PowerResource for the device that uses GPIOs that are also exported in _CRS 2) The kernel driver for the device uses the GPIOs exported in _CRS for its own purposes. This means the state of the pin is out of sync between platform firmware and the kernel. The Linux ELAN I2C touchcsreen driver (https://source.chromium.org/chromiumos/chromiumos/codesearch/+/main:src/third_party/kernel/upstream/drivers/input/touchscreen/elants_i2c.c;l=1429) is one example of this. Therefore, add disable_gpio_export_in_crs to all brya variants that use the drivers/i2c/generic or drivers/i2c/hid chip drivers. Change-Id: Ib4475bd0dc885e230911de6298fd95baa868ef29 Signed-off-by: Tim Wawrzynczak <twawrzynczak@chromium.org> Reviewed-on: https://review.coreboot.org/c/coreboot/+/60175 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Raul Rangel <rrangel@chromium.org>
This commit is contained in:
parent
8140691742
commit
a7e85d43c8
|
@ -215,6 +215,7 @@ chip soc/intel/alderlake
|
||||||
register "stop_gpio" =
|
register "stop_gpio" =
|
||||||
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
||||||
register "has_power_resource" = "1"
|
register "has_power_resource" = "1"
|
||||||
|
register "disable_gpio_export_in_crs" = "1"
|
||||||
device i2c 10 on end
|
device i2c 10 on end
|
||||||
end
|
end
|
||||||
chip drivers/i2c/hid
|
chip drivers/i2c/hid
|
||||||
|
@ -229,6 +230,7 @@ chip soc/intel/alderlake
|
||||||
"ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_C0)"
|
"ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_C0)"
|
||||||
register "generic.enable_delay_ms" = "1"
|
register "generic.enable_delay_ms" = "1"
|
||||||
register "generic.has_power_resource" = "1"
|
register "generic.has_power_resource" = "1"
|
||||||
|
register "generic.disable_gpio_export_in_crs" = "1"
|
||||||
register "hid_desc_reg_offset" = "0x01"
|
register "hid_desc_reg_offset" = "0x01"
|
||||||
device i2c 40 on end
|
device i2c 40 on end
|
||||||
end
|
end
|
||||||
|
|
|
@ -215,6 +215,7 @@ chip soc/intel/alderlake
|
||||||
register "stop_gpio" =
|
register "stop_gpio" =
|
||||||
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
||||||
register "has_power_resource" = "1"
|
register "has_power_resource" = "1"
|
||||||
|
register "disable_gpio_export_in_crs" = "1"
|
||||||
device i2c 10 on end
|
device i2c 10 on end
|
||||||
end
|
end
|
||||||
chip drivers/i2c/hid
|
chip drivers/i2c/hid
|
||||||
|
@ -229,6 +230,7 @@ chip soc/intel/alderlake
|
||||||
"ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_C0)"
|
"ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_C0)"
|
||||||
register "generic.enable_delay_ms" = "1"
|
register "generic.enable_delay_ms" = "1"
|
||||||
register "generic.has_power_resource" = "1"
|
register "generic.has_power_resource" = "1"
|
||||||
|
register "generic.disable_gpio_export_in_crs" = "1"
|
||||||
register "hid_desc_reg_offset" = "0x01"
|
register "hid_desc_reg_offset" = "0x01"
|
||||||
device i2c 40 on end
|
device i2c 40 on end
|
||||||
end
|
end
|
||||||
|
|
|
@ -260,6 +260,7 @@ chip soc/intel/alderlake
|
||||||
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
||||||
register "generic.stop_off_delay_ms" = "1"
|
register "generic.stop_off_delay_ms" = "1"
|
||||||
register "generic.has_power_resource" = "1"
|
register "generic.has_power_resource" = "1"
|
||||||
|
register "generic.disable_gpio_export_in_crs" = "1"
|
||||||
register "hid_desc_reg_offset" = "0x01"
|
register "hid_desc_reg_offset" = "0x01"
|
||||||
device i2c 10 on end
|
device i2c 10 on end
|
||||||
end
|
end
|
||||||
|
|
|
@ -217,6 +217,7 @@ chip soc/intel/alderlake
|
||||||
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
||||||
register "generic.stop_off_delay_ms" = "1"
|
register "generic.stop_off_delay_ms" = "1"
|
||||||
register "generic.has_power_resource" = "1"
|
register "generic.has_power_resource" = "1"
|
||||||
|
register "generic.disable_gpio_export_in_crs" = "1"
|
||||||
register "hid_desc_reg_offset" = "0x01"
|
register "hid_desc_reg_offset" = "0x01"
|
||||||
device i2c 0x15 on end
|
device i2c 0x15 on end
|
||||||
end
|
end
|
||||||
|
|
|
@ -173,6 +173,7 @@ chip soc/intel/alderlake
|
||||||
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
||||||
register "generic.stop_off_delay_ms" = "1"
|
register "generic.stop_off_delay_ms" = "1"
|
||||||
register "generic.has_power_resource" = "1"
|
register "generic.has_power_resource" = "1"
|
||||||
|
register "generic.disable_gpio_export_in_crs" = "1"
|
||||||
register "hid_desc_reg_offset" = "0x01"
|
register "hid_desc_reg_offset" = "0x01"
|
||||||
device i2c 0x15 on end
|
device i2c 0x15 on end
|
||||||
end
|
end
|
||||||
|
|
|
@ -425,6 +425,7 @@ chip soc/intel/alderlake
|
||||||
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
||||||
register "generic.stop_off_delay_ms" = "1"
|
register "generic.stop_off_delay_ms" = "1"
|
||||||
register "generic.has_power_resource" = "1"
|
register "generic.has_power_resource" = "1"
|
||||||
|
register "generic.disable_gpio_export_in_crs" = "1"
|
||||||
register "hid_desc_reg_offset" = "0x01"
|
register "hid_desc_reg_offset" = "0x01"
|
||||||
device i2c 0x10 on end
|
device i2c 0x10 on end
|
||||||
end
|
end
|
||||||
|
|
|
@ -236,6 +236,7 @@ chip soc/intel/alderlake
|
||||||
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
||||||
register "generic.stop_off_delay_ms" = "1"
|
register "generic.stop_off_delay_ms" = "1"
|
||||||
register "generic.has_power_resource" = "1"
|
register "generic.has_power_resource" = "1"
|
||||||
|
register "generic.disable_gpio_export_in_crs" = "1"
|
||||||
register "hid_desc_reg_offset" = "0x01"
|
register "hid_desc_reg_offset" = "0x01"
|
||||||
device i2c 0x10 on end
|
device i2c 0x10 on end
|
||||||
end
|
end
|
||||||
|
@ -251,6 +252,7 @@ chip soc/intel/alderlake
|
||||||
"ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_C0)"
|
"ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_C0)"
|
||||||
register "generic.enable_delay_ms" = "1"
|
register "generic.enable_delay_ms" = "1"
|
||||||
register "generic.has_power_resource" = "1"
|
register "generic.has_power_resource" = "1"
|
||||||
|
register "generic.disable_gpio_export_in_crs" = "1"
|
||||||
register "hid_desc_reg_offset" = "0x01"
|
register "hid_desc_reg_offset" = "0x01"
|
||||||
device i2c 0x40 on end
|
device i2c 0x40 on end
|
||||||
end
|
end
|
||||||
|
|
|
@ -223,6 +223,7 @@ chip soc/intel/alderlake
|
||||||
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
||||||
register "generic.stop_off_delay_ms" = "1"
|
register "generic.stop_off_delay_ms" = "1"
|
||||||
register "generic.has_power_resource" = "1"
|
register "generic.has_power_resource" = "1"
|
||||||
|
register "generic.disable_gpio_export_in_crs" = "1"
|
||||||
register "hid_desc_reg_offset" = "0x01"
|
register "hid_desc_reg_offset" = "0x01"
|
||||||
device i2c 0x10 on end
|
device i2c 0x10 on end
|
||||||
end
|
end
|
||||||
|
@ -238,6 +239,7 @@ chip soc/intel/alderlake
|
||||||
"ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_C0)"
|
"ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_C0)"
|
||||||
register "generic.enable_delay_ms" = "1"
|
register "generic.enable_delay_ms" = "1"
|
||||||
register "generic.has_power_resource" = "1"
|
register "generic.has_power_resource" = "1"
|
||||||
|
register "generic.disable_gpio_export_in_crs" = "1"
|
||||||
register "hid_desc_reg_offset" = "0x01"
|
register "hid_desc_reg_offset" = "0x01"
|
||||||
device i2c 0x40 on end
|
device i2c 0x40 on end
|
||||||
end
|
end
|
||||||
|
|
|
@ -386,6 +386,7 @@ chip soc/intel/alderlake
|
||||||
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
||||||
register "generic.stop_off_delay_ms" = "1"
|
register "generic.stop_off_delay_ms" = "1"
|
||||||
register "generic.has_power_resource" = "1"
|
register "generic.has_power_resource" = "1"
|
||||||
|
register "generic.disable_gpio_export_in_crs" = "1"
|
||||||
register "hid_desc_reg_offset" = "0x01"
|
register "hid_desc_reg_offset" = "0x01"
|
||||||
device i2c 0x15 on end
|
device i2c 0x15 on end
|
||||||
end
|
end
|
||||||
|
|
|
@ -270,6 +270,7 @@ chip soc/intel/alderlake
|
||||||
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
||||||
register "generic.stop_off_delay_ms" = "1"
|
register "generic.stop_off_delay_ms" = "1"
|
||||||
register "generic.has_power_resource" = "1"
|
register "generic.has_power_resource" = "1"
|
||||||
|
register "generic.disable_gpio_export_in_crs" = "1"
|
||||||
register "hid_desc_reg_offset" = "0x01"
|
register "hid_desc_reg_offset" = "0x01"
|
||||||
device i2c 0x15 on end
|
device i2c 0x15 on end
|
||||||
end
|
end
|
||||||
|
|
|
@ -292,6 +292,7 @@ chip soc/intel/alderlake
|
||||||
# Parameter T4 >= 1ms
|
# Parameter T4 >= 1ms
|
||||||
register "generic.stop_off_delay_ms" = "1"
|
register "generic.stop_off_delay_ms" = "1"
|
||||||
register "generic.has_power_resource" = "1"
|
register "generic.has_power_resource" = "1"
|
||||||
|
register "generic.disable_gpio_export_in_crs" = "1"
|
||||||
register "hid_desc_reg_offset" = "0x01"
|
register "hid_desc_reg_offset" = "0x01"
|
||||||
device i2c 5d on end
|
device i2c 5d on end
|
||||||
end
|
end
|
||||||
|
@ -307,6 +308,7 @@ chip soc/intel/alderlake
|
||||||
"ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_C0)"
|
"ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_C0)"
|
||||||
register "enable_delay_ms" = "1"
|
register "enable_delay_ms" = "1"
|
||||||
register "has_power_resource" = "1"
|
register "has_power_resource" = "1"
|
||||||
|
register "disable_gpio_export_in_crs" = "1"
|
||||||
device i2c 10 on end
|
device i2c 10 on end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -292,6 +292,7 @@ chip soc/intel/alderlake
|
||||||
# Parameter T4 >= 1ms
|
# Parameter T4 >= 1ms
|
||||||
register "generic.stop_off_delay_ms" = "1"
|
register "generic.stop_off_delay_ms" = "1"
|
||||||
register "generic.has_power_resource" = "1"
|
register "generic.has_power_resource" = "1"
|
||||||
|
register "generic.disable_gpio_export_in_crs" = "1"
|
||||||
register "hid_desc_reg_offset" = "0x01"
|
register "hid_desc_reg_offset" = "0x01"
|
||||||
device i2c 5d on end
|
device i2c 5d on end
|
||||||
end
|
end
|
||||||
|
@ -307,6 +308,7 @@ chip soc/intel/alderlake
|
||||||
"ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_C0)"
|
"ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_C0)"
|
||||||
register "enable_delay_ms" = "1"
|
register "enable_delay_ms" = "1"
|
||||||
register "has_power_resource" = "1"
|
register "has_power_resource" = "1"
|
||||||
|
register "disable_gpio_export_in_crs" = "1"
|
||||||
device i2c 10 on end
|
device i2c 10 on end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -191,6 +191,7 @@ chip soc/intel/alderlake
|
||||||
register "stop_gpio" =
|
register "stop_gpio" =
|
||||||
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
"ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
|
||||||
register "has_power_resource" = "1"
|
register "has_power_resource" = "1"
|
||||||
|
register "disable_gpio_export_in_crs" = "1"
|
||||||
device i2c 10 on end
|
device i2c 10 on end
|
||||||
end
|
end
|
||||||
chip drivers/i2c/hid
|
chip drivers/i2c/hid
|
||||||
|
@ -205,6 +206,7 @@ chip soc/intel/alderlake
|
||||||
"ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_C0)"
|
"ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_C0)"
|
||||||
register "generic.enable_delay_ms" = "1"
|
register "generic.enable_delay_ms" = "1"
|
||||||
register "generic.has_power_resource" = "1"
|
register "generic.has_power_resource" = "1"
|
||||||
|
register "generic.disable_gpio_export_in_crs" = "1"
|
||||||
register "hid_desc_reg_offset" = "0x01"
|
register "hid_desc_reg_offset" = "0x01"
|
||||||
device i2c 40 on end
|
device i2c 40 on end
|
||||||
end
|
end
|
||||||
|
|
Loading…
Reference in New Issue