Delete BdsLib dependency from Android loaders

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

Posted by Leif Lindholm 7 years ago

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

Re: [edk2] [PATCH 0/2] Delete BdsLib dependency from Android loaders

Posted by Ard Biesheuvel 7 years ago

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

Re: [edk2] [PATCH 0/2] Delete BdsLib dependency from Android loaders

Posted by Leif Lindholm 7 years ago

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