[PATCH] misc: qcom_geni: Add minicore support
Varadarajan Narayanan
varadarajan.narayanan at oss.qualcomm.com
Fri Jul 10 13:37:25 CEST 2026
The qcom_geni driver reads an ELF from storage and configures a set of
registers and programs the firmware to the GENI Serial Engine (GENI-SE)
wrapper device for the expected functionality.
Unlike the GENI-SE wrapper found in MSM SoCs, the IPQ5210's GENI-SE
wrapper is pre-configured for one of the functions defined in 'enum
geni_se_protocol_type'. Hence, the firmware download is not needed.
Only the register configuration part is needed.
Earlier, the boot stages before U-Boot would configure the GENI-SE (to
access UART/SPI etc). Since for IPQ5210 U-Boot SPL, the previous stage
(i.e. boot ROM) doesn't do that modify the driver to do the register
configuration part alone without reading an ELF from the storage.
Reviewed-by: Balaji Selvanathan <balaji.selvanathan at oss.qualcomm.com>
Reviewed-by: Simon Glass <sjg at chromium.org>
Signed-off-by: Varadarajan Narayanan <varadarajan.narayanan at oss.qualcomm.com>
---
Address the feedback in [1].
- Separate out minicore related changes from the SPL changes
- Avoid CONFIG_QCOM_GENI_MINICORE check in multiple places
- Move struct qup_mini_core_info and related defines to separate header
- Undo changes w.r.t. EVENT_SPY_SIMPLE
- Rebased on https://source.denx.de/u-boot/custodians/u-boot-snapdragon.git
to use Micheal's EVENT siupport in SPL
1 - https://lore.kernel.org/u-boot/9f40675f-e39d-4849-a6f5-7dcd7aeb251a@linaro.org/
---
drivers/misc/Kconfig | 6 +++
drivers/misc/Makefile | 1 +
drivers/misc/qcom_geni-minicore.c | 103 ++++++++++++++++++++++++++++++++++++++
drivers/misc/qcom_geni.c | 85 +++++++++++++++++++++++++++----
include/soc/qcom/geni-se.h | 5 ++
include/soc/qcom/qup-fw-load.h | 1 +
6 files changed, 190 insertions(+), 11 deletions(-)
diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
index bde5c640de8..0b7894f131e 100644
--- a/drivers/misc/Kconfig
+++ b/drivers/misc/Kconfig
@@ -93,6 +93,12 @@ config QCOM_GENI
for providing a common interface for various peripherals like UART, I2C, SPI,
etc.
+config QCOM_GENI_MINICORE
+ bool "Support minicores in Qualcomm Generic Interface (GENI) driver"
+ depends on QCOM_GENI
+ help
+ Enable support for minicores in Qualcomm GENI and its peripherals.
+
config ROCKCHIP_EFUSE
bool "Rockchip e-fuse support"
depends on MISC
diff --git a/drivers/misc/Makefile b/drivers/misc/Makefile
index e2170212e5a..2daccbba74d 100644
--- a/drivers/misc/Makefile
+++ b/drivers/misc/Makefile
@@ -66,6 +66,7 @@ obj-$(CONFIG_QFW_SMBIOS) += qfw_smbios.o
obj-$(CONFIG_SANDBOX) += qfw_sandbox.o
endif
obj-$(CONFIG_QCOM_GENI) += qcom_geni.o
+obj-$(CONFIG_QCOM_GENI_MINICORE) += qcom_geni-minicore.o
obj-$(CONFIG_$(PHASE_)ROCKCHIP_EFUSE) += rockchip-efuse.o
obj-$(CONFIG_$(PHASE_)ROCKCHIP_OTP) += rockchip-otp.o
obj-$(CONFIG_$(PHASE_)ROCKCHIP_IODOMAIN) += rockchip-io-domain.o
diff --git a/drivers/misc/qcom_geni-minicore.c b/drivers/misc/qcom_geni-minicore.c
new file mode 100644
index 00000000000..12c57bdd8b2
--- /dev/null
+++ b/drivers/misc/qcom_geni-minicore.c
@@ -0,0 +1,103 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
+ */
+
+#include <soc/qcom/geni-se.h>
+#include <soc/qcom/qup-fw-load.h>
+#include <soc/qcom/minicore.h>
+
+/*
+ * Register configuration for the QUP minicores to setup the corresponding
+ * functionality of SPI/I2C/UART.
+ */
+static u8 cfg_reg_idx[] = {
+ /* 0 to 18 */
+ 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18,
+ /* 64 to 113 */
+ 64, 65, 66, 67, 68, 69, 70, 71, 72, 73, 74, 75, 76, 77, 78, 79, 80,
+ 81, 82, 83, 84, 85, 86, 87, 88, 89, 90, 91, 92, 93, 94, 95, 96, 97,
+ 98, 99, 100, 101, 102, 103, 104, 105, 106, 107, 108, 109, 110, 111,
+ 112, 113,
+};
+
+static u32 spi_cfg_val[] = {
+ /* 0 to 18 */
+ 0x00000000, 0x00000400, 0x00000000, 0x00000000, 0x00240E78, 0x00011088,
+ 0x00240007, 0x00000000, 0x00000000, 0x0001000A, 0x00000300, 0x00000000,
+ 0x00000000, 0x00000000, 0x00154400, 0x001483A0, 0x00AA8128, 0x00641002,
+ 0x00004000,
+ /* 64 to 113 */
+ 0x00000201, 0x0001FE05, 0x0002C2E7, 0x0A435C00, 0x0010011A, 0x08800000,
+ 0x00000000, 0x100CAC00, 0x00000000, 0x00000000, 0x00000000, 0x00000000,
+ 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x000018E4, 0x00000000,
+ 0x00000003, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000,
+ 0x0007F807, 0x000FFEFE, 0x00000000, 0x00000000, 0x00000000, 0x00000000,
+ 0x00200000, 0x00000004, 0x00000009, 0x0007F807, 0x000FFEFE, 0x00000000,
+ 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000,
+ 0x00000000, 0x00000000, 0x00C0033F, 0x00000000, 0x00000000, 0x00000000,
+ 0x00000000, 0x00000055,
+};
+
+static u32 uart_cfg_val[] = {
+ /* 0 to 18 */
+ 0x00000024, 0x00000000, 0x00000024, 0x00000000, 0x00019A00, 0x00400000,
+ 0x00E00000, 0x00010020, 0x00000000, 0x00000000, 0x00000300, 0x00000700,
+ 0x00000400, 0x00000000, 0x00000000, 0x00C00000, 0x00000000, 0x00C00024,
+ 0x00000B00,
+ /* 64 to 113 */
+ 0x00020231, 0x0000CE05, 0x000360E7, 0x0941E6A8, 0x00100510, 0x42C01E51,
+ 0x00000401, 0x002E8400, 0x1694581A, 0x00000000, 0x00000000, 0x00000000,
+ 0x00000000, 0x00000000, 0x00000000, 0x00000001, 0x0000031C, 0x00000000,
+ 0x0000000F, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000,
+ 0x00081C06, 0x00004010, 0x00000000, 0x00000000, 0x00000000, 0x00000000,
+ 0x00000000, 0x0000000D, 0x00000000, 0x00081C06, 0x00004010, 0x00000000,
+ 0x00000000, 0x00000000, 0x00000000, 0x00C02415, 0x0000000E, 0x00000001,
+ 0x00000001, 0x00000000, 0x00C00000, 0x00000000, 0x00000000, 0x00000000,
+ 0x00000000, 0x00000055,
+};
+
+static u32 i2c_cfg_val[] = {
+ /* 0 to 18 */
+ 0x00000090, 0x00000000, 0x00000090, 0x00000000, 0x00038028, 0x00084080,
+ 0x00000343, 0x00010000, 0x00000000, 0x00001A00, 0x00000100, 0x00000000,
+ 0x00000000, 0x00000000, 0x00808008, 0x001C0020, 0x00000000, 0x00020000,
+ 0x00000000,
+ /* 64 to 113 */
+ 0x00000201, 0x0001FC01, 0x00036222, 0x09C01FFC, 0x00100120, 0x02C00000,
+ 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000,
+ 0x00000001, 0x00000000, 0x00000000, 0x00000000, 0x00000409, 0x00000003,
+ 0x00000002, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000,
+ 0x0007F8FE, 0x000FFEFE, 0x00000000, 0x00000000, 0x00000000, 0x00000000,
+ 0x00000000, 0x00000000, 0x00000001, 0x0007F807, 0x000FFEFE, 0x00000000,
+ 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000,
+ 0x00000000, 0x00000000, 0x00C00000, 0x00000000, 0x00000000, 0x00000000,
+ 0x00000000, 0x00000055,
+};
+
+struct qup_mini_core_info qup_mini_cores[] = {
+ {
+ .serial_protocol = GENI_SE_SPI,
+ .fw_version = 0xb02,
+ .cfg_version = 0x9,
+ .cfg_count = ARRAY_SIZE(spi_cfg_val),
+ .cfg_val = spi_cfg_val,
+ .cfg_idx = cfg_reg_idx,
+ }, {
+ .serial_protocol = GENI_SE_UART,
+ .fw_version = 0x405,
+ .cfg_version = 0xa,
+ .cfg_count = ARRAY_SIZE(uart_cfg_val),
+ .cfg_val = uart_cfg_val,
+ .cfg_idx = cfg_reg_idx,
+ }, {
+ .serial_protocol = GENI_SE_I2C,
+ .fw_version = 0x204,
+ .cfg_version = 0x9,
+ .cfg_count = ARRAY_SIZE(i2c_cfg_val),
+ .cfg_val = i2c_cfg_val,
+ .cfg_idx = cfg_reg_idx,
+ }, {
+ .serial_protocol = GENI_SE_INVALID_PROTO,
+ },
+};
diff --git a/drivers/misc/qcom_geni.c b/drivers/misc/qcom_geni.c
index a62ae6a2478..f3133f85885 100644
--- a/drivers/misc/qcom_geni.c
+++ b/drivers/misc/qcom_geni.c
@@ -21,6 +21,7 @@
#include <linux/printk.h>
#include <soc/qcom/geni-se.h>
#include <soc/qcom/qup-fw-load.h>
+#include <soc/qcom/minicore.h>
#include <dm/device_compat.h>
struct qup_se_rsc {
@@ -34,8 +35,11 @@ struct qup_se_rsc {
struct geni_se_plat {
bool need_firmware_load;
+ bool is_mini_core;
};
+static int qcom_geni_fw_initialise(void);
+
/**
* geni_enable_interrupts() Enable interrupts.
* @rsc: Pointer to a structure representing SE-related resources.
@@ -163,16 +167,54 @@ static void geni_config_common_control(struct qup_se_rsc *rsc)
COMMON_CSR_SLV_CLK_CGC_ON_BMASK);
}
-static int load_se_firmware(struct qup_se_rsc *rsc, struct elf_se_hdr *hdr)
+static int load_se_firmware(struct qup_se_rsc *rsc, bool elf, void *info)
{
+ struct geni_se_plat *plat = dev_get_plat(rsc->dev->parent);
+ struct elf_se_hdr *hdr, tmp_hdr;
const u32 *fw_val_arr, *cfg_val_arr;
const u8 *cfg_idx_arr;
u32 i, reg_value, mask, ramn_cnt;
int ret;
- fw_val_arr = (const u32 *)((u8 *)hdr + hdr->fw_offset);
- cfg_idx_arr = (const u8 *)hdr + hdr->cfg_idx_offset;
- cfg_val_arr = (const u32 *)((u8 *)hdr + hdr->cfg_val_offset);
+ if (elf) {
+ hdr = info;
+ fw_val_arr = (const u32 *)((u8 *)hdr + hdr->fw_offset);
+ cfg_idx_arr = (const u8 *)hdr + hdr->cfg_idx_offset;
+ cfg_val_arr = (const u32 *)((u8 *)hdr + hdr->cfg_val_offset);
+ } else if (plat->is_mini_core) {
+ /*
+ * Minicore controllers come with pre-configured functionality
+ * and don't need a firmware download and just need the register
+ * configuration. Hence, skipping the firmware part and setting
+ * up just the register configuration related information.
+ */
+ struct qup_mini_core_info *qmc = info;
+
+ for (; qmc->serial_protocol != GENI_SE_INVALID_PROTO; qmc++)
+ if (qmc->serial_protocol == rsc->protocol)
+ break;
+
+ if (qmc->serial_protocol == GENI_SE_INVALID_PROTO) {
+ dev_err(rsc->dev, "Invalid MINICORE protocol (%d)\n",
+ rsc->protocol);
+ return -EINVAL;
+ }
+
+ tmp_hdr.magic = MAGIC_NUM_SE;
+ tmp_hdr.version = 1;
+ tmp_hdr.serial_protocol = rsc->protocol;
+ tmp_hdr.fw_version = qmc->fw_version;
+ tmp_hdr.cfg_version = qmc->cfg_version;
+ tmp_hdr.fw_size_in_items = qmc->cfg_ram_count;
+ tmp_hdr.cfg_size_in_items = qmc->cfg_count;
+ hdr = &tmp_hdr;
+ fw_val_arr = (const u32 *)qmc->cfg_ram;
+ cfg_idx_arr = (const u8 *)qmc->cfg_idx;
+ cfg_val_arr = (const u32 *)qmc->cfg_val;
+ } else {
+ dev_err(rsc->dev, "Neither fw nor register settings found\n");
+ return -EINVAL;
+ }
geni_config_common_control(rsc);
@@ -350,8 +392,9 @@ int qcom_geni_load_firmware(phys_addr_t qup_base,
{
struct qup_se_rsc rsc;
struct elf_se_hdr *hdr;
+ bool elf;
int ret;
- void *fw;
+ void *fw, *info;
rsc.dev = dev;
rsc.base = qup_base;
@@ -377,15 +420,26 @@ int qcom_geni_load_firmware(phys_addr_t qup_base,
/* The firmware blob is the private data of the GENI wrapper (parent) */
fw = dev_get_priv(dev->parent);
- ret = read_elf(&rsc, fw, &hdr);
- if (ret) {
- dev_err(dev, "Failed to read ELF: %d\n", ret);
- return ret;
+ if (IS_ELF(*(Elf32_Ehdr *)fw)) {
+ ret = read_elf(&rsc, fw, &hdr);
+ if (ret) {
+ dev_err(dev, "Failed to read ELF: %d\n", ret);
+ return ret;
+ }
+ elf = true;
+ info = hdr;
+ } else {
+ elf = false;
+ info = fw;
+ if (!IS_ENABLED(CONFIG_QCOM_GENI_MINICORE)) {
+ dev_err(dev, "Error: f/w ELF not found and minicore support disabled\n");
+ return -EINVAL;
+ }
}
dev_info(dev, "Loading QUP firmware...\n");
- return load_se_firmware(&rsc, hdr);
+ return load_se_firmware(&rsc, elf, info);
}
/*
@@ -414,6 +468,9 @@ static int geni_se_of_to_plat(struct udevice *dev)
if (proto == GENI_SE_INVALID_PROTO)
plat->need_firmware_load = true;
+
+ if (readl(res.start + SE_HW_PARAM_2) & GENI_USE_MINICORES)
+ plat->is_mini_core = true;
}
return 0;
@@ -473,7 +530,7 @@ static int probe_children_load_firmware(struct udevice *dev)
ret = 0;
/* Find the device for this ofnode, or bind it */
if (device_find_global_by_ofnode(child, &child_dev))
- ret = lists_bind_fdt(dev, child, &child_dev, NULL, false);
+ ret = lists_bind_fdt(dev, child, &child_dev, NULL, false);
if (ret) {
/* Skip nodes that don't have drivers */
debug("Failed to probe child %s: %d\n", ofnode_get_name(child), ret);
@@ -518,6 +575,11 @@ static int qcom_geni_fw_initialise(void)
return 0;
}
+ if (plat->is_mini_core) {
+ fw_buf = qup_mini_cores;
+ goto mini_core;
+ }
+
ret = find_qupfw_part(&blk_dev, &part_info);
if (ret) {
pr_err("QUP firmware partition not found\n");
@@ -544,6 +606,7 @@ static int qcom_geni_fw_initialise(void)
return 0;
}
+mini_core:
/*
* OK! Firmware is loaded, now bind and probe remaining children. They will attempt to load
* firmware during probe. Do this for each GENI SE wrapper that needs firmware loading.
diff --git a/include/soc/qcom/geni-se.h b/include/soc/qcom/geni-se.h
index fc9a8e82cd8..3063b37010d 100644
--- a/include/soc/qcom/geni-se.h
+++ b/include/soc/qcom/geni-se.h
@@ -77,8 +77,12 @@ enum geni_se_protocol_type {
#define SE_IRQ_EN 0xe1c
#define SE_HW_PARAM_0 0xe24
#define SE_HW_PARAM_1 0xe28
+#define SE_HW_PARAM_2 0xe2c
#define SE_DMA_GENERAL_CFG 0xe30
+/* SE_HW_PARAM_2 fields */
+#define GENI_USE_MINICORES BIT(12)
+
/* GENI_DFS_IF_CFG fields */
#define DFS_IF_EN BIT(0)
@@ -248,6 +252,7 @@ enum geni_se_protocol_type {
/* SE_HW_PARAM_0 fields */
#define TX_FIFO_WIDTH_MSK GENMASK(29, 24)
#define TX_FIFO_WIDTH_SHFT 24
+
/*
* For QUP HW Version >= 3.10 Tx fifo depth support is increased
* to 256bytes and corresponding bits are 16 to 23
diff --git a/include/soc/qcom/qup-fw-load.h b/include/soc/qcom/qup-fw-load.h
index a67a93c72a4..df8a615b46b 100644
--- a/include/soc/qcom/qup-fw-load.h
+++ b/include/soc/qcom/qup-fw-load.h
@@ -14,6 +14,7 @@
#define GENI_INIT_CFG_REVISION 0x0
#define GENI_S_INIT_CFG_REVISION 0x4
#define GENI_FORCE_DEFAULT_REG 0x20
+#define GENI_OUTPUT_CTRL 0x24
#define GENI_CGC_CTRL 0x28
#define GENI_CFG_REG0 0x100
---
base-commit: 2e266ec527c20dabd53dc1b19950ce5b77cf2f24
change-id: 20260710-minicore-b5b7229e6f6e
Best regards,
--
Varadarajan Narayanan <varadarajan.narayanan at oss.qualcomm.com>
More information about the U-Boot
mailing list