u-boot.git/lib/efi_loader, branch v2020.04

efi_loader: definition of GetNextVariableName()

2020-03-22T17:38:02+00:00

'vendor' is both an input and an output parameter. So it cannot be
constant.

Fixes: 0bda81bfdc5c ("efi_loader: use const efi_guid_t * for variable services")
Signed-off-by: Heinrich Schuchardt

efi_loader: fix freestanding memmove()

2020-03-22T10:06:23+00:00

For EFI binaries we have to provide an implementation of memmove() in
efi_freestanding.c.

Before this patch the memmove() function was copying in the wrong
direction.

Signed-off-by: Heinrich Schuchardt

efi_loader: description efi_convert_pointer()

2020-03-22T10:06:23+00:00

Correct the description of function efi_convert_pointer().

Signed-off-by: Heinrich Schuchardt

efi_loader: simplify logical expression in efi_disk_add_dev()

2020-03-22T10:06:23+00:00

To check if a variable is non-zero there is no need for '!= 0'.

Signed-off-by: Heinrich Schuchardt

efi_loader: correct reported length in GetNextVariable()

2020-03-22T10:06:23+00:00

The runtime service GetNextVariable() returns the length of the next
variable including the closing 0x0000. This length should be in bytes.

Comparing the output of EDK2 and U-Boot shows that this is currently not
correctly implemented:

EDK2:
OsIndicationsSupported: 46
PlatformLang: 26
PlatformLangCodes: 36

U-Boot:
OsIndicationsSupported: 23
PlatformLang: 13
PlatformLangCodes: 18

Provide correct length in GetNextVariable().

Fixes: d99a87f84b75 ("efi_loader: implement GetNextVariableName()")
Signed-off-by: Heinrich Schuchardt

efi_loader: fix function descriptions in efi_disk.c

2020-03-22T10:06:23+00:00

Use Sphinx style for function descriptions.

Signed-off-by: Heinrich Schuchardt

efi_loader: description of efi_variable.c

2020-03-22T10:06:23+00:00

Correct the file description.

Signed-off-by: Heinrich Schuchardt

efi_loader: export a couple of protocol related functions

2020-03-17T07:29:52+00:00

This is a preparatory patch.
Those functions will be used in an implementation of UEFI firmware
management protocol as part of my capsule update patch.

Signed-off-by: AKASHI Takahiro

efi_loader: unnecessary assignment in efi_queue_event

2020-03-11T07:23:05+00:00

The assigned value NULL is never used.

Signed-off-by: Heinrich Schuchardt

efi_loader: get_memory_map: return parameters whenever possible

2020-03-11T07:23:05+00:00

Currently, if GetMemoryMap API returns EFI_BUFFER_TOO_SMALL, it doesn't
set valid values to other parameters, descriptor_size and
descriptor_version, except memory_map_size.
Some efi applications, however, may use those value; in particular,
xen uses descriptor_size to calculate a size of buffer to be allocated.

While UEFI specification is ambiguous in this point, it would be better
to address this issue proactively to maximize the compatibility with
existing efi applications.

With this patch, for example, xen.efi (and hence linux kernel) can be
started via bootefi without modification.

Signed-off-by: AKASHI Takahiro 
Reviewed-by: Heinrich Schuchardt