Re: [edk2] [PATCH 0/6] OvmfPkg, ArmVirtPkg: remove compatibility with IntelFrameworkModulePkg BDS

2016-05-25 Thread Laszlo Ersek 
On 05/24/16 23:15, Jordan Justen wrote:
> Series Reviewed-by: Jordan Justen 

Thanks all for the reviews. Commit range
8caa3caaed4b32d699b79c6d5aaa606b52d740e7..d85c5e31ed2b550dd801f82e4ddb5f7583332098.

Cheers
Laszlo

> On 2016-05-17 12:59:10, Laszlo Ersek wrote:
>> Mainly triggered by
>> 
>> from about a week ago. In patch #2 the reasons are listed in more
>> detail.
>>
>> Public branch: .
>>
>> Cc: Ard Biesheuvel 
>> Cc: Gary Ching-Pang Lin 
>> Cc: Jordan Justen 
>> Cc: Ruiyu Ni 
>>
>> Thanks
>> Laszlo
>>
>> Laszlo Ersek (6):
>>   OvmfPkg/README: refer to MdeModulePkg & PlatformBootManagerLib in
>> examples
>>   OvmfPkg: remove USE_OLD_BDS build fallback macro
>>   OvmfPkg: remove PlatformBdsLib instance
>>   OvmfPkg: remove QemuBootOrderLib instance
>>   OvmfPkg, ArmVirtPkg: clean up SetBootOrderFromQemu() parameter list
>>   OvmfPkg, ArmVirtPkg: rename QemuNewBootOrderLib to QemuBootOrderLib
>>
>>  ArmVirtPkg/ArmVirtQemu.dsc   |2 +-
>>  ArmVirtPkg/ArmVirtQemuKernel.dsc |2 +-
>>  OvmfPkg/OvmfPkgIa32.dsc  |   21 +-
>>  OvmfPkg/OvmfPkgIa32X64.dsc   |   21 +-
>>  OvmfPkg/OvmfPkgX64.dsc   |   21 +-
>>  OvmfPkg/OvmfPkgIa32.fdf  |4 -
>>  OvmfPkg/OvmfPkgIa32X64.fdf   |4 -
>>  OvmfPkg/OvmfPkgX64.fdf   |4 -
>>  OvmfPkg/Library/PlatformBdsLib/PlatformBdsLib.inf|   73 -
>>  OvmfPkg/Library/QemuBootOrderLib/QemuBootOrderLib.inf|8 +-
>>  OvmfPkg/Library/QemuNewBootOrderLib/QemuBootOrderLib.inf |   68 -
>>  OvmfPkg/Include/Library/QemuBootOrderLib.h   |   11 +-
>>  OvmfPkg/Library/PlatformBdsLib/BdsPlatform.h |  292 ---
>>  OvmfPkg/Library/QemuNewBootOrderLib/ExtraRootBusMap.h|   40 -
>>  ArmVirtPkg/Library/PlatformBootManagerLib/PlatformBm.c   |2 +-
>>  OvmfPkg/Library/PlatformBdsLib/BdsPlatform.c | 1592 
>> 
>>  OvmfPkg/Library/PlatformBdsLib/PlatformData.c|   51 -
>>  OvmfPkg/Library/PlatformBdsLib/QemuKernel.c  |  170 --
>>  OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c |2 +-
>>  OvmfPkg/Library/QemuBootOrderLib/QemuBootOrderLib.c  |  163 +-
>>  OvmfPkg/Library/QemuNewBootOrderLib/ExtraRootBusMap.c|  313 
>>  OvmfPkg/Library/QemuNewBootOrderLib/QemuBootOrderLib.c   | 1954 
>> 
>>  OvmfPkg/README   |4 +-
>>  23 files changed, 126 insertions(+), 4696 deletions(-)
>>  delete mode 100644 OvmfPkg/Library/PlatformBdsLib/PlatformBdsLib.inf
>>  delete mode 100644 OvmfPkg/Library/QemuNewBootOrderLib/QemuBootOrderLib.inf
>>  delete mode 100644 OvmfPkg/Library/PlatformBdsLib/BdsPlatform.h
>>  delete mode 100644 OvmfPkg/Library/QemuNewBootOrderLib/ExtraRootBusMap.h
>>  delete mode 100644 OvmfPkg/Library/PlatformBdsLib/BdsPlatform.c
>>  delete mode 100644 OvmfPkg/Library/PlatformBdsLib/PlatformData.c
>>  delete mode 100644 OvmfPkg/Library/PlatformBdsLib/QemuKernel.c
>>  delete mode 100644 OvmfPkg/Library/QemuNewBootOrderLib/ExtraRootBusMap.c
>>  delete mode 100644 OvmfPkg/Library/QemuNewBootOrderLib/QemuBootOrderLib.c
>>
>> -- 
>> 1.8.3.1
>>
> ___
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
> 

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

Re: [edk2] [PATCH 0/6] OvmfPkg, ArmVirtPkg: remove compatibility with IntelFrameworkModulePkg BDS

2016-05-24 Thread Jordan Justen 
Series Reviewed-by: Jordan Justen 

On 2016-05-17 12:59:10, Laszlo Ersek wrote:
> Mainly triggered by
> 
> from about a week ago. In patch #2 the reasons are listed in more
> detail.
> 
> Public branch: .
> 
> Cc: Ard Biesheuvel 
> Cc: Gary Ching-Pang Lin 
> Cc: Jordan Justen 
> Cc: Ruiyu Ni 
> 
> Thanks
> Laszlo
> 
> Laszlo Ersek (6):
>   OvmfPkg/README: refer to MdeModulePkg & PlatformBootManagerLib in
> examples
>   OvmfPkg: remove USE_OLD_BDS build fallback macro
>   OvmfPkg: remove PlatformBdsLib instance
>   OvmfPkg: remove QemuBootOrderLib instance
>   OvmfPkg, ArmVirtPkg: clean up SetBootOrderFromQemu() parameter list
>   OvmfPkg, ArmVirtPkg: rename QemuNewBootOrderLib to QemuBootOrderLib
> 
>  ArmVirtPkg/ArmVirtQemu.dsc   |2 +-
>  ArmVirtPkg/ArmVirtQemuKernel.dsc |2 +-
>  OvmfPkg/OvmfPkgIa32.dsc  |   21 +-
>  OvmfPkg/OvmfPkgIa32X64.dsc   |   21 +-
>  OvmfPkg/OvmfPkgX64.dsc   |   21 +-
>  OvmfPkg/OvmfPkgIa32.fdf  |4 -
>  OvmfPkg/OvmfPkgIa32X64.fdf   |4 -
>  OvmfPkg/OvmfPkgX64.fdf   |4 -
>  OvmfPkg/Library/PlatformBdsLib/PlatformBdsLib.inf|   73 -
>  OvmfPkg/Library/QemuBootOrderLib/QemuBootOrderLib.inf|8 +-
>  OvmfPkg/Library/QemuNewBootOrderLib/QemuBootOrderLib.inf |   68 -
>  OvmfPkg/Include/Library/QemuBootOrderLib.h   |   11 +-
>  OvmfPkg/Library/PlatformBdsLib/BdsPlatform.h |  292 ---
>  OvmfPkg/Library/QemuNewBootOrderLib/ExtraRootBusMap.h|   40 -
>  ArmVirtPkg/Library/PlatformBootManagerLib/PlatformBm.c   |2 +-
>  OvmfPkg/Library/PlatformBdsLib/BdsPlatform.c | 1592 
> 
>  OvmfPkg/Library/PlatformBdsLib/PlatformData.c|   51 -
>  OvmfPkg/Library/PlatformBdsLib/QemuKernel.c  |  170 --
>  OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c |2 +-
>  OvmfPkg/Library/QemuBootOrderLib/QemuBootOrderLib.c  |  163 +-
>  OvmfPkg/Library/QemuNewBootOrderLib/ExtraRootBusMap.c|  313 
>  OvmfPkg/Library/QemuNewBootOrderLib/QemuBootOrderLib.c   | 1954 
> 
>  OvmfPkg/README   |4 +-
>  23 files changed, 126 insertions(+), 4696 deletions(-)
>  delete mode 100644 OvmfPkg/Library/PlatformBdsLib/PlatformBdsLib.inf
>  delete mode 100644 OvmfPkg/Library/QemuNewBootOrderLib/QemuBootOrderLib.inf
>  delete mode 100644 OvmfPkg/Library/PlatformBdsLib/BdsPlatform.h
>  delete mode 100644 OvmfPkg/Library/QemuNewBootOrderLib/ExtraRootBusMap.h
>  delete mode 100644 OvmfPkg/Library/PlatformBdsLib/BdsPlatform.c
>  delete mode 100644 OvmfPkg/Library/PlatformBdsLib/PlatformData.c
>  delete mode 100644 OvmfPkg/Library/PlatformBdsLib/QemuKernel.c
>  delete mode 100644 OvmfPkg/Library/QemuNewBootOrderLib/ExtraRootBusMap.c
>  delete mode 100644 OvmfPkg/Library/QemuNewBootOrderLib/QemuBootOrderLib.c
> 
> -- 
> 1.8.3.1
> 
___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Re: [edk2] [PATCH 0/6] OvmfPkg, ArmVirtPkg: remove compatibility with IntelFrameworkModulePkg BDS

2016-05-23 Thread Laszlo Ersek 
Jordan,

On 05/17/16 21:59, Laszlo Ersek wrote:
> Mainly triggered by
> 
> from about a week ago. In patch #2 the reasons are listed in more
> detail.
> 
> Public branch: .
> 
> Cc: Ard Biesheuvel 
> Cc: Gary Ching-Pang Lin 
> Cc: Jordan Justen 
> Cc: Ruiyu Ni 
> 
> Thanks
> Laszlo
> 
> Laszlo Ersek (6):
>   OvmfPkg/README: refer to MdeModulePkg & PlatformBootManagerLib in
> examples
>   OvmfPkg: remove USE_OLD_BDS build fallback macro
>   OvmfPkg: remove PlatformBdsLib instance
>   OvmfPkg: remove QemuBootOrderLib instance
>   OvmfPkg, ArmVirtPkg: clean up SetBootOrderFromQemu() parameter list
>   OvmfPkg, ArmVirtPkg: rename QemuNewBootOrderLib to QemuBootOrderLib

can you please review this series?

Thanks
Laszlo

>  ArmVirtPkg/ArmVirtQemu.dsc   |2 +-
>  ArmVirtPkg/ArmVirtQemuKernel.dsc |2 +-
>  OvmfPkg/OvmfPkgIa32.dsc  |   21 +-
>  OvmfPkg/OvmfPkgIa32X64.dsc   |   21 +-
>  OvmfPkg/OvmfPkgX64.dsc   |   21 +-
>  OvmfPkg/OvmfPkgIa32.fdf  |4 -
>  OvmfPkg/OvmfPkgIa32X64.fdf   |4 -
>  OvmfPkg/OvmfPkgX64.fdf   |4 -
>  OvmfPkg/Library/PlatformBdsLib/PlatformBdsLib.inf|   73 -
>  OvmfPkg/Library/QemuBootOrderLib/QemuBootOrderLib.inf|8 +-
>  OvmfPkg/Library/QemuNewBootOrderLib/QemuBootOrderLib.inf |   68 -
>  OvmfPkg/Include/Library/QemuBootOrderLib.h   |   11 +-
>  OvmfPkg/Library/PlatformBdsLib/BdsPlatform.h |  292 ---
>  OvmfPkg/Library/QemuNewBootOrderLib/ExtraRootBusMap.h|   40 -
>  ArmVirtPkg/Library/PlatformBootManagerLib/PlatformBm.c   |2 +-
>  OvmfPkg/Library/PlatformBdsLib/BdsPlatform.c | 1592 
> 
>  OvmfPkg/Library/PlatformBdsLib/PlatformData.c|   51 -
>  OvmfPkg/Library/PlatformBdsLib/QemuKernel.c  |  170 --
>  OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c |2 +-
>  OvmfPkg/Library/QemuBootOrderLib/QemuBootOrderLib.c  |  163 +-
>  OvmfPkg/Library/QemuNewBootOrderLib/ExtraRootBusMap.c|  313 
>  OvmfPkg/Library/QemuNewBootOrderLib/QemuBootOrderLib.c   | 1954 
> 
>  OvmfPkg/README   |4 +-
>  23 files changed, 126 insertions(+), 4696 deletions(-)
>  delete mode 100644 OvmfPkg/Library/PlatformBdsLib/PlatformBdsLib.inf
>  delete mode 100644 OvmfPkg/Library/QemuNewBootOrderLib/QemuBootOrderLib.inf
>  delete mode 100644 OvmfPkg/Library/PlatformBdsLib/BdsPlatform.h
>  delete mode 100644 OvmfPkg/Library/QemuNewBootOrderLib/ExtraRootBusMap.h
>  delete mode 100644 OvmfPkg/Library/PlatformBdsLib/BdsPlatform.c
>  delete mode 100644 OvmfPkg/Library/PlatformBdsLib/PlatformData.c
>  delete mode 100644 OvmfPkg/Library/PlatformBdsLib/QemuKernel.c
>  delete mode 100644 OvmfPkg/Library/QemuNewBootOrderLib/ExtraRootBusMap.c
>  delete mode 100644 OvmfPkg/Library/QemuNewBootOrderLib/QemuBootOrderLib.c
> 

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

Re: [edk2] [PATCH 0/6] OvmfPkg, ArmVirtPkg: remove compatibility with IntelFrameworkModulePkg BDS

2016-05-17 Thread Ard Biesheuvel 
On 17 May 2016 at 21:59, Laszlo Ersek  wrote:
> Mainly triggered by
> 
> from about a week ago. In patch #2 the reasons are listed in more
> detail.
>
> Public branch: .
>
> Cc: Ard Biesheuvel 
> Cc: Gary Ching-Pang Lin 
> Cc: Jordan Justen 
> Cc: Ruiyu Ni 
>

For patches 5 and 6

Reviewed-by: Ard Biesheuvel 
___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

[edk2] [PATCH 0/6] OvmfPkg, ArmVirtPkg: remove compatibility with IntelFrameworkModulePkg BDS

2016-05-17 Thread Laszlo Ersek 
Mainly triggered by

from about a week ago. In patch #2 the reasons are listed in more
detail.

Public branch: .

Cc: Ard Biesheuvel 
Cc: Gary Ching-Pang Lin 
Cc: Jordan Justen 
Cc: Ruiyu Ni 

Thanks
Laszlo

Laszlo Ersek (6):
  OvmfPkg/README: refer to MdeModulePkg & PlatformBootManagerLib in
examples
  OvmfPkg: remove USE_OLD_BDS build fallback macro
  OvmfPkg: remove PlatformBdsLib instance
  OvmfPkg: remove QemuBootOrderLib instance
  OvmfPkg, ArmVirtPkg: clean up SetBootOrderFromQemu() parameter list
  OvmfPkg, ArmVirtPkg: rename QemuNewBootOrderLib to QemuBootOrderLib

 ArmVirtPkg/ArmVirtQemu.dsc   |2 +-
 ArmVirtPkg/ArmVirtQemuKernel.dsc |2 +-
 OvmfPkg/OvmfPkgIa32.dsc  |   21 +-
 OvmfPkg/OvmfPkgIa32X64.dsc   |   21 +-
 OvmfPkg/OvmfPkgX64.dsc   |   21 +-
 OvmfPkg/OvmfPkgIa32.fdf  |4 -
 OvmfPkg/OvmfPkgIa32X64.fdf   |4 -
 OvmfPkg/OvmfPkgX64.fdf   |4 -
 OvmfPkg/Library/PlatformBdsLib/PlatformBdsLib.inf|   73 -
 OvmfPkg/Library/QemuBootOrderLib/QemuBootOrderLib.inf|8 +-
 OvmfPkg/Library/QemuNewBootOrderLib/QemuBootOrderLib.inf |   68 -
 OvmfPkg/Include/Library/QemuBootOrderLib.h   |   11 +-
 OvmfPkg/Library/PlatformBdsLib/BdsPlatform.h |  292 ---
 OvmfPkg/Library/QemuNewBootOrderLib/ExtraRootBusMap.h|   40 -
 ArmVirtPkg/Library/PlatformBootManagerLib/PlatformBm.c   |2 +-
 OvmfPkg/Library/PlatformBdsLib/BdsPlatform.c | 1592 

 OvmfPkg/Library/PlatformBdsLib/PlatformData.c|   51 -
 OvmfPkg/Library/PlatformBdsLib/QemuKernel.c  |  170 --
 OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c |2 +-
 OvmfPkg/Library/QemuBootOrderLib/QemuBootOrderLib.c  |  163 +-
 OvmfPkg/Library/QemuNewBootOrderLib/ExtraRootBusMap.c|  313 
 OvmfPkg/Library/QemuNewBootOrderLib/QemuBootOrderLib.c   | 1954 

 OvmfPkg/README   |4 +-
 23 files changed, 126 insertions(+), 4696 deletions(-)
 delete mode 100644 OvmfPkg/Library/PlatformBdsLib/PlatformBdsLib.inf
 delete mode 100644 OvmfPkg/Library/QemuNewBootOrderLib/QemuBootOrderLib.inf
 delete mode 100644 OvmfPkg/Library/PlatformBdsLib/BdsPlatform.h
 delete mode 100644 OvmfPkg/Library/QemuNewBootOrderLib/ExtraRootBusMap.h
 delete mode 100644 OvmfPkg/Library/PlatformBdsLib/BdsPlatform.c
 delete mode 100644 OvmfPkg/Library/PlatformBdsLib/PlatformData.c
 delete mode 100644 OvmfPkg/Library/PlatformBdsLib/QemuKernel.c
 delete mode 100644 OvmfPkg/Library/QemuNewBootOrderLib/ExtraRootBusMap.c
 delete mode 100644 OvmfPkg/Library/QemuNewBootOrderLib/QemuBootOrderLib.c

-- 
1.8.3.1

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