diff options
author | Prasad Malisetty <pmaliset@codeaurora.org> | 2021-08-28 06:53:37 +0530 |
---|---|---|
committer | Shelley Chen <shchen@google.com> | 2022-08-01 18:04:54 +0000 |
commit | 857d3781e2f70894f3d5b5064e70ff3c275a6149 (patch) | |
tree | c5901895dfedc3ac858c503ac7c93741a252d90f /payloads/libpayload | |
parent | 16611f31eb462312068a16ccbf66972bfb5c6262 (diff) |
libpayload/pci: Add pci_map_bus function for Qualcomm platform
Add 'pci_map_bus' function and PCIE_QCOM config for Qualcomm platform.
BUG=b:182963902,b:216686574,b:181098581
TEST=Verified on Qualcomm sc7280 development board with NVMe endpoint
(Koixa NVMe, Model-KBG40ZPZ256G with FW AEGA0102). Confirmed NVMe is
getting detected in response to 'storage init' command in depthcharge
CLI prompt.
Output logs:
->dpch: storage init
Initializing NVMe controller 1e0f:0001
Identified NVMe model KBG40ZPZ256G TOSHIBA MEMORY
Added NVMe drive "NVMe Namespace 1" lbasize:512, count:0x1dcf32b0
* 0: NVMe Namespace 1
1 devices total
Also verified NVMe boot path that is depthcharge is able to load the
kernel image from NVMe storage.
Change-Id: I7d1217502cbd7d4d0cdd298919ae82435630d61c
Signed-off-by: Prasad Malisetty <quic_pmaliset@quicinc.com>
Signed-off-by: Veerabhadrarao Badiganti <quic_vbadigan@quicinc.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/57615
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'payloads/libpayload')
-rw-r--r-- | payloads/libpayload/Kconfig | 5 | ||||
-rw-r--r-- | payloads/libpayload/drivers/Makefile.inc | 1 | ||||
-rw-r--r-- | payloads/libpayload/drivers/pci_qcom.c | 126 |
3 files changed, 132 insertions, 0 deletions
diff --git a/payloads/libpayload/Kconfig b/payloads/libpayload/Kconfig index 7c5a95bafd..fcef18a7f0 100644 --- a/payloads/libpayload/Kconfig +++ b/payloads/libpayload/Kconfig @@ -418,6 +418,11 @@ config PCIE_MEDIATEK depends on PCI && !PCI_IO_OPS default n +config PCIE_QCOM + bool "Support for PCIe devices on Qualcomm platforms" + depends on PCI && !PCI_IO_OPS + default n + config NVRAM bool "Support for reading/writing NVRAM bytes" depends on ARCH_X86 # for now diff --git a/payloads/libpayload/drivers/Makefile.inc b/payloads/libpayload/drivers/Makefile.inc index 204760757b..23471b83c4 100644 --- a/payloads/libpayload/drivers/Makefile.inc +++ b/payloads/libpayload/drivers/Makefile.inc @@ -37,6 +37,7 @@ libc-$(CONFIG_LP_PCI) += pci_map_bus_ops.c endif libc-$(CONFIG_LP_PCIE_MEDIATEK) += pcie_mediatek.c +libc-$(CONFIG_LP_PCIE_QCOM) += pci_qcom.c libc-$(CONFIG_LP_SPEAKER) += speaker.c diff --git a/payloads/libpayload/drivers/pci_qcom.c b/payloads/libpayload/drivers/pci_qcom.c new file mode 100644 index 0000000000..ee159b952c --- /dev/null +++ b/payloads/libpayload/drivers/pci_qcom.c @@ -0,0 +1,126 @@ +/* SPDX-License-Identifier: GPL-2.0-only */ + +#include <libpayload.h> +#include <pci.h> + +/* + * iATU Unroll-specific register definitions + */ +#define PCIE_ATU_UNR_REGION_CTRL1 0x00 +#define PCIE_ATU_UNR_REGION_CTRL2 0x04 +#define PCIE_ATU_UNR_LOWER_BASE 0x08 +#define PCIE_ATU_UNR_UPPER_BASE 0x0C +#define PCIE_ATU_UNR_LIMIT 0x10 +#define PCIE_ATU_UNR_LOWER_TARGET 0x14 +#define PCIE_ATU_UNR_UPPER_TARGET 0x18 +#define PCIE_ATU_REGION_INDEX0 0x0 +#define PCIE_ATU_TYPE_CFG0 0x4 +#define PCIE_ATU_TYPE_CFG1 0x5 +#define PCIE_ATU_ENABLE BIT(31) +#define ATU_CTRL2 PCIE_ATU_UNR_REGION_CTRL2 +#define ATU_ENABLE PCIE_ATU_ENABLE + +#define PCIE_ATU_BUS(x) (((x) & 0xff) << 24) +#define PCIE_ATU_DEV(x) (((x) & 0x1f) << 19) +#define PCIE_ATU_FUNC(x) (((x) & 0x7) << 16) +#define LINK_WAIT_IATU_US 1000 +#define LINK_WAIT_MAX_IATU_RETRIES 5 + +/* Register address builder */ +#define PCIE_GET_ATU_OUTB_UNR_REG_OFFSET(region) ((region) << 9) + +#define lower_32_bits(n) ((u32)(n)) +#define upper_32_bits(n) ((u32)(((n) >> 16) >> 16)) + +/* + * ATU & endpoint config space base address offsets relative to + * PCIe controller base address. + */ +#define QCOM_ATU_BASE_OFFSET 0x1000 +#define QCOM_EP_CFG_OFFSET 0x100000 +#define QCOM_EP_CFG_SIZE 0x1000 /* 4K */ + +static void dw_pcie_writel_iatu(void *atu_base, unsigned short index, + uint32_t reg, uint32_t val) +{ + uint32_t offset = PCIE_GET_ATU_OUTB_UNR_REG_OFFSET(index); + + write32(atu_base + offset + reg, val); +} + +static uint32_t dw_pcie_readl_iatu(void *atu_base, unsigned short index, + uint32_t reg) +{ + uint32_t offset = PCIE_GET_ATU_OUTB_UNR_REG_OFFSET(index); + + return read32(atu_base + offset + reg); +} + +static void dw_pcie_prog_outbound_atu(void *atu_base, unsigned short index, + unsigned int type, uint64_t cfg_addr, + uint64_t pcie_addr, uint32_t cfg_size) +{ + dw_pcie_writel_iatu(atu_base, index, PCIE_ATU_UNR_LOWER_BASE, + lower_32_bits(cfg_addr)); + dw_pcie_writel_iatu(atu_base, index, PCIE_ATU_UNR_UPPER_BASE, + upper_32_bits(cfg_addr)); + dw_pcie_writel_iatu(atu_base, index, PCIE_ATU_UNR_LIMIT, + lower_32_bits(cfg_addr + cfg_size - 1)); + dw_pcie_writel_iatu(atu_base, index, PCIE_ATU_UNR_LOWER_TARGET, + lower_32_bits(pcie_addr)); + dw_pcie_writel_iatu(atu_base, index, PCIE_ATU_UNR_UPPER_TARGET, + upper_32_bits(pcie_addr)); + dw_pcie_writel_iatu(atu_base, index, PCIE_ATU_UNR_REGION_CTRL1, type); + dw_pcie_writel_iatu(atu_base, index, PCIE_ATU_UNR_REGION_CTRL2, + PCIE_ATU_ENABLE); + /* + * Make sure ATU enable takes effect before any subsequent config + * and I/O accesses. + */ + if (retry(LINK_WAIT_MAX_IATU_RETRIES, + (dw_pcie_readl_iatu(atu_base, index, ATU_CTRL2) & ATU_ENABLE), + udelay(LINK_WAIT_IATU_US))) + return; + + printf("outbound iATU is couldn't be enabled after 5ms\n"); +} + +/* Get PCIe MMIO configuration space base address */ +uintptr_t pci_map_bus(pcidev_t dev) +{ + unsigned int atu_type, busdev; + uint32_t config_size; + void *cntrlr_base, *config_base, *atu_base; + unsigned int current_bus = PCI_BUS(dev); + unsigned int devfn = (PCI_SLOT(dev) << 3) | PCI_FUNC(dev); + static pcidev_t current_dev; + + /* + * Extract PCIe controller base from coreboot and derive the ATU and + * endpoint config base addresses from it. + */ + cntrlr_base = (void *)lib_sysinfo.pcie_ctrl_base; + config_base = (void *)cntrlr_base + QCOM_EP_CFG_OFFSET; + config_size = (uint32_t)QCOM_EP_CFG_SIZE; + atu_base = (void *)cntrlr_base + QCOM_ATU_BASE_OFFSET; + + /* + * Cache the dev. For same dev, ATU mapping is not needed for each + * request. + */ + if (current_dev == dev) + goto out; + + current_dev = dev; + + busdev = PCIE_ATU_BUS(current_bus) | + PCIE_ATU_DEV(PCI_SLOT(dev)) | + PCIE_ATU_FUNC(PCI_FUNC(dev)); + + atu_type = current_bus == 1 ? PCIE_ATU_TYPE_CFG0 : PCIE_ATU_TYPE_CFG1; + + dw_pcie_prog_outbound_atu(atu_base, PCIE_ATU_REGION_INDEX0, atu_type, + (uint64_t)config_base, busdev, config_size); +out: + return (uintptr_t)config_base + (QCOM_EP_CFG_SIZE * devfn); +} |