diff options
| author | Simon Glass <sjg@chromium.org> | 2018-05-23 15:34:04 -0600 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2018-06-25 20:50:14 +0000 |
| commit | 4f16049f17a4dcbf329d6b30f0d00f0a7f5490bf (patch) | |
| tree | fc85f157c406d075af7b9423fe9193aae2b2d72c /src/mainboard/google/kahlee/mainboard.c | |
| parent | 57ccb9c5e859f515aa4f07e6bf81a9f2ae58a988 (diff) | |
| download | coreboot-4f16049f17a4dcbf329d6b30f0d00f0a7f5490bf.tar.xz | |
mb/google/kahlee/variants/grunt: Select low-power mode for BayHub720
Put the PCIe clock pins in power-saving mode for the BayHub eMMC bridge to
save power. This requires use of an additional register (Misc control
register 2) and another bit in the existing 'protect' register. The naming
of bit 0 of that register is incorrect, based on the latest datasheet
(14 June 2018) so fix that too.
BUG=b:73726008
BRANCH=none
TEST=boot without this patch:
iotools mem_read32 0xfed80e00
0x0046ffff
With this patch:
$ iotools mem_read32 0xfed80e00
0x00463fff
Also see that the PCIe clock stops when eMMC is idle and can be started by
starting disk activity.
Change-Id: I5ad1467b2e2e151215d2dfd2ce48cd4a451fe480
Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-on: https://review.coreboot.org/26515
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/mainboard/google/kahlee/mainboard.c')
| -rw-r--r-- | src/mainboard/google/kahlee/mainboard.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/mainboard/google/kahlee/mainboard.c b/src/mainboard/google/kahlee/mainboard.c index 6ba69a3665..cf600d2cec 100644 --- a/src/mainboard/google/kahlee/mainboard.c +++ b/src/mainboard/google/kahlee/mainboard.c @@ -18,6 +18,7 @@ #include <console/console.h> #include <device/device.h> #include <arch/acpi.h> +#include <arch/io.h> #include <amdblocks/agesawrapper.h> #include <amdblocks/amd_pci_util.h> #include <cbmem.h> @@ -143,6 +144,12 @@ static void mainboard_init(void *chip_info) /* Set GenIntDisable so that GPIO 90 is configured as a GPIO. */ pm_write8(PM_PCIB_CFG, pm_read8(PM_PCIB_CFG) | PM_GENINT_DISABLE); + + /* Set low-power mode for BayHub eMMC bridge's PCIe clock. */ + clrsetbits_le32((uint32_t *)(MISC_MMIO_BASE + GPP_CLK_CNTRL), + GPP_CLK2_CLOCK_REQ_MAP_MASK, + GPP_CLK2_CLOCK_REQ_MAP_CLK_REQ2 << + GPP_CLK2_CLOCK_REQ_MAP_SHIFT); } /************************************************* |