[PATCH v4 0/2] eficonfig: add support for URI based boot option

[Ilias Apalodimas]

Hi Sughosh,


On Tue, 1 Jul 2025 at 13:43, Sughosh Ganu <sughosh.ganu at linaro.org> wrote:
>
>
> Add support for adding URI based boot option through the eficonfig
> command. Also add a basic test case for checking validity of the URI's
> being passed to the wget function.
>
> Changes since V3:
> * Do away with boot_file_option member of struct
>   eficonfig_boot_option - patch 1.
> * Use the uri pointer being non-null in
>   eficonfig_process_show_file_option() for selecting menu items -
>   patch 1.
> * Put a check for lo.file_path in eficonfig_edit_boot_option() before
>   calling fill_file_info() - patch 1

Overall this works. I can add an HTTP option and boot it. However,
there's a difference from your v1/v2. The URI is not visible anymore
when you try to edit that option. Am I missing anything?

Thanks
/Ilias
>
>
> Sughosh Ganu (2):
>   cmd: eficonfig: add support for URI device path based boot options
>   test: wget: add a test case for validating URI
>
>  cmd/eficonfig.c      | 129 ++++++++++++++++++++++++++++++++++++++++---
>  include/efi_config.h |   2 +
>  test/cmd/wget.c      |  20 +++++++
>  3 files changed, 143 insertions(+), 8 deletions(-)
>
> --
> 2.34.1
>
>