diff options
author | Rizwan Qureshi <rizwan.qureshi@intel.com> | 2019-02-17 11:31:21 +0530 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2019-02-27 11:05:13 +0000 |
commit | 8aadab7e96854bf8d43f117724089dc8d5869efd (patch) | |
tree | f9cbd40e118a2e6c2e1fd5a2533cdbe384f7c0a3 /Makefile.inc | |
parent | e64c25ca1a550132e1ad35208d6a71d553d57879 (diff) | |
download | coreboot-8aadab7e96854bf8d43f117724089dc8d5869efd.tar.xz |
soc/intel/cannonlake: Add a config for configuring SD_VDD1_PWR_EN#
SD controller in CNL-PCH provides a ability to configure the behavior of
SD_VDD1_PWR_EN# as an active high or low signal. FSP provides an UPD
"SdCardPowerEnableActiveHigh" to control the same.
However, for platforms using SD_VDD1_PWR_EN# as active high, the SDXC
card connector is always powered and may impact system power. This is because
SD_VDD1_PWR_EN# does not de-assert during SDXC D3 or when SD card is not
inserted.
Workaround is to change the pad ownership of SD_VDD1_PWR_EN to GPIO and
force the TX buffer to low in _PS3. And restore the pad mode to native
function in _PS0.
Hence add a Kconfig option to update the UPD, which the board can select
based on how the SD_VDD1_PWR_EN is implemented on it. And, the workaround
gets applied based on this config.
BUG=b:123350329
Change-Id: Iee262d7ecdf8c31362aec3d95dd9b3e8359e0c25
Signed-off-by: Rizwan Qureshi <rizwan.qureshi@intel.com>
Reviewed-on: https://review.coreboot.org/c/31445
Reviewed-by: Furquan Shaikh <furquan@google.com>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'Makefile.inc')
0 files changed, 0 insertions, 0 deletions