From 7e3883633a528f9bb3db3bf97864aee01c2de4a5 Mon Sep 17 00:00:00 2001 From: Ren Kuo Date: Wed, 15 Oct 2025 14:45:44 +0800 Subject: [PATCH] mb/google/brox/var/jubilant: Apply fw_config to enable/disable I2C1 As a result of hardware BOM design, U51 (power gate for touchscreen) would be required to remove on non-touch SKU. The change will cause the I2C1 touchscreen devices probe ERROR of non-touch SKU since no power for I2C bus pull-high resistors.The ERROR is waiting I2C stop condition time out then bootperf test will get fail. The CL apply fw_config field 19 - PANEL_PWR_SEQ_CTRL for I2C1: 0...disable (non-touch sku) 1...enable (touch sku) Turn off I2C1 for non-touch sku, and keep I2C1 is on for touch sku. It will avoid the touchscreen probing error on I2c1. BUG=b:447513390 TEST=Check boot to kernel time is 1,376 sec under spec and without I2C probe error in ap log of non-touch sku. Check touchscreen device works well of touch sku. Change-Id: I72a68177a90cea88fe283d8499b8378c64206fa2 Signed-off-by: Ren Kuo Reviewed-on: https://review.coreboot.org/c/coreboot/+/89591 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel Reviewed-by: David Wu --- src/mainboard/google/brox/variants/jubilant/overridetree.cb | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/mainboard/google/brox/variants/jubilant/overridetree.cb b/src/mainboard/google/brox/variants/jubilant/overridetree.cb index 25bedf870e..1d667507a8 100644 --- a/src/mainboard/google/brox/variants/jubilant/overridetree.cb +++ b/src/mainboard/google/brox/variants/jubilant/overridetree.cb @@ -16,6 +16,10 @@ fw_config option FPMCU_ABSENT 0 option FPMCU_NUVOTON 1 end + field PANEL_PWR 19 + option DISABLE 0 + option ENABLE 1 + end end chip soc/intel/alderlake @@ -402,6 +406,7 @@ chip soc/intel/alderlake register "hid_desc_reg_offset" = "0x01" device i2c 10 on end end + probe PANEL_PWR ENABLE end device ref i2c2 on chip drivers/i2c/generic