[coreboot-gerrit] Change in coreboot[master]: mainboard/google/eve: Remove 'probed' setting from rt5663 codec

Duncan Laurie (Code Review) gerrit at coreboot.org
Sat Apr 15 12:59:12 CEST 2017


Duncan Laurie has submitted this change and it was merged. ( https://review.coreboot.org/19304 )

Change subject: mainboard/google/eve: Remove 'probed' setting from rt5663 codec
......................................................................


mainboard/google/eve: Remove 'probed' setting from rt5663 codec

Remove the 'probed' setting from the Realtek 5663 headset codec I2C
device.  This was added when we had a hardware issue that was preventing
I2C operation because the clock/data lines were swapped.

With new and/or reworked hardware this is no longer a problem and we do
not want the I2C layer in the kernel to talk to the device before the
rt5663 driver.

BUG=b:35585307
BRANCH=none
TEST=Boot on Eve and verify rt5663 driver still loads properly.

Change-Id: Ice38889e8f5d3fd1307056cab10fbe3f4e197749
Signed-off-by: Duncan Laurie <dlaurie at chromium.org>
Reviewed-on: https://review.coreboot.org/19304
Tested-by: build bot (Jenkins)
Reviewed-by: Furquan Shaikh <furquan at google.com>
Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
M src/mainboard/google/eve/devicetree.cb
1 file changed, 0 insertions(+), 1 deletion(-)

Approvals:
  Paul Menzel: Looks good to me, but someone else must approve
  build bot (Jenkins): Verified
  Furquan Shaikh: Looks good to me, approved



diff --git a/src/mainboard/google/eve/devicetree.cb b/src/mainboard/google/eve/devicetree.cb
index 99ddb2e..3de759c 100644
--- a/src/mainboard/google/eve/devicetree.cb
+++ b/src/mainboard/google/eve/devicetree.cb
@@ -277,7 +277,6 @@
 				register "name" = ""RT53""
 				register "desc" = ""Realtek RT5663""
 				register "irq_gpio" = "ACPI_GPIO_IRQ_EDGE_BOTH(GPP_D9)"
-				register "probed" = "1"
 				device i2c 13 on end
 			end
 			chip drivers/i2c/generic

-- 
To view, visit https://review.coreboot.org/19304
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ice38889e8f5d3fd1307056cab10fbe3f4e197749
Gerrit-PatchSet: 2
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Duncan Laurie <dlaurie at chromium.org>
Gerrit-Reviewer: Duncan Laurie <dlaurie at chromium.org>
Gerrit-Reviewer: Furquan Shaikh <furquan at google.com>
Gerrit-Reviewer: Paul Menzel <paulepanter at users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins)



More information about the coreboot-gerrit mailing list