* [PATCH v8 edk-platforms 0/2] add platform boot options
@ 2018-06-04 5:44 Haojian Zhuang
2018-06-04 5:44 ` [PATCH v8 edk-platforms 1/2] Platform/HiKey960: register predefined " Haojian Zhuang
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Haojian Zhuang @ 2018-06-04 5:44 UTC (permalink / raw)
To: edk2-devel; +Cc: Haojian Zhuang, Laszlo Ersek, Leif Lindholm, Ard Biesheuvel
Changelog:
v8:
* Rebase for virtual keyboard driver.
v7:
* Fix memory leakage on DevicePath.
v6:
* Remove redundant definition -- "GRUB_FILE_NAME".
v5:
* Avoid to merge device path and grub's file path in driver.
Merge them directly in DSC file.
* Avoid duplicated code to create boot options.
* Use goto to handle error condition.
v4:
* Add BootCount parameter in the interface.
v3:
* Move in initilization of boot entry.
* Update the name of interface in platform boot manager protocol.
v2:
* Update with platform boot manager protocol.
Haojian Zhuang (2):
Platform/HiKey960: register predefined boot options
Platform/HiKey: create 4 boot options
Platform/Hisilicon/HiKey/HiKey.dec | 10 +-
Platform/Hisilicon/{HiKey/HiKey.dec => HiKey960/HiKey960.dec} | 19 ++-
Platform/Hisilicon/HiKey/HiKey.dsc | 7 +
Platform/Hisilicon/HiKey960/HiKey960.dsc | 6 +
Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.inf | 11 ++
Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.inf | 11 ++
Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.h | 2 +
Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.h | 20 +--
Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.c | 164 +++++++++++++++++++
Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.c | 166 ++++++++++++++++++++
10 files changed, 384 insertions(+), 32 deletions(-)
copy Platform/Hisilicon/{HiKey/HiKey.dec => HiKey960/HiKey960.dec} (54%)
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
--
2.7.4
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v8 edk-platforms 1/2] Platform/HiKey960: register predefined boot options
2018-06-04 5:44 [PATCH v8 edk-platforms 0/2] add platform boot options Haojian Zhuang
@ 2018-06-04 5:44 ` Haojian Zhuang
2018-06-04 5:44 ` [PATCH v8 edk-platforms 2/2] Platform/HiKey: create 4 " Haojian Zhuang
2018-06-05 14:56 ` [PATCH v8 edk-platforms 0/2] add platform " Leif Lindholm
2 siblings, 0 replies; 4+ messages in thread
From: Haojian Zhuang @ 2018-06-04 5:44 UTC (permalink / raw)
To: edk2-devel; +Cc: Haojian Zhuang, Laszlo Ersek, Leif Lindholm, Ard Biesheuvel
Create 4 boot options on HiKey960 platform. They're "Boot from SD",
"Grub", "Android Boot" and "Android Fastboot".
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Acked-by: Laszlo Ersek <lersek@redhat.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>
---
Platform/Hisilicon/HiKey960/HiKey960.dec | 35 +++++
Platform/Hisilicon/HiKey960/HiKey960.dsc | 6 +
Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.inf | 11 ++
Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.h | 20 +--
Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.c | 166 ++++++++++++++++++++
5 files changed, 220 insertions(+), 18 deletions(-)
diff --git a/Platform/Hisilicon/HiKey960/HiKey960.dec b/Platform/Hisilicon/HiKey960/HiKey960.dec
new file mode 100644
index 000000000000..aa5a0caf52e5
--- /dev/null
+++ b/Platform/Hisilicon/HiKey960/HiKey960.dec
@@ -0,0 +1,35 @@
+#
+# Copyright (c) 2018, Linaro Limited. All rights reserved.
+#
+# This program and the accompanying materials
+# are licensed and made available under the terms and conditions of the BSD License
+# which accompanies this distribution. The full text of the license may be found at
+# http://opensource.org/licenses/bsd-license.php
+#
+# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
+# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
+#
+
+[Defines]
+ DEC_SPECIFICATION = 0x0001001a
+ PACKAGE_NAME = HiKey960
+ PACKAGE_GUID = 1892b5b5-d18d-47a3-8fab-e3ae6b4226b0
+ PACKAGE_VERSION = 0.1
+
+################################################################################
+#
+# Include Section - list of Include Paths that are provided by this package.
+# Comments are used for Keywords and Module Types.
+#
+# Supported Module Types:
+# BASE SEC PEI_CORE PEIM DXE_CORE DXE_DRIVER DXE_RUNTIME_DRIVER DXE_SMM_DRIVER DXE_SAL_DRIVER UEFI_DRIVER UEFI_APPLICATION
+#
+################################################################################
+[Guids.common]
+ gHiKey960TokenSpaceGuid = { 0x99a14446, 0xaad7, 0xe460, {0xb4, 0xe5, 0x1f, 0x79, 0xaa, 0xa4, 0x93, 0xfd } }
+
+[PcdsFixedAtBuild.common]
+ gHiKey960TokenSpaceGuid.PcdAndroidBootDevicePath|L""|VOID*|0x00000001
+ gHiKey960TokenSpaceGuid.PcdAndroidBootFile|{ 0x36, 0x8b, 0x73, 0x3a, 0xc5, 0xb9, 0x63, 0x47, 0xab, 0xbd, 0x6c, 0xbd, 0x4b, 0x25, 0xf9, 0xff }|VOID*|0x00000002
+ gHiKey960TokenSpaceGuid.PcdAndroidFastbootFile|{ 0x2a, 0x50, 0x88, 0x95, 0x70, 0x53, 0xe3, 0x11, 0x86, 0x31, 0xd7, 0xc5, 0x95, 0x13, 0x64, 0xc8 }|VOID*|0x00000003
+ gHiKey960TokenSpaceGuid.PcdSdBootDevicePath|L""|VOID*|0x00000004
diff --git a/Platform/Hisilicon/HiKey960/HiKey960.dsc b/Platform/Hisilicon/HiKey960/HiKey960.dsc
index 79e68754976d..be13714b9c19 100644
--- a/Platform/Hisilicon/HiKey960/HiKey960.dsc
+++ b/Platform/Hisilicon/HiKey960/HiKey960.dsc
@@ -129,6 +129,12 @@ [PcdsFixedAtBuild.common]
gEmbeddedTokenSpaceGuid.PcdAndroidFastbootUsbVendorId|0x18d1
gEmbeddedTokenSpaceGuid.PcdAndroidFastbootUsbProductId|0xd00d
+ #
+ # Android Loader
+ #
+ gHiKey960TokenSpaceGuid.PcdAndroidBootDevicePath|L"VenHw(0D51905B-B77E-452A-A2C0-ECA0CC8D514A,00003BFF0000000000)/UFS(0x0,0x3)/HD(7,GPT,D3340696-9B95-4C64-8DF6-E6D4548FBA41,0x12100,0x4000)/\\EFI\\BOOT\\GRUBAA64.EFI"
+ gHiKey960TokenSpaceGuid.PcdSdBootDevicePath|L"VenHw(0D51905B-B77E-452A-A2C0-ECA0CC8D514A,00F037FF0000000000)/SD(0x0)"
+
################################################################################
#
# Components Section - list of all EDK II Modules needed by this Platform
diff --git a/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.inf b/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.inf
index 46a9a5803e3d..54105bdb3c94 100644
--- a/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.inf
+++ b/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.inf
@@ -25,6 +25,7 @@ [Packages]
EmbeddedPkg/EmbeddedPkg.dec
MdeModulePkg/MdeModulePkg.dec
MdePkg/MdePkg.dec
+ Platform/Hisilicon/HiKey960/HiKey960.dec
Silicon/Hisilicon/Hi3660/Hi3660.dec
[LibraryClasses]
@@ -34,13 +35,23 @@ [LibraryClasses]
IoLib
PcdLib
TimerLib
+ UefiBootManagerLib
UefiDriverEntryPoint
UefiLib
[Protocols]
+ gEfiDevicePathFromTextProtocolGuid
+ gEfiLoadedImageProtocolGuid
gEmbeddedGpioProtocolGuid
+ gPlatformBootManagerProtocolGuid
gPlatformVirtualKeyboardProtocolGuid
+[Pcd]
+ gHiKey960TokenSpaceGuid.PcdAndroidBootDevicePath
+ gHiKey960TokenSpaceGuid.PcdAndroidBootFile
+ gHiKey960TokenSpaceGuid.PcdAndroidFastbootFile
+ gHiKey960TokenSpaceGuid.PcdSdBootDevicePath
+
[Guids]
gEfiEndOfDxeEventGroupGuid
diff --git a/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.h b/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.h
index 211eea55aa54..630aabcb2397 100644
--- a/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.h
+++ b/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.h
@@ -15,30 +15,14 @@
#ifndef __HIKEY960DXE_H__
#define __HIKEY960DXE_H__
-#include <Guid/EventGroup.h>
-
-#include <Hi3660.h>
-
-#include <Library/BaseLib.h>
-#include <Library/BaseMemoryLib.h>
-#include <Library/CacheMaintenanceLib.h>
-#include <Library/DebugLib.h>
-#include <Library/IoLib.h>
-#include <Library/PrintLib.h>
-#include <Library/SerialPortLib.h>
-#include <Library/TimerLib.h>
-#include <Library/UefiBootServicesTableLib.h>
-#include <Library/UefiLib.h>
-
-#include <Protocol/EmbeddedGpio.h>
-#include <Protocol/PlatformVirtualKeyboard.h>
-
#define ADB_REBOOT_ADDRESS 0x32100000
#define ADB_REBOOT_BOOTLOADER 0x77665500
#define ADB_REBOOT_NONE 0x77665501
#define DETECT_SW_FASTBOOT 68 // GPIO8_4
+#define HIKEY960_BOOT_OPTION_NUM 4
+
enum {
BOOT_MODE_RECOVERY = 0,
BOOT_MODE_MASK = 1,
diff --git a/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.c b/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.c
index b205be97db29..47ec879861bd 100644
--- a/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.c
+++ b/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.c
@@ -18,14 +18,24 @@
#include <Library/BaseLib.h>
#include <Library/BaseMemoryLib.h>
+#include <Library/CacheMaintenanceLib.h>
#include <Library/DebugLib.h>
+#include <Library/DevicePathLib.h>
#include <Library/IoLib.h>
+#include <Library/MemoryAllocationLib.h>
#include <Library/PrintLib.h>
#include <Library/SerialPortLib.h>
#include <Library/TimerLib.h>
+#include <Library/UefiBootManagerLib.h>
#include <Library/UefiBootServicesTableLib.h>
#include <Library/UefiLib.h>
+#include <Protocol/DevicePathFromText.h>
+#include <Protocol/EmbeddedGpio.h>
+#include <Protocol/LoadedImage.h>
+#include <Protocol/PlatformBootManager.h>
+#include <Protocol/PlatformVirtualKeyboard.h>
+
#include "HiKey960Dxe.h"
STATIC EMBEDDED_GPIO *mGpio;
@@ -172,6 +182,152 @@ OnEndOfDxe (
}
}
+STATIC
+EFI_STATUS
+CreatePlatformBootOptionFromPath (
+ IN CHAR16 *PathStr,
+ IN CHAR16 *Description,
+ IN OUT EFI_BOOT_MANAGER_LOAD_OPTION *BootOption
+ )
+{
+ EFI_STATUS Status;
+ EFI_DEVICE_PATH *DevicePath;
+
+ DevicePath = (EFI_DEVICE_PATH *)ConvertTextToDevicePath (PathStr);
+ ASSERT (DevicePath != NULL);
+ Status = EfiBootManagerInitializeLoadOption (
+ BootOption,
+ LoadOptionNumberUnassigned,
+ LoadOptionTypeBoot,
+ LOAD_OPTION_ACTIVE,
+ Description,
+ DevicePath,
+ NULL,
+ 0
+ );
+ FreePool (DevicePath);
+ return Status;
+}
+
+STATIC
+EFI_STATUS
+CreatePlatformBootOptionFromGuid (
+ IN EFI_GUID *FileGuid,
+ IN CHAR16 *Description,
+ IN OUT EFI_BOOT_MANAGER_LOAD_OPTION *BootOption
+ )
+{
+ EFI_STATUS Status;
+ EFI_DEVICE_PATH *DevicePath;
+ EFI_DEVICE_PATH *TempDevicePath;
+ EFI_LOADED_IMAGE_PROTOCOL *LoadedImage;
+ MEDIA_FW_VOL_FILEPATH_DEVICE_PATH FileNode;
+
+ Status = gBS->HandleProtocol (
+ gImageHandle,
+ &gEfiLoadedImageProtocolGuid,
+ (VOID **) &LoadedImage
+ );
+ ASSERT_EFI_ERROR (Status);
+ EfiInitializeFwVolDevicepathNode (&FileNode, FileGuid);
+ TempDevicePath = DevicePathFromHandle (LoadedImage->DeviceHandle);
+ ASSERT (TempDevicePath != NULL);
+ DevicePath = AppendDevicePathNode (
+ TempDevicePath,
+ (EFI_DEVICE_PATH_PROTOCOL *) &FileNode
+ );
+ ASSERT (DevicePath != NULL);
+ Status = EfiBootManagerInitializeLoadOption (
+ BootOption,
+ LoadOptionNumberUnassigned,
+ LoadOptionTypeBoot,
+ LOAD_OPTION_ACTIVE,
+ Description,
+ DevicePath,
+ NULL,
+ 0
+ );
+ FreePool (DevicePath);
+ return Status;
+}
+
+STATIC
+EFI_STATUS
+GetPlatformBootOptionsAndKeys (
+ OUT UINTN *BootCount,
+ OUT EFI_BOOT_MANAGER_LOAD_OPTION **BootOptions,
+ OUT EFI_INPUT_KEY **BootKeys
+ )
+{
+ EFI_GUID *FileGuid;
+ CHAR16 *PathStr;
+ EFI_STATUS Status;
+ UINTN Size;
+
+ Size = sizeof (EFI_BOOT_MANAGER_LOAD_OPTION) * HIKEY960_BOOT_OPTION_NUM;
+ *BootOptions = (EFI_BOOT_MANAGER_LOAD_OPTION *)AllocateZeroPool (Size);
+ if (*BootOptions == NULL) {
+ DEBUG ((DEBUG_ERROR, "Failed to allocate memory for BootOptions\n"));
+ return EFI_OUT_OF_RESOURCES;
+ }
+ Size = sizeof (EFI_INPUT_KEY) * HIKEY960_BOOT_OPTION_NUM;
+ *BootKeys = (EFI_INPUT_KEY *)AllocateZeroPool (Size);
+ if (*BootKeys == NULL) {
+ DEBUG ((DEBUG_ERROR, "Failed to allocate memory for BootKeys\n"));
+ Status = EFI_OUT_OF_RESOURCES;
+ goto Error;
+ }
+
+ PathStr = (CHAR16 *)PcdGetPtr (PcdSdBootDevicePath);
+ ASSERT (PathStr != NULL);
+ Status = CreatePlatformBootOptionFromPath (
+ PathStr,
+ L"Boot from SD",
+ &(*BootOptions)[0]
+ );
+ ASSERT_EFI_ERROR (Status);
+
+ PathStr = (CHAR16 *)PcdGetPtr (PcdAndroidBootDevicePath);
+ ASSERT (PathStr != NULL);
+ Status = CreatePlatformBootOptionFromPath (
+ PathStr,
+ L"Grub",
+ &(*BootOptions)[1]
+ );
+ ASSERT_EFI_ERROR (Status);
+
+ FileGuid = PcdGetPtr (PcdAndroidBootFile);
+ ASSERT (FileGuid != NULL);
+ Status = CreatePlatformBootOptionFromGuid (
+ FileGuid,
+ L"Android Boot",
+ &(*BootOptions)[2]
+ );
+ ASSERT_EFI_ERROR (Status);
+
+ FileGuid = PcdGetPtr (PcdAndroidFastbootFile);
+ ASSERT (FileGuid != NULL);
+ Status = CreatePlatformBootOptionFromGuid (
+ FileGuid,
+ L"Android Fastboot",
+ &(*BootOptions)[3]
+ );
+ ASSERT_EFI_ERROR (Status);
+ (*BootKeys)[3].ScanCode = SCAN_NULL;
+ (*BootKeys)[3].UnicodeChar = 'f';
+
+ *BootCount = 4;
+
+ return EFI_SUCCESS;
+Error:
+ FreePool (*BootOptions);
+ return Status;
+}
+
+PLATFORM_BOOT_MANAGER_PROTOCOL mPlatformBootManager = {
+ GetPlatformBootOptionsAndKeys
+};
+
EFI_STATUS
EFIAPI
VirtualKeyboardRegister (
@@ -297,5 +453,15 @@ HiKey960EntryPoint (
EFI_NATIVE_INTERFACE,
&mVirtualKeyboard
);
+ if (EFI_ERROR (Status)) {
+ return Status;
+ }
+
+ Status = gBS->InstallProtocolInterface (
+ &ImageHandle,
+ &gPlatformBootManagerProtocolGuid,
+ EFI_NATIVE_INTERFACE,
+ &mPlatformBootManager
+ );
return Status;
}
--
2.7.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH v8 edk-platforms 2/2] Platform/HiKey: create 4 boot options
2018-06-04 5:44 [PATCH v8 edk-platforms 0/2] add platform boot options Haojian Zhuang
2018-06-04 5:44 ` [PATCH v8 edk-platforms 1/2] Platform/HiKey960: register predefined " Haojian Zhuang
@ 2018-06-04 5:44 ` Haojian Zhuang
2018-06-05 14:56 ` [PATCH v8 edk-platforms 0/2] add platform " Leif Lindholm
2 siblings, 0 replies; 4+ messages in thread
From: Haojian Zhuang @ 2018-06-04 5:44 UTC (permalink / raw)
To: edk2-devel; +Cc: Haojian Zhuang, Laszlo Ersek, Leif Lindholm, Ard Biesheuvel
Create 4 predefined boot options for HiKey. They're
"Boot from SD", "Grub", "Android Boot" and "Android Fastboot".
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Acked-by: Laszlo Ersek <lersek@redhat.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>
---
Platform/Hisilicon/HiKey/HiKey.dec | 10 +-
Platform/Hisilicon/HiKey/HiKey.dsc | 7 +
Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.inf | 11 ++
Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.h | 2 +
Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.c | 164 ++++++++++++++++++++
5 files changed, 190 insertions(+), 4 deletions(-)
diff --git a/Platform/Hisilicon/HiKey/HiKey.dec b/Platform/Hisilicon/HiKey/HiKey.dec
index 537138eb45a1..d5de1a165910 100644
--- a/Platform/Hisilicon/HiKey/HiKey.dec
+++ b/Platform/Hisilicon/HiKey/HiKey.dec
@@ -1,5 +1,5 @@
#
-# Copyright (c) 2014-2017, Linaro Limited. All rights reserved.
+# Copyright (c) 2014-2018, Linaro Limited. All rights reserved.
#
# This program and the accompanying materials
# are licensed and made available under the terms and conditions of the BSD License
@@ -11,7 +11,7 @@
#
[Defines]
- DEC_SPECIFICATION = 0x00010019
+ DEC_SPECIFICATION = 0x0001001a
PACKAGE_NAME = HiKey
PACKAGE_GUID = d6db414d-ea67-4312-94d5-9c9e5b224d25
PACKAGE_VERSION = 0.1
@@ -32,5 +32,7 @@ [Guids.common]
gHiKeyTokenSpaceGuid = { 0x91148425, 0xcdd2, 0x4830, { 0x8b, 0xd0, 0xc6, 0x1c, 0x6d, 0xea, 0x36, 0x21 } }
[PcdsFixedAtBuild.common]
- gHiKeyTokenSpaceGuid.PcdAndroidFastbootNvmDevicePath|L""|VOID*|0x00000001
- gHiKeyTokenSpaceGuid.PcdArmFastbootFlashLimit|L""|VOID*|0x00000002
+ gHiKeyTokenSpaceGuid.PcdAndroidBootDevicePath|L""|VOID*|0x00000001
+ gHiKeyTokenSpaceGuid.PcdAndroidBootFile|{ 0x36, 0x8b, 0x73, 0x3a, 0xc5, 0xb9, 0x63, 0x47, 0xab, 0xbd, 0x6c, 0xbd, 0x4b, 0x25, 0xf9, 0xff }|VOID*|0x00000002
+ gHiKeyTokenSpaceGuid.PcdAndroidFastbootFile|{ 0x2a, 0x50, 0x88, 0x95, 0x70, 0x53, 0xe3, 0x11, 0x86, 0x31, 0xd7, 0xc5, 0x95, 0x13, 0x64, 0xc8 }|VOID*|0x00000003
+ gHiKeyTokenSpaceGuid.PcdSdBootDevicePath|L""|VOID*|0x00000004
diff --git a/Platform/Hisilicon/HiKey/HiKey.dsc b/Platform/Hisilicon/HiKey/HiKey.dsc
index 83dd68a820b1..9c1fc2e1b40d 100644
--- a/Platform/Hisilicon/HiKey/HiKey.dsc
+++ b/Platform/Hisilicon/HiKey/HiKey.dsc
@@ -138,6 +138,13 @@ [PcdsFixedAtBuild.common]
gEmbeddedTokenSpaceGuid.PcdAndroidFastbootUsbVendorId|0x18d1
gEmbeddedTokenSpaceGuid.PcdAndroidFastbootUsbProductId|0xd00d
+ #
+ # Android Loader
+ #
+ gHiKeyTokenSpaceGuid.PcdAndroidBootDevicePath|L"VenHw(0D51905B-B77E-452A-A2C0-ECA0CC8D514A,00D023F70000000000)/eMMC(0x0)/Ctrl(0x0)/HD(6,GPT,5C0F213C-17E1-4149-88C8-8B50FB4EC70E,0x7000,0x20000)/\\EFI\\BOOT\\GRUBAA64.EFI"
+ gHiKeyTokenSpaceGuid.PcdSdBootDevicePath|L"VenHw(0D51905B-B77E-452A-A2C0-ECA0CC8D514A,00E023F70000000000)/SD(0x0)"
+
+
################################################################################
#
# Components Section - list of all EDK II Modules needed by this Platform
diff --git a/Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.inf b/Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.inf
index 41aa7f8081ed..4c4644a0fe51 100644
--- a/Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.inf
+++ b/Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.inf
@@ -26,18 +26,29 @@ [Packages]
EmbeddedPkg/EmbeddedPkg.dec
MdeModulePkg/MdeModulePkg.dec
MdePkg/MdePkg.dec
+ Platform/Hisilicon/HiKey/HiKey.dec
[LibraryClasses]
CacheMaintenanceLib
DebugLib
IoLib
+ UefiBootManagerLib
UefiLib
UefiDriverEntryPoint
[Protocols]
+ gEfiDevicePathFromTextProtocolGuid
+ gEfiLoadedImageProtocolGuid
gEmbeddedGpioProtocolGuid
+ gPlatformBootManagerProtocolGuid
gPlatformVirtualKeyboardProtocolGuid
+[Pcd]
+ gHiKeyTokenSpaceGuid.PcdAndroidBootDevicePath
+ gHiKeyTokenSpaceGuid.PcdAndroidBootFile
+ gHiKeyTokenSpaceGuid.PcdAndroidFastbootFile
+ gHiKeyTokenSpaceGuid.PcdSdBootDevicePath
+
[Guids]
gEfiEndOfDxeEventGroupGuid
diff --git a/Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.h b/Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.h
index 104048135f27..0ababfa32eab 100644
--- a/Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.h
+++ b/Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.h
@@ -21,4 +21,6 @@
#define ADB_REBOOT_BOOTLOADER 0x77665500
#define ADB_REBOOT_NONE 0x77665501
+#define HIKEY_BOOT_OPTION_NUM 4
+
#endif /* __HIKEYDXE_H__ */
diff --git a/Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.c b/Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.c
index 832615f9cc80..2162b68529bd 100644
--- a/Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.c
+++ b/Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.c
@@ -14,11 +14,19 @@
#include <Library/CacheMaintenanceLib.h>
#include <Library/DebugLib.h>
+#include <Library/DevicePathLib.h>
#include <Library/IoLib.h>
+#include <Library/MemoryAllocationLib.h>
+#include <Library/PcdLib.h>
+#include <Library/UefiBootManagerLib.h>
#include <Library/UefiBootServicesTableLib.h>
#include <Library/UefiLib.h>
+#include <Protocol/DevicePath.h>
+#include <Protocol/DevicePathFromText.h>
#include <Protocol/EmbeddedGpio.h>
+#include <Protocol/LoadedImage.h>
+#include <Protocol/PlatformBootManager.h>
#include <Protocol/PlatformVirtualKeyboard.h>
#include <Hi6220.h>
@@ -99,6 +107,152 @@ HiKeyInitPeripherals (
return EFI_SUCCESS;
}
+STATIC
+EFI_STATUS
+CreatePlatformBootOptionFromPath (
+ IN CHAR16 *PathStr,
+ IN CHAR16 *Description,
+ IN OUT EFI_BOOT_MANAGER_LOAD_OPTION *BootOption
+ )
+{
+ EFI_STATUS Status;
+ EFI_DEVICE_PATH *DevicePath;
+
+ DevicePath = (EFI_DEVICE_PATH *)ConvertTextToDevicePath (PathStr);
+ ASSERT (DevicePath != NULL);
+ Status = EfiBootManagerInitializeLoadOption (
+ BootOption,
+ LoadOptionNumberUnassigned,
+ LoadOptionTypeBoot,
+ LOAD_OPTION_ACTIVE,
+ Description,
+ DevicePath,
+ NULL,
+ 0
+ );
+ FreePool (DevicePath);
+ return Status;
+}
+
+STATIC
+EFI_STATUS
+CreatePlatformBootOptionFromGuid (
+ IN EFI_GUID *FileGuid,
+ IN CHAR16 *Description,
+ IN OUT EFI_BOOT_MANAGER_LOAD_OPTION *BootOption
+ )
+{
+ EFI_STATUS Status;
+ EFI_DEVICE_PATH *DevicePath;
+ EFI_DEVICE_PATH *TempDevicePath;
+ EFI_LOADED_IMAGE_PROTOCOL *LoadedImage;
+ MEDIA_FW_VOL_FILEPATH_DEVICE_PATH FileNode;
+
+ Status = gBS->HandleProtocol (
+ gImageHandle,
+ &gEfiLoadedImageProtocolGuid,
+ (VOID **) &LoadedImage
+ );
+ ASSERT_EFI_ERROR (Status);
+ EfiInitializeFwVolDevicepathNode (&FileNode, FileGuid);
+ TempDevicePath = DevicePathFromHandle (LoadedImage->DeviceHandle);
+ ASSERT (TempDevicePath != NULL);
+ DevicePath = AppendDevicePathNode (
+ TempDevicePath,
+ (EFI_DEVICE_PATH_PROTOCOL *) &FileNode
+ );
+ ASSERT (DevicePath != NULL);
+ Status = EfiBootManagerInitializeLoadOption (
+ BootOption,
+ LoadOptionNumberUnassigned,
+ LoadOptionTypeBoot,
+ LOAD_OPTION_ACTIVE,
+ Description,
+ DevicePath,
+ NULL,
+ 0
+ );
+ FreePool (DevicePath);
+ return Status;
+}
+
+STATIC
+EFI_STATUS
+GetPlatformBootOptionsAndKeys (
+ OUT UINTN *BootCount,
+ OUT EFI_BOOT_MANAGER_LOAD_OPTION **BootOptions,
+ OUT EFI_INPUT_KEY **BootKeys
+ )
+{
+ EFI_GUID *FileGuid;
+ CHAR16 *PathStr;
+ EFI_STATUS Status;
+ UINTN Size;
+
+ Size = sizeof (EFI_BOOT_MANAGER_LOAD_OPTION) * HIKEY_BOOT_OPTION_NUM;
+ *BootOptions = (EFI_BOOT_MANAGER_LOAD_OPTION *)AllocateZeroPool (Size);
+ if (*BootOptions == NULL) {
+ DEBUG ((DEBUG_ERROR, "Failed to allocate memory for BootOptions\n"));
+ return EFI_OUT_OF_RESOURCES;
+ }
+ Size = sizeof (EFI_INPUT_KEY) * HIKEY_BOOT_OPTION_NUM;
+ *BootKeys = (EFI_INPUT_KEY *)AllocateZeroPool (Size);
+ if (*BootKeys == NULL) {
+ DEBUG ((DEBUG_ERROR, "Failed to allocate memory for BootKeys\n"));
+ Status = EFI_OUT_OF_RESOURCES;
+ goto Error;
+ }
+
+ PathStr = (CHAR16 *)PcdGetPtr (PcdSdBootDevicePath);
+ ASSERT (PathStr != NULL);
+ Status = CreatePlatformBootOptionFromPath (
+ PathStr,
+ L"Boot from SD",
+ &(*BootOptions)[0]
+ );
+ ASSERT_EFI_ERROR (Status);
+
+ PathStr = (CHAR16 *)PcdGetPtr (PcdAndroidBootDevicePath);
+ ASSERT (PathStr != NULL);
+ Status = CreatePlatformBootOptionFromPath (
+ PathStr,
+ L"Grub",
+ &(*BootOptions)[1]
+ );
+ ASSERT_EFI_ERROR (Status);
+
+ FileGuid = PcdGetPtr (PcdAndroidBootFile);
+ ASSERT (FileGuid != NULL);
+ Status = CreatePlatformBootOptionFromGuid (
+ FileGuid,
+ L"Android Boot",
+ &(*BootOptions)[2]
+ );
+ ASSERT_EFI_ERROR (Status);
+
+ FileGuid = PcdGetPtr (PcdAndroidFastbootFile);
+ ASSERT (FileGuid != NULL);
+ Status = CreatePlatformBootOptionFromGuid (
+ FileGuid,
+ L"Android Fastboot",
+ &(*BootOptions)[3]
+ );
+ ASSERT_EFI_ERROR (Status);
+ (*BootKeys)[3].ScanCode = SCAN_NULL;
+ (*BootKeys)[3].UnicodeChar = 'f';
+
+ *BootCount = 4;
+
+ return EFI_SUCCESS;
+Error:
+ FreePool (*BootOptions);
+ return Status;
+}
+
+PLATFORM_BOOT_MANAGER_PROTOCOL mPlatformBootManager = {
+ GetPlatformBootOptionsAndKeys
+};
+
EFI_STATUS
EFIAPI
VirtualKeyboardRegister (
@@ -203,5 +357,15 @@ HiKeyEntryPoint (
EFI_NATIVE_INTERFACE,
&mVirtualKeyboard
);
+ if (EFI_ERROR (Status)) {
+ return Status;
+ }
+
+ Status = gBS->InstallProtocolInterface (
+ &ImageHandle,
+ &gPlatformBootManagerProtocolGuid,
+ EFI_NATIVE_INTERFACE,
+ &mPlatformBootManager
+ );
return Status;
}
--
2.7.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v8 edk-platforms 0/2] add platform boot options
2018-06-04 5:44 [PATCH v8 edk-platforms 0/2] add platform boot options Haojian Zhuang
2018-06-04 5:44 ` [PATCH v8 edk-platforms 1/2] Platform/HiKey960: register predefined " Haojian Zhuang
2018-06-04 5:44 ` [PATCH v8 edk-platforms 2/2] Platform/HiKey: create 4 " Haojian Zhuang
@ 2018-06-05 14:56 ` Leif Lindholm
2 siblings, 0 replies; 4+ messages in thread
From: Leif Lindholm @ 2018-06-05 14:56 UTC (permalink / raw)
To: Haojian Zhuang; +Cc: edk2-devel, Laszlo Ersek, Ard Biesheuvel
Thanks Haoijian,
Series Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
Pushed as 9bb89f3323..da3e09e28d.
On Mon, Jun 04, 2018 at 01:44:36PM +0800, Haojian Zhuang wrote:
> Changelog:
> v8:
> * Rebase for virtual keyboard driver.
> v7:
> * Fix memory leakage on DevicePath.
> v6:
> * Remove redundant definition -- "GRUB_FILE_NAME".
> v5:
> * Avoid to merge device path and grub's file path in driver.
> Merge them directly in DSC file.
> * Avoid duplicated code to create boot options.
> * Use goto to handle error condition.
> v4:
> * Add BootCount parameter in the interface.
> v3:
> * Move in initilization of boot entry.
> * Update the name of interface in platform boot manager protocol.
> v2:
> * Update with platform boot manager protocol.
>
> Haojian Zhuang (2):
> Platform/HiKey960: register predefined boot options
> Platform/HiKey: create 4 boot options
>
> Platform/Hisilicon/HiKey/HiKey.dec | 10 +-
> Platform/Hisilicon/{HiKey/HiKey.dec => HiKey960/HiKey960.dec} | 19 ++-
> Platform/Hisilicon/HiKey/HiKey.dsc | 7 +
> Platform/Hisilicon/HiKey960/HiKey960.dsc | 6 +
> Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.inf | 11 ++
> Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.inf | 11 ++
> Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.h | 2 +
> Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.h | 20 +--
> Platform/Hisilicon/HiKey/HiKeyDxe/HiKeyDxe.c | 164 +++++++++++++++++++
> Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.c | 166 ++++++++++++++++++++
> 10 files changed, 384 insertions(+), 32 deletions(-)
> copy Platform/Hisilicon/{HiKey/HiKey.dec => HiKey960/HiKey960.dec} (54%)
>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Cc: Leif Lindholm <leif.lindholm@linaro.org>
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> --
> 2.7.4
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-06-05 14:56 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-06-04 5:44 [PATCH v8 edk-platforms 0/2] add platform boot options Haojian Zhuang
2018-06-04 5:44 ` [PATCH v8 edk-platforms 1/2] Platform/HiKey960: register predefined " Haojian Zhuang
2018-06-04 5:44 ` [PATCH v8 edk-platforms 2/2] Platform/HiKey: create 4 " Haojian Zhuang
2018-06-05 14:56 ` [PATCH v8 edk-platforms 0/2] add platform " Leif Lindholm
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox