Re: Pull request for efi-2022-07-rc4-4

2022-06-05 Thread Tom Rini
On Sat, Jun 04, 2022 at 02:43:13PM +0200, Heinrich Schuchardt wrote:

> Dear Tom,
> 
> The following changes since commit 90189ecd59cdf14afbe6014be5c068e599b65a72:
> 
>   Merge branch '2022-06-03-assorted-fixes' (2022-06-03 15:42:13 -0400)
> 
> are available in the Git repository at:
> 
>   https://source.denx.de/u-boot/custodians/u-boot-efi.git
> tags/efi-2022-07-rc4-4
> 
> for you to fetch changes up to 8645aefc8b699f900d97cbadd55b7f492099c61e:
> 
>   efi: test/py: authenticate fit capsules (2022-06-04 08:43:55 +0200)
> 
> 
> Gitlab CI showed no issues:
> https://source.denx.de/u-boot/custodians/u-boot-efi/-/pipelines/12196
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature


Pull request for efi-2022-07-rc4-4

2022-06-04 Thread Heinrich Schuchardt

Dear Tom,

The following changes since commit 90189ecd59cdf14afbe6014be5c068e599b65a72:

  Merge branch '2022-06-03-assorted-fixes' (2022-06-03 15:42:13 -0400)

are available in the Git repository at:

  https://source.denx.de/u-boot/custodians/u-boot-efi.git
tags/efi-2022-07-rc4-4

for you to fetch changes up to 8645aefc8b699f900d97cbadd55b7f492099c61e:

  efi: test/py: authenticate fit capsules (2022-06-04 08:43:55 +0200)


Gitlab CI showed no issues:
https://source.denx.de/u-boot/custodians/u-boot-efi/-/pipelines/12196


Pull request for efi-2022-07-rc4-4

UEFI:

* Fix the implementation of the firmware management protocol
* Fix the unit tests for signed update capsules


Chris Packham (1):
  doc: environment: Fix typo

Sughosh Ganu (2):
  EFI: Populate descriptor_count value only when image_info_size is
not zero
  EFI: Update the documentation to reflect the correct value of
OsIndications

Vincent Stehlé (2):
  test/py: efi_capsule: repair image authentication test
  efi: test/py: authenticate fit capsules

 doc/develop/uefi/uefi.rst
  |   2 +-
 doc/usage/environment.rst
  |   2 +-
 lib/efi_loader/efi_firmware.c
  | 120 ++--
 test/py/tests/test_efi_capsule/conftest.py
  |  25 +++-
 test/py/tests/test_efi_capsule/test_capsule_firmware_signed_fit.py
  | 257 ++
 ...st_capsule_firmware_signed.py =>
test_capsule_firmware_signed_raw.py} |  14 +-
 6 files changed, 362 insertions(+), 58 deletions(-)
 create mode 100644
test/py/tests/test_efi_capsule/test_capsule_firmware_signed_fit.py
 rename test/py/tests/test_efi_capsule/{test_capsule_firmware_signed.py
=> test_capsule_firmware_signed_raw.py} (96%)


Re: Pull request for efi-2022-07-rc4-3

2022-05-31 Thread Tom Rini
On Tue, May 31, 2022 at 06:57:18PM +0200, Heinrich Schuchardt wrote:

> Dear Tom,
> 
> this replaces pull request for efi-2022-07-rc4-2.
> 
> The following changes since commit 4fe629d2e8bbda5f265e870d771b92be26e885f6:
> 
>   Merge https://source.denx.de/u-boot/custodians/u-boot-riscv
> (2022-05-27 08:50:42 -0400)
> 
> are available in the Git repository at:
> 
>   https://source.denx.de/u-boot/custodians/u-boot-efi.git
> tags/efi-2022-07-rc4-3
> 
> for you to fetch changes up to e85727ddb68b3b23382ea2d78d857ffa870bb0b7:
> 
>   bootmenu: use utf-8 for menu title (2022-05-31 07:57:02 +0200)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature


Re: Pull request for efi-2022-07-rc4-3

2022-05-31 Thread Pali Rohár
On Tuesday 31 May 2022 18:57:18 Heinrich Schuchardt wrote:
> Dear Tom,
> 
> this replaces pull request for efi-2022-07-rc4-2.
> 
> The following changes since commit 4fe629d2e8bbda5f265e870d771b92be26e885f6:
> 
>   Merge https://source.denx.de/u-boot/custodians/u-boot-riscv
> (2022-05-27 08:50:42 -0400)
> 
> are available in the Git repository at:
> 
>   https://source.denx.de/u-boot/custodians/u-boot-efi.git
> tags/efi-2022-07-rc4-3
> 
> for you to fetch changes up to e85727ddb68b3b23382ea2d78d857ffa870bb0b7:
> 
>   bootmenu: use utf-8 for menu title (2022-05-31 07:57:02 +0200)

Tested-by: Pali Rohár 

Bootmenu on n900 now works correctly.

  *** U-Boot Boot Menu ***

  Attached kernel
  Internal eMMC
  External SD card
  U-Boot boot order
  U-Boot console


  Press UP/DOWN to move, ENTER to select, ESC/CTRL+C to quit

> --------------------
> Pull request for efi-2022-07-rc4-3
> 
> UEFI:
> 
> * fix a problem in loading an image from a short-path
> * fix building the bootmenu command for CONFIG_EFI_LOADER=n
> * correct the bootefi command syntax
> * add firmware management protocol to the documentation
> 
> Others:
> 
> * bootmenu: fix bootmenu title handling
> 
> 
> AKASHI Takahiro (2):
>   efi_loader: disk: add efi_disk_is_removable()
>   efi_loader: bootmgr: fix a problem in loading an image from a
> short-path
> 
> Bin Meng (2):
>   doc: sandbox: Correct the memory size config option
>   doc: sandbox: Add a note of disabling LTO when using gdb
> 
> Heinrich Schuchardt (6):
>   disk: incorrect message in is_gpt_valid()
>   test: restore timeout after bootmenu unit test
>   doc/build/gcc: add more required packages
>   test: fix pylint warnings in test_efi_selftest.py
>   test: don't change console timeout in EFI selftest.
>   cmd/bootefi: correct command syntax
> 
> Masahisa Kojima (3):
>   bootmenu: U-Boot console is enabled as default
>   lib/charset: fix compile warnings
>   bootmenu: use utf-8 for menu title
> 
> Sean Anderson (2):
>   valgrind: Disable on Risc-V
>   doc: sandbox: Add additional valgrind documentation
> 
> Vincent Stehlé (2):
>   efi: fix documentation warnings
>   doc/efi: add firmware management protocol to the documentation
> 
>  Kconfig|   1 +
>  boot/Kconfig   |   7 +++
>  cmd/Kconfig|  10 
>  cmd/bootefi.c  |  21 +---
>  cmd/bootmenu.c |  47 
>  disk/part_efi.c|   4 +-
>  doc/api/efi.rst|   6 +++
>  doc/arch/sandbox.rst   | 107
> +
>  doc/build/gcc.rst  |   8 +--
>  doc/usage/cmd/bootefi.rst  |   8 +--
>  include/charset.h  |   2 +-
>  include/efi_loader.h   |   3 ++
>  lib/efi_loader/efi_bootmgr.c   | 107
> +++--
>  lib/efi_loader/efi_disk.c  |  27 ++
>  lib/efi_loader/efi_firmware.c  |  10 ++--
>  test/py/tests/test_bootmenu.py |  66 +++
>  test/py/tests/test_efi_selftest.py |  91 ++-
>  17 files changed, 359 insertions(+), 166 deletions(-)


Pull request for efi-2022-07-rc4-3

2022-05-31 Thread Heinrich Schuchardt

Dear Tom,

this replaces pull request for efi-2022-07-rc4-2.

The following changes since commit 4fe629d2e8bbda5f265e870d771b92be26e885f6:

  Merge https://source.denx.de/u-boot/custodians/u-boot-riscv
(2022-05-27 08:50:42 -0400)

are available in the Git repository at:

  https://source.denx.de/u-boot/custodians/u-boot-efi.git
tags/efi-2022-07-rc4-3

for you to fetch changes up to e85727ddb68b3b23382ea2d78d857ffa870bb0b7:

  bootmenu: use utf-8 for menu title (2022-05-31 07:57:02 +0200)


Pull request for efi-2022-07-rc4-3

UEFI:

* fix a problem in loading an image from a short-path
* fix building the bootmenu command for CONFIG_EFI_LOADER=n
* correct the bootefi command syntax
* add firmware management protocol to the documentation

Others:

* bootmenu: fix bootmenu title handling


AKASHI Takahiro (2):
  efi_loader: disk: add efi_disk_is_removable()
  efi_loader: bootmgr: fix a problem in loading an image from a
short-path

Bin Meng (2):
  doc: sandbox: Correct the memory size config option
  doc: sandbox: Add a note of disabling LTO when using gdb

Heinrich Schuchardt (6):
  disk: incorrect message in is_gpt_valid()
  test: restore timeout after bootmenu unit test
  doc/build/gcc: add more required packages
  test: fix pylint warnings in test_efi_selftest.py
  test: don't change console timeout in EFI selftest.
  cmd/bootefi: correct command syntax

Masahisa Kojima (3):
  bootmenu: U-Boot console is enabled as default
  lib/charset: fix compile warnings
  bootmenu: use utf-8 for menu title

Sean Anderson (2):
  valgrind: Disable on Risc-V
  doc: sandbox: Add additional valgrind documentation

Vincent Stehlé (2):
  efi: fix documentation warnings
  doc/efi: add firmware management protocol to the documentation

 Kconfig|   1 +
 boot/Kconfig   |   7 +++
 cmd/Kconfig|  10 
 cmd/bootefi.c  |  21 +---
 cmd/bootmenu.c |  47 
 disk/part_efi.c|   4 +-
 doc/api/efi.rst|   6 +++
 doc/arch/sandbox.rst   | 107
+
 doc/build/gcc.rst  |   8 +--
 doc/usage/cmd/bootefi.rst  |   8 +--
 include/charset.h  |   2 +-
 include/efi_loader.h   |   3 ++
 lib/efi_loader/efi_bootmgr.c   | 107
+++--
 lib/efi_loader/efi_disk.c  |  27 ++
 lib/efi_loader/efi_firmware.c  |  10 ++--
 test/py/tests/test_bootmenu.py |  66 +++
 test/py/tests/test_efi_selftest.py |  91 ++-
 17 files changed, 359 insertions(+), 166 deletions(-)


Re: Pull request for efi-2022-07-rc4-2

2022-05-31 Thread Tom Rini
On Mon, May 30, 2022 at 10:56:13AM +0200, Pali Rohár wrote:
> On Monday 30 May 2022 09:32:53 Masahisa Kojima wrote:
> > Hi Heinrich,
> > 
> > On Mon, 30 May 2022 at 01:16, Heinrich Schuchardt  
> > wrote:
> > >
> > > Dear Tom,
> > >
> > > this replaces pull request for efi-2022-07-rc4.
> > >
> > > The following changes since commit 
> > > 4fe629d2e8bbda5f265e870d771b92be26e885f6:
> > >
> > >Merge https://source.denx.de/u-boot/custodians/u-boot-riscv
> > > (2022-05-27 08:50:42 -0400)
> > >
> > > are available in the Git repository at:
> > >
> > >https://source.denx.de/u-boot/custodians/u-boot-efi.git
> > > tags/efi-2022-07-rc4-2
> > >
> > > for you to fetch changes up to de8471a30eb399d46412b2037ba76cfa258c6115:
> > >
> > >bootmenu: use utf-8 for menu title (2022-05-29 13:18:18 +0200)
> > >
> > > Gitlab CI showed no issues:
> > > https://source.denx.de/u-boot/custodians/u-boot-efi/-/pipelines/12157
> > >
> > > 
> > > Pull request for efi-2022-07-rc4-2
> > >
> > > UEFI:
> > >
> > > * fix a problem in loading an image from a short-path
> > > * fix building the bootmenu command for CONFIG_EFI_LOADER=n
> > > * correct the bootefi command syntax
> > > * add firmware management protocol to the documentation
> > >
> > > Others:
> > >
> > > * bootmenu: fix bootmenu title handling
> > >
> > > 
> > > AKASHI Takahiro (2):
> > >efi_loader: disk: add efi_disk_is_removable()
> > >efi_loader: bootmgr: fix a problem in loading an image from a
> > > short-path
> > >
> > > Bin Meng (2):
> > >doc: sandbox: Correct the memory size config option
> > >doc: sandbox: Add a note of disabling LTO when using gdb
> > >
> > > Heinrich Schuchardt (6):
> > >disk: incorrect message in is_gpt_valid()
> > >test: restore timeout after bootmenu unit test
> > >doc/build/gcc: add more required packages
> > >test: fix pylint warnings in test_efi_selftest.py
> > >test: don't change console timeout in EFI selftest.
> > >cmd/bootefi: correct command syntax
> > >
> > > Masahisa Kojima (3):
> > >bootmenu: U-Boot console is enabled as default
> > >lib/charset: fix compile warnings
> > >bootmenu: use utf-8 for menu title
> > 
> > With your fix to the last patch, the bootmenu output
> > is as below.
> > 
> >   *** U-Boot Boot Menu ***
> > 
> >   Boot 1. kernel=bootm 0x8200
> >   Boot 2. kernel=bootm 0x8300
> >   Reset board=reset
> >   U-Boot console
> > 
> >   Press UP/DOWN to move, ENTER to select, ESC/CTRL+C to quit
> > 
> > The menu title includes both the title and command of "bootmenu_x".
> > Is it intended?
> 
> Hello! This is really not intended.
> 
> Issue is in commit:
> https://source.denx.de/u-boot/custodians/u-boot-efi/-/commit/de8471a30eb399d46412b2037ba76cfa258c6115
> 
> There is line:
> 184 + entry->title = strdup(option);
> 
> But it rather should be:
> 184 + entry->title = strndup(option, sep-option);

Thanks Pali.  Waiting for a re-spin.

-- 
Tom


signature.asc
Description: PGP signature


Re: Pull request for efi-2022-07-rc4-2

2022-05-30 Thread Pali Rohár
On Monday 30 May 2022 09:32:53 Masahisa Kojima wrote:
> Hi Heinrich,
> 
> On Mon, 30 May 2022 at 01:16, Heinrich Schuchardt  wrote:
> >
> > Dear Tom,
> >
> > this replaces pull request for efi-2022-07-rc4.
> >
> > The following changes since commit 4fe629d2e8bbda5f265e870d771b92be26e885f6:
> >
> >Merge https://source.denx.de/u-boot/custodians/u-boot-riscv
> > (2022-05-27 08:50:42 -0400)
> >
> > are available in the Git repository at:
> >
> >https://source.denx.de/u-boot/custodians/u-boot-efi.git
> > tags/efi-2022-07-rc4-2
> >
> > for you to fetch changes up to de8471a30eb399d46412b2037ba76cfa258c6115:
> >
> >bootmenu: use utf-8 for menu title (2022-05-29 13:18:18 +0200)
> >
> > Gitlab CI showed no issues:
> > https://source.denx.de/u-boot/custodians/u-boot-efi/-/pipelines/12157
> >
> > 
> > Pull request for efi-2022-07-rc4-2
> >
> > UEFI:
> >
> > * fix a problem in loading an image from a short-path
> > * fix building the bootmenu command for CONFIG_EFI_LOADER=n
> > * correct the bootefi command syntax
> > * add firmware management protocol to the documentation
> >
> > Others:
> >
> > * bootmenu: fix bootmenu title handling
> >
> > 
> > AKASHI Takahiro (2):
> >efi_loader: disk: add efi_disk_is_removable()
> >efi_loader: bootmgr: fix a problem in loading an image from a
> > short-path
> >
> > Bin Meng (2):
> >doc: sandbox: Correct the memory size config option
> >doc: sandbox: Add a note of disabling LTO when using gdb
> >
> > Heinrich Schuchardt (6):
> >disk: incorrect message in is_gpt_valid()
> >test: restore timeout after bootmenu unit test
> >doc/build/gcc: add more required packages
> >test: fix pylint warnings in test_efi_selftest.py
> >test: don't change console timeout in EFI selftest.
> >cmd/bootefi: correct command syntax
> >
> > Masahisa Kojima (3):
> >bootmenu: U-Boot console is enabled as default
> >lib/charset: fix compile warnings
> >bootmenu: use utf-8 for menu title
> 
> With your fix to the last patch, the bootmenu output
> is as below.
> 
>   *** U-Boot Boot Menu ***
> 
>   Boot 1. kernel=bootm 0x8200
>   Boot 2. kernel=bootm 0x8300
>   Reset board=reset
>   U-Boot console
> 
>   Press UP/DOWN to move, ENTER to select, ESC/CTRL+C to quit
> 
> The menu title includes both the title and command of "bootmenu_x".
> Is it intended?

Hello! This is really not intended.

Issue is in commit:
https://source.denx.de/u-boot/custodians/u-boot-efi/-/commit/de8471a30eb399d46412b2037ba76cfa258c6115

There is line:
184 +   entry->title = strdup(option);

But it rather should be:
184 +   entry->title = strndup(option, sep-option);

> Thanks,
> Masahias Kojima
> 
> >
> > Sean Anderson (2):
> >valgrind: Disable on Risc-V
> >doc: sandbox: Add additional valgrind documentation
> >
> > Vincent Stehlé (2):
> >efi: fix documentation warnings
> >doc/efi: add firmware management protocol to the documentation
> >
> >   Kconfig|   1 +
> >   boot/Kconfig   |   7 +++
> >   cmd/Kconfig|  10 
> >   cmd/bootefi.c  |  21 +---
> >   cmd/bootmenu.c |  47 
> >   disk/part_efi.c|   4 +-
> >   doc/api/efi.rst|   6 +++
> >   doc/arch/sandbox.rst   | 107
> > +
> >   doc/build/gcc.rst  |   8 +--
> >   doc/usage/cmd/bootefi.rst  |   8 +--
> >   include/charset.h  |   2 +-
> >   include/efi_loader.h   |   3 ++
> >   lib/efi_loader/efi_bootmgr.c   | 107
> > +++--
> >   lib/efi_loader/efi_disk.c  |  27 ++
> >   lib/efi_loader/efi_firmware.c  |  10 ++--
> >   test/py/tests/test_bootmenu.py |  66 +++
> >   test/py/tests/test_efi_selftest.py |  91 ++-
> >   17 files changed, 359 insertions(+), 166 deletions(-)


Re: Pull request for efi-2022-07-rc4-2

2022-05-29 Thread Masahisa Kojima
Hi Heinrich,

On Mon, 30 May 2022 at 01:16, Heinrich Schuchardt  wrote:
>
> Dear Tom,
>
> this replaces pull request for efi-2022-07-rc4.
>
> The following changes since commit 4fe629d2e8bbda5f265e870d771b92be26e885f6:
>
>Merge https://source.denx.de/u-boot/custodians/u-boot-riscv
> (2022-05-27 08:50:42 -0400)
>
> are available in the Git repository at:
>
>https://source.denx.de/u-boot/custodians/u-boot-efi.git
> tags/efi-2022-07-rc4-2
>
> for you to fetch changes up to de8471a30eb399d46412b2037ba76cfa258c6115:
>
>bootmenu: use utf-8 for menu title (2022-05-29 13:18:18 +0200)
>
> Gitlab CI showed no issues:
> https://source.denx.de/u-boot/custodians/u-boot-efi/-/pipelines/12157
>
> --------------------
> Pull request for efi-2022-07-rc4-2
>
> UEFI:
>
> * fix a problem in loading an image from a short-path
> * fix building the bootmenu command for CONFIG_EFI_LOADER=n
> * correct the bootefi command syntax
> * add firmware management protocol to the documentation
>
> Others:
>
> * bootmenu: fix bootmenu title handling
>
> 
> AKASHI Takahiro (2):
>efi_loader: disk: add efi_disk_is_removable()
>efi_loader: bootmgr: fix a problem in loading an image from a
> short-path
>
> Bin Meng (2):
>doc: sandbox: Correct the memory size config option
>doc: sandbox: Add a note of disabling LTO when using gdb
>
> Heinrich Schuchardt (6):
>disk: incorrect message in is_gpt_valid()
>test: restore timeout after bootmenu unit test
>doc/build/gcc: add more required packages
>test: fix pylint warnings in test_efi_selftest.py
>test: don't change console timeout in EFI selftest.
>cmd/bootefi: correct command syntax
>
> Masahisa Kojima (3):
>bootmenu: U-Boot console is enabled as default
>lib/charset: fix compile warnings
>bootmenu: use utf-8 for menu title

With your fix to the last patch, the bootmenu output
is as below.

  *** U-Boot Boot Menu ***

  Boot 1. kernel=bootm 0x8200
  Boot 2. kernel=bootm 0x8300
  Reset board=reset
  U-Boot console

  Press UP/DOWN to move, ENTER to select, ESC/CTRL+C to quit

The menu title includes both the title and command of "bootmenu_x".
Is it intended?

Thanks,
Masahias Kojima

>
> Sean Anderson (2):
>valgrind: Disable on Risc-V
>doc: sandbox: Add additional valgrind documentation
>
> Vincent Stehlé (2):
>efi: fix documentation warnings
>doc/efi: add firmware management protocol to the documentation
>
>   Kconfig|   1 +
>   boot/Kconfig   |   7 +++
>   cmd/Kconfig|  10 
>   cmd/bootefi.c  |  21 +---
>   cmd/bootmenu.c |  47 
>   disk/part_efi.c|   4 +-
>   doc/api/efi.rst|   6 +++
>   doc/arch/sandbox.rst   | 107
> +
>   doc/build/gcc.rst  |   8 +--
>   doc/usage/cmd/bootefi.rst  |   8 +--
>   include/charset.h  |   2 +-
>   include/efi_loader.h   |   3 ++
>   lib/efi_loader/efi_bootmgr.c   | 107
> +++--
>   lib/efi_loader/efi_disk.c  |  27 ++
>   lib/efi_loader/efi_firmware.c  |  10 ++--
>   test/py/tests/test_bootmenu.py |  66 +++
>   test/py/tests/test_efi_selftest.py |  91 ++-
>   17 files changed, 359 insertions(+), 166 deletions(-)


Pull request for efi-2022-07-rc4-2

2022-05-29 Thread Heinrich Schuchardt

Dear Tom,

this replaces pull request for efi-2022-07-rc4.

The following changes since commit 4fe629d2e8bbda5f265e870d771b92be26e885f6:

  Merge https://source.denx.de/u-boot/custodians/u-boot-riscv
(2022-05-27 08:50:42 -0400)

are available in the Git repository at:

  https://source.denx.de/u-boot/custodians/u-boot-efi.git
tags/efi-2022-07-rc4-2

for you to fetch changes up to de8471a30eb399d46412b2037ba76cfa258c6115:

  bootmenu: use utf-8 for menu title (2022-05-29 13:18:18 +0200)

Gitlab CI showed no issues:
https://source.denx.de/u-boot/custodians/u-boot-efi/-/pipelines/12157


Pull request for efi-2022-07-rc4-2

UEFI:

* fix a problem in loading an image from a short-path
* fix building the bootmenu command for CONFIG_EFI_LOADER=n
* correct the bootefi command syntax
* add firmware management protocol to the documentation

Others:

* bootmenu: fix bootmenu title handling


AKASHI Takahiro (2):
  efi_loader: disk: add efi_disk_is_removable()
  efi_loader: bootmgr: fix a problem in loading an image from a
short-path

Bin Meng (2):
  doc: sandbox: Correct the memory size config option
  doc: sandbox: Add a note of disabling LTO when using gdb

Heinrich Schuchardt (6):
  disk: incorrect message in is_gpt_valid()
  test: restore timeout after bootmenu unit test
  doc/build/gcc: add more required packages
  test: fix pylint warnings in test_efi_selftest.py
  test: don't change console timeout in EFI selftest.
  cmd/bootefi: correct command syntax

Masahisa Kojima (3):
  bootmenu: U-Boot console is enabled as default
  lib/charset: fix compile warnings
  bootmenu: use utf-8 for menu title

Sean Anderson (2):
  valgrind: Disable on Risc-V
  doc: sandbox: Add additional valgrind documentation

Vincent Stehlé (2):
  efi: fix documentation warnings
  doc/efi: add firmware management protocol to the documentation

 Kconfig|   1 +
 boot/Kconfig   |   7 +++
 cmd/Kconfig|  10 
 cmd/bootefi.c  |  21 +---
 cmd/bootmenu.c |  47 
 disk/part_efi.c|   4 +-
 doc/api/efi.rst|   6 +++
 doc/arch/sandbox.rst   | 107
+
 doc/build/gcc.rst  |   8 +--
 doc/usage/cmd/bootefi.rst  |   8 +--
 include/charset.h  |   2 +-
 include/efi_loader.h   |   3 ++
 lib/efi_loader/efi_bootmgr.c   | 107
+++--
 lib/efi_loader/efi_disk.c  |  27 ++
 lib/efi_loader/efi_firmware.c  |  10 ++--
 test/py/tests/test_bootmenu.py |  66 +++
 test/py/tests/test_efi_selftest.py |  91 ++-
 17 files changed, 359 insertions(+), 166 deletions(-)


Re: Pull request for efi-2022-07-rc4

2022-05-26 Thread Tom Rini
On Thu, May 26, 2022 at 07:13:01PM +0900, Masahisa Kojima wrote:
> Hi Heinrich, Tom,
> 
> On Thu, 26 May 2022 at 18:56, Heinrich Schuchardt  wrote:
> >
> > Dear Tom,
> >
> > The following changes since commit 7e0edcadb09d55d5319fdc862041fd1b874476f5:
> >
> >Merge branch 'master' of
> > https://source.denx.de/u-boot/custodians/u-boot-sunxi (2022-05-24
> > 23:29:00 -0400)
> >
> > are available in the Git repository at:
> >
> >https://source.denx.de/u-boot/custodians/u-boot-efi.git
> > tags/efi-2022-07-rc4
> >
> > for you to fetch changes up to 019ba3a6a644cceba46d92d42a1ed0677885a929:
> >
> >doc/build/gcc: add more required packages (2022-05-26 10:15:58 +0200)
> >
> > Gitlab CI showed no problems:
> > https://source.denx.de/u-boot/custodians/u-boot-efi/-/pipelines/12127
> >
> > 
> > Pull request for efi-2022-07-rc4
> >
> > UEFI:
> >
> > * fix a problem in loading an image from a short-path
> > * fix building the bootmenu command for CONFIG_EFI_LOADER=n
> > * correct the bootefi command syntax
> > * add firmware management protocol to the documentation
> >
> > Others:
> >
> > * bootmenu: fix bootmenu title handling
> >
> > 
> > AKASHI Takahiro (2):
> >efi_loader: disk: add efi_disk_is_removable()
> >efi_loader: bootmgr: fix a problem in loading an image from a
> > short-path
> >
> > Bin Meng (2):
> >doc: sandbox: Correct the memory size config option
> >doc: sandbox: Add a note of disabling LTO when using gdb
> >
> > Heinrich Schuchardt (6):
> >disk: incorrect message in is_gpt_valid()
> >test: restore timeout after bootmenu unit test
> >test: fix pylint warnings in test_efi_selftest.py
> >test: don't change console timeout in EFI selftest.
> >cmd/bootefi: correct command syntax
> >doc/build/gcc: add more required packages
> >
> > Masahisa Kojima (2):
> >bootmenu: fix bootmenu title handling
> 
> I have sent the revised version of the patch.
> Could you ignore "bootmenu: fix bootmenu title handling"
> and apply the revised patch here?
> https://patchwork.ozlabs.org/project/uboot/patch/20220526100938.9558-2-masahisa.koj...@linaro.org/

I'll wait for an updated PR, thanks.

-- 
Tom


signature.asc
Description: PGP signature


Re: Pull request for efi-2022-07-rc4

2022-05-26 Thread Masahisa Kojima
Hi Heinrich, Tom,

On Thu, 26 May 2022 at 18:56, Heinrich Schuchardt  wrote:
>
> Dear Tom,
>
> The following changes since commit 7e0edcadb09d55d5319fdc862041fd1b874476f5:
>
>Merge branch 'master' of
> https://source.denx.de/u-boot/custodians/u-boot-sunxi (2022-05-24
> 23:29:00 -0400)
>
> are available in the Git repository at:
>
>https://source.denx.de/u-boot/custodians/u-boot-efi.git
> tags/efi-2022-07-rc4
>
> for you to fetch changes up to 019ba3a6a644cceba46d92d42a1ed0677885a929:
>
>doc/build/gcc: add more required packages (2022-05-26 10:15:58 +0200)
>
> Gitlab CI showed no problems:
> https://source.denx.de/u-boot/custodians/u-boot-efi/-/pipelines/12127
>
> --------------------
> Pull request for efi-2022-07-rc4
>
> UEFI:
>
> * fix a problem in loading an image from a short-path
> * fix building the bootmenu command for CONFIG_EFI_LOADER=n
> * correct the bootefi command syntax
> * add firmware management protocol to the documentation
>
> Others:
>
> * bootmenu: fix bootmenu title handling
>
> 
> AKASHI Takahiro (2):
>efi_loader: disk: add efi_disk_is_removable()
>efi_loader: bootmgr: fix a problem in loading an image from a
> short-path
>
> Bin Meng (2):
>doc: sandbox: Correct the memory size config option
>doc: sandbox: Add a note of disabling LTO when using gdb
>
> Heinrich Schuchardt (6):
>disk: incorrect message in is_gpt_valid()
>test: restore timeout after bootmenu unit test
>test: fix pylint warnings in test_efi_selftest.py
>test: don't change console timeout in EFI selftest.
>cmd/bootefi: correct command syntax
>doc/build/gcc: add more required packages
>
> Masahisa Kojima (2):
>bootmenu: fix bootmenu title handling

I have sent the revised version of the patch.
Could you ignore "bootmenu: fix bootmenu title handling"
and apply the revised patch here?
https://patchwork.ozlabs.org/project/uboot/patch/20220526100938.9558-2-masahisa.koj...@linaro.org/

Thanks,
Masahisa Kojima

>lib/charset: fix compile warnings
>
> Vincent Stehlé (2):
>efi: fix documentation warnings
>doc/efi: add firmware management protocol to the documentation
>
>   cmd/bootefi.c  |  21 +---
>   cmd/bootmenu.c |  48 +++--
>   disk/part_efi.c|   4 +-
>   doc/api/efi.rst|   6 +++
>   doc/arch/sandbox.rst   |   5 +-
>   doc/build/gcc.rst  |   8 +--
>   doc/usage/cmd/bootefi.rst  |   8 +--
>   include/charset.h  |   2 +-
>   include/efi_loader.h   |   3 ++
>   lib/efi_loader/efi_bootmgr.c   | 107
> +++--
>   lib/efi_loader/efi_disk.c  |  27 ++
>   lib/efi_loader/efi_firmware.c  |  10 ++--
>   test/py/tests/test_bootmenu.py |  66 +++
>   test/py/tests/test_efi_selftest.py |  91 ++-
>   14 files changed, 266 insertions(+), 140 deletions(-)


Pull request for efi-2022-07-rc4

2022-05-26 Thread Heinrich Schuchardt

Dear Tom,

The following changes since commit 7e0edcadb09d55d5319fdc862041fd1b874476f5:

  Merge branch 'master' of
https://source.denx.de/u-boot/custodians/u-boot-sunxi (2022-05-24
23:29:00 -0400)

are available in the Git repository at:

  https://source.denx.de/u-boot/custodians/u-boot-efi.git
tags/efi-2022-07-rc4

for you to fetch changes up to 019ba3a6a644cceba46d92d42a1ed0677885a929:

  doc/build/gcc: add more required packages (2022-05-26 10:15:58 +0200)

Gitlab CI showed no problems:
https://source.denx.de/u-boot/custodians/u-boot-efi/-/pipelines/12127


Pull request for efi-2022-07-rc4

UEFI:

* fix a problem in loading an image from a short-path
* fix building the bootmenu command for CONFIG_EFI_LOADER=n
* correct the bootefi command syntax
* add firmware management protocol to the documentation

Others:

* bootmenu: fix bootmenu title handling


AKASHI Takahiro (2):
  efi_loader: disk: add efi_disk_is_removable()
  efi_loader: bootmgr: fix a problem in loading an image from a
short-path

Bin Meng (2):
  doc: sandbox: Correct the memory size config option
  doc: sandbox: Add a note of disabling LTO when using gdb

Heinrich Schuchardt (6):
  disk: incorrect message in is_gpt_valid()
  test: restore timeout after bootmenu unit test
  test: fix pylint warnings in test_efi_selftest.py
  test: don't change console timeout in EFI selftest.
  cmd/bootefi: correct command syntax
  doc/build/gcc: add more required packages

Masahisa Kojima (2):
  bootmenu: fix bootmenu title handling
  lib/charset: fix compile warnings

Vincent Stehlé (2):
  efi: fix documentation warnings
  doc/efi: add firmware management protocol to the documentation

 cmd/bootefi.c  |  21 +---
 cmd/bootmenu.c |  48 +++--
 disk/part_efi.c|   4 +-
 doc/api/efi.rst|   6 +++
 doc/arch/sandbox.rst   |   5 +-
 doc/build/gcc.rst  |   8 +--
 doc/usage/cmd/bootefi.rst  |   8 +--
 include/charset.h  |   2 +-
 include/efi_loader.h   |   3 ++
 lib/efi_loader/efi_bootmgr.c   | 107
+++--
 lib/efi_loader/efi_disk.c  |  27 ++
 lib/efi_loader/efi_firmware.c  |  10 ++--
 test/py/tests/test_bootmenu.py |  66 +++
 test/py/tests/test_efi_selftest.py |  91 ++-
 14 files changed, 266 insertions(+), 140 deletions(-)