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_...@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',

Reply via email to