[RFC 11/13] fs: remove explicit efi configuration dependency

[Heinrich Schuchardt]

Am 26. Oktober 2023 07:30:50 MESZ schrieb AKASHI Takahiro <takahiro.akashi at linaro.org>:
>Now it is clear that the feature actually depends on efi interfaces,
>not "bootefi" command. efi_set_bootdev() will automatically be nullified
>if necessary efi component is disabled.
>
>Signed-off-by: AKASHI Takahiro <takahiro.akashi at linaro.org>
>---
> fs/fs.c | 7 +++----
> 1 file changed, 3 insertions(+), 4 deletions(-)
>
>diff --git a/fs/fs.c b/fs/fs.c
>index 4cb4310c9cc2..70cdb594c4c8 100644
>--- a/fs/fs.c
>+++ b/fs/fs.c
>@@ -791,10 +791,9 @@ int do_load(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[],
> 		return 1;
> 	}
> 
>-	if (IS_ENABLED(CONFIG_CMD_BOOTEFI))
>-		efi_set_bootdev(argv[1], (argc > 2) ? argv[2] : "",
>-				(argc > 4) ? argv[4] : "", map_sysmem(addr, 0),
>-				len_read);
>+	efi_set_bootdev(argv[1], (argc > 2) ? argv[2] : "",

This function should not exist for CONFIG_EFI_LOADER=n. There are other places where the function is invoked. Please, review all of them.

Best regards

Heinrich


>+			(argc > 4) ? argv[4] : "", map_sysmem(addr, 0),
>+			len_read);
> 
> 	printf("%llu bytes read in %lu ms", len_read, time);
> 	if (time > 0) {