mbox series

[U-Boot,0/3] Clean up make process for EFI payload

Message ID 20170905011939.5794-1-xypron.glpk@gmx.de
Headers show
Series Clean up make process for EFI payload | expand

Message

Heinrich Schuchardt Sept. 5, 2017, 1:19 a.m. UTC 
Currently we have a single EFI application that we can deliver
integrated into U-Boot. It is a hello world application that
can be called with 'bootefi hello'.

The Makefiles do not easily accomodate further EFI payloads
like a unit test for the EFI API or an EFI shell.

This patch series changes Makefile.lib to allow additional
EFI payloads. Required fixes to the helloworld app are
provided.

Further work will be needed to accomodate EFI payloads built
out of multiple source files.

Heinrich Schuchardt (3):
  efi_loader: rename __efi_hello_world_*
  scripts/Makefile.lib: generalize building built in EFI app
  efi_loader: usage of always in Makefile

 cmd/bootefi.c                  |  4 ++--
 include/asm-generic/sections.h |  4 ++--
 lib/efi_loader/Makefile        |  5 +++--
 scripts/Makefile.lib           | 28 +++++++++++++++-------------
 4 files changed, 22 insertions(+), 19 deletions(-)

Comments

Andy Shevchenko Sept. 5, 2017, 2:23 p.m. UTC | #1 
On Tue, 2017-09-05 at 03:19 +0200, Heinrich Schuchardt wrote:
> Currently we have a single EFI application that we can deliver
> integrated into U-Boot. It is a hello world application that
> can be called with 'bootefi hello'.
> 
> The Makefiles do not easily accomodate further EFI payloads
> like a unit test for the EFI API or an EFI shell.
> 
> This patch series changes Makefile.lib to allow additional
> EFI payloads. Required fixes to the helloworld app are
> provided.
> 
> Further work will be needed to accomodate EFI payloads built
> out of multiple source files.

Please, amend .gitignore, if needed, according to the changes.

> 
> Heinrich Schuchardt (3):
>   efi_loader: rename __efi_hello_world_*
>   scripts/Makefile.lib: generalize building built in EFI app
>   efi_loader: usage of always in Makefile
> 
>  cmd/bootefi.c                  |  4 ++--
>  include/asm-generic/sections.h |  4 ++--
>  lib/efi_loader/Makefile        |  5 +++--
>  scripts/Makefile.lib           | 28 +++++++++++++++-------------
>  4 files changed, 22 insertions(+), 19 deletions(-)
>
Heinrich Schuchardt Sept. 6, 2017, 1:50 a.m. UTC | #2 
On 09/05/2017 04:23 PM, Andy Shevchenko wrote:
> On Tue, 2017-09-05 at 03:19 +0200, Heinrich Schuchardt wrote:
>> Currently we have a single EFI application that we can deliver
>> integrated into U-Boot. It is a hello world application that
>> can be called with 'bootefi hello'.
>>
>> The Makefiles do not easily accomodate further EFI payloads
>> like a unit test for the EFI API or an EFI shell.
>>
>> This patch series changes Makefile.lib to allow additional
>> EFI payloads. Required fixes to the helloworld app are
>> provided.
>>
>> Further work will be needed to accomodate EFI payloads built
>> out of multiple source files.
> 
> Please, amend .gitignore, if needed, according to the changes.

In this old version of the patch series no new files are created.

In the v2 version dependency files are created/updated which we want to
add to the source repository keep.

Best regards

Heinrich

> 
>>
>> Heinrich Schuchardt (3):
>>   efi_loader: rename __efi_hello_world_*
>>   scripts/Makefile.lib: generalize building built in EFI app
>>   efi_loader: usage of always in Makefile
>>
>>  cmd/bootefi.c                  |  4 ++--
>>  include/asm-generic/sections.h |  4 ++--
>>  lib/efi_loader/Makefile        |  5 +++--
>>  scripts/Makefile.lib           | 28 +++++++++++++++-------------
>>  4 files changed, 22 insertions(+), 19 deletions(-)
>>
>