[U-Boot] [PATCH v2] efi: Update README.efi to clarify build and test instructions

[Bin Meng]

Hi Igor,

On Mon, Aug 17, 2015 at 3:10 PM, Stoppa, Igor <igor.stoppa at intel.com> wrote:
> Hello Bin,
> a couple more of last-minute comments below:
>
> On 15 August 2015 at 04:44, Bin Meng <bmeng.cn at gmail.com> wrote:
>
>
>> +For building an EFI payload, run:
>>     make qemu-x86_defconfig
>
> Since qemu-x86_defconfig was an example, maybe this could be written as:
>
> make <selected defconfig, i.e. qemu-x86_defconfig>
> change EFI settings according to what described above
> make
>

OK.

>
>>  Trying it out
>
> Would it be worth mentioning somewhere here that the
> payload/application obtained out of the build process should be
> deployed as
> /efi/boot/bootx64.efi or /efi/boot/bootia32.efi, to make it load
> automatically from the BIOS?
>

No, I don't think this should be documented. This is EFI spec, not
U-Boot. And if we document this, what if someone else (not you) who
don't like the automatically loading the application complains? Each
doc should just describe its own scope.

> Strictly speaking this is not U-Boot business, but it might simplify
> the life of the users to state it explicitly.
>

Yep, and it is a balance. I am not saying current doc is perfect, but
we just simply cannot document everything, or overdocumeting.

> One other thing that I haven't fully investigated yet (so maybe it's
> already explained elsewhere) is: after the application/payload is
> built for x86 EFI,
> how to deal with the environment partition?
>

What environment partition?

> If this is the end of the x86/efi specific documentation and one
> should refer to something else, it would be good to have a
> link/reference to the next doc.
>

Regards,
Bin