u-boot.git/lib/efi_loader, branch v2017.11

efi_loader: Disable env_save() call on boot

2017-10-29T11:59:08+00:00

With the introduction of EFI variable support, we also wanted to persist
these EFI variables. However, the way it was implemented we ended up
persisting all U-Boot environment variables on every EFI boot.

That could potentially lead to unexpected side effects because variables
that were not supposed to be written to persisted env get written. It also
means we may end up writing the environment more often than we should.

For this release, let's just disable EFI variable persistence and instead
implement it properly for the next one.

Reported-by: Heinrich Schuchardt 
Fixes: ad644e7c182 ("efi_loader: efi variable support")
Signed-off-by: Alexander Graf 
Acked-by: Rob Clark 
Reviewed-by: Heinrich Schuchardt

efi_loader: console support for color attributes

2017-10-12T15:24:25+00:00

Shell.efi uses this, and supporting color attributes makes things look
nicer.  Map the EFI fg/bg color attributes to ANSI escape sequences.
Not all colors have a perfect match, but spec just says "Devices
supporting a different number of text colors are required to emulate the
above colors to the best of the device’s capabilities".

Signed-off-by: Rob Clark 
Tested-by: Heinrich Schuchardt 
Reviewed-by: Alexander Graf 
[agraf: s/unsigned/unsigned int/]
Signed-off-by: Alexander Graf

efi_loader: Add mem-mapped for fallback

2017-10-12T15:22:58+00:00

When we don't have a real device/image path, such as 'bootefi hello',
construct a mem-mapped device-path.

This fixes 'bootefi hello' after devicepath refactoring.

Fixes: 95c5553ea2 ("efi_loader: refactor boot device and loaded_image handling")
Signed-off-by: Rob Clark 
Acked-by: Heinrich Schuchardt 
Signed-off-by: Alexander Graf

efi_loader: don't increment part twice per loop

2017-10-10T16:14:05+00:00

Correct a mistake in the part number handling of commit
16a73b249d138fedeb188710533902ed7aac1ddc and only increment part once
per loop.

Signed-off-by: Jonathan Gray 
Tested-by: Peter Robinson 
Signed-off-by: Alexander Graf

efi_loader: search all possible disk partitions

2017-10-10T07:56:04+00:00

When searching for partitions don't stop if a partition is not present
for a given partition number as there may be valid partitions after.

Search for up to MAX_SEARCH_PARTITIONS matching the other callers of
part_get_info().

This allows OpenBSD to boot via the efi_loader on rpi_3 again after
changes made after U-Boot 2017.09.  With MBR partitioning OpenBSD will
by default use the fourth partition for the 0xA6 (OpenBSD) partition.

Signed-off-by: Jonathan Gray 
Signed-off-by: Alexander Graf

efi_loader: comments for functions add missing @return

2017-10-09T05:00:38+00:00

For some functions the @return description is missing.

Fix typo.

Signed-off-by: Heinrich Schuchardt 
Reviewed-by: Simon Glass 
Signed-off-by: Alexander Graf

efi_loader: Fix disk dp's for pre-DM/legacy devices

2017-10-09T05:00:38+00:00

This fixes an issue with OpenBSD's bootloader, and I think should also
fix a similar issue with grub2 on legacy devices.  In the legacy case
we were creating disk objects for the partitions, but not also the
parent device.

Reported-by: Jonathan Gray 
Signed-off-by: Rob Clark 
Signed-off-by: Alexander Graf

efi_loader: avoid NULL dereference in efi_dp_match

2017-10-09T05:00:37+00:00

When calling bootefi hello twice a kernel dump occurs.

Neither bootefi hello nor bootefi selftest have an image
device patch. So do not try to dereference the NULL
value.

Fixes: 95c5553ea26 efi_loader: refactor boot device and loaded_image handling
Signed-off-by: Heinrich Schuchardt 
Reviewed-by: Simon Glass 
Signed-off-by: Alexander Graf

efi_loader: fill return values in SimpleNetworkProtocol

2017-10-09T05:00:35+00:00

In the receive function all return values should be filled.

Signed-off-by: Heinrich Schuchardt 
Reviewed-by: Simon Glass 
Signed-off-by: Alexander Graf

efi_loader: size fields in SimpleNetworkProtocol

2017-10-09T05:00:34+00:00

The size fields in the Simple Network Protocol are all
UINTN in the UEFI spec. So use size_t.

Provide a function description of the receive function.

Signed-off-by: Heinrich Schuchardt 
Reviewed-by: Simon Glass 
Signed-off-by: Alexander Graf