Updates the QFW driver to use the driver model, splitting the driver
into qfw_pio and qfw_mmio (for non-x86) in their own uclass.

Signed-off-by: Asherah Connor <[email protected]>
---

Changes in v4:
- PIO definitions are now #defines
- qfw_*_plat structs are no longer in header files
- Remove yield/pause in DMA wait loop
- Change struct udevice *qfw_get_dev(void) to int qfw_get_dev(struct
  udevice **)

 arch/arm/Kconfig         |   1 +
 arch/x86/cpu/qemu/cpu.c  |   9 +-
 arch/x86/cpu/qemu/qemu.c |  47 +-------
 arch/x86/cpu/qfw_cpu.c   |  11 +-
 cmd/qfw.c                |  52 ++++-----
 common/Makefile          |   2 +
 common/qfw.c             | 105 +++++++++++++++++
 drivers/misc/Makefile    |   6 +-
 drivers/misc/qfw.c       | 238 ++++++++++++++-------------------------
 drivers/misc/qfw_mmio.c  | 119 ++++++++++++++++++++
 drivers/misc/qfw_pio.c   |  69 ++++++++++++
 include/dm/uclass-id.h   |   1 +
 include/qfw.h            |  61 ++++++----
 13 files changed, 466 insertions(+), 255 deletions(-)
 create mode 100644 common/qfw.c
 create mode 100644 drivers/misc/qfw_mmio.c
 create mode 100644 drivers/misc/qfw_pio.c

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index d51abbeaf0..cd01dc458a 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -937,6 +937,7 @@ config ARCH_QEMU
        imply DM_RNG
        imply DM_RTC
        imply RTC_PL031
+       imply CMD_QFW
 
 config ARCH_RMOBILE
        bool "Renesas ARM SoCs"
diff --git a/arch/x86/cpu/qemu/cpu.c b/arch/x86/cpu/qemu/cpu.c
index 9ce86b379c..c78e374644 100644
--- a/arch/x86/cpu/qemu/cpu.c
+++ b/arch/x86/cpu/qemu/cpu.c
@@ -22,7 +22,14 @@ int cpu_qemu_get_desc(const struct udevice *dev, char *buf, 
int size)
 
 static int cpu_qemu_get_count(const struct udevice *dev)
 {
-       return qemu_fwcfg_online_cpus();
+       int ret;
+       struct udevice *qfw_dev;
+
+       ret = qfw_get_dev(&qfw_dev);
+       if (ret)
+               return ret;
+
+       return qemu_fwcfg_online_cpus(qfw_dev);
 }
 
 static const struct cpu_ops cpu_qemu_ops = {
diff --git a/arch/x86/cpu/qemu/qemu.c b/arch/x86/cpu/qemu/qemu.c
index 044a429c13..605f51e1b8 100644
--- a/arch/x86/cpu/qemu/qemu.c
+++ b/arch/x86/cpu/qemu/qemu.c
@@ -8,6 +8,7 @@
 #include <init.h>
 #include <pci.h>
 #include <qfw.h>
+#include <dm/platdata.h>
 #include <asm/irq.h>
 #include <asm/post.h>
 #include <asm/processor.h>
@@ -18,46 +19,10 @@ static bool i440fx;
 
 #ifdef CONFIG_QFW
 
-/* on x86, the qfw registers are all IO ports */
-#define FW_CONTROL_PORT        0x510
-#define FW_DATA_PORT           0x511
-#define FW_DMA_PORT_LOW        0x514
-#define FW_DMA_PORT_HIGH       0x518
-
-static void qemu_x86_fwcfg_read_entry_pio(uint16_t entry,
-               uint32_t size, void *address)
-{
-       uint32_t i = 0;
-       uint8_t *data = address;
-
-       /*
-        * writting FW_CFG_INVALID will cause read operation to resume at
-        * last offset, otherwise read will start at offset 0
-        *
-        * Note: on platform where the control register is IO port, the
-        * endianness is little endian.
-        */
-       if (entry != FW_CFG_INVALID)
-               outw(cpu_to_le16(entry), FW_CONTROL_PORT);
-
-       /* the endianness of data register is string-preserving */
-       while (size--)
-               data[i++] = inb(FW_DATA_PORT);
-}
-
-static void qemu_x86_fwcfg_read_entry_dma(struct fw_cfg_dma_access *dma)
-{
-       /* the DMA address register is big endian */
-       outl(cpu_to_be32((uintptr_t)dma), FW_DMA_PORT_HIGH);
-
-       while (be32_to_cpu(dma->control) & ~FW_CFG_DMA_ERROR)
-               __asm__ __volatile__ ("pause");
-}
-
-static struct fw_cfg_arch_ops fwcfg_x86_ops = {
-       .arch_read_pio = qemu_x86_fwcfg_read_entry_pio,
-       .arch_read_dma = qemu_x86_fwcfg_read_entry_dma
+U_BOOT_DRVINFO(x86_qfw_pio) = {
+       .name = "qfw_pio",
 };
+
 #endif
 
 static void enable_pm_piix(void)
@@ -132,10 +97,6 @@ static void qemu_chipset_init(void)
 
                enable_pm_ich9();
        }
-
-#ifdef CONFIG_QFW
-       qemu_fwcfg_init(&fwcfg_x86_ops);
-#endif
 }
 
 #if !CONFIG_IS_ENABLED(SPL_X86_32BIT_INIT)
diff --git a/arch/x86/cpu/qfw_cpu.c b/arch/x86/cpu/qfw_cpu.c
index b959eaddde..9700908064 100644
--- a/arch/x86/cpu/qfw_cpu.c
+++ b/arch/x86/cpu/qfw_cpu.c
@@ -18,7 +18,7 @@ int qemu_cpu_fixup(void)
        int cpu_num;
        int cpu_online;
        struct uclass *uc;
-       struct udevice *dev, *pdev;
+       struct udevice *dev, *pdev, *qfwdev;
        struct cpu_plat *plat;
        char *cpu;
 
@@ -39,6 +39,13 @@ int qemu_cpu_fixup(void)
                return -ENODEV;
        }
 
+       /* get qfw dev */
+       ret = qfw_get_dev(&qfwdev);
+       if (ret) {
+               printf("unable to find qfw device\n");
+               return ret;
+       }
+
        /* calculate cpus that are already bound */
        cpu_num = 0;
        for (uclass_find_first_device(UCLASS_CPU, &dev);
@@ -48,7 +55,7 @@ int qemu_cpu_fixup(void)
        }
 
        /* get actual cpu number */
-       cpu_online = qemu_fwcfg_online_cpus();
+       cpu_online = qemu_fwcfg_online_cpus(qfwdev);
        if (cpu_online < 0) {
                printf("unable to get online cpu number: %d\n", cpu_online);
                return cpu_online;
diff --git a/cmd/qfw.c b/cmd/qfw.c
index bb571487f0..87af408a8d 100644
--- a/cmd/qfw.c
+++ b/cmd/qfw.c
@@ -8,19 +8,22 @@
 #include <env.h>
 #include <errno.h>
 #include <qfw.h>
+#include <dm.h>
+
+static struct udevice *qfw_dev;
 
 /*
  * This function prepares kernel for zboot. It loads kernel data
  * to 'load_addr', initrd to 'initrd_addr' and kernel command
  * line using qemu fw_cfg interface.
  */
-static int qemu_fwcfg_setup_kernel(void *load_addr, void *initrd_addr)
+static int qemu_fwcfg_cmd_setup_kernel(void *load_addr, void *initrd_addr)
 {
        char *data_addr;
        uint32_t setup_size, kernel_size, cmdline_size, initrd_size;
 
-       qemu_fwcfg_read_entry(FW_CFG_SETUP_SIZE, 4, &setup_size);
-       qemu_fwcfg_read_entry(FW_CFG_KERNEL_SIZE, 4, &kernel_size);
+       qfw_read_entry(qfw_dev, FW_CFG_SETUP_SIZE, 4, &setup_size);
+       qfw_read_entry(qfw_dev, FW_CFG_KERNEL_SIZE, 4, &kernel_size);
 
        if (setup_size == 0 || kernel_size == 0) {
                printf("warning: no kernel available\n");
@@ -28,28 +31,28 @@ static int qemu_fwcfg_setup_kernel(void *load_addr, void 
*initrd_addr)
        }
 
        data_addr = load_addr;
-       qemu_fwcfg_read_entry(FW_CFG_SETUP_DATA,
-                             le32_to_cpu(setup_size), data_addr);
+       qfw_read_entry(qfw_dev, FW_CFG_SETUP_DATA,
+                      le32_to_cpu(setup_size), data_addr);
        data_addr += le32_to_cpu(setup_size);
 
-       qemu_fwcfg_read_entry(FW_CFG_KERNEL_DATA,
-                             le32_to_cpu(kernel_size), data_addr);
+       qfw_read_entry(qfw_dev, FW_CFG_KERNEL_DATA,
+                      le32_to_cpu(kernel_size), data_addr);
        data_addr += le32_to_cpu(kernel_size);
 
        data_addr = initrd_addr;
-       qemu_fwcfg_read_entry(FW_CFG_INITRD_SIZE, 4, &initrd_size);
+       qfw_read_entry(qfw_dev, FW_CFG_INITRD_SIZE, 4, &initrd_size);
        if (initrd_size == 0) {
                printf("warning: no initrd available\n");
        } else {
-               qemu_fwcfg_read_entry(FW_CFG_INITRD_DATA,
-                                     le32_to_cpu(initrd_size), data_addr);
+               qfw_read_entry(qfw_dev, FW_CFG_INITRD_DATA,
+                              le32_to_cpu(initrd_size), data_addr);
                data_addr += le32_to_cpu(initrd_size);
        }
 
-       qemu_fwcfg_read_entry(FW_CFG_CMDLINE_SIZE, 4, &cmdline_size);
+       qfw_read_entry(qfw_dev, FW_CFG_CMDLINE_SIZE, 4, &cmdline_size);
        if (cmdline_size) {
-               qemu_fwcfg_read_entry(FW_CFG_CMDLINE_DATA,
-                                     le32_to_cpu(cmdline_size), data_addr);
+               qfw_read_entry(qfw_dev, FW_CFG_CMDLINE_DATA,
+                              le32_to_cpu(cmdline_size), data_addr);
                /*
                 * if kernel cmdline only contains '\0', (e.g. no -append
                 * when invoking qemu), do not update bootargs
@@ -72,19 +75,18 @@ static int qemu_fwcfg_setup_kernel(void *load_addr, void 
*initrd_addr)
        return 0;
 }
 
-static int qemu_fwcfg_list_firmware(void)
+static int qemu_fwcfg_cmd_list_firmware(void)
 {
        int ret;
        struct fw_cfg_file_iter iter;
        struct fw_file *file;
 
        /* make sure fw_list is loaded */
-       ret = qemu_fwcfg_read_firmware_list();
+       ret = qemu_fwcfg_read_firmware_list(qfw_dev);
        if (ret)
                return ret;
 
-
-       for (file = qemu_fwcfg_file_iter_init(&iter);
+       for (file = qemu_fwcfg_file_iter_init(qfw_dev, &iter);
             !qemu_fwcfg_file_iter_end(&iter);
             file = qemu_fwcfg_file_iter_next(&iter)) {
                printf("%-56s\n", file->cfg.name);
@@ -96,7 +98,7 @@ static int qemu_fwcfg_list_firmware(void)
 static int qemu_fwcfg_do_list(struct cmd_tbl *cmdtp, int flag,
                              int argc, char *const argv[])
 {
-       if (qemu_fwcfg_list_firmware() < 0)
+       if (qemu_fwcfg_cmd_list_firmware() < 0)
                return CMD_RET_FAILURE;
 
        return 0;
@@ -105,14 +107,7 @@ static int qemu_fwcfg_do_list(struct cmd_tbl *cmdtp, int 
flag,
 static int qemu_fwcfg_do_cpus(struct cmd_tbl *cmdtp, int flag,
                              int argc, char *const argv[])
 {
-       int ret = qemu_fwcfg_online_cpus();
-       if (ret < 0) {
-               printf("QEMU fw_cfg interface not found\n");
-               return CMD_RET_FAILURE;
-       }
-
-       printf("%d cpu(s) online\n", qemu_fwcfg_online_cpus());
-
+       printf("%d cpu(s) online\n", qemu_fwcfg_online_cpus(qfw_dev));
        return 0;
 }
 
@@ -153,7 +148,7 @@ static int qemu_fwcfg_do_load(struct cmd_tbl *cmdtp, int 
flag,
                return CMD_RET_FAILURE;
        }
 
-       return qemu_fwcfg_setup_kernel(load_addr, initrd_addr);
+       return qemu_fwcfg_cmd_setup_kernel(load_addr, initrd_addr);
 }
 
 static struct cmd_tbl fwcfg_commands[] = {
@@ -168,7 +163,8 @@ static int do_qemu_fw(struct cmd_tbl *cmdtp, int flag, int 
argc,
        int ret;
        struct cmd_tbl *fwcfg_cmd;
 
-       if (!qemu_fwcfg_present()) {
+       ret = qfw_get_dev(&qfw_dev);
+       if (ret) {
                printf("QEMU fw_cfg interface not found\n");
                return CMD_RET_USAGE;
        }
diff --git a/common/Makefile b/common/Makefile
index daeea67cf2..f174a06c33 100644
--- a/common/Makefile
+++ b/common/Makefile
@@ -137,3 +137,5 @@ obj-$(CONFIG_CMD_LOADB) += xyzModem.o
 obj-$(CONFIG_$(SPL_TPL_)YMODEM_SUPPORT) += xyzModem.o
 
 obj-$(CONFIG_AVB_VERIFY) += avb_verify.o
+
+obj-$(CONFIG_QFW) += qfw.o
diff --git a/common/qfw.c b/common/qfw.c
new file mode 100644
index 0000000000..c0ffa20b74
--- /dev/null
+++ b/common/qfw.c
@@ -0,0 +1,105 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * (C) Copyright 2015 Miao Yan <[email protected]>
+ * (C) Copyright 2021 Asherah Connor <[email protected]>
+ */
+
+#include <dm.h>
+#include <dm/uclass.h>
+#include <qfw.h>
+#include <stdlib.h>
+
+int qfw_get_dev(struct udevice **devp)
+{
+       return uclass_first_device(UCLASS_QFW, devp);
+}
+
+int qemu_fwcfg_online_cpus(struct udevice *dev)
+{
+       u16 nb_cpus;
+
+       qfw_read_entry(dev, FW_CFG_NB_CPUS, 2, &nb_cpus);
+
+       return le16_to_cpu(nb_cpus);
+}
+
+int qemu_fwcfg_read_firmware_list(struct udevice *dev)
+{
+       int i;
+       u32 count;
+       struct fw_file *file;
+       struct list_head *entry;
+
+       struct qfw_dev *qdev = dev_get_uclass_priv(dev);
+
+       /* don't read it twice */
+       if (!list_empty(&qdev->fw_list))
+               return 0;
+
+       qfw_read_entry(dev, FW_CFG_FILE_DIR, 4, &count);
+       if (!count)
+               return 0;
+
+       count = be32_to_cpu(count);
+       for (i = 0; i < count; i++) {
+               file = malloc(sizeof(*file));
+               if (!file) {
+                       printf("error: allocating resource\n");
+                       goto err;
+               }
+               qfw_read_entry(dev, FW_CFG_INVALID,
+                              sizeof(struct fw_cfg_file), &file->cfg);
+               file->addr = 0;
+               list_add_tail(&file->list, &qdev->fw_list);
+       }
+
+       return 0;
+
+err:
+       list_for_each(entry, &qdev->fw_list) {
+               file = list_entry(entry, struct fw_file, list);
+               free(file);
+       }
+
+       return -ENOMEM;
+}
+
+struct fw_file *qemu_fwcfg_find_file(struct udevice *dev, const char *name)
+{
+       struct list_head *entry;
+       struct fw_file *file;
+
+       struct qfw_dev *qdev = dev_get_uclass_priv(dev);
+
+       list_for_each(entry, &qdev->fw_list) {
+               file = list_entry(entry, struct fw_file, list);
+               if (!strcmp(file->cfg.name, name))
+                       return file;
+       }
+
+       return NULL;
+}
+
+struct fw_file *qemu_fwcfg_file_iter_init(struct udevice *dev,
+                                         struct fw_cfg_file_iter *iter)
+{
+       struct qfw_dev *qdev = dev_get_uclass_priv(dev);
+
+       iter->entry = qdev->fw_list.next;
+       iter->end = &qdev->fw_list;
+       return list_entry((struct list_head *)iter->entry,
+                         struct fw_file, list);
+}
+
+struct fw_file *qemu_fwcfg_file_iter_next(struct fw_cfg_file_iter *iter)
+{
+       iter->entry = ((struct list_head *)iter->entry)->next;
+       return list_entry((struct list_head *)iter->entry,
+                         struct fw_file, list);
+}
+
+bool qemu_fwcfg_file_iter_end(struct fw_cfg_file_iter *iter)
+{
+       return iter->entry == iter->end;
+}
+
diff --git a/drivers/misc/Makefile b/drivers/misc/Makefile
index d737203704..2988289ea3 100644
--- a/drivers/misc/Makefile
+++ b/drivers/misc/Makefile
@@ -55,7 +55,11 @@ obj-$(CONFIG_NUVOTON_NCT6102D) += nuvoton_nct6102d.o
 obj-$(CONFIG_P2SB) += p2sb-uclass.o
 obj-$(CONFIG_PCA9551_LED) += pca9551_led.o
 obj-$(CONFIG_$(SPL_)PWRSEQ) += pwrseq-uclass.o
-obj-$(CONFIG_QFW) += qfw.o
+ifdef CONFIG_QFW
+obj-y += qfw.o
+obj-$(CONFIG_X86) += qfw_pio.o
+obj-$(CONFIG_ARM) += qfw_mmio.o
+endif
 obj-$(CONFIG_ROCKCHIP_EFUSE) += rockchip-efuse.o
 obj-$(CONFIG_ROCKCHIP_OTP) += rockchip-otp.o
 obj-$(CONFIG_SANDBOX) += syscon_sandbox.o misc_sandbox.o
diff --git a/drivers/misc/qfw.c b/drivers/misc/qfw.c
index f6eb6583ed..7ad8f0979b 100644
--- a/drivers/misc/qfw.c
+++ b/drivers/misc/qfw.c
@@ -1,25 +1,22 @@
 // SPDX-License-Identifier: GPL-2.0+
 /*
  * (C) Copyright 2015 Miao Yan <[email protected]>
+ * (C) Copyright 2021 Asherah Connor <[email protected]>
  */
 
+#define LOG_CATEGORY UCLASS_QFW
+
 #include <common.h>
 #include <command.h>
 #include <errno.h>
 #include <log.h>
 #include <malloc.h>
 #include <qfw.h>
-#include <asm/io.h>
+#include <dm.h>
+#include <misc.h>
 #ifdef CONFIG_GENERATE_ACPI_TABLE
 #include <asm/tables.h>
 #endif
-#include <linux/list.h>
-
-static bool fwcfg_present;
-static bool fwcfg_dma_present;
-static struct fw_cfg_arch_ops *fwcfg_arch_ops;
-
-static LIST_HEAD(fw_list);
 
 #ifdef CONFIG_GENERATE_ACPI_TABLE
 /*
@@ -32,7 +29,8 @@ static LIST_HEAD(fw_list);
  *          be ignored.
  * @return: 0 on success, or negative value on failure
  */
-static int bios_linker_allocate(struct bios_linker_entry *entry, ulong *addr)
+static int bios_linker_allocate(struct udevice *dev,
+                               struct bios_linker_entry *entry, ulong *addr)
 {
        uint32_t size, align;
        struct fw_file *file;
@@ -45,7 +43,7 @@ static int bios_linker_allocate(struct bios_linker_entry 
*entry, ulong *addr)
                return -EINVAL;
        }
 
-       file = qemu_fwcfg_find_file(entry->alloc.file);
+       file = qemu_fwcfg_find_file(dev, entry->alloc.file);
        if (!file) {
                printf("error: can't find file %s\n", entry->alloc.file);
                return -ENOENT;
@@ -75,8 +73,8 @@ static int bios_linker_allocate(struct bios_linker_entry 
*entry, ulong *addr)
        debug("bios_linker_allocate: allocate file %s, size %u, zone %d, align 
%u, addr 0x%lx\n",
              file->cfg.name, size, entry->alloc.zone, align, aligned_addr);
 
-       qemu_fwcfg_read_entry(be16_to_cpu(file->cfg.select),
-                             size, (void *)aligned_addr);
+       qfw_read_entry(dev, be16_to_cpu(file->cfg.select), size,
+                      (void *)aligned_addr);
        file->addr = aligned_addr;
 
        /* adjust address for low memory allocation */
@@ -94,16 +92,17 @@ static int bios_linker_allocate(struct bios_linker_entry 
*entry, ulong *addr)
  *          ACPI tables
  * @return: 0 on success, or negative value on failure
  */
-static int bios_linker_add_pointer(struct bios_linker_entry *entry)
+static int bios_linker_add_pointer(struct udevice *dev,
+                                  struct bios_linker_entry *entry)
 {
        struct fw_file *dest, *src;
        uint32_t offset = le32_to_cpu(entry->pointer.offset);
        uint64_t pointer = 0;
 
-       dest = qemu_fwcfg_find_file(entry->pointer.dest_file);
+       dest = qemu_fwcfg_find_file(dev, entry->pointer.dest_file);
        if (!dest || !dest->addr)
                return -ENOENT;
-       src = qemu_fwcfg_find_file(entry->pointer.src_file);
+       src = qemu_fwcfg_find_file(dev, entry->pointer.src_file);
        if (!src || !src->addr)
                return -ENOENT;
 
@@ -127,13 +126,14 @@ static int bios_linker_add_pointer(struct 
bios_linker_entry *entry)
  *          checksums
  * @return: 0 on success, or negative value on failure
  */
-static int bios_linker_add_checksum(struct bios_linker_entry *entry)
+static int bios_linker_add_checksum(struct udevice *dev,
+                                   struct bios_linker_entry *entry)
 {
        struct fw_file *file;
        uint8_t *data, cksum = 0;
        uint8_t *cksum_start;
 
-       file = qemu_fwcfg_find_file(entry->cksum.file);
+       file = qemu_fwcfg_find_file(dev, entry->cksum.file);
        if (!file || !file->addr)
                return -ENOENT;
 
@@ -149,20 +149,27 @@ static int bios_linker_add_checksum(struct 
bios_linker_entry *entry)
 /* This function loads and patches ACPI tables provided by QEMU */
 ulong write_acpi_tables(ulong addr)
 {
-       int i, ret = 0;
+       int i, ret;
        struct fw_file *file;
        struct bios_linker_entry *table_loader;
        struct bios_linker_entry *entry;
        uint32_t size;
+       struct udevice *dev;
+
+       ret = qfw_get_dev(&dev);
+       if (ret) {
+               printf("error: no qfw\n");
+               return addr;
+       }
 
        /* make sure fw_list is loaded */
-       ret = qemu_fwcfg_read_firmware_list();
+       ret = qemu_fwcfg_read_firmware_list(dev);
        if (ret) {
                printf("error: can't read firmware file list\n");
                return addr;
        }
 
-       file = qemu_fwcfg_find_file("etc/table-loader");
+       file = qemu_fwcfg_find_file(dev, "etc/table-loader");
        if (!file) {
                printf("error: can't find etc/table-loader\n");
                return addr;
@@ -180,24 +187,23 @@ ulong write_acpi_tables(ulong addr)
                return addr;
        }
 
-       qemu_fwcfg_read_entry(be16_to_cpu(file->cfg.select),
-                             size, table_loader);
+       qfw_read_entry(dev, be16_to_cpu(file->cfg.select), size, table_loader);
 
        for (i = 0; i < (size / sizeof(*entry)); i++) {
                entry = table_loader + i;
                switch (le32_to_cpu(entry->command)) {
                case BIOS_LINKER_LOADER_COMMAND_ALLOCATE:
-                       ret = bios_linker_allocate(entry, &addr);
+                       ret = bios_linker_allocate(dev, entry, &addr);
                        if (ret)
                                goto out;
                        break;
                case BIOS_LINKER_LOADER_COMMAND_ADD_POINTER:
-                       ret = bios_linker_add_pointer(entry);
+                       ret = bios_linker_add_pointer(dev, entry);
                        if (ret)
                                goto out;
                        break;
                case BIOS_LINKER_LOADER_COMMAND_ADD_CHECKSUM:
-                       ret = bios_linker_add_checksum(entry);
+                       ret = bios_linker_add_checksum(dev, entry);
                        if (ret)
                                goto out;
                        break;
@@ -209,7 +215,7 @@ ulong write_acpi_tables(ulong addr)
 out:
        if (ret) {
                struct fw_cfg_file_iter iter;
-               for (file = qemu_fwcfg_file_iter_init(&iter);
+               for (file = qemu_fwcfg_file_iter_init(dev, &iter);
                     !qemu_fwcfg_file_iter_end(&iter);
                     file = qemu_fwcfg_file_iter_next(&iter)) {
                        if (file->addr) {
@@ -225,170 +231,90 @@ out:
 
 ulong acpi_get_rsdp_addr(void)
 {
+       int ret;
        struct fw_file *file;
+       struct udevice *dev;
 
-       file = qemu_fwcfg_find_file("etc/acpi/rsdp");
+       ret = qfw_get_dev(&dev);
+       if (ret) {
+               printf("error: no qfw\n");
+               return 0;
+       }
+
+       file = qemu_fwcfg_find_file(dev, "etc/acpi/rsdp");
        return file->addr;
 }
 #endif
 
-/* Read configuration item using fw_cfg PIO interface */
-static void qemu_fwcfg_read_entry_pio(uint16_t entry,
-               uint32_t size, void *address)
+static void qfw_read_entry_io(struct qfw_dev *qdev, u16 entry, u32 size,
+                             void *address)
 {
-       debug("qemu_fwcfg_read_entry_pio: entry 0x%x, size %u address %p\n",
-             entry, size, address);
+       struct dm_qfw_ops *ops = dm_qfw_get_ops(qdev->dev);
 
-       return fwcfg_arch_ops->arch_read_pio(entry, size, address);
+       debug("%s: entry 0x%x, size %u address %p\n", __func__, entry, size,
+             address);
+
+       ops->read_entry_io(qdev->dev, entry, size, address);
 }
 
-/* Read configuration item using fw_cfg DMA interface */
-static void qemu_fwcfg_read_entry_dma(uint16_t entry,
-               uint32_t size, void *address)
+static void qfw_read_entry_dma(struct qfw_dev *qdev, u16 entry, u32 size,
+                              void *address)
 {
-       struct fw_cfg_dma_access dma;
+       struct dm_qfw_ops *ops = dm_qfw_get_ops(qdev->dev);
 
-       dma.length = cpu_to_be32(size);
-       dma.address = cpu_to_be64((uintptr_t)address);
-       dma.control = cpu_to_be32(FW_CFG_DMA_READ);
+       struct qfw_dma dma = {
+               .length = cpu_to_be32(size),
+               .address = cpu_to_be64((uintptr_t)address),
+               .control = cpu_to_be32(FW_CFG_DMA_READ),
+       };
 
        /*
-        * writting FW_CFG_INVALID will cause read operation to resume at
-        * last offset, otherwise read will start at offset 0
+        * writing FW_CFG_INVALID will cause read operation to resume at last
+        * offset, otherwise read will start at offset 0
         */
        if (entry != FW_CFG_INVALID)
                dma.control |= cpu_to_be32(FW_CFG_DMA_SELECT | (entry << 16));
 
-       barrier();
-
-       debug("qemu_fwcfg_read_entry_dma: entry 0x%x, size %u address %p, 
control 0x%x\n",
+       debug("%s: entry 0x%x, size %u address %p, control 0x%x\n", __func__,
              entry, size, address, be32_to_cpu(dma.control));
 
-       fwcfg_arch_ops->arch_read_dma(&dma);
-}
+       barrier();
 
-bool qemu_fwcfg_present(void)
-{
-       return fwcfg_present;
+       ops->read_entry_dma(qdev->dev, &dma);
 }
 
-bool qemu_fwcfg_dma_present(void)
+void qfw_read_entry(struct udevice *dev, u16 entry, u32 length, void *address)
 {
-       return fwcfg_dma_present;
-}
+       struct qfw_dev *qdev = dev_get_uclass_priv(dev);
 
-void qemu_fwcfg_read_entry(uint16_t entry, uint32_t length, void *address)
-{
-       if (fwcfg_dma_present)
-               qemu_fwcfg_read_entry_dma(entry, length, address);
+       if (qdev->dma_present)
+               qfw_read_entry_dma(qdev, entry, length, address);
        else
-               qemu_fwcfg_read_entry_pio(entry, length, address);
+               qfw_read_entry_io(qdev, entry, length, address);
 }
 
-int qemu_fwcfg_online_cpus(void)
+int qfw_register(struct udevice *dev)
 {
-       uint16_t nb_cpus;
+       struct qfw_dev *qdev = dev_get_uclass_priv(dev);
+       u32 qemu, dma_enabled;
 
-       if (!fwcfg_present)
+       qdev->dev = dev;
+       INIT_LIST_HEAD(&qdev->fw_list);
+
+       qfw_read_entry_io(qdev, FW_CFG_SIGNATURE, 4, &qemu);
+       if (be32_to_cpu(qemu) != QEMU_FW_CFG_SIGNATURE)
                return -ENODEV;
 
-       qemu_fwcfg_read_entry(FW_CFG_NB_CPUS, 2, &nb_cpus);
-
-       return le16_to_cpu(nb_cpus);
-}
-
-int qemu_fwcfg_read_firmware_list(void)
-{
-       int i;
-       uint32_t count;
-       struct fw_file *file;
-       struct list_head *entry;
-
-       /* don't read it twice */
-       if (!list_empty(&fw_list))
-               return 0;
-
-       qemu_fwcfg_read_entry(FW_CFG_FILE_DIR, 4, &count);
-       if (!count)
-               return 0;
-
-       count = be32_to_cpu(count);
-       for (i = 0; i < count; i++) {
-               file = malloc(sizeof(*file));
-               if (!file) {
-                       printf("error: allocating resource\n");
-                       goto err;
-               }
-               qemu_fwcfg_read_entry(FW_CFG_INVALID,
-                                     sizeof(struct fw_cfg_file), &file->cfg);
-               file->addr = 0;
-               list_add_tail(&file->list, &fw_list);
-       }
+       qfw_read_entry_io(qdev, FW_CFG_ID, 1, &dma_enabled);
+       if (dma_enabled & FW_CFG_DMA_ENABLED)
+               qdev->dma_present = true;
 
        return 0;
-
-err:
-       list_for_each(entry, &fw_list) {
-               file = list_entry(entry, struct fw_file, list);
-               free(file);
-       }
-
-       return -ENOMEM;
-}
-
-struct fw_file *qemu_fwcfg_find_file(const char *name)
-{
-       struct list_head *entry;
-       struct fw_file *file;
-
-       list_for_each(entry, &fw_list) {
-               file = list_entry(entry, struct fw_file, list);
-               if (!strcmp(file->cfg.name, name))
-                       return file;
-       }
-
-       return NULL;
-}
-
-struct fw_file *qemu_fwcfg_file_iter_init(struct fw_cfg_file_iter *iter)
-{
-       iter->entry = fw_list.next;
-       return list_entry((struct list_head *)iter->entry,
-                         struct fw_file, list);
 }
 
-struct fw_file *qemu_fwcfg_file_iter_next(struct fw_cfg_file_iter *iter)
-{
-       iter->entry = ((struct list_head *)iter->entry)->next;
-       return list_entry((struct list_head *)iter->entry,
-                         struct fw_file, list);
-}
-
-bool qemu_fwcfg_file_iter_end(struct fw_cfg_file_iter *iter)
-{
-       return iter->entry == &fw_list;
-}
-
-void qemu_fwcfg_init(struct fw_cfg_arch_ops *ops)
-{
-       uint32_t qemu;
-       uint32_t dma_enabled;
+UCLASS_DRIVER(qfw) = {
+       .id             = UCLASS_QFW,
+       .name           = "qfw",
+       .per_device_auto        = sizeof(struct qfw_dev),
+};
 
-       fwcfg_present = false;
-       fwcfg_dma_present = false;
-       fwcfg_arch_ops = NULL;
-
-       if (!ops || !ops->arch_read_pio || !ops->arch_read_dma)
-               return;
-       fwcfg_arch_ops = ops;
-
-       qemu_fwcfg_read_entry_pio(FW_CFG_SIGNATURE, 4, &qemu);
-       if (be32_to_cpu(qemu) == QEMU_FW_CFG_SIGNATURE)
-               fwcfg_present = true;
-
-       if (fwcfg_present) {
-               qemu_fwcfg_read_entry_pio(FW_CFG_ID, 1, &dma_enabled);
-               if (dma_enabled & FW_CFG_DMA_ENABLED)
-                       fwcfg_dma_present = true;
-       }
-}
diff --git a/drivers/misc/qfw_mmio.c b/drivers/misc/qfw_mmio.c
new file mode 100644
index 0000000000..f397384054
--- /dev/null
+++ b/drivers/misc/qfw_mmio.c
@@ -0,0 +1,119 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * MMIO interface for QFW
+ *
+ * (C) Copyright 2015 Miao Yan <[email protected]>
+ * (C) Copyright 2021 Asherah Connor <[email protected]>
+ */
+
+#define LOG_CATEGORY UCLASS_QFW
+
+#include <asm/types.h>
+#include <asm/io.h>
+#include <dm.h>
+#include <dm/device.h>
+#include <qfw.h>
+
+struct qfw_mmio {
+       /*
+        * Each access to the 64-bit data register can be 8/16/32/64 bits wide.
+        */
+       union {
+               u8 data8;
+               u16 data16;
+               u32 data32;
+               u64 data64;
+       };
+       u16 selector;
+       u8 padding[6];
+       u64 dma;
+};
+
+struct qfw_mmio_plat {
+       volatile struct qfw_mmio *mmio;
+};
+
+static void qfw_mmio_read_entry_io(struct udevice *dev, u16 entry, u32 size,
+                                  void *address)
+{
+       struct qfw_mmio_plat *plat = dev_get_plat(dev);
+
+       /*
+        * writing FW_CFG_INVALID will cause read operation to resume at last
+        * offset, otherwise read will start at offset 0
+        *
+        * Note: on platform where the control register is MMIO, the register
+        * is big endian.
+        */
+       if (entry != FW_CFG_INVALID)
+               plat->mmio->selector = cpu_to_be16(entry);
+
+       /* the endianness of data register is string-preserving */
+       while (size >= 8) {
+               *(u64 *)address = plat->mmio->data64;
+               address += 8;
+               size -= 8;
+       }
+       while (size >= 4) {
+               *(u32 *)address = plat->mmio->data32;
+               address += 4;
+               size -= 4;
+       }
+       while (size >= 2) {
+               *(u16 *)address = plat->mmio->data16;
+               address += 2;
+               size -= 2;
+       }
+       while (size >= 1) {
+               *(u8 *)address = plat->mmio->data8;
+               address += 1;
+               size -= 1;
+       }
+}
+
+/* Read configuration item using fw_cfg DMA interface */
+static void qfw_mmio_read_entry_dma(struct udevice *dev, struct qfw_dma *dma)
+{
+       struct qfw_mmio_plat *plat = dev_get_plat(dev);
+
+       /* the DMA address register is big-endian */
+       plat->mmio->dma = cpu_to_be64((uintptr_t)dma);
+
+       while (be32_to_cpu(dma->control) & ~FW_CFG_DMA_ERROR);
+}
+
+static int qfw_mmio_of_to_plat(struct udevice *dev)
+{
+       struct qfw_mmio_plat *plat = dev_get_plat(dev);
+
+       plat->mmio = map_physmem(dev_read_addr(dev),
+                                sizeof(struct qfw_mmio),
+                                MAP_NOCACHE);
+
+       return 0;
+}
+
+static int qfw_mmio_probe(struct udevice *dev)
+{
+       return qfw_register(dev);
+}
+
+static struct dm_qfw_ops qfw_mmio_ops = {
+       .read_entry_io = qfw_mmio_read_entry_io,
+       .read_entry_dma = qfw_mmio_read_entry_dma,
+};
+
+static const struct udevice_id qfw_mmio_ids[] = {
+       { .compatible = "qemu,fw-cfg-mmio" },
+       {}
+};
+
+U_BOOT_DRIVER(qfw_mmio) = {
+       .name   = "qfw_mmio",
+       .id     = UCLASS_QFW,
+       .of_match       = qfw_mmio_ids,
+       .plat_auto      = sizeof(struct qfw_mmio_plat),
+       .of_to_plat     = qfw_mmio_of_to_plat,
+       .probe  = qfw_mmio_probe,
+       .ops    = &qfw_mmio_ops,
+};
diff --git a/drivers/misc/qfw_pio.c b/drivers/misc/qfw_pio.c
new file mode 100644
index 0000000000..e2f628d338
--- /dev/null
+++ b/drivers/misc/qfw_pio.c
@@ -0,0 +1,69 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * PIO interface for QFW
+ *
+ * (C) Copyright 2015 Miao Yan <[email protected]>
+ * (C) Copyright 2021 Asherah Connor <[email protected]>
+ */
+
+#define LOG_CATEGORY UCLASS_QFW
+
+#include <asm/io.h>
+#include <dm/device.h>
+#include <qfw.h>
+
+/*
+ * PIO ports are correct for x86, which appears to be the only arch that uses
+ * PIO.
+ */
+#define FW_CONTROL_PORT      0x510
+#define FW_DATA_PORT         0x511
+#define FW_DMA_PORT_LOW      0x514
+#define FW_DMA_PORT_HIGH     0x518
+
+static void qfw_pio_read_entry_io(struct udevice *dev, u16 entry, u32 size,
+                                 void *address)
+{
+       /*
+        * writing FW_CFG_INVALID will cause read operation to resume at last
+        * offset, otherwise read will start at offset 0
+        *
+        * Note: on platform where the control register is IO port, the
+        * endianness is little endian.
+        */
+       if (entry != FW_CFG_INVALID)
+               outw(cpu_to_le16(entry), FW_CONTROL_PORT);
+
+       /* the endianness of data register is string-preserving */
+       u32 i = 0;
+       u8 *data = address;
+
+       while (size--)
+               data[i++] = inb(FW_DATA_PORT);
+}
+
+/* Read configuration item using fw_cfg DMA interface */
+static void qfw_pio_read_entry_dma(struct udevice *dev, struct qfw_dma *dma)
+{
+       /* the DMA address register is big-endian */
+       outl(cpu_to_be32((uintptr_t)dma), FW_DMA_PORT_HIGH);
+
+       while (be32_to_cpu(dma->control) & ~FW_CFG_DMA_ERROR);
+}
+
+static int qfw_pio_probe(struct udevice *dev)
+{
+       return qfw_register(dev);
+}
+
+static struct dm_qfw_ops qfw_pio_ops = {
+       .read_entry_io = qfw_pio_read_entry_io,
+       .read_entry_dma = qfw_pio_read_entry_dma,
+};
+
+U_BOOT_DRIVER(qfw_pio) = {
+       .name   = "qfw_pio",
+       .id     = UCLASS_QFW,
+       .probe  = qfw_pio_probe,
+       .ops    = &qfw_pio_ops,
+};
diff --git a/include/dm/uclass-id.h b/include/dm/uclass-id.h
index d75de368c5..d800f679d5 100644
--- a/include/dm/uclass-id.h
+++ b/include/dm/uclass-id.h
@@ -90,6 +90,7 @@ enum uclass_id {
        UCLASS_POWER_DOMAIN,    /* (SoC) Power domains */
        UCLASS_PWM,             /* Pulse-width modulator */
        UCLASS_PWRSEQ,          /* Power sequence device */
+       UCLASS_QFW,             /* QEMU firmware config device */
        UCLASS_RAM,             /* RAM controller */
        UCLASS_REGULATOR,       /* Regulator device */
        UCLASS_REMOTEPROC,      /* Remote Processor device */
diff --git a/include/qfw.h b/include/qfw.h
index cea8e11d44..7b389dec25 100644
--- a/include/qfw.h
+++ b/include/qfw.h
@@ -82,19 +82,7 @@ struct fw_file {
 };
 
 struct fw_cfg_file_iter {
-       struct list_head *entry; /* structure to iterate file list */
-};
-
-struct fw_cfg_dma_access {
-       __be32 control;
-       __be32 length;
-       __be64 address;
-};
-
-struct fw_cfg_arch_ops {
-       void (*arch_read_pio)(uint16_t selector, uint32_t size,
-                       void *address);
-       void (*arch_read_dma)(struct fw_cfg_dma_access *dma);
+       struct list_head *entry, *end; /* structures to iterate file list */
 };
 
 struct bios_linker_entry {
@@ -146,32 +134,57 @@ struct bios_linker_entry {
        };
 } __packed;
 
+struct qfw_dma {
+       __be32 control;
+       __be32 length;
+       __be64 address;
+};
+
+struct qfw_dev {
+       struct udevice *dev;
+       bool dma_present;
+       struct list_head fw_list;
+};
+
+struct dm_qfw_ops {
+       void (*read_entry_io)(struct udevice *dev, u16 entry, u32 size,
+                             void *address);
+       void (*read_entry_dma)(struct udevice *dev, struct qfw_dma *dma);
+};
+
+#define dm_qfw_get_ops(dev) \
+               ((struct dm_qfw_ops *)(dev)->driver->ops)
+
+int qfw_register(struct udevice *dev);
+
+struct udevice;
+
 /**
- * Initialize QEMU fw_cfg interface
+ * Get QEMU firmware config device.
  *
- * @ops: arch specific read operations
+ * @devp   Pointer to be filled with address of the qfw device.
+ *
+ * @return 0 on success, -ENODEV if the device is not available.
  */
-void qemu_fwcfg_init(struct fw_cfg_arch_ops *ops);
+int qfw_get_dev(struct udevice **devp);
 
-void qemu_fwcfg_read_entry(uint16_t entry, uint32_t length, void *address);
-int qemu_fwcfg_read_firmware_list(void);
-struct fw_file *qemu_fwcfg_find_file(const char *name);
+void qfw_read_entry(struct udevice *dev, u16 entry, u32 length, void *address);
+int qemu_fwcfg_read_firmware_list(struct udevice *dev);
+struct fw_file *qemu_fwcfg_find_file(struct udevice *dev, const char *name);
 
 /**
  * Get system cpu number
  *
  * @return:   cpu number in system
  */
-int qemu_fwcfg_online_cpus(void);
+int qemu_fwcfg_online_cpus(struct udevice *dev);
 
 /* helper functions to iterate firmware file list */
-struct fw_file *qemu_fwcfg_file_iter_init(struct fw_cfg_file_iter *iter);
+struct fw_file *qemu_fwcfg_file_iter_init(struct udevice *dev,
+                                         struct fw_cfg_file_iter *iter);
 struct fw_file *qemu_fwcfg_file_iter_next(struct fw_cfg_file_iter *iter);
 bool qemu_fwcfg_file_iter_end(struct fw_cfg_file_iter *iter);
 
-bool qemu_fwcfg_present(void);
-bool qemu_fwcfg_dma_present(void);
-
 /**
  * qemu_cpu_fixup() - Fix up the CPUs for QEMU
  *
-- 
2.20.1

Reply via email to