diff mbox series

[edk2] ArmPlatformPkg/LcdGraphicsOutputDxe: remove VExpressPkg dependency

Message ID 20171204205052.10197-1-ard.biesheuvel@linaro.org
State Accepted
Commit c1c72a46b98159df009dfb7566cdbf49f34453ca
Headers show
Series [edk2] ArmPlatformPkg/LcdGraphicsOutputDxe: remove VExpressPkg dependency | expand

Commit Message

Ard Biesheuvel Dec. 4, 2017, 8:50 p.m. UTC
The two instantiations of LcdGraphicsOutputDxe reference VExpressPkg.dec
without actually relying on anything it defines. In preparation of
moving out VExpressPkg into edk2-platforms, drop these references so we
can keep LcdGraphicsOutputDxe in EDK2.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>

---
This is a preparatory patch for my rebase of the ArmPlatformPkg cleanup
series, allowing future contributions to the LCD output driver to remain
in EDK2, while forcing VExpress specific glue code to be factored out
correctly and contributed to edk2-platforms.

 ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/HdLcdGraphicsOutputDxe.inf    | 1 -
 ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/PL111LcdGraphicsOutputDxe.inf | 1 -
 2 files changed, 2 deletions(-)

-- 
2.11.0

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

Comments

Leif Lindholm Dec. 4, 2017, 9:34 p.m. UTC | #1
On Mon, Dec 04, 2017 at 08:50:52PM +0000, Ard Biesheuvel wrote:
> The two instantiations of LcdGraphicsOutputDxe reference VExpressPkg.dec

> without actually relying on anything it defines. In preparation of

> moving out VExpressPkg into edk2-platforms, drop these references so we

> can keep LcdGraphicsOutputDxe in EDK2.

> 

> Contributed-under: TianoCore Contribution Agreement 1.1

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


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


> ---

> This is a preparatory patch for my rebase of the ArmPlatformPkg cleanup

> series, allowing future contributions to the LCD output driver to remain

> in EDK2, while forcing VExpress specific glue code to be factored out

> correctly and contributed to edk2-platforms.

> 

>  ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/HdLcdGraphicsOutputDxe.inf    | 1 -

>  ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/PL111LcdGraphicsOutputDxe.inf | 1 -

>  2 files changed, 2 deletions(-)

> 

> diff --git a/ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/HdLcdGraphicsOutputDxe.inf b/ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/HdLcdGraphicsOutputDxe.inf

> index 462d1fa402d7..896fc588b275 100644

> --- a/ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/HdLcdGraphicsOutputDxe.inf

> +++ b/ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/HdLcdGraphicsOutputDxe.inf

> @@ -31,7 +31,6 @@ [Packages]

>    MdePkg/MdePkg.dec

>    MdeModulePkg/MdeModulePkg.dec

>    ArmPkg/ArmPkg.dec

> -  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressPkg.dec

>    ArmPlatformPkg/ArmPlatformPkg.dec

>  

>  [LibraryClasses]

> diff --git a/ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/PL111LcdGraphicsOutputDxe.inf b/ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/PL111LcdGraphicsOutputDxe.inf

> index 003cc2ffa912..39e42bcbab2a 100644

> --- a/ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/PL111LcdGraphicsOutputDxe.inf

> +++ b/ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/PL111LcdGraphicsOutputDxe.inf

> @@ -30,7 +30,6 @@ [Packages]

>    MdePkg/MdePkg.dec

>    MdeModulePkg/MdeModulePkg.dec

>    ArmPkg/ArmPkg.dec

> -  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressPkg.dec

>    ArmPlatformPkg/ArmPlatformPkg.dec

>  

>  [LibraryClasses]

> -- 

> 2.11.0

> 

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Ard Biesheuvel Dec. 4, 2017, 10:02 p.m. UTC | #2
On 4 December 2017 at 21:34, Leif Lindholm <leif.lindholm@linaro.org> wrote:
> On Mon, Dec 04, 2017 at 08:50:52PM +0000, Ard Biesheuvel wrote:

>> The two instantiations of LcdGraphicsOutputDxe reference VExpressPkg.dec

>> without actually relying on anything it defines. In preparation of

>> moving out VExpressPkg into edk2-platforms, drop these references so we

>> can keep LcdGraphicsOutputDxe in EDK2.

>>

>> Contributed-under: TianoCore Contribution Agreement 1.1

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

>

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

>


Thanks. Pushed as c1c72a46b981
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
diff mbox series

Patch

diff --git a/ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/HdLcdGraphicsOutputDxe.inf b/ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/HdLcdGraphicsOutputDxe.inf
index 462d1fa402d7..896fc588b275 100644
--- a/ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/HdLcdGraphicsOutputDxe.inf
+++ b/ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/HdLcdGraphicsOutputDxe.inf
@@ -31,7 +31,6 @@  [Packages]
   MdePkg/MdePkg.dec
   MdeModulePkg/MdeModulePkg.dec
   ArmPkg/ArmPkg.dec
-  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressPkg.dec
   ArmPlatformPkg/ArmPlatformPkg.dec
 
 [LibraryClasses]
diff --git a/ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/PL111LcdGraphicsOutputDxe.inf b/ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/PL111LcdGraphicsOutputDxe.inf
index 003cc2ffa912..39e42bcbab2a 100644
--- a/ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/PL111LcdGraphicsOutputDxe.inf
+++ b/ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/PL111LcdGraphicsOutputDxe.inf
@@ -30,7 +30,6 @@  [Packages]
   MdePkg/MdePkg.dec
   MdeModulePkg/MdeModulePkg.dec
   ArmPkg/ArmPkg.dec
-  ArmPlatformPkg/ArmVExpressPkg/ArmVExpressPkg.dec
   ArmPlatformPkg/ArmPlatformPkg.dec
 
 [LibraryClasses]