[edk2-devel][edk2-platforms][PATCH V2 0/2] Add MinPlatformPkg NOOPT build option

Oram, Isaac W posted 2 patches 1 year, 8 months ago
Failed in applying to current master (apply log)
.../MinPlatformPkg/Include/Dsc/CoreDxeLib.dsc | 66 ++++++-------------
.../MinPlatformPkg/Include/Dsc/CorePeiLib.dsc | 29 ++------
.../Intel/MinPlatformPkg/MinPlatformPkg.dsc   |  2 +-
3 files changed, 27 insertions(+), 70 deletions(-)
[edk2-devel][edk2-platforms][PATCH V2 0/2] Add MinPlatformPkg NOOPT build option
Posted by Oram, Isaac W 1 year, 8 months ago
V2:
Wrap DXE performance library with feature PCD

V1:
Add the NOOPT build option.
Use the same libraries for DEBUG and NOOPT Clean up some duplication and coding style issues with the include files.

Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Eric Dong <eric.dong@intel.com>
Signed-off-by: Isaac Oram <isaac.w.oram@intel.com>

Isaac Oram (2):
  MinPlatformPkg/Build: Add NOOPT build
  MinPlatformPkg/Build: Reduce duplication

 .../MinPlatformPkg/Include/Dsc/CoreDxeLib.dsc | 66 ++++++-------------
 .../MinPlatformPkg/Include/Dsc/CorePeiLib.dsc | 29 ++------
 .../Intel/MinPlatformPkg/MinPlatformPkg.dsc   |  2 +-
 3 files changed, 27 insertions(+), 70 deletions(-)

-- 
2.36.1.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#91548): https://edk2.groups.io/g/devel/message/91548
Mute This Topic: https://groups.io/mt/92490676/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel][edk2-platforms][PATCH V2 0/2] Add MinPlatformPkg NOOPT build option
Posted by Nate DeSimone 1 year, 8 months ago
For the series...

Reviewed-by: Nate DeSimone <nathaniel.l.desimone@intel.com>

> -----Original Message-----
> From: Oram, Isaac W <isaac.w.oram@intel.com>
> Sent: Tuesday, July 19, 2022 12:43 PM
> To: devel@edk2.groups.io
> Cc: Oram, Isaac W <isaac.w.oram@intel.com>; Chiu, Chasel
> <chasel.chiu@intel.com>; Desimone, Nathaniel L
> <nathaniel.l.desimone@intel.com>; Gao, Liming
> <gaoliming@byosoft.com.cn>; Dong, Eric <eric.dong@intel.com>
> Subject: [edk2-devel][edk2-platforms][PATCH V2 0/2] Add MinPlatformPkg
> NOOPT build option
> 
> V2:
> Wrap DXE performance library with feature PCD
> 
> V1:
> Add the NOOPT build option.
> Use the same libraries for DEBUG and NOOPT Clean up some duplication and
> coding style issues with the include files.
> 
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Eric Dong <eric.dong@intel.com>
> Signed-off-by: Isaac Oram <isaac.w.oram@intel.com>
> 
> Isaac Oram (2):
>   MinPlatformPkg/Build: Add NOOPT build
>   MinPlatformPkg/Build: Reduce duplication
> 
>  .../MinPlatformPkg/Include/Dsc/CoreDxeLib.dsc | 66 ++++++-------------
> .../MinPlatformPkg/Include/Dsc/CorePeiLib.dsc | 29 ++------
>  .../Intel/MinPlatformPkg/MinPlatformPkg.dsc   |  2 +-
>  3 files changed, 27 insertions(+), 70 deletions(-)
> 
> --
> 2.36.1.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#91595): https://edk2.groups.io/g/devel/message/91595
Mute This Topic: https://groups.io/mt/92490676/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel][edk2-platforms][PATCH V2 0/2] Add MinPlatformPkg NOOPT build option
Posted by Oram, Isaac W 1 year, 8 months ago
Pushed as 0f836bd4e3..773ac505b7

-----Original Message-----
From: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com> 
Sent: Wednesday, July 20, 2022 2:07 PM
To: Oram, Isaac W <isaac.w.oram@intel.com>; devel@edk2.groups.io
Cc: Chiu, Chasel <chasel.chiu@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>; Dong, Eric <eric.dong@intel.com>
Subject: RE: [edk2-devel][edk2-platforms][PATCH V2 0/2] Add MinPlatformPkg NOOPT build option

For the series...

Reviewed-by: Nate DeSimone <nathaniel.l.desimone@intel.com>

> -----Original Message-----
> From: Oram, Isaac W <isaac.w.oram@intel.com>
> Sent: Tuesday, July 19, 2022 12:43 PM
> To: devel@edk2.groups.io
> Cc: Oram, Isaac W <isaac.w.oram@intel.com>; Chiu, Chasel 
> <chasel.chiu@intel.com>; Desimone, Nathaniel L 
> <nathaniel.l.desimone@intel.com>; Gao, Liming 
> <gaoliming@byosoft.com.cn>; Dong, Eric <eric.dong@intel.com>
> Subject: [edk2-devel][edk2-platforms][PATCH V2 0/2] Add MinPlatformPkg 
> NOOPT build option
> 
> V2:
> Wrap DXE performance library with feature PCD
> 
> V1:
> Add the NOOPT build option.
> Use the same libraries for DEBUG and NOOPT Clean up some duplication 
> and coding style issues with the include files.
> 
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Eric Dong <eric.dong@intel.com>
> Signed-off-by: Isaac Oram <isaac.w.oram@intel.com>
> 
> Isaac Oram (2):
>   MinPlatformPkg/Build: Add NOOPT build
>   MinPlatformPkg/Build: Reduce duplication
> 
>  .../MinPlatformPkg/Include/Dsc/CoreDxeLib.dsc | 66 
> ++++++------------- .../MinPlatformPkg/Include/Dsc/CorePeiLib.dsc | 29 ++------
>  .../Intel/MinPlatformPkg/MinPlatformPkg.dsc   |  2 +-
>  3 files changed, 27 insertions(+), 70 deletions(-)
> 
> --
> 2.36.1.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#91599): https://edk2.groups.io/g/devel/message/91599
Mute This Topic: https://groups.io/mt/92490676/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel][edk2-platforms][PATCH V2 0/2] Add MinPlatformPkg NOOPT build option
Posted by Chiu, Chasel 1 year, 8 months ago
For this series, Reviewed-by: Chasel Chiu <chasel.chiu@intel.com>

Thanks,
Chasel


> -----Original Message-----
> From: Oram, Isaac W <isaac.w.oram@intel.com>
> Sent: Tuesday, July 19, 2022 12:43 PM
> To: devel@edk2.groups.io
> Cc: Oram, Isaac W <isaac.w.oram@intel.com>; Chiu, Chasel
> <chasel.chiu@intel.com>; Desimone, Nathaniel L
> <nathaniel.l.desimone@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>;
> Dong, Eric <eric.dong@intel.com>
> Subject: [edk2-devel][edk2-platforms][PATCH V2 0/2] Add MinPlatformPkg
> NOOPT build option
> 
> V2:
> Wrap DXE performance library with feature PCD
> 
> V1:
> Add the NOOPT build option.
> Use the same libraries for DEBUG and NOOPT Clean up some duplication and
> coding style issues with the include files.
> 
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Eric Dong <eric.dong@intel.com>
> Signed-off-by: Isaac Oram <isaac.w.oram@intel.com>
> 
> Isaac Oram (2):
>   MinPlatformPkg/Build: Add NOOPT build
>   MinPlatformPkg/Build: Reduce duplication
> 
>  .../MinPlatformPkg/Include/Dsc/CoreDxeLib.dsc | 66 ++++++-------------
>   .../MinPlatformPkg/Include/Dsc/CorePeiLib.dsc | 29 ++------
>  .../Intel/MinPlatformPkg/MinPlatformPkg.dsc   |  2 +-
>  3 files changed, 27 insertions(+), 70 deletions(-)
> 
> --
> 2.36.1.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#91555): https://edk2.groups.io/g/devel/message/91555
Mute This Topic: https://groups.io/mt/92490676/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-