U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: "Vincent Stehlé" <vincent.stehle@arm.com>
To: Heinrich Schuchardt <xypron.glpk@gmx.de>
Cc: u-boot@lists.denx.de, Ilias Apalodimas <ilias.apalodimas@linaro.org>
Subject: Re: [PATCH 0/2] efi: fix and test hii keyboard layout pointer computation
Date: Mon, 18 May 2026 11:48:27 +0200	[thread overview]
Message-ID: <agrga7jakYcRlzEK@debian> (raw)
In-Reply-To: <B11344DB-FD58-478D-A9BD-5081C48116D3@gmx.de>

On Tue, May 12, 2026 at 09:05:29PM +0200, Heinrich Schuchardt wrote:
> Am 12. Mai 2026 19:40:32 MESZ schrieb "Vincent Stehlé" <vincent.stehle@arm.com>:
> >- Fix one scaled pointer arithmetic bug in EFI Loader HII implementation.
> >- Enhance the existing EFI HII unit test to catch the bug.
> >
> >The unit test can run on the sandbox with the following command:
> >
> >  ./u-boot -T -c "setenv efi_selftest HII database protocols; \
> >		  bootefi selftest"
> >
> >Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>
> >---
> >Vincent Stehlé (2):
> >      efi_loader: fix hii keyboard layout pointer computation
> >      efi_selftest: test hii keyboard layouts more
> >
> > lib/efi_loader/efi_hii.c                 |  3 +-
> > lib/efi_selftest/efi_selftest_hii.c      | 67 ++++++++++++++++++--------------
> > lib/efi_selftest/efi_selftest_hii_data.c | 12 ++++++
> > 3 files changed, 51 insertions(+), 31 deletions(-)
> >---
> >base-commit: 5732bd0f457b4c671e46574d64d4acb099c0f0a5
> >change-id: 20260512-layout-pointer-29d1cd52c070
> >
> >Best regards,
> 
> Hello Vincent,
> 
> Off the list:
> 
> Did you find any real use for HII protocols in U-Boot except for running the SCT?
> 
> Are HII based EFI applications a thing?

Hi Heinrich,

A fair question indeed, as the HII protocols do not seem to be used that much in
practice.

The UEFI Shell seems to be using them, though, which should be a sufficient
reason to support them, I think.

Those protocols are used as well a little in systemd-boot and iPXE for example,
but as far as I can tell, they are not used at all in Shim, GRUB, Linux, Xen,
the BSDs or Android GBL.

Best regards,
Vincent.

> 
> Best regards
> 
> Heinrich

  reply	other threads:[~2026-05-18  9:48 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-05-12 17:40 [PATCH 0/2] efi: fix and test hii keyboard layout pointer computation Vincent Stehlé
2026-05-12 17:40 ` [PATCH 1/2] efi_loader: fix " Vincent Stehlé
2026-05-12 17:40 ` [PATCH 2/2] efi_selftest: test hii keyboard layouts more Vincent Stehlé
2026-05-12 19:05 ` [PATCH 0/2] efi: fix and test hii keyboard layout pointer computation Heinrich Schuchardt
2026-05-18  9:48   ` Vincent Stehlé [this message]
2026-05-18 10:51     ` Michael Brown

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=agrga7jakYcRlzEK@debian \
    --to=vincent.stehle@arm.com \
    --cc=ilias.apalodimas@linaro.org \
    --cc=u-boot@lists.denx.de \
    --cc=xypron.glpk@gmx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox