Masahisa Kojima 1dd705cf99 efi: use 32-bit alignment for efi_guid_t
Current U-Boot implements 64-bit boundary for efi_guid_t structure.
It follows the UEFI specification, page 21 of the UEFI Specification v2.10
says about EFI_GUID:
  128-bit buffer containing a unique identifier value. Unless
  otherwise specified, aligned on a 64-bit boundary.

On the other hand, page 163 of the UEFI specification v2.10 and
EDK2 reference implementation both define EFI_GUID as
struct { u32 a; u16; b; u16 c; u8 d[8]; }; and so the implied
alignment is 32-bit not 64-bit like U-Boot efi_guid_t.

Due to this alignment difference, EDK2 application "CapsuleApp.efi -P"
does not work as expected.
This calls EFI_FIRMWARE_MANAGEMENT_PROTOCOL.GetImageInfo()
and dump the EFI_FIRMWARE_IMAGE_DESCRIPTOR structure,
offsetof(EFI_FIRMWARE_IMAGE_DESCRIPTOR, ImageTypeId) is different,
8 in U-Boot and 4 in EDK2(CapsuleApp.efi).
Here is the wrong EFI_GUID dump.
  wrong dump : ImageTypeId - 00000000-7D83-058B-D550-474CA19560D8
  expected   : ImageTypeId - 058B7D83-50D5-4C47-A195-60D86AD341C4

EFI_FIRMWARE_IMAGE_DESCRIPTOR structure is defined in UEFI specification:
  typedef struct {
          UINT8 ImageIndex;
          EFI_GUID ImageTypeId;
          UINT64 ImageId
          <snip>

  } EFI_FIRMWARE_IMAGE_DESCRIPTOR;

There was the relevant patch for linux kernel to use 32-bit alignment
for efi_guid_t [1].
U-Boot should get aligned to EDK2 reference implementation and
linux kernel.

Due to this alignment change, efi_hii_ref structure in include/efi_api.h
is affected, but it is not used in the current U-Boot code.

[1] https://lore.kernel.org/all/20190202094119.13230-5-ard.biesheuvel@linaro.org/

Cc: Ilias Apalodimas <ilias.apalodimas@linaro.org>
Signed-off-by: Masahisa Kojima <masahisa.kojima@linaro.org>
Reviewed-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
Reviewed-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>
2023-02-10 13:05:39 +01:00
..
2022-07-05 14:37:16 +02:00
2023-02-07 14:33:49 -05:00
2022-11-28 13:06:39 -05:00
2022-10-30 20:07:16 +01:00
2022-09-29 16:07:57 -04:00
2023-02-06 17:10:15 -05:00
2023-02-07 14:33:50 -05:00
2022-09-24 10:47:01 -04:00
2022-07-26 02:30:56 -06:00
2022-09-29 16:07:57 -04:00
2022-04-06 14:01:42 -04:00
2022-03-30 13:02:55 -04:00
2022-10-20 08:58:25 -04:00
2022-11-02 08:42:03 +01:00
2023-02-10 13:05:39 +01:00
2023-02-06 13:04:52 -05:00
2022-03-15 16:19:29 -04:00
2022-04-06 14:01:42 -04:00
2022-10-31 14:47:32 -04:00
2023-01-16 18:26:50 -05:00
2023-01-11 15:02:24 -05:00
2022-08-04 16:18:47 -04:00
2022-08-31 12:16:01 -04:00
2022-10-18 09:32:53 +08:00
2023-01-20 12:27:06 -05:00
2022-07-26 11:29:00 +02:00
2022-06-08 13:59:53 -04:00
2022-06-28 17:03:32 -04:00
2022-07-26 11:29:02 +02:00
2023-01-27 14:47:57 -05:00
2022-04-28 09:26:44 -04:00
2022-10-30 20:07:17 +01:00
2022-08-10 13:38:30 -04:00
2023-02-06 13:04:53 -05:00
2023-01-16 18:26:50 -05:00
2023-01-31 22:02:27 +09:00
2023-02-02 14:44:53 -05:00
2023-01-23 18:11:40 -05:00
2022-06-08 14:00:22 -04:00
2022-10-29 07:36:33 -06:00
2022-12-07 16:04:17 -05:00
2023-01-16 18:01:10 +08:00
2022-07-08 12:20:28 -04:00
2022-12-09 14:10:28 -05:00
2022-04-01 15:03:13 -04:00
2022-09-13 12:08:40 -04:00
2022-10-16 12:23:22 +02:00
2022-04-06 14:03:17 -04:00
2022-09-24 10:47:01 -04:00
2022-12-07 16:04:17 -05:00
2022-12-07 16:04:17 -05:00
2022-12-22 10:31:49 -05:00
2022-12-22 10:31:49 -05:00
2023-01-16 18:26:50 -05:00
2022-09-18 10:26:49 +02:00