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

2022-07-20 Thread Oram, Isaac W
Pushed as 0f836bd4e3..773ac505b7

-Original Message-
From: Desimone, Nathaniel L  
Sent: Wednesday, July 20, 2022 2:07 PM
To: Oram, Isaac W ; devel@edk2.groups.io
Cc: Chiu, Chasel ; Gao, Liming 
; Dong, Eric 
Subject: RE: [edk2-devel][edk2-platforms][PATCH V2 0/2] Add MinPlatformPkg 
NOOPT build option

For the series...

Reviewed-by: Nate DeSimone 

> -Original Message-
> From: Oram, Isaac W 
> Sent: Tuesday, July 19, 2022 12:43 PM
> To: devel@edk2.groups.io
> Cc: Oram, Isaac W ; Chiu, Chasel 
> ; Desimone, Nathaniel L 
> ; Gao, Liming 
> ; Dong, Eric 
> 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 
> Cc: Nate DeSimone 
> Cc: Liming Gao 
> Cc: Eric Dong 
> Signed-off-by: Isaac Oram 
> 
> 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/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




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

2022-07-20 Thread Nate DeSimone
For the series...

Reviewed-by: Nate DeSimone 

> -Original Message-
> From: Oram, Isaac W 
> Sent: Tuesday, July 19, 2022 12:43 PM
> To: devel@edk2.groups.io
> Cc: Oram, Isaac W ; Chiu, Chasel
> ; Desimone, Nathaniel L
> ; Gao, Liming
> ; Dong, Eric 
> 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 
> Cc: Nate DeSimone 
> Cc: Liming Gao 
> Cc: Eric Dong 
> Signed-off-by: Isaac Oram 
> 
> 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/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




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

2022-07-19 Thread Chiu, Chasel


For this series, Reviewed-by: Chasel Chiu 

Thanks,
Chasel


> -Original Message-
> From: Oram, Isaac W 
> Sent: Tuesday, July 19, 2022 12:43 PM
> To: devel@edk2.groups.io
> Cc: Oram, Isaac W ; Chiu, Chasel
> ; Desimone, Nathaniel L
> ; Gao, Liming ;
> Dong, Eric 
> 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 
> Cc: Nate DeSimone 
> Cc: Liming Gao 
> Cc: Eric Dong 
> Signed-off-by: Isaac Oram 
> 
> 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/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




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

2022-07-19 Thread Oram, Isaac W
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 
Cc: Nate DeSimone 
Cc: Liming Gao 
Cc: Eric Dong 
Signed-off-by: Isaac Oram 

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/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-