[PATCH v2 32/41] bootstd: Add a system bootdev for strange boot methods

From: Simon Glass <sjg@chromium.org>
To: U-Boot Mailing List <u-boot@lists.denx.de>
Cc: "Michal Simek" <michal.simek@xilinx.com>,
	"Heinrich Schuchardt" <xypron.glpk@gmx.de>,
	"Tom Rini" <trini@konsulko.com>,
	"Ilias Apalodimas" <ilias.apalodimas@linaro.org>,
	"Daniel Schwierzeck" <daniel.schwierzeck@gmail.com>,
	"Steffen Jaeckel" <jaeckel-floss@eyet-services.de>,
	"Marek Behún" <marek.behun@nic.cz>,
	"Lukas Auer" <lukas.auer@aisec.fraunhofer.de>,
	"Dennis Gilmore" <dennis@ausil.us>,
	"Simon Glass" <sjg@chromium.org>
Subject: [PATCH v2 32/41] bootstd: Add a system bootdev for strange boot methods
Date: Sat, 23 Oct 2021 17:26:26 -0600	[thread overview]
Message-ID: <20211023232635.9195-23-sjg@chromium.org> (raw)
In-Reply-To: <20211023232635.9195-1-sjg@chromium.org>

Some boot methods don't act on a single bootdev but instead do their own
thing. An example is EFI bootmgr which scan various devices using its own
logic. Add a bootdev to handle this.

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

(no changes since v1)

 boot/Makefile         |  2 +-
 boot/bootstd-uclass.c |  6 ++++
 boot/system_bootdev.c | 64 +++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 71 insertions(+), 1 deletion(-)
 create mode 100644 boot/system_bootdev.c

diff --git a/boot/Makefile b/boot/Makefile
index c2345435201..795665f7ce5 100644
--- a/boot/Makefile
+++ b/boot/Makefile
@@ -23,7 +23,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
+obj-$(CONFIG_$(SPL_TPL_)BOOTSTD) += bootdev-uclass.o system_bootdev.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 7dddd72bc4a..3952560eee5 100644
--- a/boot/bootstd-uclass.c
+++ b/boot/bootstd-uclass.c
@@ -143,6 +143,12 @@ 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
new file mode 100644
index 00000000000..2cf7cd4dc11
--- /dev/null
+++ b/boot/system_bootdev.c
@@ -0,0 +1,64 @@
+// 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>
+ */
+
+#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,
+};
-- 
2.33.0.1079.g6e70778dc9-goog

