diff mbox

[U-Boot,2/3] x86: Add a reference to README.efi

Message ID 1468740226-27470-2-git-send-email-sjg@chromium.org
State Accepted
Delegated to: Bin Meng
Headers show

Commit Message

Simon Glass July 17, 2016, 7:23 a.m. UTC
UEFI is commonly used on x86. Add a reference to U-Boot's support for this
in the x86 README.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

 doc/README.x86 | 11 +++++++++++
 1 file changed, 11 insertions(+)

Comments

Bin Meng July 18, 2016, 1:27 a.m. UTC | #1
On Sun, Jul 17, 2016 at 3:23 PM, Simon Glass <sjg@chromium.org> wrote:
> UEFI is commonly used on x86. Add a reference to U-Boot's support for this
> in the x86 README.
>
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
>
>  doc/README.x86 | 11 +++++++++++
>  1 file changed, 11 insertions(+)
>

Reviewed-by: Bin Meng <bmeng.cn@gmail.com>

But one issue below:

> diff --git a/doc/README.x86 b/doc/README.x86
> index 7d694b1..04e7a54 100644
> --- a/doc/README.x86
> +++ b/doc/README.x86
> @@ -1035,10 +1035,21 @@ command from the OS.
>  For other platform boards, ACPI support status can be checked by examining their
>  board defconfig files to see if CONFIG_GENERATE_ACPI_TABLE is set to y.
>
> +EFI Support
> +-----------
> +U-Boot supports booting as a 32-bit or 64-bit EFI payload, e.g. with UEFI.
> +This is enabled with CONFIG_EFI_STUB. U-Boot can also run as an EFI
> +application, with CONFIG_EFI_APP. The CONFIG_EFI_LOADER option, where U-Booot
> +provides an EFI environment to the kernel (i.e. replaces UEFI completely but
> +provides the same EFI run-time services) is not currently supports on x86.

is not currently "supported"

I can fix this when applying

> +
> +See README.efi for details of EFI support in U-Boot.
> +
>  TODO List
>  ---------
>  - Audio
>  - Chrome OS verified boot
> +- Support for CONFIG_EFI_LOADER
>
>  References
>  ----------
> --

Regards,
Bin
Simon Glass July 18, 2016, 11:55 a.m. UTC | #2
Hi Bin,

On 17 July 2016 at 19:27, Bin Meng <bmeng.cn@gmail.com> wrote:
> On Sun, Jul 17, 2016 at 3:23 PM, Simon Glass <sjg@chromium.org> wrote:
>> UEFI is commonly used on x86. Add a reference to U-Boot's support for this
>> in the x86 README.
>>
>> Signed-off-by: Simon Glass <sjg@chromium.org>
>> ---
>>
>>  doc/README.x86 | 11 +++++++++++
>>  1 file changed, 11 insertions(+)
>>
>
> Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
>
> But one issue below:
>
>> diff --git a/doc/README.x86 b/doc/README.x86
>> index 7d694b1..04e7a54 100644
>> --- a/doc/README.x86
>> +++ b/doc/README.x86
>> @@ -1035,10 +1035,21 @@ command from the OS.
>>  For other platform boards, ACPI support status can be checked by examining their
>>  board defconfig files to see if CONFIG_GENERATE_ACPI_TABLE is set to y.
>>
>> +EFI Support
>> +-----------
>> +U-Boot supports booting as a 32-bit or 64-bit EFI payload, e.g. with UEFI.
>> +This is enabled with CONFIG_EFI_STUB. U-Boot can also run as an EFI
>> +application, with CONFIG_EFI_APP. The CONFIG_EFI_LOADER option, where U-Booot
>> +provides an EFI environment to the kernel (i.e. replaces UEFI completely but
>> +provides the same EFI run-time services) is not currently supports on x86.
>
> is not currently "supported"
>
> I can fix this when applying

OK thank you.

>
>> +
>> +See README.efi for details of EFI support in U-Boot.
>> +
>>  TODO List
>>  ---------
>>  - Audio
>>  - Chrome OS verified boot
>> +- Support for CONFIG_EFI_LOADER
>>
>>  References
>>  ----------
>> --
>
> Regards,
> Bin

Regards,
Simon
Bin Meng July 19, 2016, 3:40 a.m. UTC | #3
On Mon, Jul 18, 2016 at 9:27 AM, Bin Meng <bmeng.cn@gmail.com> wrote:
> On Sun, Jul 17, 2016 at 3:23 PM, Simon Glass <sjg@chromium.org> wrote:
>> UEFI is commonly used on x86. Add a reference to U-Boot's support for this
>> in the x86 README.
>>
>> Signed-off-by: Simon Glass <sjg@chromium.org>
>> ---
>>
>>  doc/README.x86 | 11 +++++++++++
>>  1 file changed, 11 insertions(+)
>>
>
> Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
>
> But one issue below:
>
>> diff --git a/doc/README.x86 b/doc/README.x86
>> index 7d694b1..04e7a54 100644
>> --- a/doc/README.x86
>> +++ b/doc/README.x86
>> @@ -1035,10 +1035,21 @@ command from the OS.
>>  For other platform boards, ACPI support status can be checked by examining their
>>  board defconfig files to see if CONFIG_GENERATE_ACPI_TABLE is set to y.
>>
>> +EFI Support
>> +-----------
>> +U-Boot supports booting as a 32-bit or 64-bit EFI payload, e.g. with UEFI.
>> +This is enabled with CONFIG_EFI_STUB. U-Boot can also run as an EFI
>> +application, with CONFIG_EFI_APP. The CONFIG_EFI_LOADER option, where U-Booot
>> +provides an EFI environment to the kernel (i.e. replaces UEFI completely but
>> +provides the same EFI run-time services) is not currently supports on x86.
>
> is not currently "supported"
>
> I can fix this when applying
>

Fixed this, and

applied to u-boot-x86, thanks!
diff mbox

Patch

diff --git a/doc/README.x86 b/doc/README.x86
index 7d694b1..04e7a54 100644
--- a/doc/README.x86
+++ b/doc/README.x86
@@ -1035,10 +1035,21 @@  command from the OS.
 For other platform boards, ACPI support status can be checked by examining their
 board defconfig files to see if CONFIG_GENERATE_ACPI_TABLE is set to y.
 
+EFI Support
+-----------
+U-Boot supports booting as a 32-bit or 64-bit EFI payload, e.g. with UEFI.
+This is enabled with CONFIG_EFI_STUB. U-Boot can also run as an EFI
+application, with CONFIG_EFI_APP. The CONFIG_EFI_LOADER option, where U-Booot
+provides an EFI environment to the kernel (i.e. replaces UEFI completely but
+provides the same EFI run-time services) is not currently supports on x86.
+
+See README.efi for details of EFI support in U-Boot.
+
 TODO List
 ---------
 - Audio
 - Chrome OS verified boot
+- Support for CONFIG_EFI_LOADER
 
 References
 ----------