On 3/16/24 22:50, Janne Grunau via B4 Relay wrote:
From: Andre Przywara <andre.przyw...@arm.com>

UEFI relies entirely on unicode output, which actual fonts displayed on
the screen might not be ready for.

Add a test displaying some international characters, to reveal missing
glyphs, especially in our builtin fonts.
This would be needed to be manually checked on the screen for
correctness.

Signed-off-by: Andre Przywara <andre.przyw...@arm.com>
Suggested-by: Heinrich Schuchardt <xypron.g...@gmx.de>
Signed-off-by: Janne Grunau <j...@jannau.net>
---
  lib/efi_selftest/efi_selftest_textoutput.c | 21 +++++++++++++++++++++
  1 file changed, 21 insertions(+)

diff --git a/lib/efi_selftest/efi_selftest_textoutput.c 
b/lib/efi_selftest/efi_selftest_textoutput.c
index cc44b38bc2..917903473d 100644
--- a/lib/efi_selftest/efi_selftest_textoutput.c
+++ b/lib/efi_selftest/efi_selftest_textoutput.c
@@ -31,6 +31,21 @@ static int execute(void)
                0xD804, 0xDC22,
                0};

+       const u16 text[] =
+u"This should render international characters as described\n"
+u"U+00D6 \u00D6 - Latin capital letter O with diaresis\n"
+u"U+00DF \u00DF - Latin small letter sharp s\n"
+u"U+00E5 \u00E5 - Latin small letter a with ring above\n"
+u"U+00E9 \u00E9 - Latin small letter e with acute\n"
+u"U+00F1 \u00F1 - Latin small letter n with tilde\n"
+u"U+00F6 \u00F6 - Latin small letter o with diaresis\n"
+u"The following characters will render as '?' with bitmap fonts\n"
+u"U+00F8 \u00F8 - Latin small letter o with stroke\n"
+u"U+03AC \u03AC - Greek small letter alpha with tonus\n"
+u"U+03BB \u03BB - Greek small letter lambda\n"
+u"U+03C2 \u03C2 - Greek small letter final sigma\n"
+u"U+1F19 \u1F19 - Greek capital letter epsilon with dasia\n";

The strings should be indented by two tabs.

Otherwise:

Reviewed-by: Heinrich Schuchardt <xypron.g...@gmx.de>

+
        /* SetAttribute */
        efi_st_printf("\nColor palette\n");
        for (foreground = 0; foreground < 0x10; ++foreground) {
@@ -119,6 +134,12 @@ static int execute(void)
                return EFI_ST_FAILURE;
        }
        efi_st_printf("\n");
+       ret = con_out->output_string(con_out, text);
+       if (ret != EFI_ST_SUCCESS) {
+               efi_st_error("OutputString failed for international chars\n");
+               return EFI_ST_FAILURE;
+       }
+       efi_st_printf("\n");

        return EFI_ST_SUCCESS;
  }


Reply via email to