On 18 March 2016 at 11:59, Matt Fleming [off-list ref] wrote:

On Thu, 10 Mar, at 12:40:01PM, Ard Biesheuvel wrote:

quoted

This patch redeclares efi_boot_services_t::install_configuration_table
as a function pointer so that the boot service is callable as a function.

Signed-off-by: Ard Biesheuvel <redacted>
---
 include/linux/efi.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Any chance you could fold this patch into the one that actually uses
install_configuration_table() on ARM? That way it's clear to see where
the users are.

Sure. Patch #2 actually does the same for locate_handle()

quoted

diff --git a/include/linux/efi.h b/include/linux/efi.h
index e747eb08b2be..8ba6eb7863aa 100644
--- a/include/linux/efi.h
+++ b/include/linux/efi.h

@@ -283,7 +283,7 @@ typedef struct {
      void *register_protocol_notify;
      void *locate_handle;
      void *locate_device_path;
-     void *install_configuration_table;
+     efi_status_t (*install_configuration_table)(efi_guid_t *, void *);
      void *load_image;
      void *start_image;
      void *exit;
--

1.9.1