From ce37e47f46e52090529a91a45d1c914d159643c8 Mon Sep 17 00:00:00 2001 From: "Zhuo-hao.Lee" Date: Wed, 27 Jul 2016 13:18:22 +0800 Subject: [PATCH] skylake: fix VSDIO is at 0.8V when SDCard is not inserted 1. Enable SoC SD_CMD/D* signals pull-down of 20k when SD-card is removed. When SD-card is disconnected, the pull-down is disabled. 2. Provide path for weak leakage from buffers of SD_CMD/D* signal to be grounded. Thus dropping voltage on the SD_CMD/D* signals to ~0V. BUG=chrome-os-partner:54421 TEST=no power leakage when SDCard isn't inserted on skylake platform Change-Id: I567199b172841125f8916a61a76005cfdaa62eb8 Signed-off-by: Zhuo-hao.Lee Reviewed-on: https://review.coreboot.org/15910 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel Reviewed-by: Duncan Laurie --- src/soc/intel/skylake/acpi/scs.asl | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/src/soc/intel/skylake/acpi/scs.asl b/src/soc/intel/skylake/acpi/scs.asl index 56f27a97ea..69bc82a143 100644 --- a/src/soc/intel/skylake/acpi/scs.asl +++ b/src/soc/intel/skylake/acpi/scs.asl @@ -88,6 +88,14 @@ Device (SDXC) Method (_PS0, 0, Serialized) { + /* Disable 20K pull-down on CLK, CMD and DAT lines */ + ^^PCRA (PID_GPIOCOM3, 0x4c4, 0xFFFFEFFF) + ^^PCRA (PID_GPIOCOM3, 0x4cc, 0xFFFFEFFF) + ^^PCRA (PID_GPIOCOM3, 0x4d4, 0xFFFFEFFF) + ^^PCRA (PID_GPIOCOM3, 0x4dc, 0xFFFFEFFF) + ^^PCRA (PID_GPIOCOM3, 0x4e4, 0xFFFFEFFF) + ^^PCRA (PID_GPIOCOM3, 0x4f4, 0xFFFFEFFF) + /* Disable Power Gate */ Store (0, ^PGEN) @@ -113,6 +121,14 @@ Device (SDXC) Store (3, Local0) Store (Local0, ^D0D3) Store (^D0D3, Local0) + + /* Enable 20K pull-down on CLK, CMD and DAT lines */ + ^^PCRO (PID_GPIOCOM3, 0x4c4, 0x00001000) + ^^PCRO (PID_GPIOCOM3, 0x4cc, 0x00001000) + ^^PCRO (PID_GPIOCOM3, 0x4d4, 0x00001000) + ^^PCRO (PID_GPIOCOM3, 0x4dc, 0x00001000) + ^^PCRO (PID_GPIOCOM3, 0x4e4, 0x00001000) + ^^PCRO (PID_GPIOCOM3, 0x4f4, 0x00001000) } Device (CARD)