diff mbox series

[U-Boot,01/18] x86: doc: Fix reference to EFI doc in U-Boot

Message ID 1528637118-32739-2-git-send-email-bmeng.cn@gmail.com
State Superseded
Delegated to: Bin Meng
Headers show
Series x86: efi: Fixes and enhancements to application and payload support | expand

Commit Message

Bin Meng June 10, 2018, 1:25 p.m. UTC
Since commit f3b5056c4e72 ("efi_loader: split README.efi into two
separate documents"), the original README.efi was renamed to
README.u-boot_on_efi, but x86 doc still refers to the old one.

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

 doc/README.x86 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Heinrich Schuchardt June 10, 2018, 1:39 p.m. UTC | #1
On 06/10/2018 03:25 PM, Bin Meng wrote:
> Since commit f3b5056c4e72 ("efi_loader: split README.efi into two
> separate documents"), the original README.efi was renamed to
> README.u-boot_on_efi, but x86 doc still refers to the old one.
> 
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
> ---
> 
>  doc/README.x86 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/doc/README.x86 b/doc/README.x86
> index 78664c3..865a965 100644
> --- a/doc/README.x86
> +++ b/doc/README.x86
> @@ -1139,7 +1139,7 @@ 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 supported on x86.
>  
> -See README.efi for details of EFI support in U-Boot.
> +See  for details of EFI support in U-Boot.

Thanks for updating the references.

The chapter "EFI Support" refers both to running U-Boot as an EFI
payload as well as using U-Boot as an EFI platform. So both
doc/README.u-boot_on_efi and doc/README.uefi should be referenced.

But I would prefer if you could also move the sentences concerning EFI
that has spilled over into "64-bit Support" into the "EFI Support" chapter.

Best regards

Heinrich

>  
>  64-bit Support
>  --------------
>
Bin Meng June 10, 2018, 2:01 p.m. UTC | #2
Hi Heinrich,

On Sun, Jun 10, 2018 at 9:39 PM, Heinrich Schuchardt <xypron.glpk@gmx.de> wrote:
> On 06/10/2018 03:25 PM, Bin Meng wrote:
>> Since commit f3b5056c4e72 ("efi_loader: split README.efi into two
>> separate documents"), the original README.efi was renamed to
>> README.u-boot_on_efi, but x86 doc still refers to the old one.
>>
>> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
>> ---
>>
>>  doc/README.x86 | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/doc/README.x86 b/doc/README.x86
>> index 78664c3..865a965 100644
>> --- a/doc/README.x86
>> +++ b/doc/README.x86
>> @@ -1139,7 +1139,7 @@ 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 supported on x86.
>>
>> -See README.efi for details of EFI support in U-Boot.
>> +See  for details of EFI support in U-Boot.
>
> Thanks for updating the references.
>
> The chapter "EFI Support" refers both to running U-Boot as an EFI
> payload as well as using U-Boot as an EFI platform. So both
> doc/README.u-boot_on_efi and doc/README.uefi should be referenced.
>

OK, will do in v2. One question: the doc says: CONFIG_EFI_LOADER
option is not currently supported on x86. Is this still the case, or
outdated? (I haven't tried the EFI loader on x86 or ARM yet)

> But I would prefer if you could also move the sentences concerning EFI
> that has spilled over into "64-bit Support" into the "EFI Support" chapter.
>

Sure.

Regards,
Bin
Heinrich Schuchardt June 10, 2018, 5:46 p.m. UTC | #3
On 06/10/2018 04:01 PM, Bin Meng wrote:
> Hi Heinrich,
> 
> On Sun, Jun 10, 2018 at 9:39 PM, Heinrich Schuchardt <xypron.glpk@gmx.de> wrote:
>> On 06/10/2018 03:25 PM, Bin Meng wrote:
>>> Since commit f3b5056c4e72 ("efi_loader: split README.efi into two
>>> separate documents"), the original README.efi was renamed to
>>> README.u-boot_on_efi, but x86 doc still refers to the old one.
>>>
>>> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
>>> ---
>>>
>>>  doc/README.x86 | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/doc/README.x86 b/doc/README.x86
>>> index 78664c3..865a965 100644
>>> --- a/doc/README.x86
>>> +++ b/doc/README.x86
>>> @@ -1139,7 +1139,7 @@ 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 supported on x86.
>>>
>>> -See README.efi for details of EFI support in U-Boot.
>>> +See  for details of EFI support in U-Boot.
>>
>> Thanks for updating the references.
>>
>> The chapter "EFI Support" refers both to running U-Boot as an EFI
>> payload as well as using U-Boot as an EFI platform. So both
>> doc/README.u-boot_on_efi and doc/README.uefi should be referenced.
>>
> 
> OK, will do in v2. One question: the doc says: CONFIG_EFI_LOADER
> option is not currently supported on x86. Is this still the case, or
> outdated? (I haven't tried the EFI loader on x86 or ARM yet)

qemu-x86_defconfig has CONFIG_EFI_LOADER enabled by default.
bootefi selftest is successful.

But loading Linux fails by trying to use video memory as normal memory.

E.g. memory reservations according to e820 table are missing.

Patch https://patchwork.ozlabs.org/patch/860400/ is not sufficient to
resolve the issue.

I lack the knowledge which memory regions have to be reserved on the
platform.

Best regards

Heinrich

> 
>> But I would prefer if you could also move the sentences concerning EFI
>> that has spilled over into "64-bit Support" into the "EFI Support" chapter.
>>
> 
> Sure.
> 
> Regards,
> Bin
>
diff mbox series

Patch

diff --git a/doc/README.x86 b/doc/README.x86
index 78664c3..865a965 100644
--- a/doc/README.x86
+++ b/doc/README.x86
@@ -1139,7 +1139,7 @@  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 supported on x86.
 
-See README.efi for details of EFI support in U-Boot.
+See README.u-boot_on_efi for details of EFI support in U-Boot.
 
 64-bit Support
 --------------