From: Alexander Graf <agraf@suse.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 5/8] arm: efi: Add a hello world test program
Date: Tue, 4 Oct 2016 17:50:10 +0200 [thread overview]
Message-ID: <8A822FA8-DD6F-4E16-A51B-C60956412C65@suse.de> (raw)
In-Reply-To: <CAPnjgZ28KihPP-72JiR6reKYGP5oW11=PPqbL3km7dc58tyNrw@mail.gmail.com>
> Am 04.10.2016 um 17:37 schrieb Simon Glass <sjg@chromium.org>:
>
> Hi Alex,
>
>> On 3 October 2016 at 21:15, Alexander Graf <agraf@suse.de> wrote:
>>
>>
>> Am 03.10.2016 um 23:50 schrieb Simon Glass <sjg@chromium.org>:
>>
>> Hi,
>>
>> On 27 September 2016 at 15:28, Tom Rini <trini@konsulko.com> wrote:
>>
>> On Mon, Sep 26, 2016 at 09:36:19AM +0200, Alexander Graf wrote:
>>
>>
>>
>> On 25.09.16 23:27, Simon Glass wrote:
>>
>> It is useful to have a basic sanity check for EFI loader support. Add a
>>
>> 'bootefi hello' command which loads HelloWord.efi and runs it under U-Boot.
>>
>>
>> Signed-off-by: Simon Glass <sjg@chromium.org>
>>
>> ---
>>
>>
>> Changes in v2: None
>>
>>
>> arch/arm/lib/HelloWorld32.efi | Bin 0 -> 11712 bytes
>>
>>
>> IIRC U-Boot as a whole is GPL licensed, which means that any binaries
>>
>> shipped inside would also need to be GPL compatibly licensed which again
>>
>> means that the source code (and build instructions?) for this .efi file
>>
>> would need to be part of the tree, no?
>>
>>
>> Yeah, I'm not super comfortable with this.
>>
>>
>> Do you think we should drop these binary patches? I could always put
>> the binaries somewhere along with instructions on how to get them.
>>
>>
>> I think that's the best option, yes. You can always just add a url to the
>> readme to point people into the right direction.
>
> OK. One problem is that we cannot write a test for it unless we
> actually run an EFI application.
Well, you could always provide a binary disk image that you run in qemu as test case. That one doesn't have to be gpl compliant thn because it's not derived work :).
>
>>
>>
>> I do think it is useful to be able to test the platform though.
>>
>>
>> I don't disagree, but I would argue that for the average u-boot user it
>> brings no additional value ;). And people like you who know how to enable a
>> new architecture probably also know how to get a file into their target's
>> memory.
>
> I wonder if we can build our own hello world application? I think I
> did it once. But there is EFI library code that we would need to bring
> in (perhaps a small amount).
We could. The main problem is the PE header.
Maybe we can trick around that with bincopy -O binary though. Hmm :).
Alex
>
> Regards,
> Simon
next prev parent reply other threads:[~2016-10-04 15:50 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-25 21:27 [U-Boot] [PATCH v2 1/8] x86: Add implementations of setjmp() and longjmp() Simon Glass
2016-09-25 21:27 ` [U-Boot] [PATCH v2 2/8] efi: Use asmlinkage for EFIAPI Simon Glass
2016-09-26 8:50 ` Bin Meng
2016-10-13 14:35 ` [U-Boot] [U-Boot,v2,2/8] " Alexander Graf
2016-09-25 21:27 ` [U-Boot] [PATCH v2 3/8] efi: Fix missing EFIAPI specifiers Simon Glass
2016-10-13 14:35 ` [U-Boot] [U-Boot,v2,3/8] " Alexander Graf
2016-09-25 21:27 ` [U-Boot] [PATCH v2 4/8] x86: Tidy up selection of building the EFI stub Simon Glass
2016-09-26 8:50 ` Bin Meng
2016-09-27 0:35 ` Simon Glass
2016-09-27 2:44 ` Bin Meng
2016-09-27 17:55 ` Simon Glass
2016-09-28 1:23 ` Bin Meng
2016-09-28 14:43 ` Simon Glass
2016-09-29 3:37 ` Bin Meng
2016-09-29 5:08 ` Alexander Graf
2016-09-29 5:37 ` Bin Meng
2016-09-29 7:13 ` Alexander Graf
2016-09-29 7:28 ` Bin Meng
2016-09-29 8:06 ` Alexander Graf
2016-09-29 8:24 ` Bin Meng
2016-09-25 21:27 ` [U-Boot] [PATCH v2 5/8] arm: efi: Add a hello world test program Simon Glass
2016-09-26 7:36 ` Alexander Graf
2016-09-27 21:28 ` Tom Rini
2016-10-03 21:50 ` Simon Glass
2016-10-04 3:15 ` Alexander Graf
2016-10-04 15:37 ` Simon Glass
2016-10-04 15:50 ` Alexander Graf [this message]
2016-10-18 20:37 ` Simon Glass
2016-10-19 7:07 ` Alexander Graf
2016-11-07 15:45 ` Simon Glass
2016-09-26 8:50 ` Bin Meng
2016-09-25 21:27 ` [U-Boot] [PATCH v2 6/8] x86: efi: Add EFI loader support for x86 Simon Glass
2016-10-13 14:35 ` [U-Boot] [U-Boot, v2, " Alexander Graf
2016-09-25 21:27 ` [U-Boot] [PATCH v2 7/8] x86: efi: Add a hello world test program Simon Glass
2016-09-25 21:27 ` [U-Boot] [PATCH v2 8/8] x86: Enable EFI loader support Simon Glass
2016-09-26 7:00 ` [U-Boot] [PATCH v2 1/8] x86: Add implementations of setjmp() and longjmp() Bin Meng
2016-09-26 7:05 ` Alexander Graf
2016-09-26 7:21 ` Bin Meng
2016-09-26 7:28 ` Alexander Graf
2016-09-27 0:34 ` Simon Glass
2016-09-27 2:59 ` Bin Meng
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8A822FA8-DD6F-4E16-A51B-C60956412C65@suse.de \
--to=agraf@suse.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.