public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [PATCH edk2-platforms 1/2] Platform/ARM: remove DmaLib library class resolution
@ 2017-08-30  9:52 Ard Biesheuvel
  2017-08-30  9:52 ` [PATCH edk2-platforms 2/2] Platform/HiKey: " Ard Biesheuvel
  2017-08-30 13:07 ` [PATCH edk2-platforms 1/2] Platform/ARM: " Leif Lindholm
  0 siblings, 2 replies; 4+ messages in thread
From: Ard Biesheuvel @ 2017-08-30  9:52 UTC (permalink / raw)
  To: edk2-devel, leif.lindholm; +Cc: Ard Biesheuvel

Nothing in the ARM builds depends on DmaLib anymore, so let's remove
the resolution for it.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
---
 Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc | 1 -
 1 file changed, 1 deletion(-)

diff --git a/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc b/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc
index a630d3c600cc..770007d1c893 100644
--- a/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc
+++ b/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc
@@ -70,7 +70,6 @@
   DefaultExceptionHandlerLib|ArmPkg/Library/DefaultExceptionHandlerLib/DefaultExceptionHandlerLib.inf
   CpuExceptionHandlerLib|ArmPkg/Library/ArmExceptionLib/ArmExceptionLib.inf
   ArmDisassemblerLib|ArmPkg/Library/ArmDisassemblerLib/ArmDisassemblerLib.inf
-  DmaLib|ArmPkg/Library/ArmDmaLib/ArmDmaLib.inf
   ArmGicLib|ArmPkg/Drivers/ArmGic/ArmGicLib.inf
   ArmGicArchLib|ArmPkg/Library/ArmGicArchLib/ArmGicArchLib.inf
   ArmPlatformStackLib|ArmPlatformPkg/Library/ArmPlatformStackLib/ArmPlatformStackLib.inf
-- 
2.11.0



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH edk2-platforms 2/2] Platform/HiKey: remove DmaLib library class resolution
  2017-08-30  9:52 [PATCH edk2-platforms 1/2] Platform/ARM: remove DmaLib library class resolution Ard Biesheuvel
@ 2017-08-30  9:52 ` Ard Biesheuvel
  2017-08-30 13:07 ` [PATCH edk2-platforms 1/2] Platform/ARM: " Leif Lindholm
  1 sibling, 0 replies; 4+ messages in thread
From: Ard Biesheuvel @ 2017-08-30  9:52 UTC (permalink / raw)
  To: edk2-devel, leif.lindholm; +Cc: Ard Biesheuvel

The HiKey platform does not incorporate any drivers that depend on
the DmaLib library class so we can remove the resolution for it.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
---
 Platform/Hisilicon/HiKey/HiKey.dsc | 1 -
 1 file changed, 1 deletion(-)

diff --git a/Platform/Hisilicon/HiKey/HiKey.dsc b/Platform/Hisilicon/HiKey/HiKey.dsc
index 9accf781c25f..968e8acc0883 100644
--- a/Platform/Hisilicon/HiKey/HiKey.dsc
+++ b/Platform/Hisilicon/HiKey/HiKey.dsc
@@ -108,7 +108,6 @@
 
   # USB Requirements
   UefiUsbLib|MdePkg/Library/UefiUsbLib/UefiUsbLib.inf
-  DmaLib|ArmPkg/Library/ArmDmaLib/ArmDmaLib.inf
 
   # Network Libraries
   UefiScsiLib|MdePkg/Library/UefiScsiLib/UefiScsiLib.inf
-- 
2.11.0



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH edk2-platforms 1/2] Platform/ARM: remove DmaLib library class resolution
  2017-08-30  9:52 [PATCH edk2-platforms 1/2] Platform/ARM: remove DmaLib library class resolution Ard Biesheuvel
  2017-08-30  9:52 ` [PATCH edk2-platforms 2/2] Platform/HiKey: " Ard Biesheuvel
@ 2017-08-30 13:07 ` Leif Lindholm
  2017-08-30 13:21   ` Ard Biesheuvel
  1 sibling, 1 reply; 4+ messages in thread
From: Leif Lindholm @ 2017-08-30 13:07 UTC (permalink / raw)
  To: Ard Biesheuvel; +Cc: edk2-devel

On Wed, Aug 30, 2017 at 10:52:07AM +0100, Ard Biesheuvel wrote:
> Nothing in the ARM builds depends on DmaLib anymore, so let's remove
> the resolution for it.
> 
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>

For the series:
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>

> ---
>  Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc b/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc
> index a630d3c600cc..770007d1c893 100644
> --- a/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc
> +++ b/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc
> @@ -70,7 +70,6 @@
>    DefaultExceptionHandlerLib|ArmPkg/Library/DefaultExceptionHandlerLib/DefaultExceptionHandlerLib.inf
>    CpuExceptionHandlerLib|ArmPkg/Library/ArmExceptionLib/ArmExceptionLib.inf
>    ArmDisassemblerLib|ArmPkg/Library/ArmDisassemblerLib/ArmDisassemblerLib.inf
> -  DmaLib|ArmPkg/Library/ArmDmaLib/ArmDmaLib.inf
>    ArmGicLib|ArmPkg/Drivers/ArmGic/ArmGicLib.inf
>    ArmGicArchLib|ArmPkg/Library/ArmGicArchLib/ArmGicArchLib.inf
>    ArmPlatformStackLib|ArmPlatformPkg/Library/ArmPlatformStackLib/ArmPlatformStackLib.inf
> -- 
> 2.11.0
> 


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH edk2-platforms 1/2] Platform/ARM: remove DmaLib library class resolution
  2017-08-30 13:07 ` [PATCH edk2-platforms 1/2] Platform/ARM: " Leif Lindholm
@ 2017-08-30 13:21   ` Ard Biesheuvel
  0 siblings, 0 replies; 4+ messages in thread
From: Ard Biesheuvel @ 2017-08-30 13:21 UTC (permalink / raw)
  To: Leif Lindholm; +Cc: edk2-devel@lists.01.org

On 30 August 2017 at 14:07, Leif Lindholm <leif.lindholm@linaro.org> wrote:
> On Wed, Aug 30, 2017 at 10:52:07AM +0100, Ard Biesheuvel wrote:
>> Nothing in the ARM builds depends on DmaLib anymore, so let's remove
>> the resolution for it.
>>
>> Contributed-under: TianoCore Contribution Agreement 1.1
>> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>
> For the series:
> Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
>

Thanks.

Pushed as

924c62678ad1 Platform/ARM: remove DmaLib library class resolution
8b89ae57eec0 Platform/HiKey: remove DmaLib library class resolution


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-08-30 13:18 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-08-30  9:52 [PATCH edk2-platforms 1/2] Platform/ARM: remove DmaLib library class resolution Ard Biesheuvel
2017-08-30  9:52 ` [PATCH edk2-platforms 2/2] Platform/HiKey: " Ard Biesheuvel
2017-08-30 13:07 ` [PATCH edk2-platforms 1/2] Platform/ARM: " Leif Lindholm
2017-08-30 13:21   ` Ard Biesheuvel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox