]> git.ipfire.org Git - thirdparty/u-boot.git/commitdiff
boot: Remove legacy extension board support
authorKory Maincent (TI.com) <kory.maincent@bootlin.com>
Thu, 30 Oct 2025 16:45:09 +0000 (17:45 +0100)
committerTom Rini <trini@konsulko.com>
Mon, 3 Nov 2025 16:02:39 +0000 (10:02 -0600)
Remove the legacy extension board implementation now that all boards
have been converted to use the new UCLASS-based framework. This
eliminates lines of legacy code while preserving functionality
through the modern driver model approach.

Update the bootstd tests, due to the removal of extension hunter.

Signed-off-by: Kory Maincent (TI.com) <kory.maincent@bootlin.com>
19 files changed:
MAINTAINERS
arch/Kconfig
arch/arm/mach-imx/imx8m/Kconfig
arch/arm/mach-omap2/am33xx/Kconfig
arch/arm/mach-omap2/omap5/Kconfig
arch/arm/mach-sunxi/Kconfig
board/sandbox/sandbox.c
board/ti/common/Kconfig
board/ti/common/Makefile
boot/Kconfig
boot/Makefile
boot/extension-uclass.c
boot/extension.c [deleted file]
cmd/Kconfig
cmd/extension_board.c
include/extension_board.h
test/boot/bootdev.c
test/boot/bootflow.c
test/boot/bootstd_common.h

index a63ffa14ef500b2c8377baca5a6d7174a86afa4a..a75e59453d7f71a8536d695c93d75e0c9c57708d 100644 (file)
@@ -1188,7 +1188,6 @@ S:        Maintained
 F:     board/sunxi/chip.c
 F:     board/ti/common/cape_detect.c
 F:     boot/extension-uclass.c
-F:     boot/extension.c
 F:     cmd/extension_board.c
 F:     include/extension_board.h
 
index 5bb65a29f8daf965d36e7fcdd6cededf04dca1f3..7e05e0c2263470ff397e09729f07f43170b79063 100644 (file)
@@ -215,7 +215,7 @@ config SANDBOX
        select SYSRESET_CMD_POWEROFF if CMD_POWEROFF
        select SYS_CACHE_SHIFT_4
        select IRQ
-       select SUPPORT_DM_EXTENSION_SCAN if CMDLINE
+       select SUPPORT_EXTENSION_SCAN if CMDLINE
        select SUPPORT_ACPI
        imply BITREVERSE
        select BLOBLIST
index 5324eff308459916ae93dcf120a74e18a56c7a61..e7bc154b8053f8eb0e99411fe4879ab06d923532 100644 (file)
@@ -359,14 +359,14 @@ config TARGET_IMX8MM_CL_IOT_GATE
        select IMX8MM
        select SUPPORT_SPL
        select IMX8M_LPDDR4
-       select SUPPORT_DM_EXTENSION_SCAN
+       select SUPPORT_EXTENSION_SCAN
 
 config TARGET_IMX8MM_CL_IOT_GATE_OPTEE
        bool "CompuLab iot-gate-imx8 with optee support"
        select IMX8MM
        select SUPPORT_SPL
        select IMX8M_LPDDR4
-       select SUPPORT_DM_EXTENSION_SCAN
+       select SUPPORT_EXTENSION_SCAN
 
 config TARGET_IMX8MP_RSB3720A1_4G
        bool "Support i.MX8MP RSB3720A1 4G"
index 77f7938305be47ef2af5734a921fb7a5693f34e0..dff4f1cf202f4a59157cfd6c666d3601a80c9f2a 100644 (file)
@@ -15,7 +15,7 @@ config TARGET_AM335X_EVM
        select DM_GPIO
        select DM_SERIAL
        select TI_I2C_BOARD_DETECT
-       select SUPPORT_DM_EXTENSION_SCAN
+       select SUPPORT_EXTENSION_SCAN
        imply CMD_DM
        imply SPL_DM
        imply SPL_DM_SEQ_ALIAS
index 819490a8cf87987ab262a23d5becd1f289b23e83..5394529658bc5e1e6258bcc513f2926a6f0720b6 100644 (file)
@@ -38,7 +38,7 @@ config TARGET_AM57XX_EVM
        select CMD_DDR3
        select DRA7XX
        select TI_I2C_BOARD_DETECT
-       select SUPPORT_DM_EXTENSION_SCAN
+       select SUPPORT_EXTENSION_SCAN
        imply DM_THERMAL
        imply SCSI
        imply SPL_THERMAL
index f5696199516f3332f0b7ccc03db38977c80e36ee..b04ec671696243ccfd803d4b519211b719381808 100644 (file)
@@ -1223,7 +1223,7 @@ config BLUETOOTH_DT_DEVICE_FIXUP
 
 config CHIP_DIP_SCAN
        bool "Enable DIPs detection for CHIP board"
-       select SUPPORT_DM_EXTENSION_SCAN
+       select SUPPORT_EXTENSION_SCAN
        select W1
        select W1_GPIO
        select W1_EEPROM
index d46db6d492f3b979c0700e4904d66929971fbbb4..d0bb3e3bb487dcb1161d5d883267a91dc8079429 100644 (file)
@@ -110,7 +110,7 @@ int ft_board_setup(void *fdt, struct bd_info *bd)
        return fdt_add_mem_rsv(fdt, 0x00d02000, 0x4000);
 }
 
-#if CONFIG_IS_ENABLED(SUPPORT_DM_EXTENSION_SCAN) && \
+#if CONFIG_IS_ENABLED(SUPPORT_EXTENSION_SCAN) && \
        !CONFIG_IS_ENABLED(XPL_BUILD)
 static int sandbox_extension_board_scan(struct udevice *dev,
                                        struct alist *extension_list)
index feb05b4bf95173b904ed92c6bf64ded133b074ef..9512c5c23f1adafe53cfb006d8d074f8293b0848 100644 (file)
@@ -20,7 +20,7 @@ config CAPE_EEPROM_BUS_NUM
        int "Cape EEPROM's I2C bus address"
        range 0 8
        default 2
-       depends on SUPPORT_DM_EXTENSION_SCAN
+       depends on SUPPORT_EXTENSION_SCAN
 
 config TI_COMMON_CMD_OPTIONS
        bool "Enable cmd options on TI platforms"
index b42273d3a5af3a179cc5fa61f92a63c4b754833e..f58935b4103927ebbead3d18d71bee3d5e52c065 100644 (file)
@@ -2,6 +2,6 @@
 # Copyright (C) 2015-2016 Texas Instruments Incorporated - https://www.ti.com/
 
 obj-${CONFIG_TI_I2C_BOARD_DETECT} += board_detect.o
-obj-${CONFIG_$(PHASE_)SUPPORT_DM_EXTENSION_SCAN} += cape_detect.o
+obj-${CONFIG_$(PHASE_)SUPPORT_EXTENSION_SCAN} += cape_detect.o
 obj-${CONFIG_OF_LIBFDT} += fdt_ops.o
 obj-${CONFIG_ARCH_K3} += k3-ddr.o
index 08d666d4b93b0a81a48103b86f0ae400a7eba750..a75f6b641ed1c0ee1033b3f0a33fe6f11da68794 100644 (file)
@@ -1910,10 +1910,6 @@ config SUPPORT_EXTENSION_SCAN
        select OF_LIBFDT_OVERLAY
         bool
 
-config SUPPORT_DM_EXTENSION_SCAN
-       select OF_LIBFDT_OVERLAY
-        bool
-
 config USE_BOOTARGS
        bool "Enable boot arguments"
        help
index aa26070fbb8d209939719a0a1e1778f5dfa495d1..7fb56e7ef3794d48bafc14ffc6ddc7a3d9cb128f 100644 (file)
@@ -9,8 +9,7 @@ obj-$(CONFIG_BOOT_RETRY) += bootretry.o
 obj-$(CONFIG_CMD_BOOTM) += bootm.o bootm_os.o
 obj-$(CONFIG_CMD_BOOTZ) += bootm.o bootm_os.o
 obj-$(CONFIG_CMD_BOOTI) += bootm.o bootm_os.o
-obj-$(CONFIG_SUPPORT_EXTENSION_SCAN) += extension.o
-obj-$(CONFIG_SUPPORT_DM_EXTENSION_SCAN) += extension-uclass.o
+obj-$(CONFIG_SUPPORT_EXTENSION_SCAN) += extension-uclass.o
 
 obj-$(CONFIG_PXE_UTILS) += pxe_utils.o
 
index 914971215af6b53f5f84f48fe357038826866ddf..35f751237ebf77ffbf340ec2889cc44b566dc51c 100644 (file)
@@ -14,7 +14,7 @@
 #include <dm/lists.h>
 #include <dm/uclass.h>
 
-struct alist *dm_extension_get_list(void)
+struct alist *extension_get_list(void)
 {
        struct udevice *dev;
 
@@ -24,7 +24,7 @@ struct alist *dm_extension_get_list(void)
        return dev_get_priv(dev);
 }
 
-int dm_extension_probe(struct udevice *dev)
+int extension_probe(struct udevice *dev)
 {
        struct alist *extension_list = dev_get_priv(dev);
 
@@ -32,7 +32,7 @@ int dm_extension_probe(struct udevice *dev)
        return 0;
 }
 
-int dm_extension_remove(struct udevice *dev)
+int extension_remove(struct udevice *dev)
 {
        struct alist *extension_list = dev_get_priv(dev);
 
@@ -40,9 +40,9 @@ int dm_extension_remove(struct udevice *dev)
        return 0;
 }
 
-int dm_extension_scan(void)
+int extension_scan(void)
 {
-       struct alist *extension_list = dm_extension_get_list();
+       struct alist *extension_list = extension_get_list();
        const struct extension_ops *ops;
        struct udevice *dev;
        int ret;
@@ -109,9 +109,9 @@ static int _extension_apply(const struct extension *extension)
        return ret;
 }
 
-int dm_extension_apply(int extension_num)
+int extension_apply(int extension_num)
 {
-       struct alist *extension_list = dm_extension_get_list();
+       struct alist *extension_list = extension_get_list();
        const struct extension *extension;
 
        if (!extension_list)
@@ -127,9 +127,9 @@ int dm_extension_apply(int extension_num)
        return _extension_apply(extension);
 }
 
-int dm_extension_apply_all(void)
+int extension_apply_all(void)
 {
-       struct alist *extension_list = dm_extension_get_list();
+       struct alist *extension_list = extension_get_list();
        const struct extension *extension;
        int ret;
 
diff --git a/boot/extension.c b/boot/extension.c
deleted file mode 100644 (file)
index 8f28a7e..0000000
+++ /dev/null
@@ -1,99 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0+
-/*
- * (C) Copyright 2025 Köry Maincent <kory.maincent@bootlin.com>
- */
-
-#include <bootdev.h>
-#include <command.h>
-#include <env.h>
-#include <extension_board.h>
-#include <fdt_support.h>
-#include <malloc.h>
-#include <mapmem.h>
-
-LIST_HEAD(extension_list);
-
-int extension_apply(struct extension *extension)
-{
-       ulong extrasize, overlay_addr;
-       struct fdt_header *blob;
-       char *overlay_cmd;
-
-       if (!working_fdt) {
-               printf("No FDT memory address configured. Please configure\n"
-                      "the FDT address via \"fdt addr <address>\" command.\n");
-               return CMD_RET_FAILURE;
-       }
-
-       overlay_cmd = env_get("extension_overlay_cmd");
-       if (!overlay_cmd) {
-               printf("Environment extension_overlay_cmd is missing\n");
-               return CMD_RET_FAILURE;
-       }
-
-       overlay_addr = env_get_hex("extension_overlay_addr", 0);
-       if (!overlay_addr) {
-               printf("Environment extension_overlay_addr is missing\n");
-               return CMD_RET_FAILURE;
-       }
-
-       env_set("extension_overlay_name", extension->overlay);
-       if (run_command(overlay_cmd, 0) != 0)
-               return CMD_RET_FAILURE;
-
-       extrasize = env_get_hex("filesize", 0);
-       if (!extrasize)
-               return CMD_RET_FAILURE;
-
-       fdt_shrink_to_minimum(working_fdt, extrasize);
-
-       blob = map_sysmem(overlay_addr, 0);
-       if (!fdt_valid(&blob))
-               return CMD_RET_FAILURE;
-
-       /* apply method prints messages on error */
-       if (fdt_overlay_apply_verbose(working_fdt, blob))
-               return CMD_RET_FAILURE;
-
-       return CMD_RET_SUCCESS;
-}
-
-int extension_scan(bool show)
-{
-       struct extension *extension, *next;
-       int extension_num;
-
-       list_for_each_entry_safe(extension, next, &extension_list, list) {
-               list_del(&extension->list);
-               free(extension);
-       }
-       extension_num = extension_board_scan(&extension_list);
-       if (show && extension_num >= 0)
-               printf("Found %d extension board(s).\n", extension_num);
-
-       /* either the number of extensions, or -ve for error */
-       return extension_num;
-}
-
-static int extension_bootdev_hunt(struct bootdev_hunter *info, bool show)
-{
-       int ret;
-
-       ret = env_set_hex("extension_overlay_addr",
-                         env_get_hex("fdtoverlay_addr_r", 0));
-       if (ret)
-               return log_msg_ret("env", ret);
-
-       ret = extension_scan(show);
-       if (ret < 0)
-               return log_msg_ret("ext", ret);
-
-       return 0;
-}
-
-/* extensions should have a uclass - for now we use UCLASS_SIMPLE_BUS uclass */
-BOOTDEV_HUNTER(extension_bootdev_hunter) = {
-       .prio           = BOOTDEVP_1_PRE_SCAN,
-       .uclass         = UCLASS_SIMPLE_BUS,
-       .hunt           = extension_bootdev_hunt,
-};
index 721bdb87c8a18863eb4cc1f08210aaab59fb2de6..986eeeba807575d689712c879df9e82ef1593b09 100644 (file)
@@ -548,7 +548,7 @@ config CMD_FDT
 config CMD_EXTENSION
        bool "Extension board management command"
        select CMD_FDT
-       depends on SUPPORT_EXTENSION_SCAN || SUPPORT_DM_EXTENSION_SCAN
+       depends on SUPPORT_EXTENSION_SCAN
        help
          Enables the "extension" command, which allows to detect
          extension boards connected to the system, and apply
index 1f1eddbe976832d7180c22a230830ce1773da46d..c373397e0fb3ea776bf2688b96482bf0fbbcc606 100644 (file)
 static int do_extension_list(struct cmd_tbl *cmdtp, int flag,
                             int argc, char *const argv[])
 {
-#if CONFIG_IS_ENABLED(SUPPORT_DM_EXTENSION_SCAN)
-       struct alist *dm_extension_list;
-#endif
+       struct alist *extension_list;
        struct extension *extension;
        int i = 0;
 
-#if CONFIG_IS_ENABLED(SUPPORT_DM_EXTENSION_SCAN)
-       dm_extension_list = dm_extension_get_list();
-
-       if (!alist_get_ptr(dm_extension_list, 0)) {
-               printf("No extension registered - Please run \"extension scan\"\n");
-               return CMD_RET_SUCCESS;
-       }
-
-       alist_for_each(extension, dm_extension_list) {
-               printf("Extension %d: %s\n", i++, extension->name);
-               printf("\tManufacturer: \t\t%s\n", extension->owner);
-               printf("\tVersion: \t\t%s\n", extension->version);
-               printf("\tDevicetree overlay: \t%s\n", extension->overlay);
-               printf("\tOther information: \t%s\n", extension->other);
-       }
-#else
-       if (list_empty(&extension_list)) {
+       extension_list = extension_get_list();
+       if (!alist_get_ptr(extension_list, 0)) {
                printf("No extension registered - Please run \"extension scan\"\n");
                return CMD_RET_SUCCESS;
        }
 
-       list_for_each_entry(extension, &extension_list, list) {
+       alist_for_each(extension, extension_list) {
                printf("Extension %d: %s\n", i++, extension->name);
                printf("\tManufacturer: \t\t%s\n", extension->owner);
                printf("\tVersion: \t\t%s\n", extension->version);
                printf("\tDevicetree overlay: \t%s\n", extension->overlay);
                printf("\tOther information: \t%s\n", extension->other);
        }
-#endif
        return CMD_RET_SUCCESS;
 }
 
@@ -55,69 +37,31 @@ static int do_extension_scan(struct cmd_tbl *cmdtp, int flag,
 {
        int extension_num;
 
-#if CONFIG_IS_ENABLED(SUPPORT_DM_EXTENSION_SCAN)
-       extension_num = dm_extension_scan();
+       extension_num = extension_scan();
        if (extension_num == -ENODEV)
                extension_num = 0;
        else if (extension_num < 0)
                return CMD_RET_FAILURE;
 
        printf("Found %d extension board(s).\n", extension_num);
-#else
-       extension_num = extension_scan(true);
-       if (extension_num < 0 && extension_num != -ENODEV)
-               return CMD_RET_FAILURE;
-#endif
-
        return CMD_RET_SUCCESS;
 }
 
 static int do_extension_apply(struct cmd_tbl *cmdtp, int flag,
                              int argc, char *const argv[])
 {
-#if !CONFIG_IS_ENABLED(SUPPORT_DM_EXTENSION_SCAN)
-       struct extension *extension = NULL;
-       struct list_head *entry;
-       int i = 0;
-#endif
-       int extension_id, ret;
+       int extension_id;
 
        if (argc < 2)
                return CMD_RET_USAGE;
 
        if (strcmp(argv[1], "all") == 0) {
-               ret = CMD_RET_FAILURE;
-#if CONFIG_IS_ENABLED(SUPPORT_DM_EXTENSION_SCAN)
-               if (dm_extension_apply_all())
+               if (extension_apply_all())
                        return CMD_RET_FAILURE;
-#else
-               list_for_each_entry(extension, &extension_list, list) {
-                       ret = extension_apply(extension);
-                       if (ret != CMD_RET_SUCCESS)
-                               break;
-               }
-#endif
        } else {
                extension_id = simple_strtol(argv[1], NULL, 10);
-#if CONFIG_IS_ENABLED(SUPPORT_DM_EXTENSION_SCAN)
-               if (dm_extension_apply(extension_id))
+               if (extension_apply(extension_id))
                        return CMD_RET_FAILURE;
-#else
-               list_for_each(entry, &extension_list) {
-                       if (i == extension_id) {
-                               extension = list_entry(entry, struct extension,  list);
-                               break;
-                       }
-                       i++;
-               }
-
-               if (!extension) {
-                       printf("Wrong extension number\n");
-                       return CMD_RET_FAILURE;
-               }
-
-               ret = extension_apply(extension);
-#endif
        }
 
        return CMD_RET_SUCCESS;
index a74a9c67c01bd55e6c21e6988224dc44e3c535f2..451e8ed832d1b1994fe0779693d9bb7a3ce20bbe 100644 (file)
 extern struct list_head extension_list;
 
 /**
- * dm_extension_get_list - Get the extension list
+ * extension_get_list - Get the extension list
  * Return: The extension alist pointer, or NULL if no such list exists.
  *
  * The caller must not free the list.
  */
-struct alist *dm_extension_get_list(void);
+struct alist *extension_get_list(void);
 
 /**
- * dm_extension_probe - Probe extension device
+ * extension_probe - Probe extension device
  * @dev: Extension device that needs to be probed
  * Return: Zero on success, negative on failure.
  */
-int dm_extension_probe(struct udevice *dev);
+int extension_probe(struct udevice *dev);
 
 /**
- * dm_extension_remove - Remove extension device
+ * extension_remove - Remove extension device
  * @dev: Extension device that needs to be removed
  * Return: Zero on success, negative on failure.
  */
-int dm_extension_remove(struct udevice *dev);
+int extension_remove(struct udevice *dev);
 
 /**
- * dm_extension_scan - Scan extension boards available.
+ * extension_scan - Scan extension boards available.
  * Return: Zero on success, negative on failure.
  */
-int dm_extension_scan(void);
+int extension_scan(void);
 
 /**
- * dm_extension_apply - Apply extension board overlay to the devicetree
+ * extension_apply - Apply extension board overlay to the devicetree
  * @extension_num: Extension number to be applied
  * Return: Zero on success, negative on failure.
  */
-int dm_extension_apply(int extension_num);
+int extension_apply(int extension_num);
 
 /**
- * dm_extension_apply_all - Apply all extension board overlays to the
+ * extension_apply_all - Apply all extension board overlays to the
  *                         devicetree
  * Return: Zero on success, negative on failure.
  */
-int dm_extension_apply_all(void);
+int extension_apply_all(void);
 
 /**
  * extension - Description fields of an extension board
- * @list: List head
  * @name: Name of the extension
  * @owner: Owner of the extension
  * @version: Version of the extension
@@ -66,7 +65,6 @@ int dm_extension_apply_all(void);
  * @other: Other information of this extension
  */
 struct extension {
-       struct list_head list;
        char name[32];
        char owner[32];
        char version[32];
@@ -92,38 +90,12 @@ struct extension_ops {
        U_BOOT_DRIVER(_name) = { \
                .name = #_name, \
                .id = UCLASS_EXTENSION, \
-               .probe = dm_extension_probe, \
-               .remove = dm_extension_remove, \
+               .probe = extension_probe, \
+               .remove = extension_remove, \
                .ops = &(struct extension_ops) { \
                       .scan = _scan_func, \
                       }, \
                .priv_auto = sizeof(struct alist), \
        }
 
-/**
- * extension_board_scan - Add system-specific function to scan extension board.
- * @param extension_list       List of extension board information to update.
- * Return: the number of extension.
- *
- * This function is called if CONFIG_CMD_EXTENSION is defined.
- * Needs to fill the list extension_list with elements.
- * Each element need to be allocated to an extension structure.
- *
- */
-int extension_board_scan(struct list_head *extension_list);
-
-/**
- * extension_apply - Apply extension board overlay to the devicetree
- * @extension: Extension to be applied
- * Return: Zero on success, negative on failure.
- */
-int extension_apply(struct extension *extension);
-
-/**
- * extension_scan - Scan extension boards available.
- * @show: Flag to enable verbose log
- * Return: Zero on success, negative on failure.
- */
-int extension_scan(bool show);
-
 #endif /* __EXTENSION_SUPPORT_H */
index a5f3d4462a94a9d0e8dcca25a38546b6a36ea9a4..0820bf10ee083da69b5349c5dddc51947c47b9fd 100644 (file)
@@ -380,7 +380,6 @@ static int bootdev_test_hunter(struct unit_test_state *uts)
        ut_assert_nextline("Prio  Used  Uclass           Hunter");
        ut_assert_nextlinen("----");
        ut_assert_nextline("   6        ethernet         eth_bootdev");
-       ut_assert_nextline("   1        simple_bus       (none)");
        ut_assert_nextline("   5        ide              ide_bootdev");
        ut_assert_nextline("   2        mmc              mmc_bootdev");
        ut_assert_nextline("   4        nvme             nvme_bootdev");
@@ -389,15 +388,15 @@ static int bootdev_test_hunter(struct unit_test_state *uts)
        ut_assert_nextline("   4        spi_flash        sf_bootdev");
        ut_assert_nextline("   5        usb              usb_bootdev");
        ut_assert_nextline("   4        virtio           virtio_bootdev");
-       ut_assert_nextline("(total hunters: 10)");
+       ut_assert_nextline("(total hunters: 9)");
        ut_assert_console_end();
 
        ut_assertok(bootdev_hunt("usb1", false));
        ut_assert_skip_to_line("Bus usb@1: 5 USB Device(s) found");
        ut_assert_console_end();
 
-       /* USB is 7th in the list, so bit 8 */
-       ut_asserteq(BIT(8), std->hunters_used);
+       /* USB is 8th in the list, so bit 7 */
+       ut_asserteq(BIT(7), std->hunters_used);
 
        return 0;
 }
@@ -418,7 +417,7 @@ static int bootdev_test_cmd_hunt(struct unit_test_state *uts)
        ut_assert_nextline("Prio  Used  Uclass           Hunter");
        ut_assert_nextlinen("----");
        ut_assert_nextline("   6        ethernet         eth_bootdev");
-       ut_assert_skip_to_line("(total hunters: 10)");
+       ut_assert_skip_to_line("(total hunters: 9)");
        ut_assert_console_end();
 
        /* Use the MMC hunter and see that it updates */
@@ -426,7 +425,7 @@ static int bootdev_test_cmd_hunt(struct unit_test_state *uts)
        ut_assertok(run_command("bootdev hunt -l", 0));
        ut_assert_skip_to_line("   5        ide              ide_bootdev");
        ut_assert_nextline("   2     *  mmc              mmc_bootdev");
-       ut_assert_skip_to_line("(total hunters: 10)");
+       ut_assert_skip_to_line("(total hunters: 9)");
        ut_assert_console_end();
 
        /* Scan all hunters */
@@ -436,8 +435,6 @@ static int bootdev_test_cmd_hunt(struct unit_test_state *uts)
        ut_assert_nextline("Hunting with: ethernet");
 
        /* This is the extension feature which has no uclass at present */
-       ut_assert_nextline("Hunting with: simple_bus");
-       ut_assert_nextline("Found 2 extension board(s).");
        ut_assert_nextline("Hunting with: ide");
 
        /* mmc hunter has already been used so should not run again */
@@ -457,7 +454,6 @@ static int bootdev_test_cmd_hunt(struct unit_test_state *uts)
        ut_assert_nextlinen("Prio");
        ut_assert_nextlinen("----");
        ut_assert_nextline("   6     *  ethernet         eth_bootdev");
-       ut_assert_nextline("   1     *  simple_bus       (none)");
        ut_assert_nextline("   5     *  ide              ide_bootdev");
        ut_assert_nextline("   2     *  mmc              mmc_bootdev");
        ut_assert_nextline("   4     *  nvme             nvme_bootdev");
@@ -466,7 +462,7 @@ static int bootdev_test_cmd_hunt(struct unit_test_state *uts)
        ut_assert_nextline("   4     *  spi_flash        sf_bootdev");
        ut_assert_nextline("   5     *  usb              usb_bootdev");
        ut_assert_nextline("   4     *  virtio           virtio_bootdev");
-       ut_assert_nextline("(total hunters: 10)");
+       ut_assert_nextline("(total hunters: 9)");
        ut_assert_console_end();
 
        ut_asserteq(GENMASK(MAX_HUNTER, 0), std->hunters_used);
@@ -490,7 +486,7 @@ static int bootdev_test_hunt_scan(struct unit_test_state *uts)
        ut_assertok(bootflow_scan_first(NULL, NULL, &iter,
                                        BOOTFLOWIF_SHOW | BOOTFLOWIF_HUNT |
                                        BOOTFLOWIF_SKIP_GLOBAL, &bflow));
-       ut_asserteq(BIT(MMC_HUNTER) | BIT(1), std->hunters_used);
+       ut_asserteq(BIT(MMC_HUNTER), std->hunters_used);
 
        return 0;
 }
@@ -650,8 +646,8 @@ static int bootdev_test_next_label(struct unit_test_state *uts)
        ut_asserteq_str("scsi.id0lun0.bootdev", dev->name);
        ut_asserteq(BOOTFLOW_METHF_SINGLE_UCLASS, mflags);
 
-       /* SCSI is 7th in the list, so bit 6 */
-       ut_asserteq(BIT(MMC_HUNTER) | BIT(6), std->hunters_used);
+       /* SCSI is 6th in the list, so bit 5 */
+       ut_asserteq(BIT(MMC_HUNTER) | BIT(5), std->hunters_used);
 
        ut_assertok(bootdev_next_label(&iter, &dev, &mflags));
        ut_assert_console_end();
@@ -661,7 +657,7 @@ static int bootdev_test_next_label(struct unit_test_state *uts)
                    mflags);
 
        /* dhcp: Ethernet is first so bit 0 */
-       ut_asserteq(BIT(MMC_HUNTER) | BIT(6) | BIT(0), std->hunters_used);
+       ut_asserteq(BIT(MMC_HUNTER) | BIT(5) | BIT(0), std->hunters_used);
 
        ut_assertok(bootdev_next_label(&iter, &dev, &mflags));
        ut_assert_console_end();
@@ -671,7 +667,7 @@ static int bootdev_test_next_label(struct unit_test_state *uts)
                    mflags);
 
        /* pxe: Ethernet is first so bit 0 */
-       ut_asserteq(BIT(MMC_HUNTER) | BIT(6) | BIT(0), std->hunters_used);
+       ut_asserteq(BIT(MMC_HUNTER) | BIT(5) | BIT(0), std->hunters_used);
 
        mflags = 123;
        ut_asserteq(-ENODEV, bootdev_next_label(&iter, &dev, &mflags));
@@ -679,7 +675,7 @@ static int bootdev_test_next_label(struct unit_test_state *uts)
        ut_assert_console_end();
 
        /* no change */
-       ut_asserteq(BIT(MMC_HUNTER) | BIT(6) | BIT(0), std->hunters_used);
+       ut_asserteq(BIT(MMC_HUNTER) | BIT(5) | BIT(0), std->hunters_used);
 
        return 0;
 }
@@ -724,12 +720,10 @@ static int bootdev_test_next_prio(struct unit_test_state *uts)
 
        ut_assertok(bootdev_next_prio(&iter, &dev));
        ut_asserteq_str("mmc2.bootdev", dev->name);
-       ut_assert_nextline("Hunting with: simple_bus");
-       ut_assert_nextline("Found 2 extension board(s).");
        ut_assert_nextline("Hunting with: mmc");
        ut_assert_console_end();
 
-       ut_asserteq(BIT(MMC_HUNTER) | BIT(1), std->hunters_used);
+       ut_asserteq(BIT(MMC_HUNTER), std->hunters_used);
 
        ut_assertok(bootdev_next_prio(&iter, &dev));
        ut_asserteq_str("mmc1.bootdev", dev->name);
index 8de5a310add38798f3a52ea6cd796bdffdb16bd6..e6a2537c15d5cc331f397ba23c78091bed807d74 100644 (file)
@@ -799,7 +799,7 @@ static int bootflow_cmd_hunt_single(struct unit_test_state *uts)
        ut_assert_console_end();
 
        /* check that the hunter was used */
-       ut_asserteq(BIT(MMC_HUNTER) | BIT(1), std->hunters_used);
+       ut_asserteq(BIT(MMC_HUNTER), std->hunters_used);
 
        return 0;
 }
@@ -821,14 +821,12 @@ static int bootflow_cmd_hunt_label(struct unit_test_state *uts)
        ut_assertok(run_command("bootflow scan -l mmc", 0));
 
        /* check that the hunter was used */
-       ut_asserteq(BIT(MMC_HUNTER) | BIT(1), std->hunters_used);
+       ut_asserteq(BIT(MMC_HUNTER), std->hunters_used);
 
        /* check that we got the mmc1 bootflow */
        ut_assert_nextline("Scanning for bootflows with label 'mmc'");
        ut_assert_nextlinen("Seq");
        ut_assert_nextlinen("---");
-       ut_assert_nextline("Hunting with: simple_bus");
-       ut_assert_nextline("Found 2 extension board(s).");
        ut_assert_nextline("Hunting with: mmc");
        ut_assert_nextline("Scanning bootdev 'mmc2.bootdev':");
        ut_assert_nextline("Scanning bootdev 'mmc1.bootdev':");
index c61698adc02827e35df1fc15e328242f8c0fe857..dd769313a84295d00a4504bd3016c2ca1fe01313 100644 (file)
@@ -21,8 +21,8 @@
 #define TEST_VERNUM            0x00010002
 
 enum {
-       MAX_HUNTER      = 9,
-       MMC_HUNTER      = 3,    /* ID of MMC hunter */
+       MAX_HUNTER      = 8,
+       MMC_HUNTER      = 2,    /* ID of MMC hunter */
 };
 
 struct unit_test_state;