Re: [edk2] [PATCH] ArmPlatformPkg ArmVirtPkg: reorganize PL011 code

2017-11-16 Thread Leif Lindholm
On Thu, Nov 16, 2017 at 05:38:10PM +, Ard Biesheuvel wrote: > On 16 November 2017 at 17:36, Leif Lindholm wrote: > > On Thu, Nov 16, 2017 at 05:12:16PM +, Ard Biesheuvel wrote: > >> The PL011 code in ArmPlatformPkg is organized in a weird way: there is > >> a

Re: [edk2] [PATCH] ArmPlatformPkg ArmVirtPkg: reorganize PL011 code

2017-11-16 Thread Ard Biesheuvel
On 16 November 2017 at 17:36, Leif Lindholm wrote: > On Thu, Nov 16, 2017 at 05:12:16PM +, Ard Biesheuvel wrote: >> The PL011 code in ArmPlatformPkg is organized in a weird way: there is >> a single PL011Uart.h header file under Include/Drivers containing both >>

Re: [edk2] [PATCH] ArmPlatformPkg ArmVirtPkg: reorganize PL011 code

2017-11-16 Thread Leif Lindholm
On Thu, Nov 16, 2017 at 05:12:16PM +, Ard Biesheuvel wrote: > The PL011 code in ArmPlatformPkg is organized in a weird way: there is > a single PL011Uart.h header file under Include/Drivers containing both > register definitions and function entry points. The PL011Uart library > itself is in

[edk2] [PATCH] ArmPlatformPkg ArmVirtPkg: reorganize PL011 code

2017-11-16 Thread Ard Biesheuvel
The PL011 code in ArmPlatformPkg is organized in a weird way: there is a single PL011Uart.h header file under Include/Drivers containing both register definitions and function entry points. The PL011Uart library itself is in Drivers/ but it is actually a library. So let's clean this up: add a new