public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Alexander Graf <agraf@suse.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 02/14] include/efi_api.h: Add more detailed API definitions
Date: Mon, 1 Feb 2016 23:46:45 +0100	[thread overview]
Message-ID: <56AFE055.4090002@suse.de> (raw)
In-Reply-To: <CAPnjgZ2A6HZqtMo=qfH3hrGRZquAgnCg=PAkQYjdsbFj8yrR5g@mail.gmail.com>



On 01/31/2016 04:17 PM, Simon Glass wrote:
> On 14 January 2016 at 22:06, Alexander Graf <agraf@suse.de> wrote:
>> The EFI API header is great, but missing a good chunk of function prototype,
>> GUID defines and enum declarations.
>>
>> This patch extends it to cover more of the EFI API. It's still not 100%
>> complete, but sufficient enough for our EFI payload interface.
>>
>> Signed-off-by: Alexander Graf <agraf@suse.de>
>> ---
>>   include/efi_api.h | 197 ++++++++++++++++++++++++++++++++++++++++++++----------
>>   1 file changed, 160 insertions(+), 37 deletions(-)
> Reviewed-by: Simon Glass <sjg@chromium.org>
>
> But does the notify_function parameter need EFIAPI also?

Yes, probably. The reason nobody realized it is that there's nobody 
calling it today :). Only 32bit arm and ia64 grub2 make use of it. Linux 
doesn't use the callback at all.

But I'll change it.


Alex

  reply	other threads:[~2016-02-01 22:46 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-15  5:06 [U-Boot] [PATCH 00/14] EFI payload / application support v2 Alexander Graf
2016-01-15  5:06 ` [U-Boot] [PATCH 01/14] disk/part.c: Expose list of available block drivers Alexander Graf
2016-01-15  5:06 ` [U-Boot] [PATCH 02/14] include/efi_api.h: Add more detailed API definitions Alexander Graf
2016-01-31 15:17   ` Simon Glass
2016-02-01 22:46     ` Alexander Graf [this message]
2016-01-15  5:06 ` [U-Boot] [PATCH 03/14] efi_loader: Add PE image loader Alexander Graf
2016-01-31 15:18   ` Simon Glass
2016-02-01 22:58     ` Alexander Graf
2016-01-15  5:06 ` [U-Boot] [PATCH 04/14] efi_loader: Add boot time services Alexander Graf
2016-01-20  0:16   ` Leif Lindholm
2016-02-02  1:52     ` Alexander Graf
2016-01-31 15:19   ` Simon Glass
2016-02-01 23:45     ` Alexander Graf
2016-02-01 23:54       ` Simon Glass
2016-02-02  0:02         ` Alexander Graf
2016-01-15  5:06 ` [U-Boot] [PATCH 05/14] efi_loader: Add console interface Alexander Graf
2016-01-31 15:19   ` Simon Glass
2016-01-15  5:06 ` [U-Boot] [PATCH 06/14] efi_loader: Add runtime services Alexander Graf
2016-01-21 17:20   ` Leif Lindholm
2016-01-31 15:20   ` Simon Glass
2016-02-01 23:57     ` Alexander Graf
2016-01-15  5:06 ` [U-Boot] [PATCH 07/14] efi_loader: Add disk interfaces Alexander Graf
2016-01-31 15:23   ` Simon Glass
2016-02-02  0:32     ` Alexander Graf
2016-01-15  5:06 ` [U-Boot] [PATCH 08/14] efi_loader: Add "bootefi" command Alexander Graf
2016-01-31 15:23   ` Simon Glass
2016-01-15  5:06 ` [U-Boot] [PATCH 09/14] efi_loader: Implement memory allocation and map Alexander Graf
2016-01-31 15:23   ` Simon Glass
2016-02-02  0:59     ` Alexander Graf
2016-01-15  5:06 ` [U-Boot] [PATCH 10/14] arm64: Allow exceptions to return Alexander Graf
2016-01-15  5:06 ` [U-Boot] [PATCH 11/14] arm64: Allow EFI payload code to take exceptions Alexander Graf
2016-01-15  5:06 ` [U-Boot] [PATCH 12/14] efi_loader: Add DCACHE_OFF support for arm64 Alexander Graf
2016-01-15  5:06 ` [U-Boot] [PATCH 13/14] efi_loader: hook up in build environment Alexander Graf
2016-01-31 15:24   ` Simon Glass
2016-01-15  5:06 ` [U-Boot] [PATCH 14/14] efi_loader: Add distro boot script for removable media Alexander Graf
2016-01-31 15:24   ` Simon Glass
2016-02-02  1:05     ` Alexander Graf
2016-01-31 15:17 ` [U-Boot] [PATCH 00/14] EFI payload / application support v2 Simon Glass
2016-01-31 21:43   ` Alexander Graf
2016-02-01  2:52     ` Simon Glass
2016-02-01  3:25       ` Simon Glass
2016-02-01 21:38       ` Alexander Graf
2016-02-02  0:02         ` Simon Glass
2016-02-02  0:16           ` Alexander Graf
2016-02-02  0:28             ` Simon Glass

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=56AFE055.4090002@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox