test: Add test for new command pci_mps

Adds a test for the new pci_mps command to ensure that it can set the
Maximum Payload Size (MPS) of all devices to 256 bytes in the sandbox
environment. Enables the pci_mps command in the sandbox environment so
that this test can be run.

Signed-off-by: Stephen Carlson <stcarlso@linux.microsoft.com>

diff --git a/MAINTAINERS b/MAINTAINERS
index 91d40ea4b6e5653b249e5dc6225784ffbdb5a581..d2e245e5e90b12092783183dec9c7916c6dee02e 100644 (file)
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1249,6 +1249,12 @@ M:       Heiko Schocher <hs@denx.de>
 S:     Maintained
 F:     drivers/pci/pci_mpc85xx.c
 
+PCI MPS
+M:     Stephen Carlson <stcarlso@linux.microsoft.com>
+S:     Maintained
+F:     cmd/pci_mps.c
+F:     test/cmd/pci_mps.c
+
 POWER
 M:     Jaehoon Chung <jh80.chung@samsung.com>
 S:     Maintained

diff --git a/configs/sandbox_defconfig b/configs/sandbox_defconfig
index b0f588da32e2e0363b5f95f124baef9383d080e1..2141c3d4698946d96bdcd69ffa2cb9c6ab73140b 100644 (file)
--- a/configs/sandbox_defconfig
+++ b/configs/sandbox_defconfig
@@ -78,6 +78,7 @@ CONFIG_CMD_MMC=y
 CONFIG_CMD_MUX=y
 CONFIG_CMD_OSD=y
 CONFIG_CMD_PCI=y
+CONFIG_CMD_PCI_MPS=y
 CONFIG_CMD_READ=y
 CONFIG_CMD_REMOTEPROC=y
 CONFIG_CMD_SPI=y

diff --git a/include/test/suites.h b/include/test/suites.h
index 7c4960c004d7a2ba4da5037808e7ef21f2b1fc65..7349ce5aa60f1b472496d821e27d282f82ffa27d 100644 (file)
--- a/include/test/suites.h
+++ b/include/test/suites.h
@@ -48,6 +48,8 @@ int do_ut_mem(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[]);
 int do_ut_optee(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[]);
 int do_ut_overlay(struct cmd_tbl *cmdtp, int flag, int argc,
                  char *const argv[]);
+int do_ut_pci_mps(struct cmd_tbl *cmdtp, int flag, int argc,
+                 char *const argv[]);
 int do_ut_print(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[]);
 int do_ut_seama(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[]);
 int do_ut_setexpr(struct cmd_tbl *cmdtp, int flag, int argc,

diff --git a/test/cmd/Makefile b/test/cmd/Makefile
index 7848f348bc48f24e43a7c3e444412774dbf51a49..055adc65a25e635931ddeb106ec6d0b0cb8f88ee 100644 (file)
--- a/test/cmd/Makefile
+++ b/test/cmd/Makefile
@@ -14,6 +14,9 @@ obj-$(CONFIG_CMD_FDT) += fdt.o
 obj-$(CONFIG_CONSOLE_TRUETYPE) += font.o
 obj-$(CONFIG_CMD_LOADM) += loadm.o
 obj-$(CONFIG_CMD_MEM_SEARCH) += mem_search.o
+ifdef CONFIG_CMD_PCI
+obj-$(CONFIG_CMD_PCI_MPS) += pci_mps.o
+endif
 obj-$(CONFIG_CMD_PINMUX) += pinmux.o
 obj-$(CONFIG_CMD_PWM) += pwm.o
 obj-$(CONFIG_CMD_SEAMA) += seama.o

diff --git a/test/cmd/pci_mps.c b/test/cmd/pci_mps.c
new file mode 100644 (file)
index 0000000..fd96f4f
--- /dev/null
+++ b/test/cmd/pci_mps.c
@@ -0,0 +1,42 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Tests that the PCI Maximum Payload Size (MPS) command can set the sandbox
+ * PCI Express device to safe mode and determine the correct payload size.
+ *
+ * Copyright 2023 Microsoft
+ * Written by Stephen Carlson <stcarlso@linux.microsoft.com>
+ */
+
+#include <common.h>
+#include <console.h>
+#include <test/suites.h>
+#include <test/ut.h>
+
+#define PCI_MPS_TEST(_name, _flags) UNIT_TEST(_name, _flags, pci_mps_test)
+
+/* Test "pci_mps" command in safe "s" mode */
+static int test_pci_mps_safe(struct unit_test_state *uts)
+{
+       /* Enumerate PCI Express first */
+       ut_assertok(run_command("pci e", 0));
+       ut_assert_console_end();
+
+       /* Test pci_mps s */
+       ut_assertok(run_command("pci_mps s", 0));
+       ut_assert_nextline("Setting MPS of all devices to 256B");
+       ut_assert_console_end();
+
+       return 0;
+}
+
+PCI_MPS_TEST(test_pci_mps_safe, UT_TESTF_CONSOLE_REC);
+
+int do_ut_pci_mps(struct cmd_tbl *cmdtp, int flag, int argc,
+                 char * const argv[])
+{
+       struct unit_test *tests = UNIT_TEST_SUITE_START(pci_mps_test);
+       const int n = UNIT_TEST_SUITE_COUNT(pci_mps_test);
+
+       return cmd_ut_category("cmd_pci_mps", "pci_mps_test_", tests, n,
+                              argc, argv);
+}

diff --git a/test/cmd_ut.c b/test/cmd_ut.c
index 409c22bfd2438b41c9ada778913a70e0f2f6ade0..d440da833a9c106edd4bb7fe6f68717fcf329bd8 100644 (file)
--- a/test/cmd_ut.c
+++ b/test/cmd_ut.c
@@ -110,6 +110,9 @@ static struct cmd_tbl cmd_ut_sub[] = {
 #ifdef CONFIG_CMD_LOADM
        U_BOOT_CMD_MKENT(loadm, CONFIG_SYS_MAXARGS, 1, do_ut_loadm, "", ""),
 #endif
+#ifdef CONFIG_CMD_PCI_MPS
+       U_BOOT_CMD_MKENT(pci_mps, CONFIG_SYS_MAXARGS, 1, do_ut_pci_mps, "", ""),
+#endif
 #ifdef CONFIG_CMD_SEAMA
        U_BOOT_CMD_MKENT(seama, CONFIG_SYS_MAXARGS, 1, do_ut_seama, "", ""),
 #endif
@@ -209,6 +212,9 @@ static char ut_help_text[] =
 #endif
 #ifdef CONFIG_UT_OVERLAY
        "\noverlay - device tree overlays"
+#endif
+#ifdef CONFIG_CMD_PCI_MPS
+       "\npci_mps - PCI Express Maximum Payload Size"
 #endif
        "\nprint  - printing things to the console"
        "\nsetexpr - setexpr command"