Akshu Agrawal has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/30725
Change subject: mb/google/kahlee/liara: Add 20ms delay to capture ......................................................................
mb/google/kahlee/liara: Add 20ms delay to capture
define wakeup-delay-ms to 20ms. This avoids the pop noise heard at the start of capture.
BUG=b:119926436 TEST=with kernel patch https://lore.kernel.org/patchwork/patch/1029806/ no pop sound heard at start of capture
Change-Id: I2593afa69cfb955f6a2b695406855e0f31f28468 Signed-off-by: Akshu Agrawal akshu.agrawal@amd.com --- M src/mainboard/google/kahlee/variants/liara/devicetree.cb 1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/25/30725/1
diff --git a/src/mainboard/google/kahlee/variants/liara/devicetree.cb b/src/mainboard/google/kahlee/variants/liara/devicetree.cb index bc6c643..a929c46 100644 --- a/src/mainboard/google/kahlee/variants/liara/devicetree.cb +++ b/src/mainboard/google/kahlee/variants/liara/devicetree.cb @@ -97,6 +97,7 @@ end #domain device mmio 0xfedc2000 on chip drivers/generic/adau7002 + register "wakeup_delay" = "20" device generic 0.0 on end end chip drivers/i2c/da7219
Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30725 )
Change subject: mb/google/kahlee/liara: Add 20ms delay to capture ......................................................................
Patch Set 2: Code-Review+2
Martin Roth has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/30725 )
Change subject: mb/google/kahlee/liara: Add 20ms delay to capture ......................................................................
mb/google/kahlee/liara: Add 20ms delay to capture
define wakeup-delay-ms to 20ms. This avoids the pop noise heard at the start of capture.
BUG=b:119926436 TEST=with kernel patch https://lore.kernel.org/patchwork/patch/1029806/ no pop sound heard at start of capture
Change-Id: I2593afa69cfb955f6a2b695406855e0f31f28468 Signed-off-by: Akshu Agrawal akshu.agrawal@amd.com Reviewed-on: https://review.coreboot.org/c/30725 Reviewed-by: Martin Roth martinroth@google.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/mainboard/google/kahlee/variants/liara/devicetree.cb 1 file changed, 1 insertion(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Martin Roth: Looks good to me, approved
diff --git a/src/mainboard/google/kahlee/variants/liara/devicetree.cb b/src/mainboard/google/kahlee/variants/liara/devicetree.cb index bc6c643..a929c46 100644 --- a/src/mainboard/google/kahlee/variants/liara/devicetree.cb +++ b/src/mainboard/google/kahlee/variants/liara/devicetree.cb @@ -97,6 +97,7 @@ end #domain device mmio 0xfedc2000 on chip drivers/generic/adau7002 + register "wakeup_delay" = "20" device generic 0.0 on end end chip drivers/i2c/da7219
-
Akshu Agrawal (Code Review) -
Martin Roth (Code Review)