mbox series

[edk2,0/2] Delete BdsLib dependency from Android loaders

Message ID 20171125132735.3507-1-leif.lindholm@linaro.org
Headers show
Series Delete BdsLib dependency from Android loaders | expand

Message

Leif Lindholm Nov. 25, 2017, 1:27 p.m. UTC 
AndroidBoot and AndroidFastboot (as well as their shared component
AndroidBootImgLib) have minor dependencies on BdsLib.

First fix up a missing mapping in EmbeddedPkg.dsc to let the modules
build standalone. Then get rid of the use of BdsLib.

Leif Lindholm (2):
  EmbeddedPkg: add UefiRuntimeLib resolution to package .dsc
  EmbeddedPkg: get rid of BdsLib dependency from Android*Boot

 EmbeddedPkg/Application/AndroidBoot/AndroidBootApp.c             | 4 ++--
 EmbeddedPkg/Application/AndroidBoot/AndroidBootApp.inf           | 1 -
 EmbeddedPkg/Application/AndroidFastboot/Arm/BootAndroidBootImg.c | 1 -
 EmbeddedPkg/EmbeddedPkg.dsc                                      | 7 ++-----
 EmbeddedPkg/Library/AndroidBootImgLib/AndroidBootImgLib.inf      | 1 +
 5 files changed, 5 insertions(+), 9 deletions(-)

-- 
2.11.0

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Comments

Ard Biesheuvel Nov. 25, 2017, 3:25 p.m. UTC | #1 
On 25 November 2017 at 13:27, Leif Lindholm <leif.lindholm@linaro.org> wrote:
> AndroidBoot and AndroidFastboot (as well as their shared component

> AndroidBootImgLib) have minor dependencies on BdsLib.

>

> First fix up a missing mapping in EmbeddedPkg.dsc to let the modules

> build standalone. Then get rid of the use of BdsLib.

>

> Leif Lindholm (2):

>   EmbeddedPkg: add UefiRuntimeLib resolution to package .dsc

>   EmbeddedPkg: get rid of BdsLib dependency from Android*Boot

>


Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>


>  EmbeddedPkg/Application/AndroidBoot/AndroidBootApp.c             | 4 ++--

>  EmbeddedPkg/Application/AndroidBoot/AndroidBootApp.inf           | 1 -

>  EmbeddedPkg/Application/AndroidFastboot/Arm/BootAndroidBootImg.c | 1 -

>  EmbeddedPkg/EmbeddedPkg.dsc                                      | 7 ++-----

>  EmbeddedPkg/Library/AndroidBootImgLib/AndroidBootImgLib.inf      | 1 +

>  5 files changed, 5 insertions(+), 9 deletions(-)

>

> --

> 2.11.0

>

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Leif Lindholm Nov. 25, 2017, 7:02 p.m. UTC | #2 
On Sat, Nov 25, 2017 at 03:25:21PM +0000, Ard Biesheuvel wrote:
> On 25 November 2017 at 13:27, Leif Lindholm <leif.lindholm@linaro.org> wrote:

> > AndroidBoot and AndroidFastboot (as well as their shared component

> > AndroidBootImgLib) have minor dependencies on BdsLib.

> >

> > First fix up a missing mapping in EmbeddedPkg.dsc to let the modules

> > build standalone. Then get rid of the use of BdsLib.

> >

> > Leif Lindholm (2):

> >   EmbeddedPkg: add UefiRuntimeLib resolution to package .dsc

> >   EmbeddedPkg: get rid of BdsLib dependency from Android*Boot

> >

> 

> Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>


Thanks!
Pushed as 108ff4a04b..a64d587294.

/
    Leif
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel