From: Simon Glass <sjg@chromium.org>
Date: Sat, 30 Jul 2022 21:52:29 +0000 (-0600)
Subject: bootstd: Drop the system bootdev
X-Git-Url: http://git.dujemihanovic.xyz/?a=commitdiff_plain;h=2ff5490d7dee933eaf0b73d4d50d76660e5da6ff;p=u-boot.git

bootstd: Drop the system bootdev

This was a work-around for the fact that global bootmeths such as EFI
bootmgr and VBE don't use a particular bootdev, or at least select it
themselves so that we don't need to scan all bootdevs when using that
bootmeth.

Drop the system bootdev entirely.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

diff --git a/boot/Makefile b/boot/Makefile
index 124065a03f..854b3913b2 100644
--- a/boot/Makefile
+++ b/boot/Makefile
@@ -18,7 +18,7 @@ obj-y += image.o image-board.o
 obj-$(CONFIG_ANDROID_AB) += android_ab.o
 obj-$(CONFIG_ANDROID_BOOT_IMAGE) += image-android.o image-android-dt.o
 
-obj-$(CONFIG_$(SPL_TPL_)BOOTSTD) += bootdev-uclass.o system_bootdev.o
+obj-$(CONFIG_$(SPL_TPL_)BOOTSTD) += bootdev-uclass.o
 obj-$(CONFIG_$(SPL_TPL_)BOOTSTD) += bootflow.o
 obj-$(CONFIG_$(SPL_TPL_)BOOTSTD) += bootmeth-uclass.o
 obj-$(CONFIG_$(SPL_TPL_)BOOTSTD) += bootstd-uclass.o
diff --git a/boot/bootstd-uclass.c b/boot/bootstd-uclass.c
index 5107b6d4c7..565c22a36e 100644
--- a/boot/bootstd-uclass.c
+++ b/boot/bootstd-uclass.c
@@ -145,12 +145,6 @@ int dm_scan_other(bool pre_reloc_only)
 		}
 	}
 
-	/* Create the system bootdev too */
-	ret = device_bind_driver(bootstd, "system_bootdev", "system-bootdev",
-				 &dev);
-	if (ret)
-		return log_msg_ret("sys", ret);
-
 	return 0;
 }
 
diff --git a/boot/system_bootdev.c b/boot/system_bootdev.c
deleted file mode 100644
index 432d203478..0000000000
--- a/boot/system_bootdev.c
+++ /dev/null
@@ -1,66 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0+
-/*
- * Bootdevice for system, used for bootmeths not tied to any partition device
- *
- * Copyright 2021 Google LLC
- * Written by Simon Glass <sjg@chromium.org>
- */
-
-#define LOG_CATEGORY UCLASS_BOOTSTD
-
-#include <common.h>
-#include <bootdev.h>
-#include <bootflow.h>
-#include <bootmeth.h>
-#include <command.h>
-#include <distro.h>
-#include <dm.h>
-#include <log.h>
-#include <net.h>
-
-static int system_get_bootflow(struct udevice *dev, struct bootflow_iter *iter,
-			       struct bootflow *bflow)
-{
-	int ret;
-
-	/* Must be an bootstd device */
-	ret = bootflow_iter_uses_system(iter);
-	if (ret)
-		return log_msg_ret("net", ret);
-
-	ret = bootmeth_check(bflow->method, iter);
-	if (ret)
-		return log_msg_ret("check", ret);
-
-	ret = bootmeth_read_bootflow(bflow->method, bflow);
-	if (ret)
-		return log_msg_ret("method", ret);
-
-	return 0;
-}
-
-static int system_bootdev_bind(struct udevice *dev)
-{
-	struct bootdev_uc_plat *ucp = dev_get_uclass_plat(dev);
-
-	ucp->prio = BOOTDEVP_6_SYSTEM;
-
-	return 0;
-}
-
-struct bootdev_ops system_bootdev_ops = {
-	.get_bootflow	= system_get_bootflow,
-};
-
-static const struct udevice_id system_bootdev_ids[] = {
-	{ .compatible = "u-boot,bootdev-system" },
-	{ }
-};
-
-U_BOOT_DRIVER(system_bootdev) = {
-	.name		= "system_bootdev",
-	.id		= UCLASS_BOOTDEV,
-	.ops		= &system_bootdev_ops,
-	.bind		= system_bootdev_bind,
-	.of_match	= system_bootdev_ids,
-};
diff --git a/test/boot/bootflow.c b/test/boot/bootflow.c
index b2f77972d1..39c2c33dea 100644
--- a/test/boot/bootflow.c
+++ b/test/boot/bootflow.c
@@ -12,7 +12,6 @@
 #include <bootstd.h>
 #include <dm.h>
 #include <asm/test.h>
-#include <dm/device-internal.h>
 #include <dm/lists.h>
 #include <test/suites.h>
 #include <test/ut.h>
@@ -337,7 +336,7 @@ static int bootflow_system(struct unit_test_state *uts)
 	ut_assert_skip_to_line(
 		"  0  efi_mgr      ready   (none)       0  <NULL>                    <NULL>");
 	ut_assert_skip_to_line("No more bootdevs");
-	ut_assert_skip_to_line("(6 bootflows, 6 valid)");
+	ut_assert_skip_to_line("(5 bootflows, 5 valid)");
 	ut_assert_console_end();
 
 	return 0;
@@ -358,10 +357,6 @@ static int bootflow_iter_disable(struct unit_test_state *uts)
 	ut_assertok(device_bind_driver(bootstd, "bootmeth_sandbox", "sandbox",
 				       &dev));
 
-	/* Add the system bootdev that it uses */
-	ut_assertok(device_bind_driver(bootstd, "system_bootdev",
-				       "system-bootdev", &dev));
-
 	ut_assertok(bootstd_test_drop_bootdev_order(uts));
 
 	bootstd_clear_glob();