[PATCH v1 16/16] test/qtest/ast2700-smc-test: Support to test AST2700

Jamin Lin via posted 16 patches 4 days, 18 hours ago
There is a newer version of this series
[PATCH v1 16/16] test/qtest/ast2700-smc-test: Support to test AST2700
Posted by Jamin Lin via 4 days, 18 hours ago
Add test_ast2700_evb function and reused testcases which are from
aspeed_smc-test.c for AST2700 testing. The base address, flash base address
and ce index of fmc_cs0 are 0x14000000, 0x100000000 and 0, respectively.
The default flash model of fmc_cs0 is "w25q01jvq" whose size is 128MB,
so set jedec_id 0xef4021.

Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
---
 tests/qtest/ast2700-smc-test.c | 598 +++++++++++++++++++++++++++++++++
 tests/qtest/meson.build        |   3 +-
 2 files changed, 600 insertions(+), 1 deletion(-)
 create mode 100644 tests/qtest/ast2700-smc-test.c

diff --git a/tests/qtest/ast2700-smc-test.c b/tests/qtest/ast2700-smc-test.c
new file mode 100644
index 0000000000..b7cfc9936d
--- /dev/null
+++ b/tests/qtest/ast2700-smc-test.c
@@ -0,0 +1,598 @@
+/*
+ * QTest testcase for the M25P80 Flash
+ * Using the ASPEED SPI Controller since
+ * AST2700.
+ *
+ * SPDX-License-Identifier: GPL-2.0-or-later
+ * Copyright (C) 2024 ASPEED Technology Inc.
+ */
+
+#include "qemu/osdep.h"
+#include "qemu/bswap.h"
+#include "libqtest-single.h"
+#include "qemu/bitops.h"
+
+/*
+ * ASPEED SPI Controller registers
+ */
+#define R_CONF              0x00
+#define   CONF_ENABLE_W0       16
+#define R_CE_CTRL           0x04
+#define   CRTL_EXTENDED0       0  /* 32 bit addressing for SPI */
+#define R_CTRL0             0x10
+#define   CTRL_IO_QUAD_IO      BIT(31)
+#define   CTRL_CE_STOP_ACTIVE  BIT(2)
+#define   CTRL_READMODE        0x0
+#define   CTRL_FREADMODE       0x1
+#define   CTRL_WRITEMODE       0x2
+#define   CTRL_USERMODE        0x3
+#define SR_WEL BIT(1)
+
+/*
+ * Flash commands
+ */
+enum {
+    JEDEC_READ = 0x9f,
+    RDSR = 0x5,
+    WRDI = 0x4,
+    BULK_ERASE = 0xc7,
+    READ = 0x03,
+    PP = 0x02,
+    WRSR = 0x1,
+    WREN = 0x6,
+    SRWD = 0x80,
+    RESET_ENABLE = 0x66,
+    RESET_MEMORY = 0x99,
+    EN_4BYTE_ADDR = 0xB7,
+    ERASE_SECTOR = 0xd8,
+};
+
+#define CTRL_IO_MODE_MASK  (BIT(31) | BIT(30) | BIT(29) | BIT(28))
+#define FLASH_PAGE_SIZE           256
+
+typedef struct TestData {
+    QTestState *s;
+    uint64_t spi_base;
+    uint64_t flash_base;
+    uint32_t jedec_id;
+    char *tmp_path;
+    uint8_t cs;
+    const char *node;
+    uint32_t page_addr;
+} TestData;
+
+/*
+ * Use an explicit bswap for the values read/wrote to the flash region
+ * as they are BE and the Aspeed CPU is LE.
+ */
+static inline uint32_t make_be32(uint32_t data)
+{
+    return bswap32(data);
+}
+
+static inline void spi_writel(const TestData *data, uint64_t offset,
+                              uint32_t value)
+{
+    qtest_writel(data->s, data->spi_base + offset, value);
+}
+
+static inline uint32_t spi_readl(const TestData *data, uint64_t offset)
+{
+    return qtest_readl(data->s, data->spi_base + offset);
+}
+
+static inline void flash_writeb(const TestData *data, uint64_t offset,
+                                uint8_t value)
+{
+    qtest_writeb(data->s, data->flash_base + offset, value);
+}
+
+static inline void flash_writel(const TestData *data, uint64_t offset,
+                                uint32_t value)
+{
+    qtest_writel(data->s, data->flash_base + offset, value);
+}
+
+static inline uint8_t flash_readb(const TestData *data, uint64_t offset)
+{
+    return qtest_readb(data->s, data->flash_base + offset);
+}
+
+static inline uint32_t flash_readl(const TestData *data, uint64_t offset)
+{
+    return qtest_readl(data->s, data->flash_base + offset);
+}
+
+static void spi_conf(const TestData *data, uint32_t value)
+{
+    uint32_t conf = spi_readl(data, R_CONF);
+
+    conf |= value;
+    spi_writel(data, R_CONF, conf);
+}
+
+static void spi_conf_remove(const TestData *data, uint32_t value)
+{
+    uint32_t conf = spi_readl(data, R_CONF);
+
+    conf &= ~value;
+    spi_writel(data, R_CONF, conf);
+}
+
+static void spi_ce_ctrl(const TestData *data, uint32_t value)
+{
+    uint32_t conf = spi_readl(data, R_CE_CTRL);
+
+    conf |= value;
+    spi_writel(data, R_CE_CTRL, conf);
+}
+
+static void spi_ctrl_setmode(const TestData *data, uint8_t mode, uint8_t cmd)
+{
+    uint32_t ctrl_reg = R_CTRL0 + data->cs * 4;
+    uint32_t ctrl = spi_readl(data, ctrl_reg);
+    ctrl &= ~(CTRL_USERMODE | 0xff << 16);
+    ctrl |= mode | (cmd << 16);
+    spi_writel(data, ctrl_reg, ctrl);
+}
+
+static void spi_ctrl_start_user(const TestData *data)
+{
+    uint32_t ctrl_reg = R_CTRL0 + data->cs * 4;
+    uint32_t ctrl = spi_readl(data, ctrl_reg);
+
+    ctrl |= CTRL_USERMODE | CTRL_CE_STOP_ACTIVE;
+    spi_writel(data, ctrl_reg, ctrl);
+
+    ctrl &= ~CTRL_CE_STOP_ACTIVE;
+    spi_writel(data, ctrl_reg, ctrl);
+}
+
+static void spi_ctrl_stop_user(const TestData *data)
+{
+    uint32_t ctrl_reg = R_CTRL0 + data->cs * 4;
+    uint32_t ctrl = spi_readl(data, ctrl_reg);
+
+    ctrl |= CTRL_USERMODE | CTRL_CE_STOP_ACTIVE;
+    spi_writel(data, ctrl_reg, ctrl);
+}
+
+static void spi_ctrl_set_io_mode(const TestData *data, uint32_t value)
+{
+    uint32_t ctrl_reg = R_CTRL0 + data->cs * 4;
+    uint32_t ctrl = spi_readl(data, ctrl_reg);
+    uint32_t mode;
+
+    mode = value & CTRL_IO_MODE_MASK;
+    ctrl &= ~CTRL_IO_MODE_MASK;
+    ctrl |= mode;
+    spi_writel(data, ctrl_reg, ctrl);
+}
+
+static void flash_reset(const TestData *data)
+{
+    spi_conf(data, 1 << (CONF_ENABLE_W0 + data->cs));
+
+    spi_ctrl_start_user(data);
+    flash_writeb(data, 0, RESET_ENABLE);
+    flash_writeb(data, 0, RESET_MEMORY);
+    flash_writeb(data, 0, WREN);
+    flash_writeb(data, 0, BULK_ERASE);
+    flash_writeb(data, 0, WRDI);
+    spi_ctrl_stop_user(data);
+
+    spi_conf_remove(data, 1 << (CONF_ENABLE_W0 + data->cs));
+}
+
+static void test_read_jedec(const void *data)
+{
+    const TestData *test_data = (const TestData *)data;
+    uint32_t jedec = 0x0;
+
+    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
+
+    spi_ctrl_start_user(test_data);
+    flash_writeb(test_data, 0, JEDEC_READ);
+    jedec |= flash_readb(test_data, 0) << 16;
+    jedec |= flash_readb(test_data, 0) << 8;
+    jedec |= flash_readb(test_data, 0);
+    spi_ctrl_stop_user(test_data);
+
+    flash_reset(test_data);
+
+    g_assert_cmphex(jedec, ==, test_data->jedec_id);
+}
+
+static void read_page(const TestData *data, uint32_t addr, uint32_t *page)
+{
+    int i;
+
+    spi_ctrl_start_user(data);
+
+    flash_writeb(data, 0, EN_4BYTE_ADDR);
+    flash_writeb(data, 0, READ);
+    flash_writel(data, 0, make_be32(addr));
+
+    /* Continuous read are supported */
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        page[i] = make_be32(flash_readl(data, 0));
+    }
+    spi_ctrl_stop_user(data);
+}
+
+static void read_page_mem(const TestData *data, uint32_t addr, uint32_t *page)
+{
+    int i;
+
+    /* move out USER mode to use direct reads from the AHB bus */
+    spi_ctrl_setmode(data, CTRL_READMODE, READ);
+
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        page[i] = make_be32(flash_readl(data, addr + i * 4));
+    }
+}
+
+static void test_erase_sector(const void *data)
+{
+    const TestData *test_data = (const TestData *)data;
+    uint32_t some_page_addr = test_data->page_addr;
+    uint32_t page[FLASH_PAGE_SIZE / 4];
+    int i;
+
+    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
+
+    /*
+     * Previous page should be full of 0xffs after backend is
+     * initialized
+     */
+    read_page(test_data, some_page_addr - FLASH_PAGE_SIZE, page);
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        g_assert_cmphex(page[i], ==, 0xffffffff);
+    }
+
+    spi_ctrl_start_user(test_data);
+    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
+    flash_writeb(test_data, 0, WREN);
+    flash_writeb(test_data, 0, PP);
+    flash_writel(test_data, 0, make_be32(some_page_addr));
+
+    /* Fill the page with its own addresses */
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        flash_writel(test_data, 0, make_be32(some_page_addr + i * 4));
+    }
+    spi_ctrl_stop_user(test_data);
+
+    /* Check the page is correctly written */
+    read_page(test_data, some_page_addr, page);
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        g_assert_cmphex(page[i], ==, some_page_addr + i * 4);
+    }
+
+    spi_ctrl_start_user(test_data);
+    flash_writeb(test_data, 0, WREN);
+    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
+    flash_writeb(test_data, 0, ERASE_SECTOR);
+    flash_writel(test_data, 0, make_be32(some_page_addr));
+    spi_ctrl_stop_user(test_data);
+
+    /* Check the page is erased */
+    read_page(test_data, some_page_addr, page);
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        g_assert_cmphex(page[i], ==, 0xffffffff);
+    }
+
+    flash_reset(test_data);
+}
+
+static void test_erase_all(const void *data)
+{
+    const TestData *test_data = (const TestData *)data;
+    uint32_t some_page_addr = test_data->page_addr;
+    uint32_t page[FLASH_PAGE_SIZE / 4];
+    int i;
+
+    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
+
+    /*
+     * Previous page should be full of 0xffs after backend is
+     * initialized
+     */
+    read_page(test_data, some_page_addr - FLASH_PAGE_SIZE, page);
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        g_assert_cmphex(page[i], ==, 0xffffffff);
+    }
+
+    spi_ctrl_start_user(test_data);
+    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
+    flash_writeb(test_data, 0, WREN);
+    flash_writeb(test_data, 0, PP);
+    flash_writel(test_data, 0, make_be32(some_page_addr));
+
+    /* Fill the page with its own addresses */
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        flash_writel(test_data, 0, make_be32(some_page_addr + i * 4));
+    }
+    spi_ctrl_stop_user(test_data);
+
+    /* Check the page is correctly written */
+    read_page(test_data, some_page_addr, page);
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        g_assert_cmphex(page[i], ==, some_page_addr + i * 4);
+    }
+
+    spi_ctrl_start_user(test_data);
+    flash_writeb(test_data, 0, WREN);
+    flash_writeb(test_data, 0, BULK_ERASE);
+    spi_ctrl_stop_user(test_data);
+
+    /* Check the page is erased */
+    read_page(test_data, some_page_addr, page);
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        g_assert_cmphex(page[i], ==, 0xffffffff);
+    }
+
+    flash_reset(test_data);
+}
+
+static void test_write_page(const void *data)
+{
+    const TestData *test_data = (const TestData *)data;
+    uint32_t my_page_addr = test_data->page_addr;
+    uint32_t some_page_addr = my_page_addr + FLASH_PAGE_SIZE;
+    uint32_t page[FLASH_PAGE_SIZE / 4];
+    int i;
+
+    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
+
+    spi_ctrl_start_user(test_data);
+    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
+    flash_writeb(test_data, 0, WREN);
+    flash_writeb(test_data, 0, PP);
+    flash_writel(test_data, 0, make_be32(my_page_addr));
+
+    /* Fill the page with its own addresses */
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        flash_writel(test_data, 0, make_be32(my_page_addr + i * 4));
+    }
+    spi_ctrl_stop_user(test_data);
+
+    /* Check what was written */
+    read_page(test_data, my_page_addr, page);
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        g_assert_cmphex(page[i], ==, my_page_addr + i * 4);
+    }
+
+    /* Check some other page. It should be full of 0xff */
+    read_page(test_data, some_page_addr, page);
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        g_assert_cmphex(page[i], ==, 0xffffffff);
+    }
+
+    flash_reset(test_data);
+}
+
+static void test_read_page_mem(const void *data)
+{
+    const TestData *test_data = (const TestData *)data;
+    uint32_t my_page_addr = test_data->page_addr;
+    uint32_t some_page_addr = my_page_addr + FLASH_PAGE_SIZE;
+    uint32_t page[FLASH_PAGE_SIZE / 4];
+    int i;
+
+    /*
+     * Enable 4BYTE mode for controller.
+     */
+    spi_ce_ctrl(test_data, 1 << (CRTL_EXTENDED0 + test_data->cs));
+
+    /* Enable 4BYTE mode for flash. */
+    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
+    spi_ctrl_start_user(test_data);
+    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
+    flash_writeb(test_data, 0, WREN);
+    flash_writeb(test_data, 0, PP);
+    flash_writel(test_data, 0, make_be32(my_page_addr));
+
+    /* Fill the page with its own addresses */
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        flash_writel(test_data, 0, make_be32(my_page_addr + i * 4));
+    }
+    spi_ctrl_stop_user(test_data);
+    spi_conf_remove(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
+
+    /* Check what was written */
+    read_page_mem(test_data, my_page_addr, page);
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        g_assert_cmphex(page[i], ==, my_page_addr + i * 4);
+    }
+
+    /* Check some other page. It should be full of 0xff */
+    read_page_mem(test_data, some_page_addr, page);
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        g_assert_cmphex(page[i], ==, 0xffffffff);
+    }
+
+    flash_reset(test_data);
+}
+
+static void test_write_page_mem(const void *data)
+{
+    const TestData *test_data = (const TestData *)data;
+    uint32_t my_page_addr = test_data->page_addr;
+    uint32_t page[FLASH_PAGE_SIZE / 4];
+    int i;
+
+    /*
+     * Enable 4BYTE mode for controller.
+     */
+    spi_ce_ctrl(test_data, 1 << (CRTL_EXTENDED0 + test_data->cs));
+
+    /* Enable 4BYTE mode for flash. */
+    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
+    spi_ctrl_start_user(test_data);
+    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
+    flash_writeb(test_data, 0, WREN);
+    spi_ctrl_stop_user(test_data);
+
+    /* move out USER mode to use direct writes to the AHB bus */
+    spi_ctrl_setmode(test_data, CTRL_WRITEMODE, PP);
+
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        flash_writel(test_data, my_page_addr + i * 4,
+               make_be32(my_page_addr + i * 4));
+    }
+
+    /* Check what was written */
+    read_page_mem(test_data, my_page_addr, page);
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        g_assert_cmphex(page[i], ==, my_page_addr + i * 4);
+    }
+
+    flash_reset(test_data);
+}
+
+static void test_read_status_reg(const void *data)
+{
+    const TestData *test_data = (const TestData *)data;
+    uint8_t r;
+
+    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
+
+    spi_ctrl_start_user(test_data);
+    flash_writeb(test_data, 0, RDSR);
+    r = flash_readb(test_data, 0);
+    spi_ctrl_stop_user(test_data);
+
+    g_assert_cmphex(r & SR_WEL, ==, 0);
+    g_assert(!qtest_qom_get_bool
+            (test_data->s, test_data->node, "write-enable"));
+
+    spi_ctrl_start_user(test_data);
+    flash_writeb(test_data, 0, WREN);
+    flash_writeb(test_data, 0, RDSR);
+    r = flash_readb(test_data, 0);
+    spi_ctrl_stop_user(test_data);
+
+    g_assert_cmphex(r & SR_WEL, ==, SR_WEL);
+    g_assert(qtest_qom_get_bool
+            (test_data->s, test_data->node, "write-enable"));
+
+    spi_ctrl_start_user(test_data);
+    flash_writeb(test_data, 0, WRDI);
+    flash_writeb(test_data, 0, RDSR);
+    r = flash_readb(test_data, 0);
+    spi_ctrl_stop_user(test_data);
+
+    g_assert_cmphex(r & SR_WEL, ==, 0);
+    g_assert(!qtest_qom_get_bool
+            (test_data->s, test_data->node, "write-enable"));
+
+    flash_reset(test_data);
+}
+
+static void test_write_page_qpi(const void *data)
+{
+    const TestData *test_data = (const TestData *)data;
+    uint32_t my_page_addr = test_data->page_addr;
+    uint32_t some_page_addr = my_page_addr + FLASH_PAGE_SIZE;
+    uint32_t page[FLASH_PAGE_SIZE / 4];
+    uint32_t page_pattern[] = {
+        0xebd8c134, 0x5da196bc, 0xae15e729, 0x5085ccdf
+    };
+    int i;
+
+    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
+
+    spi_ctrl_start_user(test_data);
+    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
+    flash_writeb(test_data, 0, WREN);
+    flash_writeb(test_data, 0, PP);
+    flash_writel(test_data, 0, make_be32(my_page_addr));
+
+    /* Set quad io mode */
+    spi_ctrl_set_io_mode(test_data, CTRL_IO_QUAD_IO);
+
+    /* Fill the page pattern */
+    for (i = 0; i < ARRAY_SIZE(page_pattern); i++) {
+        flash_writel(test_data, 0, make_be32(page_pattern[i]));
+    }
+
+    /* Fill the page with its own addresses */
+    for (; i < FLASH_PAGE_SIZE / 4; i++) {
+        flash_writel(test_data, 0, make_be32(my_page_addr + i * 4));
+    }
+
+    /* Restore io mode */
+    spi_ctrl_set_io_mode(test_data, 0);
+    spi_ctrl_stop_user(test_data);
+
+    /* Check what was written */
+    read_page(test_data, my_page_addr, page);
+    for (i = 0; i < ARRAY_SIZE(page_pattern); i++) {
+        g_assert_cmphex(page[i], ==, page_pattern[i]);
+    }
+    for (; i < FLASH_PAGE_SIZE / 4; i++) {
+        g_assert_cmphex(page[i], ==, my_page_addr + i * 4);
+    }
+
+    /* Check some other page. It should be full of 0xff */
+    read_page(test_data, some_page_addr, page);
+    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
+        g_assert_cmphex(page[i], ==, 0xffffffff);
+    }
+
+    flash_reset(test_data);
+}
+
+static void test_ast2700_evb(TestData *data)
+{
+    int ret;
+    int fd;
+
+    fd = g_file_open_tmp("qtest.m25p80.w25q01jvq.XXXXXX",
+                         &data->tmp_path, NULL);
+    g_assert(fd >= 0);
+    ret = ftruncate(fd, 128 * 1024 * 1024);
+    g_assert(ret == 0);
+    close(fd);
+
+    data->s = qtest_initf("-machine ast2700-evb "
+                          "-drive file=%s,format=raw,if=mtd",
+                          data->tmp_path);
+
+    /* fmc cs0 with w25q01jvq flash */
+    data->flash_base = 0x100000000;
+    data->spi_base = 0x14000000;
+    data->jedec_id = 0xef4021;
+    data->cs = 0;
+    data->node = "/machine/soc/fmc/ssi.0/child[0]";
+    /* beyond 64MB */
+    data->page_addr = 0x40000 * FLASH_PAGE_SIZE;
+
+    qtest_add_data_func("/ast2700/smc/read_jedec", data, test_read_jedec);
+    qtest_add_data_func("/ast2700/smc/erase_sector", data, test_erase_sector);
+    qtest_add_data_func("/ast2700/smc/erase_all",  data, test_erase_all);
+    qtest_add_data_func("/ast2700/smc/write_page", data, test_write_page);
+    qtest_add_data_func("/ast2700/smc/read_page_mem",
+                        data, test_read_page_mem);
+    qtest_add_data_func("/ast2700/smc/write_page_mem",
+                        data, test_write_page_mem);
+    qtest_add_data_func("/ast2700/smc/read_status_reg",
+                        data, test_read_status_reg);
+    qtest_add_data_func("/ast2700/smc/write_page_qpi",
+                        data, test_write_page_qpi);
+}
+
+int main(int argc, char **argv)
+{
+    TestData ast2700_evb_data;
+    int ret;
+
+    g_test_init(&argc, &argv, NULL);
+
+    test_ast2700_evb(&ast2700_evb_data);
+    ret = g_test_run();
+
+    qtest_quit(ast2700_evb_data.s);
+    unlink(ast2700_evb_data.tmp_path);
+    return ret;
+}
diff --git a/tests/qtest/meson.build b/tests/qtest/meson.build
index 41f3678cee..c7113f1e1d 100644
--- a/tests/qtest/meson.build
+++ b/tests/qtest/meson.build
@@ -210,7 +210,8 @@ qtests_aspeed = \
    'aspeed_smc-test',
    'aspeed_gpio-test']
 qtests_aspeed64 = \
-  ['ast2700-gpio-test']
+  ['ast2700-gpio-test',
+   'ast2700-smc-test']
 
 qtests_stm32l4x5 = \
   ['stm32l4x5_exti-test',
-- 
2.34.1
Re: [SPAM] [PATCH v1 16/16] test/qtest/ast2700-smc-test: Support to test AST2700
Posted by Cédric Le Goater 1 day, 10 hours ago
Hello Jamin,

On 10/18/24 07:31, Jamin Lin wrote:
> Add test_ast2700_evb function and reused testcases which are from
> aspeed_smc-test.c for AST2700 testing. The base address, flash base address
> and ce index of fmc_cs0 are 0x14000000, 0x100000000 and 0, respectively.
> The default flash model of fmc_cs0 is "w25q01jvq" whose size is 128MB,
> so set jedec_id 0xef4021.
> 
> Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>

Most of the routines below are copied from the original aspeed_smc-test.c
file (Copyright IBM).

To avoid duplication, we should move the implementation to a common file
aspeed-smc-utils.c and introduce an aspeed-smc-utils.h header file.

Thanks,

C.




> ---
>   tests/qtest/ast2700-smc-test.c | 598 +++++++++++++++++++++++++++++++++
>   tests/qtest/meson.build        |   3 +-
>   2 files changed, 600 insertions(+), 1 deletion(-)
>   create mode 100644 tests/qtest/ast2700-smc-test.c
> 
> diff --git a/tests/qtest/ast2700-smc-test.c b/tests/qtest/ast2700-smc-test.c
> new file mode 100644
> index 0000000000..b7cfc9936d
> --- /dev/null
> +++ b/tests/qtest/ast2700-smc-test.c
> @@ -0,0 +1,598 @@
> +/*
> + * QTest testcase for the M25P80 Flash
> + * Using the ASPEED SPI Controller since
> + * AST2700.
> + *
> + * SPDX-License-Identifier: GPL-2.0-or-later
> + * Copyright (C) 2024 ASPEED Technology Inc.
> + */
> +
> +#include "qemu/osdep.h"
> +#include "qemu/bswap.h"
> +#include "libqtest-single.h"
> +#include "qemu/bitops.h"
> +
> +/*
> + * ASPEED SPI Controller registers
> + */
> +#define R_CONF              0x00
> +#define   CONF_ENABLE_W0       16
> +#define R_CE_CTRL           0x04
> +#define   CRTL_EXTENDED0       0  /* 32 bit addressing for SPI */
> +#define R_CTRL0             0x10
> +#define   CTRL_IO_QUAD_IO      BIT(31)
> +#define   CTRL_CE_STOP_ACTIVE  BIT(2)
> +#define   CTRL_READMODE        0x0
> +#define   CTRL_FREADMODE       0x1
> +#define   CTRL_WRITEMODE       0x2
> +#define   CTRL_USERMODE        0x3
> +#define SR_WEL BIT(1)
> +
> +/*
> + * Flash commands
> + */
> +enum {
> +    JEDEC_READ = 0x9f,
> +    RDSR = 0x5,
> +    WRDI = 0x4,
> +    BULK_ERASE = 0xc7,
> +    READ = 0x03,
> +    PP = 0x02,
> +    WRSR = 0x1,
> +    WREN = 0x6,
> +    SRWD = 0x80,
> +    RESET_ENABLE = 0x66,
> +    RESET_MEMORY = 0x99,
> +    EN_4BYTE_ADDR = 0xB7,
> +    ERASE_SECTOR = 0xd8,
> +};
> +
> +#define CTRL_IO_MODE_MASK  (BIT(31) | BIT(30) | BIT(29) | BIT(28))
> +#define FLASH_PAGE_SIZE           256
> +
> +typedef struct TestData {
> +    QTestState *s;
> +    uint64_t spi_base;
> +    uint64_t flash_base;
> +    uint32_t jedec_id;
> +    char *tmp_path;
> +    uint8_t cs;
> +    const char *node;
> +    uint32_t page_addr;
> +} TestData;
> +
> +/*
> + * Use an explicit bswap for the values read/wrote to the flash region
> + * as they are BE and the Aspeed CPU is LE.
> + */
> +static inline uint32_t make_be32(uint32_t data)
> +{
> +    return bswap32(data);
> +}
> +
> +static inline void spi_writel(const TestData *data, uint64_t offset,
> +                              uint32_t value)
> +{
> +    qtest_writel(data->s, data->spi_base + offset, value);
> +}
> +
> +static inline uint32_t spi_readl(const TestData *data, uint64_t offset)
> +{
> +    return qtest_readl(data->s, data->spi_base + offset);
> +}
> +
> +static inline void flash_writeb(const TestData *data, uint64_t offset,
> +                                uint8_t value)
> +{
> +    qtest_writeb(data->s, data->flash_base + offset, value);
> +}
> +
> +static inline void flash_writel(const TestData *data, uint64_t offset,
> +                                uint32_t value)
> +{
> +    qtest_writel(data->s, data->flash_base + offset, value);
> +}
> +
> +static inline uint8_t flash_readb(const TestData *data, uint64_t offset)
> +{
> +    return qtest_readb(data->s, data->flash_base + offset);
> +}
> +
> +static inline uint32_t flash_readl(const TestData *data, uint64_t offset)
> +{
> +    return qtest_readl(data->s, data->flash_base + offset);
> +}
> +
> +static void spi_conf(const TestData *data, uint32_t value)
> +{
> +    uint32_t conf = spi_readl(data, R_CONF);
> +
> +    conf |= value;
> +    spi_writel(data, R_CONF, conf);
> +}
> +
> +static void spi_conf_remove(const TestData *data, uint32_t value)
> +{
> +    uint32_t conf = spi_readl(data, R_CONF);
> +
> +    conf &= ~value;
> +    spi_writel(data, R_CONF, conf);
> +}
> +
> +static void spi_ce_ctrl(const TestData *data, uint32_t value)
> +{
> +    uint32_t conf = spi_readl(data, R_CE_CTRL);
> +
> +    conf |= value;
> +    spi_writel(data, R_CE_CTRL, conf);
> +}
> +
> +static void spi_ctrl_setmode(const TestData *data, uint8_t mode, uint8_t cmd)
> +{
> +    uint32_t ctrl_reg = R_CTRL0 + data->cs * 4;
> +    uint32_t ctrl = spi_readl(data, ctrl_reg);
> +    ctrl &= ~(CTRL_USERMODE | 0xff << 16);
> +    ctrl |= mode | (cmd << 16);
> +    spi_writel(data, ctrl_reg, ctrl);
> +}
> +
> +static void spi_ctrl_start_user(const TestData *data)
> +{
> +    uint32_t ctrl_reg = R_CTRL0 + data->cs * 4;
> +    uint32_t ctrl = spi_readl(data, ctrl_reg);
> +
> +    ctrl |= CTRL_USERMODE | CTRL_CE_STOP_ACTIVE;
> +    spi_writel(data, ctrl_reg, ctrl);
> +
> +    ctrl &= ~CTRL_CE_STOP_ACTIVE;
> +    spi_writel(data, ctrl_reg, ctrl);
> +}
> +
> +static void spi_ctrl_stop_user(const TestData *data)
> +{
> +    uint32_t ctrl_reg = R_CTRL0 + data->cs * 4;
> +    uint32_t ctrl = spi_readl(data, ctrl_reg);
> +
> +    ctrl |= CTRL_USERMODE | CTRL_CE_STOP_ACTIVE;
> +    spi_writel(data, ctrl_reg, ctrl);
> +}
> +
> +static void spi_ctrl_set_io_mode(const TestData *data, uint32_t value)
> +{
> +    uint32_t ctrl_reg = R_CTRL0 + data->cs * 4;
> +    uint32_t ctrl = spi_readl(data, ctrl_reg);
> +    uint32_t mode;
> +
> +    mode = value & CTRL_IO_MODE_MASK;
> +    ctrl &= ~CTRL_IO_MODE_MASK;
> +    ctrl |= mode;
> +    spi_writel(data, ctrl_reg, ctrl);
> +}
> +
> +static void flash_reset(const TestData *data)
> +{
> +    spi_conf(data, 1 << (CONF_ENABLE_W0 + data->cs));
> +
> +    spi_ctrl_start_user(data);
> +    flash_writeb(data, 0, RESET_ENABLE);
> +    flash_writeb(data, 0, RESET_MEMORY);
> +    flash_writeb(data, 0, WREN);
> +    flash_writeb(data, 0, BULK_ERASE);
> +    flash_writeb(data, 0, WRDI);
> +    spi_ctrl_stop_user(data);
> +
> +    spi_conf_remove(data, 1 << (CONF_ENABLE_W0 + data->cs));
> +}
> +
> +static void test_read_jedec(const void *data)
> +{
> +    const TestData *test_data = (const TestData *)data;
> +    uint32_t jedec = 0x0;
> +
> +    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
> +
> +    spi_ctrl_start_user(test_data);
> +    flash_writeb(test_data, 0, JEDEC_READ);
> +    jedec |= flash_readb(test_data, 0) << 16;
> +    jedec |= flash_readb(test_data, 0) << 8;
> +    jedec |= flash_readb(test_data, 0);
> +    spi_ctrl_stop_user(test_data);
> +
> +    flash_reset(test_data);
> +
> +    g_assert_cmphex(jedec, ==, test_data->jedec_id);
> +}
> +
> +static void read_page(const TestData *data, uint32_t addr, uint32_t *page)
> +{
> +    int i;
> +
> +    spi_ctrl_start_user(data);
> +
> +    flash_writeb(data, 0, EN_4BYTE_ADDR);
> +    flash_writeb(data, 0, READ);
> +    flash_writel(data, 0, make_be32(addr));
> +
> +    /* Continuous read are supported */
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        page[i] = make_be32(flash_readl(data, 0));
> +    }
> +    spi_ctrl_stop_user(data);
> +}
> +
> +static void read_page_mem(const TestData *data, uint32_t addr, uint32_t *page)
> +{
> +    int i;
> +
> +    /* move out USER mode to use direct reads from the AHB bus */
> +    spi_ctrl_setmode(data, CTRL_READMODE, READ);
> +
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        page[i] = make_be32(flash_readl(data, addr + i * 4));
> +    }
> +}
> +
> +static void test_erase_sector(const void *data)
> +{
> +    const TestData *test_data = (const TestData *)data;
> +    uint32_t some_page_addr = test_data->page_addr;
> +    uint32_t page[FLASH_PAGE_SIZE / 4];
> +    int i;
> +
> +    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
> +
> +    /*
> +     * Previous page should be full of 0xffs after backend is
> +     * initialized
> +     */
> +    read_page(test_data, some_page_addr - FLASH_PAGE_SIZE, page);
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        g_assert_cmphex(page[i], ==, 0xffffffff);
> +    }
> +
> +    spi_ctrl_start_user(test_data);
> +    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
> +    flash_writeb(test_data, 0, WREN);
> +    flash_writeb(test_data, 0, PP);
> +    flash_writel(test_data, 0, make_be32(some_page_addr));
> +
> +    /* Fill the page with its own addresses */
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        flash_writel(test_data, 0, make_be32(some_page_addr + i * 4));
> +    }
> +    spi_ctrl_stop_user(test_data);
> +
> +    /* Check the page is correctly written */
> +    read_page(test_data, some_page_addr, page);
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        g_assert_cmphex(page[i], ==, some_page_addr + i * 4);
> +    }
> +
> +    spi_ctrl_start_user(test_data);
> +    flash_writeb(test_data, 0, WREN);
> +    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
> +    flash_writeb(test_data, 0, ERASE_SECTOR);
> +    flash_writel(test_data, 0, make_be32(some_page_addr));
> +    spi_ctrl_stop_user(test_data);
> +
> +    /* Check the page is erased */
> +    read_page(test_data, some_page_addr, page);
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        g_assert_cmphex(page[i], ==, 0xffffffff);
> +    }
> +
> +    flash_reset(test_data);
> +}
> +
> +static void test_erase_all(const void *data)
> +{
> +    const TestData *test_data = (const TestData *)data;
> +    uint32_t some_page_addr = test_data->page_addr;
> +    uint32_t page[FLASH_PAGE_SIZE / 4];
> +    int i;
> +
> +    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
> +
> +    /*
> +     * Previous page should be full of 0xffs after backend is
> +     * initialized
> +     */
> +    read_page(test_data, some_page_addr - FLASH_PAGE_SIZE, page);
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        g_assert_cmphex(page[i], ==, 0xffffffff);
> +    }
> +
> +    spi_ctrl_start_user(test_data);
> +    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
> +    flash_writeb(test_data, 0, WREN);
> +    flash_writeb(test_data, 0, PP);
> +    flash_writel(test_data, 0, make_be32(some_page_addr));
> +
> +    /* Fill the page with its own addresses */
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        flash_writel(test_data, 0, make_be32(some_page_addr + i * 4));
> +    }
> +    spi_ctrl_stop_user(test_data);
> +
> +    /* Check the page is correctly written */
> +    read_page(test_data, some_page_addr, page);
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        g_assert_cmphex(page[i], ==, some_page_addr + i * 4);
> +    }
> +
> +    spi_ctrl_start_user(test_data);
> +    flash_writeb(test_data, 0, WREN);
> +    flash_writeb(test_data, 0, BULK_ERASE);
> +    spi_ctrl_stop_user(test_data);
> +
> +    /* Check the page is erased */
> +    read_page(test_data, some_page_addr, page);
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        g_assert_cmphex(page[i], ==, 0xffffffff);
> +    }
> +
> +    flash_reset(test_data);
> +}
> +
> +static void test_write_page(const void *data)
> +{
> +    const TestData *test_data = (const TestData *)data;
> +    uint32_t my_page_addr = test_data->page_addr;
> +    uint32_t some_page_addr = my_page_addr + FLASH_PAGE_SIZE;
> +    uint32_t page[FLASH_PAGE_SIZE / 4];
> +    int i;
> +
> +    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
> +
> +    spi_ctrl_start_user(test_data);
> +    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
> +    flash_writeb(test_data, 0, WREN);
> +    flash_writeb(test_data, 0, PP);
> +    flash_writel(test_data, 0, make_be32(my_page_addr));
> +
> +    /* Fill the page with its own addresses */
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        flash_writel(test_data, 0, make_be32(my_page_addr + i * 4));
> +    }
> +    spi_ctrl_stop_user(test_data);
> +
> +    /* Check what was written */
> +    read_page(test_data, my_page_addr, page);
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        g_assert_cmphex(page[i], ==, my_page_addr + i * 4);
> +    }
> +
> +    /* Check some other page. It should be full of 0xff */
> +    read_page(test_data, some_page_addr, page);
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        g_assert_cmphex(page[i], ==, 0xffffffff);
> +    }
> +
> +    flash_reset(test_data);
> +}
> +
> +static void test_read_page_mem(const void *data)
> +{
> +    const TestData *test_data = (const TestData *)data;
> +    uint32_t my_page_addr = test_data->page_addr;
> +    uint32_t some_page_addr = my_page_addr + FLASH_PAGE_SIZE;
> +    uint32_t page[FLASH_PAGE_SIZE / 4];
> +    int i;
> +
> +    /*
> +     * Enable 4BYTE mode for controller.
> +     */
> +    spi_ce_ctrl(test_data, 1 << (CRTL_EXTENDED0 + test_data->cs));
> +
> +    /* Enable 4BYTE mode for flash. */
> +    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
> +    spi_ctrl_start_user(test_data);
> +    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
> +    flash_writeb(test_data, 0, WREN);
> +    flash_writeb(test_data, 0, PP);
> +    flash_writel(test_data, 0, make_be32(my_page_addr));
> +
> +    /* Fill the page with its own addresses */
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        flash_writel(test_data, 0, make_be32(my_page_addr + i * 4));
> +    }
> +    spi_ctrl_stop_user(test_data);
> +    spi_conf_remove(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
> +
> +    /* Check what was written */
> +    read_page_mem(test_data, my_page_addr, page);
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        g_assert_cmphex(page[i], ==, my_page_addr + i * 4);
> +    }
> +
> +    /* Check some other page. It should be full of 0xff */
> +    read_page_mem(test_data, some_page_addr, page);
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        g_assert_cmphex(page[i], ==, 0xffffffff);
> +    }
> +
> +    flash_reset(test_data);
> +}
> +
> +static void test_write_page_mem(const void *data)
> +{
> +    const TestData *test_data = (const TestData *)data;
> +    uint32_t my_page_addr = test_data->page_addr;
> +    uint32_t page[FLASH_PAGE_SIZE / 4];
> +    int i;
> +
> +    /*
> +     * Enable 4BYTE mode for controller.
> +     */
> +    spi_ce_ctrl(test_data, 1 << (CRTL_EXTENDED0 + test_data->cs));
> +
> +    /* Enable 4BYTE mode for flash. */
> +    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
> +    spi_ctrl_start_user(test_data);
> +    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
> +    flash_writeb(test_data, 0, WREN);
> +    spi_ctrl_stop_user(test_data);
> +
> +    /* move out USER mode to use direct writes to the AHB bus */
> +    spi_ctrl_setmode(test_data, CTRL_WRITEMODE, PP);
> +
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        flash_writel(test_data, my_page_addr + i * 4,
> +               make_be32(my_page_addr + i * 4));
> +    }
> +
> +    /* Check what was written */
> +    read_page_mem(test_data, my_page_addr, page);
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        g_assert_cmphex(page[i], ==, my_page_addr + i * 4);
> +    }
> +
> +    flash_reset(test_data);
> +}
> +
> +static void test_read_status_reg(const void *data)
> +{
> +    const TestData *test_data = (const TestData *)data;
> +    uint8_t r;
> +
> +    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
> +
> +    spi_ctrl_start_user(test_data);
> +    flash_writeb(test_data, 0, RDSR);
> +    r = flash_readb(test_data, 0);
> +    spi_ctrl_stop_user(test_data);
> +
> +    g_assert_cmphex(r & SR_WEL, ==, 0);
> +    g_assert(!qtest_qom_get_bool
> +            (test_data->s, test_data->node, "write-enable"));
> +
> +    spi_ctrl_start_user(test_data);
> +    flash_writeb(test_data, 0, WREN);
> +    flash_writeb(test_data, 0, RDSR);
> +    r = flash_readb(test_data, 0);
> +    spi_ctrl_stop_user(test_data);
> +
> +    g_assert_cmphex(r & SR_WEL, ==, SR_WEL);
> +    g_assert(qtest_qom_get_bool
> +            (test_data->s, test_data->node, "write-enable"));
> +
> +    spi_ctrl_start_user(test_data);
> +    flash_writeb(test_data, 0, WRDI);
> +    flash_writeb(test_data, 0, RDSR);
> +    r = flash_readb(test_data, 0);
> +    spi_ctrl_stop_user(test_data);
> +
> +    g_assert_cmphex(r & SR_WEL, ==, 0);
> +    g_assert(!qtest_qom_get_bool
> +            (test_data->s, test_data->node, "write-enable"));
> +
> +    flash_reset(test_data);
> +}
> +
> +static void test_write_page_qpi(const void *data)
> +{
> +    const TestData *test_data = (const TestData *)data;
> +    uint32_t my_page_addr = test_data->page_addr;
> +    uint32_t some_page_addr = my_page_addr + FLASH_PAGE_SIZE;
> +    uint32_t page[FLASH_PAGE_SIZE / 4];
> +    uint32_t page_pattern[] = {
> +        0xebd8c134, 0x5da196bc, 0xae15e729, 0x5085ccdf
> +    };
> +    int i;
> +
> +    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
> +
> +    spi_ctrl_start_user(test_data);
> +    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
> +    flash_writeb(test_data, 0, WREN);
> +    flash_writeb(test_data, 0, PP);
> +    flash_writel(test_data, 0, make_be32(my_page_addr));
> +
> +    /* Set quad io mode */
> +    spi_ctrl_set_io_mode(test_data, CTRL_IO_QUAD_IO);
> +
> +    /* Fill the page pattern */
> +    for (i = 0; i < ARRAY_SIZE(page_pattern); i++) {
> +        flash_writel(test_data, 0, make_be32(page_pattern[i]));
> +    }
> +
> +    /* Fill the page with its own addresses */
> +    for (; i < FLASH_PAGE_SIZE / 4; i++) {
> +        flash_writel(test_data, 0, make_be32(my_page_addr + i * 4));
> +    }
> +
> +    /* Restore io mode */
> +    spi_ctrl_set_io_mode(test_data, 0);
> +    spi_ctrl_stop_user(test_data);
> +
> +    /* Check what was written */
> +    read_page(test_data, my_page_addr, page);
> +    for (i = 0; i < ARRAY_SIZE(page_pattern); i++) {
> +        g_assert_cmphex(page[i], ==, page_pattern[i]);
> +    }
> +    for (; i < FLASH_PAGE_SIZE / 4; i++) {
> +        g_assert_cmphex(page[i], ==, my_page_addr + i * 4);
> +    }
> +
> +    /* Check some other page. It should be full of 0xff */
> +    read_page(test_data, some_page_addr, page);
> +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> +        g_assert_cmphex(page[i], ==, 0xffffffff);
> +    }
> +
> +    flash_reset(test_data);
> +}
> +
> +static void test_ast2700_evb(TestData *data)
> +{
> +    int ret;
> +    int fd;
> +
> +    fd = g_file_open_tmp("qtest.m25p80.w25q01jvq.XXXXXX",
> +                         &data->tmp_path, NULL);
> +    g_assert(fd >= 0);
> +    ret = ftruncate(fd, 128 * 1024 * 1024);
> +    g_assert(ret == 0);
> +    close(fd);
> +
> +    data->s = qtest_initf("-machine ast2700-evb "
> +                          "-drive file=%s,format=raw,if=mtd",
> +                          data->tmp_path);
> +
> +    /* fmc cs0 with w25q01jvq flash */
> +    data->flash_base = 0x100000000;
> +    data->spi_base = 0x14000000;
> +    data->jedec_id = 0xef4021;
> +    data->cs = 0;
> +    data->node = "/machine/soc/fmc/ssi.0/child[0]";
> +    /* beyond 64MB */
> +    data->page_addr = 0x40000 * FLASH_PAGE_SIZE;
> +
> +    qtest_add_data_func("/ast2700/smc/read_jedec", data, test_read_jedec);
> +    qtest_add_data_func("/ast2700/smc/erase_sector", data, test_erase_sector);
> +    qtest_add_data_func("/ast2700/smc/erase_all",  data, test_erase_all);
> +    qtest_add_data_func("/ast2700/smc/write_page", data, test_write_page);
> +    qtest_add_data_func("/ast2700/smc/read_page_mem",
> +                        data, test_read_page_mem);
> +    qtest_add_data_func("/ast2700/smc/write_page_mem",
> +                        data, test_write_page_mem);
> +    qtest_add_data_func("/ast2700/smc/read_status_reg",
> +                        data, test_read_status_reg);
> +    qtest_add_data_func("/ast2700/smc/write_page_qpi",
> +                        data, test_write_page_qpi);
> +}
> +
> +int main(int argc, char **argv)
> +{
> +    TestData ast2700_evb_data;
> +    int ret;
> +
> +    g_test_init(&argc, &argv, NULL);
> +
> +    test_ast2700_evb(&ast2700_evb_data);
> +    ret = g_test_run();
> +
> +    qtest_quit(ast2700_evb_data.s);
> +    unlink(ast2700_evb_data.tmp_path);
> +    return ret;
> +}
> diff --git a/tests/qtest/meson.build b/tests/qtest/meson.build
> index 41f3678cee..c7113f1e1d 100644
> --- a/tests/qtest/meson.build
> +++ b/tests/qtest/meson.build
> @@ -210,7 +210,8 @@ qtests_aspeed = \
>      'aspeed_smc-test',
>      'aspeed_gpio-test']
>   qtests_aspeed64 = \
> -  ['ast2700-gpio-test']
> +  ['ast2700-gpio-test',
> +   'ast2700-smc-test']
>   
>   qtests_stm32l4x5 = \
>     ['stm32l4x5_exti-test',
RE: [SPAM] [PATCH v1 16/16] test/qtest/ast2700-smc-test: Support to test AST2700
Posted by Jamin Lin 21 hours ago
Hi Cedric,

> Subject: Re: [SPAM] [PATCH v1 16/16] test/qtest/ast2700-smc-test: Support to
> test AST2700
> 
> Hello Jamin,
> 
> On 10/18/24 07:31, Jamin Lin wrote:
> > Add test_ast2700_evb function and reused testcases which are from
> > aspeed_smc-test.c for AST2700 testing. The base address, flash base
> > address and ce index of fmc_cs0 are 0x14000000, 0x100000000 and 0,
> respectively.
> > The default flash model of fmc_cs0 is "w25q01jvq" whose size is 128MB,
> > so set jedec_id 0xef4021.
> >
> > Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
> 
> Most of the routines below are copied from the original aspeed_smc-test.c file
> (Copyright IBM).
> 
> To avoid duplication, we should move the implementation to a common file
> aspeed-smc-utils.c and introduce an aspeed-smc-utils.h header file.
> 
Thanks for review and suggestion.
Okay, will move common testcases and API to aspeed-smc-utils.c.

Thanks-Jamin

> Thanks,
> 
> C.
> 
> 
> 
> 
> > ---
> >   tests/qtest/ast2700-smc-test.c | 598
> +++++++++++++++++++++++++++++++++
> >   tests/qtest/meson.build        |   3 +-
> >   2 files changed, 600 insertions(+), 1 deletion(-)
> >   create mode 100644 tests/qtest/ast2700-smc-test.c
> >
> > diff --git a/tests/qtest/ast2700-smc-test.c
> > b/tests/qtest/ast2700-smc-test.c new file mode 100644 index
> > 0000000000..b7cfc9936d
> > --- /dev/null
> > +++ b/tests/qtest/ast2700-smc-test.c
> > @@ -0,0 +1,598 @@
> > +/*
> > + * QTest testcase for the M25P80 Flash
> > + * Using the ASPEED SPI Controller since
> > + * AST2700.
> > + *
> > + * SPDX-License-Identifier: GPL-2.0-or-later
> > + * Copyright (C) 2024 ASPEED Technology Inc.
> > + */
> > +
> > +#include "qemu/osdep.h"
> > +#include "qemu/bswap.h"
> > +#include "libqtest-single.h"
> > +#include "qemu/bitops.h"
> > +
> > +/*
> > + * ASPEED SPI Controller registers
> > + */
> > +#define R_CONF              0x00
> > +#define   CONF_ENABLE_W0       16
> > +#define R_CE_CTRL           0x04
> > +#define   CRTL_EXTENDED0       0  /* 32 bit addressing for SPI */
> > +#define R_CTRL0             0x10
> > +#define   CTRL_IO_QUAD_IO      BIT(31)
> > +#define   CTRL_CE_STOP_ACTIVE  BIT(2)
> > +#define   CTRL_READMODE        0x0
> > +#define   CTRL_FREADMODE       0x1
> > +#define   CTRL_WRITEMODE       0x2
> > +#define   CTRL_USERMODE        0x3
> > +#define SR_WEL BIT(1)
> > +
> > +/*
> > + * Flash commands
> > + */
> > +enum {
> > +    JEDEC_READ = 0x9f,
> > +    RDSR = 0x5,
> > +    WRDI = 0x4,
> > +    BULK_ERASE = 0xc7,
> > +    READ = 0x03,
> > +    PP = 0x02,
> > +    WRSR = 0x1,
> > +    WREN = 0x6,
> > +    SRWD = 0x80,
> > +    RESET_ENABLE = 0x66,
> > +    RESET_MEMORY = 0x99,
> > +    EN_4BYTE_ADDR = 0xB7,
> > +    ERASE_SECTOR = 0xd8,
> > +};
> > +
> > +#define CTRL_IO_MODE_MASK  (BIT(31) | BIT(30) | BIT(29) | BIT(28))
> > +#define FLASH_PAGE_SIZE           256
> > +
> > +typedef struct TestData {
> > +    QTestState *s;
> > +    uint64_t spi_base;
> > +    uint64_t flash_base;
> > +    uint32_t jedec_id;
> > +    char *tmp_path;
> > +    uint8_t cs;
> > +    const char *node;
> > +    uint32_t page_addr;
> > +} TestData;
> > +
> > +/*
> > + * Use an explicit bswap for the values read/wrote to the flash
> > +region
> > + * as they are BE and the Aspeed CPU is LE.
> > + */
> > +static inline uint32_t make_be32(uint32_t data) {
> > +    return bswap32(data);
> > +}
> > +
> > +static inline void spi_writel(const TestData *data, uint64_t offset,
> > +                              uint32_t value) {
> > +    qtest_writel(data->s, data->spi_base + offset, value); }
> > +
> > +static inline uint32_t spi_readl(const TestData *data, uint64_t
> > +offset) {
> > +    return qtest_readl(data->s, data->spi_base + offset); }
> > +
> > +static inline void flash_writeb(const TestData *data, uint64_t offset,
> > +                                uint8_t value) {
> > +    qtest_writeb(data->s, data->flash_base + offset, value); }
> > +
> > +static inline void flash_writel(const TestData *data, uint64_t offset,
> > +                                uint32_t value) {
> > +    qtest_writel(data->s, data->flash_base + offset, value); }
> > +
> > +static inline uint8_t flash_readb(const TestData *data, uint64_t
> > +offset) {
> > +    return qtest_readb(data->s, data->flash_base + offset); }
> > +
> > +static inline uint32_t flash_readl(const TestData *data, uint64_t
> > +offset) {
> > +    return qtest_readl(data->s, data->flash_base + offset); }
> > +
> > +static void spi_conf(const TestData *data, uint32_t value) {
> > +    uint32_t conf = spi_readl(data, R_CONF);
> > +
> > +    conf |= value;
> > +    spi_writel(data, R_CONF, conf);
> > +}
> > +
> > +static void spi_conf_remove(const TestData *data, uint32_t value) {
> > +    uint32_t conf = spi_readl(data, R_CONF);
> > +
> > +    conf &= ~value;
> > +    spi_writel(data, R_CONF, conf);
> > +}
> > +
> > +static void spi_ce_ctrl(const TestData *data, uint32_t value) {
> > +    uint32_t conf = spi_readl(data, R_CE_CTRL);
> > +
> > +    conf |= value;
> > +    spi_writel(data, R_CE_CTRL, conf); }
> > +
> > +static void spi_ctrl_setmode(const TestData *data, uint8_t mode,
> > +uint8_t cmd) {
> > +    uint32_t ctrl_reg = R_CTRL0 + data->cs * 4;
> > +    uint32_t ctrl = spi_readl(data, ctrl_reg);
> > +    ctrl &= ~(CTRL_USERMODE | 0xff << 16);
> > +    ctrl |= mode | (cmd << 16);
> > +    spi_writel(data, ctrl_reg, ctrl); }
> > +
> > +static void spi_ctrl_start_user(const TestData *data) {
> > +    uint32_t ctrl_reg = R_CTRL0 + data->cs * 4;
> > +    uint32_t ctrl = spi_readl(data, ctrl_reg);
> > +
> > +    ctrl |= CTRL_USERMODE | CTRL_CE_STOP_ACTIVE;
> > +    spi_writel(data, ctrl_reg, ctrl);
> > +
> > +    ctrl &= ~CTRL_CE_STOP_ACTIVE;
> > +    spi_writel(data, ctrl_reg, ctrl); }
> > +
> > +static void spi_ctrl_stop_user(const TestData *data) {
> > +    uint32_t ctrl_reg = R_CTRL0 + data->cs * 4;
> > +    uint32_t ctrl = spi_readl(data, ctrl_reg);
> > +
> > +    ctrl |= CTRL_USERMODE | CTRL_CE_STOP_ACTIVE;
> > +    spi_writel(data, ctrl_reg, ctrl); }
> > +
> > +static void spi_ctrl_set_io_mode(const TestData *data, uint32_t
> > +value) {
> > +    uint32_t ctrl_reg = R_CTRL0 + data->cs * 4;
> > +    uint32_t ctrl = spi_readl(data, ctrl_reg);
> > +    uint32_t mode;
> > +
> > +    mode = value & CTRL_IO_MODE_MASK;
> > +    ctrl &= ~CTRL_IO_MODE_MASK;
> > +    ctrl |= mode;
> > +    spi_writel(data, ctrl_reg, ctrl); }
> > +
> > +static void flash_reset(const TestData *data) {
> > +    spi_conf(data, 1 << (CONF_ENABLE_W0 + data->cs));
> > +
> > +    spi_ctrl_start_user(data);
> > +    flash_writeb(data, 0, RESET_ENABLE);
> > +    flash_writeb(data, 0, RESET_MEMORY);
> > +    flash_writeb(data, 0, WREN);
> > +    flash_writeb(data, 0, BULK_ERASE);
> > +    flash_writeb(data, 0, WRDI);
> > +    spi_ctrl_stop_user(data);
> > +
> > +    spi_conf_remove(data, 1 << (CONF_ENABLE_W0 + data->cs)); }
> > +
> > +static void test_read_jedec(const void *data) {
> > +    const TestData *test_data = (const TestData *)data;
> > +    uint32_t jedec = 0x0;
> > +
> > +    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
> > +
> > +    spi_ctrl_start_user(test_data);
> > +    flash_writeb(test_data, 0, JEDEC_READ);
> > +    jedec |= flash_readb(test_data, 0) << 16;
> > +    jedec |= flash_readb(test_data, 0) << 8;
> > +    jedec |= flash_readb(test_data, 0);
> > +    spi_ctrl_stop_user(test_data);
> > +
> > +    flash_reset(test_data);
> > +
> > +    g_assert_cmphex(jedec, ==, test_data->jedec_id); }
> > +
> > +static void read_page(const TestData *data, uint32_t addr, uint32_t
> > +*page) {
> > +    int i;
> > +
> > +    spi_ctrl_start_user(data);
> > +
> > +    flash_writeb(data, 0, EN_4BYTE_ADDR);
> > +    flash_writeb(data, 0, READ);
> > +    flash_writel(data, 0, make_be32(addr));
> > +
> > +    /* Continuous read are supported */
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        page[i] = make_be32(flash_readl(data, 0));
> > +    }
> > +    spi_ctrl_stop_user(data);
> > +}
> > +
> > +static void read_page_mem(const TestData *data, uint32_t addr,
> > +uint32_t *page) {
> > +    int i;
> > +
> > +    /* move out USER mode to use direct reads from the AHB bus */
> > +    spi_ctrl_setmode(data, CTRL_READMODE, READ);
> > +
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        page[i] = make_be32(flash_readl(data, addr + i * 4));
> > +    }
> > +}
> > +
> > +static void test_erase_sector(const void *data) {
> > +    const TestData *test_data = (const TestData *)data;
> > +    uint32_t some_page_addr = test_data->page_addr;
> > +    uint32_t page[FLASH_PAGE_SIZE / 4];
> > +    int i;
> > +
> > +    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
> > +
> > +    /*
> > +     * Previous page should be full of 0xffs after backend is
> > +     * initialized
> > +     */
> > +    read_page(test_data, some_page_addr - FLASH_PAGE_SIZE, page);
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        g_assert_cmphex(page[i], ==, 0xffffffff);
> > +    }
> > +
> > +    spi_ctrl_start_user(test_data);
> > +    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
> > +    flash_writeb(test_data, 0, WREN);
> > +    flash_writeb(test_data, 0, PP);
> > +    flash_writel(test_data, 0, make_be32(some_page_addr));
> > +
> > +    /* Fill the page with its own addresses */
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        flash_writel(test_data, 0, make_be32(some_page_addr + i * 4));
> > +    }
> > +    spi_ctrl_stop_user(test_data);
> > +
> > +    /* Check the page is correctly written */
> > +    read_page(test_data, some_page_addr, page);
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        g_assert_cmphex(page[i], ==, some_page_addr + i * 4);
> > +    }
> > +
> > +    spi_ctrl_start_user(test_data);
> > +    flash_writeb(test_data, 0, WREN);
> > +    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
> > +    flash_writeb(test_data, 0, ERASE_SECTOR);
> > +    flash_writel(test_data, 0, make_be32(some_page_addr));
> > +    spi_ctrl_stop_user(test_data);
> > +
> > +    /* Check the page is erased */
> > +    read_page(test_data, some_page_addr, page);
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        g_assert_cmphex(page[i], ==, 0xffffffff);
> > +    }
> > +
> > +    flash_reset(test_data);
> > +}
> > +
> > +static void test_erase_all(const void *data) {
> > +    const TestData *test_data = (const TestData *)data;
> > +    uint32_t some_page_addr = test_data->page_addr;
> > +    uint32_t page[FLASH_PAGE_SIZE / 4];
> > +    int i;
> > +
> > +    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
> > +
> > +    /*
> > +     * Previous page should be full of 0xffs after backend is
> > +     * initialized
> > +     */
> > +    read_page(test_data, some_page_addr - FLASH_PAGE_SIZE, page);
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        g_assert_cmphex(page[i], ==, 0xffffffff);
> > +    }
> > +
> > +    spi_ctrl_start_user(test_data);
> > +    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
> > +    flash_writeb(test_data, 0, WREN);
> > +    flash_writeb(test_data, 0, PP);
> > +    flash_writel(test_data, 0, make_be32(some_page_addr));
> > +
> > +    /* Fill the page with its own addresses */
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        flash_writel(test_data, 0, make_be32(some_page_addr + i * 4));
> > +    }
> > +    spi_ctrl_stop_user(test_data);
> > +
> > +    /* Check the page is correctly written */
> > +    read_page(test_data, some_page_addr, page);
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        g_assert_cmphex(page[i], ==, some_page_addr + i * 4);
> > +    }
> > +
> > +    spi_ctrl_start_user(test_data);
> > +    flash_writeb(test_data, 0, WREN);
> > +    flash_writeb(test_data, 0, BULK_ERASE);
> > +    spi_ctrl_stop_user(test_data);
> > +
> > +    /* Check the page is erased */
> > +    read_page(test_data, some_page_addr, page);
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        g_assert_cmphex(page[i], ==, 0xffffffff);
> > +    }
> > +
> > +    flash_reset(test_data);
> > +}
> > +
> > +static void test_write_page(const void *data) {
> > +    const TestData *test_data = (const TestData *)data;
> > +    uint32_t my_page_addr = test_data->page_addr;
> > +    uint32_t some_page_addr = my_page_addr + FLASH_PAGE_SIZE;
> > +    uint32_t page[FLASH_PAGE_SIZE / 4];
> > +    int i;
> > +
> > +    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
> > +
> > +    spi_ctrl_start_user(test_data);
> > +    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
> > +    flash_writeb(test_data, 0, WREN);
> > +    flash_writeb(test_data, 0, PP);
> > +    flash_writel(test_data, 0, make_be32(my_page_addr));
> > +
> > +    /* Fill the page with its own addresses */
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        flash_writel(test_data, 0, make_be32(my_page_addr + i * 4));
> > +    }
> > +    spi_ctrl_stop_user(test_data);
> > +
> > +    /* Check what was written */
> > +    read_page(test_data, my_page_addr, page);
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        g_assert_cmphex(page[i], ==, my_page_addr + i * 4);
> > +    }
> > +
> > +    /* Check some other page. It should be full of 0xff */
> > +    read_page(test_data, some_page_addr, page);
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        g_assert_cmphex(page[i], ==, 0xffffffff);
> > +    }
> > +
> > +    flash_reset(test_data);
> > +}
> > +
> > +static void test_read_page_mem(const void *data) {
> > +    const TestData *test_data = (const TestData *)data;
> > +    uint32_t my_page_addr = test_data->page_addr;
> > +    uint32_t some_page_addr = my_page_addr + FLASH_PAGE_SIZE;
> > +    uint32_t page[FLASH_PAGE_SIZE / 4];
> > +    int i;
> > +
> > +    /*
> > +     * Enable 4BYTE mode for controller.
> > +     */
> > +    spi_ce_ctrl(test_data, 1 << (CRTL_EXTENDED0 + test_data->cs));
> > +
> > +    /* Enable 4BYTE mode for flash. */
> > +    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
> > +    spi_ctrl_start_user(test_data);
> > +    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
> > +    flash_writeb(test_data, 0, WREN);
> > +    flash_writeb(test_data, 0, PP);
> > +    flash_writel(test_data, 0, make_be32(my_page_addr));
> > +
> > +    /* Fill the page with its own addresses */
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        flash_writel(test_data, 0, make_be32(my_page_addr + i * 4));
> > +    }
> > +    spi_ctrl_stop_user(test_data);
> > +    spi_conf_remove(test_data, 1 << (CONF_ENABLE_W0 +
> > + test_data->cs));
> > +
> > +    /* Check what was written */
> > +    read_page_mem(test_data, my_page_addr, page);
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        g_assert_cmphex(page[i], ==, my_page_addr + i * 4);
> > +    }
> > +
> > +    /* Check some other page. It should be full of 0xff */
> > +    read_page_mem(test_data, some_page_addr, page);
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        g_assert_cmphex(page[i], ==, 0xffffffff);
> > +    }
> > +
> > +    flash_reset(test_data);
> > +}
> > +
> > +static void test_write_page_mem(const void *data) {
> > +    const TestData *test_data = (const TestData *)data;
> > +    uint32_t my_page_addr = test_data->page_addr;
> > +    uint32_t page[FLASH_PAGE_SIZE / 4];
> > +    int i;
> > +
> > +    /*
> > +     * Enable 4BYTE mode for controller.
> > +     */
> > +    spi_ce_ctrl(test_data, 1 << (CRTL_EXTENDED0 + test_data->cs));
> > +
> > +    /* Enable 4BYTE mode for flash. */
> > +    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
> > +    spi_ctrl_start_user(test_data);
> > +    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
> > +    flash_writeb(test_data, 0, WREN);
> > +    spi_ctrl_stop_user(test_data);
> > +
> > +    /* move out USER mode to use direct writes to the AHB bus */
> > +    spi_ctrl_setmode(test_data, CTRL_WRITEMODE, PP);
> > +
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        flash_writel(test_data, my_page_addr + i * 4,
> > +               make_be32(my_page_addr + i * 4));
> > +    }
> > +
> > +    /* Check what was written */
> > +    read_page_mem(test_data, my_page_addr, page);
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        g_assert_cmphex(page[i], ==, my_page_addr + i * 4);
> > +    }
> > +
> > +    flash_reset(test_data);
> > +}
> > +
> > +static void test_read_status_reg(const void *data) {
> > +    const TestData *test_data = (const TestData *)data;
> > +    uint8_t r;
> > +
> > +    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
> > +
> > +    spi_ctrl_start_user(test_data);
> > +    flash_writeb(test_data, 0, RDSR);
> > +    r = flash_readb(test_data, 0);
> > +    spi_ctrl_stop_user(test_data);
> > +
> > +    g_assert_cmphex(r & SR_WEL, ==, 0);
> > +    g_assert(!qtest_qom_get_bool
> > +            (test_data->s, test_data->node, "write-enable"));
> > +
> > +    spi_ctrl_start_user(test_data);
> > +    flash_writeb(test_data, 0, WREN);
> > +    flash_writeb(test_data, 0, RDSR);
> > +    r = flash_readb(test_data, 0);
> > +    spi_ctrl_stop_user(test_data);
> > +
> > +    g_assert_cmphex(r & SR_WEL, ==, SR_WEL);
> > +    g_assert(qtest_qom_get_bool
> > +            (test_data->s, test_data->node, "write-enable"));
> > +
> > +    spi_ctrl_start_user(test_data);
> > +    flash_writeb(test_data, 0, WRDI);
> > +    flash_writeb(test_data, 0, RDSR);
> > +    r = flash_readb(test_data, 0);
> > +    spi_ctrl_stop_user(test_data);
> > +
> > +    g_assert_cmphex(r & SR_WEL, ==, 0);
> > +    g_assert(!qtest_qom_get_bool
> > +            (test_data->s, test_data->node, "write-enable"));
> > +
> > +    flash_reset(test_data);
> > +}
> > +
> > +static void test_write_page_qpi(const void *data) {
> > +    const TestData *test_data = (const TestData *)data;
> > +    uint32_t my_page_addr = test_data->page_addr;
> > +    uint32_t some_page_addr = my_page_addr + FLASH_PAGE_SIZE;
> > +    uint32_t page[FLASH_PAGE_SIZE / 4];
> > +    uint32_t page_pattern[] = {
> > +        0xebd8c134, 0x5da196bc, 0xae15e729, 0x5085ccdf
> > +    };
> > +    int i;
> > +
> > +    spi_conf(test_data, 1 << (CONF_ENABLE_W0 + test_data->cs));
> > +
> > +    spi_ctrl_start_user(test_data);
> > +    flash_writeb(test_data, 0, EN_4BYTE_ADDR);
> > +    flash_writeb(test_data, 0, WREN);
> > +    flash_writeb(test_data, 0, PP);
> > +    flash_writel(test_data, 0, make_be32(my_page_addr));
> > +
> > +    /* Set quad io mode */
> > +    spi_ctrl_set_io_mode(test_data, CTRL_IO_QUAD_IO);
> > +
> > +    /* Fill the page pattern */
> > +    for (i = 0; i < ARRAY_SIZE(page_pattern); i++) {
> > +        flash_writel(test_data, 0, make_be32(page_pattern[i]));
> > +    }
> > +
> > +    /* Fill the page with its own addresses */
> > +    for (; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        flash_writel(test_data, 0, make_be32(my_page_addr + i * 4));
> > +    }
> > +
> > +    /* Restore io mode */
> > +    spi_ctrl_set_io_mode(test_data, 0);
> > +    spi_ctrl_stop_user(test_data);
> > +
> > +    /* Check what was written */
> > +    read_page(test_data, my_page_addr, page);
> > +    for (i = 0; i < ARRAY_SIZE(page_pattern); i++) {
> > +        g_assert_cmphex(page[i], ==, page_pattern[i]);
> > +    }
> > +    for (; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        g_assert_cmphex(page[i], ==, my_page_addr + i * 4);
> > +    }
> > +
> > +    /* Check some other page. It should be full of 0xff */
> > +    read_page(test_data, some_page_addr, page);
> > +    for (i = 0; i < FLASH_PAGE_SIZE / 4; i++) {
> > +        g_assert_cmphex(page[i], ==, 0xffffffff);
> > +    }
> > +
> > +    flash_reset(test_data);
> > +}
> > +
> > +static void test_ast2700_evb(TestData *data) {
> > +    int ret;
> > +    int fd;
> > +
> > +    fd = g_file_open_tmp("qtest.m25p80.w25q01jvq.XXXXXX",
> > +                         &data->tmp_path, NULL);
> > +    g_assert(fd >= 0);
> > +    ret = ftruncate(fd, 128 * 1024 * 1024);
> > +    g_assert(ret == 0);
> > +    close(fd);
> > +
> > +    data->s = qtest_initf("-machine ast2700-evb "
> > +                          "-drive file=%s,format=raw,if=mtd",
> > +                          data->tmp_path);
> > +
> > +    /* fmc cs0 with w25q01jvq flash */
> > +    data->flash_base = 0x100000000;
> > +    data->spi_base = 0x14000000;
> > +    data->jedec_id = 0xef4021;
> > +    data->cs = 0;
> > +    data->node = "/machine/soc/fmc/ssi.0/child[0]";
> > +    /* beyond 64MB */
> > +    data->page_addr = 0x40000 * FLASH_PAGE_SIZE;
> > +
> > +    qtest_add_data_func("/ast2700/smc/read_jedec", data,
> test_read_jedec);
> > +    qtest_add_data_func("/ast2700/smc/erase_sector", data,
> test_erase_sector);
> > +    qtest_add_data_func("/ast2700/smc/erase_all",  data,
> test_erase_all);
> > +    qtest_add_data_func("/ast2700/smc/write_page", data,
> test_write_page);
> > +    qtest_add_data_func("/ast2700/smc/read_page_mem",
> > +                        data, test_read_page_mem);
> > +    qtest_add_data_func("/ast2700/smc/write_page_mem",
> > +                        data, test_write_page_mem);
> > +    qtest_add_data_func("/ast2700/smc/read_status_reg",
> > +                        data, test_read_status_reg);
> > +    qtest_add_data_func("/ast2700/smc/write_page_qpi",
> > +                        data, test_write_page_qpi); }
> > +
> > +int main(int argc, char **argv)
> > +{
> > +    TestData ast2700_evb_data;
> > +    int ret;
> > +
> > +    g_test_init(&argc, &argv, NULL);
> > +
> > +    test_ast2700_evb(&ast2700_evb_data);
> > +    ret = g_test_run();
> > +
> > +    qtest_quit(ast2700_evb_data.s);
> > +    unlink(ast2700_evb_data.tmp_path);
> > +    return ret;
> > +}
> > diff --git a/tests/qtest/meson.build b/tests/qtest/meson.build index
> > 41f3678cee..c7113f1e1d 100644
> > --- a/tests/qtest/meson.build
> > +++ b/tests/qtest/meson.build
> > @@ -210,7 +210,8 @@ qtests_aspeed = \
> >      'aspeed_smc-test',
> >      'aspeed_gpio-test']
> >   qtests_aspeed64 = \
> > -  ['ast2700-gpio-test']
> > +  ['ast2700-gpio-test',
> > +   'ast2700-smc-test']
> >
> >   qtests_stm32l4x5 = \
> >     ['stm32l4x5_exti-test',