Hi Sughosh,
On Tue, 1 Jul 2025 at 13:43, Sughosh Ganu <sughosh.g...@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 > >
