diff options
| author | Heinrich Schuchardt <heinrich.schuchardt@canonical.com> | 2023-02-10 08:13:23 +0100 |
|---|---|---|
| committer | Heinrich Schuchardt <heinrich.schuchardt@canonical.com> | 2023-02-10 13:05:40 +0100 |
| commit | 4db17a4b12524d0ec2dc30913dbbf44f968ce8e0 (patch) | |
| tree | 80c88b96abb85cf9e8be214b9b6a3124f2662d34 /include | |
| parent | 7d840627ca38d775107dc9480f4d14ce236f95ce (diff) | |
efi_loader: fix struct efi_input_key
The UEFI specification defines filed UnicodeChar as CHAR16. We use
u16 for CHAR16 throughout our code. The change fixes the following errors:
lib/efi_loader/initrddump.c: In function ‘efi_input’:
lib/efi_loader/initrddump.c:218:38: warning:
comparison is always false due to limited range of data type
[-Wtype-limits]
218 | if (key.unicode_char >= 0xD800 && key.unicode_char <= 0xDBFF)
| ^~
lib/efi_loader/initrddump.c:218:68: warning:
comparison is always true due to limited range of data type
[-Wtype-limits]
218 | if (key.unicode_char >= 0xD800 && key.unicode_char <= 0xDBFF)
| ^~
Fixes: 867a6ac86dd8 ("efi: Add start-up library code")
Reported-by: Marek Vasut <marex@denx.de>
Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
Diffstat (limited to 'include')
| -rw-r--r-- | include/efi_api.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/efi_api.h b/include/efi_api.h index 9bd70b0f18c..e1cdaf52479 100644 --- a/include/efi_api.h +++ b/include/efi_api.h @@ -817,7 +817,7 @@ struct efi_simple_text_output_protocol { struct efi_input_key { u16 scan_code; - s16 unicode_char; + u16 unicode_char; }; #define EFI_SHIFT_STATE_INVALID 0x00000000 |