[PATCH RFC 0/6] video: Add UTF-8 support for UEFI applications

[Andre Przywara]

On Wed, 17 Jan 2024 23:24:26 +0100
Janne Grunau via B4 Relay <devnull+j.jannau.net at kernel.org> wrote:

Hi Janne,

> Andre submitted 2 years ago DM_VIDEO improvements for UEFI applications
> using UEFI's EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL but did not follow with
> suggested changes. This series takes care of the UTF-8 support which
> required to draw symbol and box drawing characters used by UEFI
> applications like grub2 and sd-boot correctly.
> 
> Compared to Andre's version this version has the following changes:
> - use and extend existing conversion functions from lib/charset.c
> - convert first to UTF-32 to support the truetype console as well
> - conversion is conditional on CONFIG_EFI_LOADER
> - use escape sequences in tests as proposed by Heinrich

many thanks for picking this up, much appreciated!
In general this looks good, I will test this later this week, and have a
closer look, once I am on a device with video output.

Cheers,
Andre

> 
> Link: https://lore.kernel.org/u-boot/20220110005638.21599-1-andre.przywara@arm.com/
> Signed-off-by: Janne Grunau <j at jannau.net>
> ---
> Andre Przywara (2):
>       efi_selftest: Add international characters test
>       efi_selftest: Add box drawing character selftest
> 
> Janne Grunau (4):
>       lib: charset: Fix utf8_to_utf32_stream() return value doc string
>       video: console: Parse UTF-8 character sequences
>       lib/charset: Map cp437 low chars (0x01 - 0x1f) from unicode
>       efi_selftest: Add symbol character selftest
> 
>  drivers/video/console_normal.c             |  6 +++--
>  drivers/video/console_rotate.c             | 16 +++++++++-----
>  drivers/video/console_truetype.c           |  8 +++----
>  drivers/video/vidconsole-uclass.c          | 18 ++++++++++-----
>  drivers/video/vidconsole_internal.h        | 15 +++++++++++++
>  include/charset.h                          |  4 ++--
>  include/cp1250.h                           | 12 ++++++++--
>  include/cp437.h                            | 12 ++++++++--
>  include/video_console.h                    | 10 +++++----
>  lib/charset.c                              |  9 +++++---
>  lib/efi_loader/efi_unicode_collation.c     |  2 +-
>  lib/efi_selftest/efi_selftest_textoutput.c | 35 ++++++++++++++++++++++++++++++
>  12 files changed, 116 insertions(+), 31 deletions(-)
> ---
> base-commit: 866ca972d6c3cabeaf6dbac431e8e08bb30b3c8e
> change-id: 20240117-vidconsole-utf8-uefi-fa23b4ac65d6
> 
> Best regards,