efi_driver: EFI block driver
authorHeinrich Schuchardt <xypron.glpk@gmx.de>
Sun, 21 Jan 2018 18:29:30 +0000 (19:29 +0100)
committerAlexander Graf <agraf@suse.de>
Mon, 22 Jan 2018 22:09:14 +0000 (23:09 +0100)
This patch provides
* a uclass for EFI drivers
* a EFI driver for block devices

For each EFI driver the uclass
* creates a handle
* adds the driver binding protocol

The uclass provides the bind, start, and stop entry points for the driver
binding protocol.

In bind() and stop() it checks if the controller implements the protocol
supported by the EFI driver. In the start() function it calls the bind()
function of the EFI driver. In the stop() function it destroys the child
controllers.

The EFI block driver binds to controllers implementing the block io
protocol.

When the bind function of the EFI block driver is called it creates a
new U-Boot block device. It installs child handles for all partitions and
installs the simple file protocol on these.

The read and write functions of the EFI block driver delegate calls to the
controller that it is bound to.

A usage example is as following:

U-Boot loads the iPXE snp.efi executable. iPXE connects an iSCSI drive and
exposes a handle with the block IO protocol. It calls ConnectController.

Now the EFI block driver installs the partitions with the simple file
protocol.

iPXE uses the simple file protocol to load Grub or the Linux Kernel.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
[agraf: add comment on calloc len]
Signed-off-by: Alexander Graf <agraf@suse.de>
cmd/bootefi.c
drivers/block/blk-uclass.c
include/blk.h
include/config_fallbacks.h
include/dm/uclass-id.h
include/efi_driver.h [new file with mode: 0644]
include/efi_loader.h
lib/Makefile
lib/efi_driver/Makefile [new file with mode: 0644]
lib/efi_driver/efi_block_device.c [new file with mode: 0644]
lib/efi_driver/efi_uclass.c [new file with mode: 0644]

index a30259c4c12c58fd5e2817afdfec0c568ab72857..51213c0293c307b843b4a1da819d9141632b9d20 100644 (file)
@@ -32,6 +32,9 @@ static void efi_init_obj_list(void)
 {
        efi_obj_list_initalized = 1;
 
+       /* Initialize EFI driver uclass */
+       efi_driver_init();
+
        efi_console_register();
 #ifdef CONFIG_PARTITIONS
        efi_disk_register();
index 010ed32d3add35c434c0d9f819edd31fcb527fa0..bfda2211f0e8b122b2d3f9909d2e54d5bb7bc005 100644 (file)
@@ -24,6 +24,7 @@ static const char *if_typename_str[IF_TYPE_COUNT] = {
        [IF_TYPE_HOST]          = "host",
        [IF_TYPE_SYSTEMACE]     = "ace",
        [IF_TYPE_NVME]          = "nvme",
+       [IF_TYPE_EFI]           = "efi",
 };
 
 static enum uclass_id if_type_uclass_id[IF_TYPE_COUNT] = {
@@ -36,8 +37,9 @@ static enum uclass_id if_type_uclass_id[IF_TYPE_COUNT] = {
        [IF_TYPE_SD]            = UCLASS_INVALID,
        [IF_TYPE_SATA]          = UCLASS_AHCI,
        [IF_TYPE_HOST]          = UCLASS_ROOT,
-       [IF_TYPE_NVME]          = UCLASS_NVME,
        [IF_TYPE_SYSTEMACE]     = UCLASS_INVALID,
+       [IF_TYPE_NVME]          = UCLASS_NVME,
+       [IF_TYPE_EFI]           = UCLASS_EFI,
 };
 
 static enum if_type if_typename_to_iftype(const char *if_typename)
index 41b4d7efa82b3f88994b95a61071310ae51462fe..69b5a98e5673f01b87c0d2ee45f77a848d19e4d4 100644 (file)
@@ -34,6 +34,7 @@ enum if_type {
        IF_TYPE_HOST,
        IF_TYPE_SYSTEMACE,
        IF_TYPE_NVME,
+       IF_TYPE_EFI,
 
        IF_TYPE_COUNT,                  /* Number of interface types */
 };
index 2c4d43d67269ff53f6ee0ce0a42672c2017129b5..524313d5aab24ec5e144ad927a9ae9f632a47b61 100644 (file)
@@ -52,6 +52,7 @@
        defined(CONFIG_MMC) || \
        defined(CONFIG_NVME) || \
        defined(CONFIG_SYSTEMACE) || \
+       (defined(CONFIG_EFI_LOADER) && !defined(CONFIG_SPL_BUILD)) || \
        defined(CONFIG_SANDBOX)
 #define HAVE_BLOCK_DEVICE
 #endif
index 3fc20834aedd1973800501bbe62e4681eb79dc60..07fabc3ce6cf67d0dc45c85ef598ecd6247c41d7 100644 (file)
@@ -34,6 +34,7 @@ enum uclass_id {
        UCLASS_CROS_EC,         /* Chrome OS EC */
        UCLASS_DISPLAY,         /* Display (e.g. DisplayPort, HDMI) */
        UCLASS_DMA,             /* Direct Memory Access */
+       UCLASS_EFI,             /* EFI managed devices */
        UCLASS_ETH,             /* Ethernet device */
        UCLASS_GPIO,            /* Bank of general-purpose I/O pins */
        UCLASS_FIRMWARE,        /* Firmware */
diff --git a/include/efi_driver.h b/include/efi_driver.h
new file mode 100644 (file)
index 0000000..2bbe26c
--- /dev/null
@@ -0,0 +1,30 @@
+/*
+ *  EFI application loader
+ *
+ *  Copyright (c) 2017 Heinrich Schuchardt
+ *
+ *  SPDX-License-Identifier:     GPL-2.0+
+ */
+
+#ifndef _EFI_DRIVER_H
+#define _EFI_DRIVER_H 1
+
+#include <common.h>
+#include <dm.h>
+#include <efi_loader.h>
+
+struct efi_driver_ops {
+       const efi_guid_t *protocol;
+       const efi_guid_t *child_protocol;
+       int (*bind)(efi_handle_t handle, void *interface);
+};
+
+/*
+ * This structure adds internal fields to the driver binding protocol.
+ */
+struct efi_driver_binding_extended_protocol {
+       struct efi_driver_binding_protocol bp;
+       const struct efi_driver_ops *ops;
+};
+
+#endif /* _EFI_DRIVER_H */
index 563c7ba3cf45ca1ea31848b606ff8deadac74fe6..21c03c5c28f88560f4932feff51be25edcda7277 100644 (file)
@@ -271,6 +271,8 @@ efi_status_t efi_get_memory_map(efi_uintn_t *memory_map_size,
 /* Adds a range into the EFI memory map */
 uint64_t efi_add_memory_map(uint64_t start, uint64_t pages, int memory_type,
                            bool overlap_only_ram);
+/* Called by board init to initialize the EFI drivers */
+int efi_driver_init(void);
 /* Called by board init to initialize the EFI memory map */
 int efi_memory_init(void);
 /* Adds new or overrides configuration table entry to the system table */
index 8cd779f8cad16d43d27084ade912b86bb5786880..0db41c19f379f1209a5eaf622d8f1867e5655e5b 100644 (file)
@@ -8,6 +8,7 @@
 ifndef CONFIG_SPL_BUILD
 
 obj-$(CONFIG_EFI) += efi/
+obj-$(CONFIG_EFI_LOADER) += efi_driver/
 obj-$(CONFIG_EFI_LOADER) += efi_loader/
 obj-$(CONFIG_EFI_LOADER) += efi_selftest/
 obj-$(CONFIG_LZMA) += lzma/
diff --git a/lib/efi_driver/Makefile b/lib/efi_driver/Makefile
new file mode 100644 (file)
index 0000000..e35529a
--- /dev/null
@@ -0,0 +1,13 @@
+#
+# (C) Copyright 2017 Heinrich Schuchardt
+#
+#  SPDX-License-Identifier:     GPL-2.0+
+#
+
+# This file only gets included with CONFIG_EFI_LOADER set, so all
+# object inclusion implicitly depends on it
+
+obj-y += efi_uclass.o
+ifeq ($(CONFIG_BLK)$(CONFIG_PARTITIONS),yy)
+obj-y += efi_block_device.o
+endif
diff --git a/lib/efi_driver/efi_block_device.c b/lib/efi_driver/efi_block_device.c
new file mode 100644 (file)
index 0000000..d9d2b14
--- /dev/null
@@ -0,0 +1,210 @@
+/*
+ *  EFI block driver
+ *
+ *  Copyright (c) 2017 Heinrich Schuchardt
+ *
+ *  SPDX-License-Identifier:     GPL-2.0+
+ *
+ * The EFI uclass creates a handle for this driver and installs the
+ * driver binding protocol on it.
+ *
+ * The EFI block driver binds to controllers implementing the block io
+ * protocol.
+ *
+ * When the bind function of the EFI block driver is called it creates a
+ * new U-Boot block device. It installs child handles for all partitions and
+ * installs the simple file protocol on these.
+ *
+ * The read and write functions of the EFI block driver delegate calls to the
+ * controller that it is bound to.
+ *
+ * A usage example is as following:
+ *
+ * U-Boot loads the iPXE snp.efi executable. iPXE connects an iSCSI drive and
+ * exposes a handle with the block IO protocol. It calls ConnectController.
+ *
+ * Now the EFI block driver installs the partitions with the simple file
+ * protocol.
+ *
+ * iPXE uses the simple file protocol to load Grub or the Linux Kernel.
+ */
+
+#include <efi_driver.h>
+#include <dm/device-internal.h>
+#include <dm/root.h>
+
+/*
+ * EFI attributes of the udevice handled by this driver.
+ *
+ * handle      handle of the controller on which this driver is installed
+ * io          block io protocol proxied by this driver
+ */
+struct efi_blk_priv {
+       efi_handle_t            handle;
+       struct efi_block_io     *io;
+};
+
+/*
+ * Read from block device
+ *
+ * @dev                device
+ * @blknr      first block to be read
+ * @blkcnt     number of blocks to read
+ * @buffer     output buffer
+ * @return     number of blocks transferred
+ */
+static ulong efi_bl_read(struct udevice *dev, lbaint_t blknr, lbaint_t blkcnt,
+                        void *buffer)
+{
+       struct efi_blk_priv *priv = dev->priv;
+       struct efi_block_io *io = priv->io;
+       efi_status_t ret;
+
+       EFI_PRINT("%s: read '%s', from block " LBAFU ", " LBAFU " blocks\n",
+                 __func__, dev->name, blknr, blkcnt);
+       ret = EFI_CALL(io->read_blocks(
+                               io, io->media->media_id, (u64)blknr,
+                               (efi_uintn_t)blkcnt *
+                               (efi_uintn_t)io->media->block_size, buffer));
+       EFI_PRINT("%s: r = %u\n", __func__,
+                 (unsigned int)(ret & ~EFI_ERROR_MASK));
+       if (ret != EFI_SUCCESS)
+               return 0;
+       return blkcnt;
+}
+
+/*
+ * Write to block device
+ *
+ * @dev                device
+ * @blknr      first block to be write
+ * @blkcnt     number of blocks to write
+ * @buffer     input buffer
+ * @return     number of blocks transferred
+ */
+static ulong efi_bl_write(struct udevice *dev, lbaint_t blknr, lbaint_t blkcnt,
+                         const void *buffer)
+{
+       struct efi_blk_priv *priv = dev->priv;
+       struct efi_block_io *io = priv->io;
+       efi_status_t ret;
+
+       EFI_PRINT("%s: write '%s', from block " LBAFU ", " LBAFU " blocks\n",
+                 __func__, dev->name, blknr, blkcnt);
+       ret = EFI_CALL(io->write_blocks(
+                               io, io->media->media_id, (u64)blknr,
+                               (efi_uintn_t)blkcnt *
+                               (efi_uintn_t)io->media->block_size,
+                               (void *)buffer));
+       EFI_PRINT("%s: r = %u\n", __func__,
+                 (unsigned int)(ret & ~EFI_ERROR_MASK));
+       if (ret != EFI_SUCCESS)
+               return 0;
+       return blkcnt;
+}
+
+/*
+ * Create partions for the block device.
+ *
+ * @handle     EFI handle of the block device
+ * @dev                udevice of the block device
+ */
+static int efi_bl_bind_partitions(efi_handle_t handle, struct udevice *dev)
+{
+       struct blk_desc *desc;
+       const char *if_typename;
+
+       desc = dev_get_uclass_platdata(dev);
+       if_typename = blk_get_if_type_name(desc->if_type);
+
+       return efi_disk_create_partitions(handle, desc, if_typename,
+                                         desc->devnum, dev->name);
+}
+
+/*
+ * Create a block device for a handle
+ *
+ * @handle     handle
+ * @interface  block io protocol
+ * @return     0 = success
+ */
+static int efi_bl_bind(efi_handle_t handle, void *interface)
+{
+       struct udevice *bdev, *parent = dm_root();
+       int ret, devnum;
+       char *name;
+       struct efi_object *obj = efi_search_obj(handle);
+       struct efi_block_io *io = interface;
+       int disks;
+       struct efi_blk_priv *priv;
+
+       EFI_PRINT("%s: handle %p, interface %p\n", __func__, handle, io);
+
+       if (!obj)
+               return -ENOENT;
+
+       devnum = blk_find_max_devnum(IF_TYPE_EFI);
+       if (devnum == -ENODEV)
+               devnum = 0;
+       else if (devnum < 0)
+               return devnum;
+
+       name = calloc(1, 18); /* strlen("efiblk#2147483648") + 1 */
+       if (!name)
+               return -ENOMEM;
+       sprintf(name, "efiblk#%d", devnum);
+
+       /* Create driver model udevice for the EFI block io device */
+       ret = blk_create_device(parent, "efi_blk", name, IF_TYPE_EFI, devnum,
+                               io->media->block_size,
+                               (lbaint_t)io->media->last_block, &bdev);
+       if (ret)
+               return ret;
+       if (!bdev)
+               return -ENOENT;
+       /* Allocate priv */
+       ret = device_probe(bdev);
+       if (ret)
+               return ret;
+       EFI_PRINT("%s: block device '%s' created\n", __func__, bdev->name);
+
+       priv = bdev->priv;
+       priv->handle = handle;
+       priv->io = interface;
+
+       ret = blk_prepare_device(bdev);
+
+       /* Create handles for the partions of the block device */
+       disks = efi_bl_bind_partitions(handle, bdev);
+       EFI_PRINT("Found %d partitions\n", disks);
+
+       return 0;
+}
+
+/* Block device driver operators */
+static const struct blk_ops efi_blk_ops = {
+       .read   = efi_bl_read,
+       .write  = efi_bl_write,
+};
+
+/* Identify as block device driver */
+U_BOOT_DRIVER(efi_blk) = {
+       .name                   = "efi_blk",
+       .id                     = UCLASS_BLK,
+       .ops                    = &efi_blk_ops,
+       .priv_auto_alloc_size   = sizeof(struct efi_blk_priv),
+};
+
+/* EFI driver operators */
+static const struct efi_driver_ops driver_ops = {
+       .protocol       = &efi_block_io_guid,
+       .child_protocol = &efi_block_io_guid,
+       .bind           = efi_bl_bind,
+};
+
+/* Identify as EFI driver */
+U_BOOT_DRIVER(efi_block) = {
+       .name           = "EFI block driver",
+       .id             = UCLASS_EFI,
+       .ops            = &driver_ops,
+};
diff --git a/lib/efi_driver/efi_uclass.c b/lib/efi_driver/efi_uclass.c
new file mode 100644 (file)
index 0000000..90797f9
--- /dev/null
@@ -0,0 +1,330 @@
+/*
+ *  Uclass for EFI drivers
+ *
+ *  Copyright (c) 2017 Heinrich Schuchardt
+ *
+ *  SPDX-License-Identifier:     GPL-2.0+
+ *
+ * For each EFI driver the uclass
+ * - creates a handle
+ * - installs the driver binding protocol
+ *
+ * The uclass provides the bind, start, and stop entry points for the driver
+ * binding protocol.
+ *
+ * In bind() and stop() it checks if the controller implements the protocol
+ * supported by the EFI driver. In the start() function it calls the bind()
+ * function of the EFI driver. In the stop() function it destroys the child
+ * controllers.
+ */
+
+#include <efi_driver.h>
+
+/*
+ * Check node type. We do not support partitions as controller handles.
+ *
+ * @handle     handle to be checked
+ * @return     status code
+ */
+static efi_status_t check_node_type(efi_handle_t handle)
+{
+       efi_status_t r, ret = EFI_SUCCESS;
+       const struct efi_device_path *dp;
+
+       /* Open the device path protocol */
+       r = EFI_CALL(systab.boottime->open_protocol(
+                       handle, &efi_guid_device_path, (void **)&dp,
+                       NULL, NULL, EFI_OPEN_PROTOCOL_GET_PROTOCOL));
+       if (r == EFI_SUCCESS && dp) {
+               /* Get the last node */
+               const struct efi_device_path *node = efi_dp_last_node(dp);
+               /* We do not support partitions as controller */
+               if (!node || node->type == DEVICE_PATH_TYPE_MEDIA_DEVICE)
+                       ret = EFI_UNSUPPORTED;
+       }
+       return ret;
+}
+
+/*
+ * Check if the driver supports the controller.
+ *
+ * @this                       driver binding protocol
+ * @controller_handle          handle of the controller
+ * @remaining_device_path      path specifying the child controller
+ * @return                     status code
+ */
+static efi_status_t EFIAPI efi_uc_supported(
+               struct efi_driver_binding_protocol *this,
+               efi_handle_t controller_handle,
+               struct efi_device_path *remaining_device_path)
+{
+       efi_status_t r, ret;
+       void *interface;
+       struct efi_driver_binding_extended_protocol *bp =
+                       (struct efi_driver_binding_extended_protocol *)this;
+
+       EFI_ENTRY("%p, %p, %ls", this, controller_handle,
+                 efi_dp_str(remaining_device_path));
+
+       ret = EFI_CALL(systab.boottime->open_protocol(
+                       controller_handle, bp->ops->protocol,
+                       &interface, this->driver_binding_handle,
+                       controller_handle, EFI_OPEN_PROTOCOL_BY_DRIVER));
+       switch (ret) {
+       case EFI_ACCESS_DENIED:
+       case EFI_ALREADY_STARTED:
+               goto out;
+       case EFI_SUCCESS:
+               break;
+       default:
+               ret = EFI_UNSUPPORTED;
+               goto out;
+       }
+
+       ret = check_node_type(controller_handle);
+
+       r = EFI_CALL(systab.boottime->close_protocol(
+                               controller_handle, bp->ops->protocol,
+                               this->driver_binding_handle,
+                               controller_handle));
+       if (r != EFI_SUCCESS)
+               ret = EFI_UNSUPPORTED;
+out:
+       return EFI_EXIT(ret);
+}
+
+/*
+ * Create child controllers and attach driver.
+ *
+ * @this                       driver binding protocol
+ * @controller_handle          handle of the controller
+ * @remaining_device_path      path specifying the child controller
+ * @return                     status code
+ */
+static efi_status_t EFIAPI efi_uc_start(
+               struct efi_driver_binding_protocol *this,
+               efi_handle_t controller_handle,
+               struct efi_device_path *remaining_device_path)
+{
+       efi_status_t r, ret;
+       void *interface = NULL;
+       struct efi_driver_binding_extended_protocol *bp =
+                       (struct efi_driver_binding_extended_protocol *)this;
+
+       EFI_ENTRY("%p, %pUl, %ls", this, controller_handle,
+                 efi_dp_str(remaining_device_path));
+
+       /* Attach driver to controller */
+       ret = EFI_CALL(systab.boottime->open_protocol(
+                       controller_handle, bp->ops->protocol,
+                       &interface, this->driver_binding_handle,
+                       controller_handle, EFI_OPEN_PROTOCOL_BY_DRIVER));
+       switch (ret) {
+       case EFI_ACCESS_DENIED:
+       case EFI_ALREADY_STARTED:
+               goto out;
+       case EFI_SUCCESS:
+               break;
+       default:
+               ret =  EFI_UNSUPPORTED;
+               goto out;
+       }
+       ret = check_node_type(controller_handle);
+       if (ret != EFI_SUCCESS) {
+               r = EFI_CALL(systab.boottime->close_protocol(
+                               controller_handle, bp->ops->protocol,
+                               this->driver_binding_handle,
+                               controller_handle));
+               if (r != EFI_SUCCESS)
+                       EFI_PRINT("Failure to close handle\n");
+               goto out;
+       }
+
+       /* TODO: driver specific stuff */
+       bp->ops->bind(controller_handle, interface);
+
+out:
+       return EFI_EXIT(ret);
+}
+
+/*
+ * Remove a single child controller from the parent controller.
+ *
+ * @controller_handle  parent controller
+ * @child_handle       child controller
+ * @return             status code
+ */
+static efi_status_t disconnect_child(efi_handle_t controller_handle,
+                                    efi_handle_t child_handle)
+{
+       efi_status_t ret;
+       efi_guid_t *guid_controller = NULL;
+       efi_guid_t *guid_child_controller = NULL;
+
+       ret = EFI_CALL(systab.boottime->close_protocol(
+                               controller_handle, guid_controller,
+                               child_handle, child_handle));
+       if (ret != EFI_SUCCESS) {
+               EFI_PRINT("Cannot close protocol\n");
+               return ret;
+       }
+       ret = EFI_CALL(systab.boottime->uninstall_protocol_interface(
+                               child_handle, guid_child_controller, NULL));
+       if (ret != EFI_SUCCESS) {
+               EFI_PRINT("Cannot uninstall protocol interface\n");
+               return ret;
+       }
+       return ret;
+}
+
+/*
+ * Remove child controllers and disconnect the controller.
+ *
+ * @this                       driver binding protocol
+ * @controller_handle          handle of the controller
+ * @number_of_children         number of child controllers to remove
+ * @child_handle_buffer                handles of the child controllers to remove
+ * @return                     status code
+ */
+static efi_status_t EFIAPI efi_uc_stop(
+               struct efi_driver_binding_protocol *this,
+               efi_handle_t controller_handle,
+               size_t number_of_children,
+               efi_handle_t *child_handle_buffer)
+{
+       efi_status_t ret;
+       efi_uintn_t count;
+       struct efi_open_protocol_info_entry *entry_buffer;
+       efi_guid_t *guid_controller = NULL;
+
+       EFI_ENTRY("%p, %pUl, %zu, %p", this, controller_handle,
+                 number_of_children, child_handle_buffer);
+
+       /* Destroy provided child controllers */
+       if (number_of_children) {
+               efi_uintn_t i;
+
+               for (i = 0; i < number_of_children; ++i) {
+                       ret = disconnect_child(controller_handle,
+                                              child_handle_buffer[i]);
+                       if (ret != EFI_SUCCESS)
+                               return ret;
+               }
+               return EFI_SUCCESS;
+       }
+
+       /* Destroy all children */
+       ret = EFI_CALL(systab.boottime->open_protocol_information(
+                                       controller_handle, guid_controller,
+                                       &entry_buffer, &count));
+       if (ret != EFI_SUCCESS)
+               goto out;
+       while (count) {
+               if (entry_buffer[--count].attributes &
+                   EFI_OPEN_PROTOCOL_BY_CHILD_CONTROLLER) {
+                       ret = disconnect_child(
+                                       controller_handle,
+                                       entry_buffer[count].agent_handle);
+                       if (ret != EFI_SUCCESS)
+                               goto out;
+               }
+       }
+       ret = EFI_CALL(systab.boottime->free_pool(entry_buffer));
+       if (ret != EFI_SUCCESS)
+               printf("%s(%u) %s: ERROR: Cannot free pool\n",
+                      __FILE__, __LINE__, __func__);
+
+       /* Detach driver from controller */
+       ret = EFI_CALL(systab.boottime->close_protocol(
+                       controller_handle, guid_controller,
+                       this->driver_binding_handle, controller_handle));
+out:
+       return EFI_EXIT(ret);
+}
+
+static efi_status_t efi_add_driver(struct driver *drv)
+{
+       efi_status_t ret;
+       const struct efi_driver_ops *ops = drv->ops;
+       struct efi_driver_binding_extended_protocol *bp;
+
+       debug("EFI: Adding driver '%s'\n", drv->name);
+       if (!ops->protocol) {
+               printf("EFI: ERROR: protocol GUID missing for driver '%s'\n",
+                      drv->name);
+               return EFI_INVALID_PARAMETER;
+       }
+       bp = calloc(1, sizeof(struct efi_driver_binding_extended_protocol));
+       if (!bp)
+               return EFI_OUT_OF_RESOURCES;
+
+       bp->bp.supported = efi_uc_supported;
+       bp->bp.start = efi_uc_start;
+       bp->bp.stop = efi_uc_stop;
+       bp->bp.version = 0xffffffff;
+       bp->ops = drv->ops;
+
+       ret = efi_create_handle(&bp->bp.driver_binding_handle);
+       if (ret != EFI_SUCCESS) {
+               free(bp);
+               goto out;
+       }
+       bp->bp.image_handle = bp->bp.driver_binding_handle;
+       ret = efi_add_protocol(bp->bp.driver_binding_handle,
+                              &efi_guid_driver_binding_protocol, bp);
+       if (ret != EFI_SUCCESS) {
+               efi_delete_handle(bp->bp.driver_binding_handle);
+               free(bp);
+               goto out;
+       }
+out:
+       return ret;
+}
+
+/*
+ * Initialize the EFI drivers.
+ * Called by board_init_r().
+ *
+ * @return     0 = success, any other value will stop further execution
+ */
+int efi_driver_init(void)
+{
+       struct driver *drv;
+       int ret = 0;
+
+       /* Save 'gd' pointer */
+       efi_save_gd();
+
+       debug("EFI: Initializing EFI driver framework\n");
+       for (drv = ll_entry_start(struct driver, driver);
+            drv < ll_entry_end(struct driver, driver); ++drv) {
+               if (drv->id == UCLASS_EFI) {
+                       ret = efi_add_driver(drv);
+                       if (ret) {
+                               printf("EFI: ERROR: failed to add driver %s\n",
+                                      drv->name);
+                               break;
+                       }
+               }
+       }
+       return ret;
+}
+
+static int efi_uc_init(struct uclass *class)
+{
+       printf("EFI: Initializing UCLASS_EFI\n");
+       return 0;
+}
+
+static int efi_uc_destroy(struct uclass *class)
+{
+       printf("Destroying  UCLASS_EFI\n");
+       return 0;
+}
+
+UCLASS_DRIVER(efi) = {
+       .name           = "efi",
+       .id             = UCLASS_EFI,
+       .init           = efi_uc_init,
+       .destroy        = efi_uc_destroy,
+};