On Thu, Jul 9, 2026 at 12:43 AM Cédric Le Goater <[email protected]> wrote: > > On 7/8/26 17:32, Bin Meng wrote: > > On Wed, Jul 8, 2026 at 7:17 PM Cédric Le Goater <[email protected]> wrote: > >> > >> Add read_page_mem_dor (CTRL_FREADMODE with DOR command and dual data > >> IO mode) and write_page_dor (user-mode DOR) tests. > >> > >> Signed-off-by: Cédric Le Goater <[email protected]> > >> --- > >> tests/qtest/aspeed-smc-utils.h | 4 +++ > >> tests/qtest/aspeed-smc-utils.c | 50 ++++++++++++++++++++++++++++++++++ > >> tests/qtest/aspeed_smc-test.c | 16 +++++++++++ > >> tests/qtest/ast2700-smc-test.c | 4 +++ > >> 4 files changed, 74 insertions(+) > >> > >> diff --git a/tests/qtest/aspeed-smc-utils.h > >> b/tests/qtest/aspeed-smc-utils.h > >> index 5a86514caf35..fbfb94e831d8 100644 > >> --- a/tests/qtest/aspeed-smc-utils.h > >> +++ b/tests/qtest/aspeed-smc-utils.h > >> @@ -44,6 +44,7 @@ > >> #define CTRL_FREADMODE 0x1 > >> #define CTRL_WRITEMODE 0x2 > >> #define CTRL_USERMODE 0x3 > >> +#define CTRL_IO_DUAL_DATA BIT(29) > > > > This macro is already defined in aspeed-smc-utils.h:43 > > I am confused. I only see this CTRL_IO_DUAL_DATA define in > hw/ssi/aspeed_smc.c. >
Sorry, my fault. I had a local tree that modified some of these qtests before. Regards, Bin
