Change in coreboot[master]: mb/google/brya/var/kinox: Add delay time for BH799BB rtd3

[Martin L Roth (Code Review)]

Martin L Roth has submitted this change. ( https://review.coreboot.org/c/coreboot/+/64686 )

Change subject: mb/google/brya/var/kinox: Add delay time for BH799BB rtd3
......................................................................

mb/google/brya/var/kinox: Add delay time for BH799BB rtd3

This CL adds the delay time into the RTD3 sequence, which will turn
off the eMMC controller (a true D3cold state) during the RTD3 sequence.
We checked power on sequence requires enable pin prior to reset pin
delay of 50ms and add delay of 20ms to meet the sequence on various
eMMC SKUs. Based on BH799BB_Preliminary_DS_R079_20201124.pdf in
chapter 7.2.

BUG=b:232327947
TEST=Build and suspend_stress_test -c 2500 pass

Signed-off-by: Dtrain Hsu 
Change-Id: I42cde5336f73a446cf5157e78f955fef8d70ae7e
Reviewed-on: https://review.coreboot.org/c/coreboot/+/64686
Tested-by: build bot (Jenkins) 
Reviewed-by: Tim Wawrzynczak 
---
M src/mainboard/google/brya/variants/kinox/overridetree.cb
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Tim Wawrzynczak: Looks good to me, approved

diff --git a/src/mainboard/google/brya/variants/kinox/overridetree.cb b/src/mainboard/google/brya/variants/kinox/overridetree.cb
index 2646b63..7aa2536 100644
--- a/src/mainboard/google/brya/variants/kinox/overridetree.cb
+++ b/src/mainboard/google/brya/variants/kinox/overridetree.cb
@@ -206,6 +206,8 @@
    			register "enable_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_H13)"
    			register "reset_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_D18)"
    			register "srcclk_pin" = "1"
+				register "reset_delay_ms" = "50"
+				register "enable_delay_ms" = "20"
    			device generic 0 alias emmc_rtd3 on end
    		end
    	end # BH799BBLN