Re: [edk2-devel] [edk2-platforms] [PATCH v2 4/4] WhiskeylakeOpenBoardPkg/UpXtreme: Add DSC and build files

2020-02-18 Thread Nate DeSimone
Hi Prince,

Please update the copyright year on build.cfg.

Thanks,
Nate

On Thu, Feb 13, 2020 at 05:55:11PM +, Agyeman, Prince wrote:
> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2191
> 
> Adds the DSC and build files necessary to build the
> UpXtreme board instance.
> 
> Key files
> =
> * build_config.cfg - Board-specific build configuration file.
> * OpenBoardPkg.dsc - The UpXtreme board description file.
> * OpenBoardPkgPcd.dsc - Used for other PCD customization.
> * OpenBoardPkg.fdf - The UpXtreme board flash file.
> * OpenBoardPkgBuildOption.dsc - Sets build options Based
>   on PCD values.
> 
> Co-authored-by: Michael Kubacki 
> Cc: Chasel Chiu 
> Cc: Nate DeSimone 
> Signed-off-by: Prince Agyeman 
> ---
>  Platform/Intel/Readme.md  |  19 +-
>  .../UpXtreme/OpenBoardPkg.dsc | 448 +++
>  .../UpXtreme/OpenBoardPkg.fdf | 708 ++
>  .../UpXtreme/OpenBoardPkgBuildOption.dsc  | 154 
>  .../UpXtreme/OpenBoardPkgPcd.dsc  | 409 ++
>  .../UpXtreme/build_config.cfg |  35 +
>  Platform/Intel/build.cfg  |   1 +
>  7 files changed, 1772 insertions(+), 2 deletions(-)
>  create mode 100644 
> Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkg.dsc
>  create mode 100644 
> Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkg.fdf
>  create mode 100644 
> Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkgBuildOption.dsc
>  create mode 100644 
> Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkgPcd.dsc
>  create mode 100644 
> Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/build_config.cfg
> 
> diff --git a/Platform/Intel/Readme.md b/Platform/Intel/Readme.md
> index 02d9517d19..55cf02a8ca 100644
> --- a/Platform/Intel/Readme.md
> +++ b/Platform/Intel/Readme.md
> @@ -59,6 +59,12 @@ A UEFI firmware implementation using MinPlatformPkg is 
> constructed using the fol
>
>  ### **Supported Hardware**
>
> + AAEON
> +
> +| Machine Name  | Supported Chipsets 
> | BoardPkg | Board Name |
> +||--||
> +| UP Xtreme | Whiskey Lake   
> | WhiskeylakeOpenBoardPkg  | UpXtreme   |
> +
>   Intel
>
>  ***Intel Reference and Validation Platform***
> @@ -233,6 +239,9 @@ return back to the minimum platform caller.
>|   || build 
> settings, environment variables.
>|   ||
>|   ||--WhiskeylakeOpenBoardPkg
> +  |   ||   |--UpXtreme
> +  |   ||   |---build_config.cfg: UpXtreme 
> specific build
> +  |   || settings 
> environment variables.
>|   ||   |--WhiskeylakeURvp
>|   ||   |---build_config.cfg: 
> WhiskeylakeURvp specific build
>|   || settings 
> environment variables.
> @@ -254,8 +263,14 @@ return back to the minimum platform caller.
>  1. This firmware project has only been tested booting to Microsoft Windows 
> 10 x64 and Ubuntu 17.10 with AHCI mode.
>
>  **WhiskeylakeOpenBoardPkg**
> -1. This firmware project has only been tested booting to Microsoft Windows 
> 10 x64 with AHCI mode and Integrated Graphic
> +1. This firmware project has mainly been tested booting to Microsoft Windows 
> 10 x64 with AHCI mode and Integrated Graphic
> Device.
> +2. UP Xtreme boards might hang during Windows 10 boot.
> +3. Not all UP Xtreme boards are supported at this time.
> +* The model below boots to x64 windows 10 home edition and Ubuntu 18.04
> +  * Xtreme Intel(R) Core(TM) i3-8145UE CPU @ 2.20GHz with 8GB RAM
> +  * Xtreme Intel(R) Core(TM) i7-8565U CPU @ 1.80GHz with 16GB RAM
> +* Intel(R) Celeron(R) CPU 4305UE @ 2.00GHz with 4GB RAM hangs during 
> memory initialization
>
>  ### **Package Builds**
>
> @@ -293,5 +308,5 @@ If you would like to help but are not sure where to start 
> some areas currently i
>   * Adding board ports for more motherboards and systems
>   * Adding Clang support
>
> -Please feel free to contact Michael Kubacki (michael.a.kubacki at intel.com) 
> and Isaac Oram (isaac.w.oram at intel.com)
> +Please feel free to contact Isaac Oram (isaac.w.oram at intel.com)
>  if you would like to discuss contribution ideas.
> diff --git a/Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkg.dsc 
> b/Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkg.dsc
> new file mode 100644
> index 00..2ab9cb03ea
> --- /dev/null
> +++ 

Re: [edk2-devel] [edk2-platforms] [PATCH v2 4/4] WhiskeylakeOpenBoardPkg/UpXtreme: Add DSC and build files

2020-02-13 Thread Chiu, Chasel


Reviewed-by: Chasel Chiu 


> -Original Message-
> From: Agyeman, Prince 
> Sent: Friday, February 14, 2020 1:55 AM
> To: devel@edk2.groups.io
> Cc: Chiu, Chasel ; Desimone, Nathaniel L
> 
> Subject: [edk2-platforms] [PATCH v2 4/4]
> WhiskeylakeOpenBoardPkg/UpXtreme: Add DSC and build files
> 
> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2191
> 
> Adds the DSC and build files necessary to build the
> UpXtreme board instance.
> 
> Key files
> =
> * build_config.cfg - Board-specific build configuration file.
> * OpenBoardPkg.dsc - The UpXtreme board description file.
> * OpenBoardPkgPcd.dsc - Used for other PCD customization.
> * OpenBoardPkg.fdf - The UpXtreme board flash file.
> * OpenBoardPkgBuildOption.dsc - Sets build options Based
>   on PCD values.
> 
> Co-authored-by: Michael Kubacki 
> Cc: Chasel Chiu 
> Cc: Nate DeSimone 
> Signed-off-by: Prince Agyeman 
> ---
>  Platform/Intel/Readme.md  |  19 +-
>  .../UpXtreme/OpenBoardPkg.dsc | 448 +++
>  .../UpXtreme/OpenBoardPkg.fdf | 708
> ++
>  .../UpXtreme/OpenBoardPkgBuildOption.dsc  | 154 
>  .../UpXtreme/OpenBoardPkgPcd.dsc  | 409 ++
>  .../UpXtreme/build_config.cfg |  35 +
>  Platform/Intel/build.cfg  |   1 +
>  7 files changed, 1772 insertions(+), 2 deletions(-)
>  create mode 100644
> Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkg.dsc
>  create mode 100644
> Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkg.fdf
>  create mode 100644
> Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkgBuildOpt
> ion.dsc
>  create mode 100644
> Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkgPcd.dsc
>  create mode 100644
> Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/build_config.cfg
> 
> diff --git a/Platform/Intel/Readme.md b/Platform/Intel/Readme.md
> index 02d9517d19..55cf02a8ca 100644
> --- a/Platform/Intel/Readme.md
> +++ b/Platform/Intel/Readme.md
> @@ -59,6 +59,12 @@ A UEFI firmware implementation using MinPlatformPkg
> is constructed using the fol
> 
>  ### **Supported Hardware**
> 
> + AAEON
> +
> +| Machine Name  | Supported Chipsets
> | BoardPkg | Board Name |
> +||
> --||
> +| UP Xtreme | Whiskey Lake
> | WhiskeylakeOpenBoardPkg  | UpXtreme   |
> +
>   Intel
> 
>  ***Intel Reference and Validation Platform***
> @@ -233,6 +239,9 @@ return back to the minimum platform caller.
>|   ||
> build settings, environment variables.
>|   ||
>|   ||--WhiskeylakeOpenBoardPkg
> +  |   ||   |--UpXtreme
> +  |   ||   |---build_config.cfg:
> UpXtreme specific build
> +  |   ||
> settings environment variables.
>|   ||   |--WhiskeylakeURvp
>|   ||   |---build_config.cfg:
> WhiskeylakeURvp specific build
>|   ||
> settings environment variables.
> @@ -254,8 +263,14 @@ return back to the minimum platform caller.
>  1. This firmware project has only been tested booting to Microsoft Windows
> 10 x64 and Ubuntu 17.10 with AHCI mode.
> 
>  **WhiskeylakeOpenBoardPkg**
> -1. This firmware project has only been tested booting to Microsoft Windows
> 10 x64 with AHCI mode and Integrated Graphic
> +1. This firmware project has mainly been tested booting to Microsoft
> Windows 10 x64 with AHCI mode and Integrated Graphic
> Device.
> +2. UP Xtreme boards might hang during Windows 10 boot.
> +3. Not all UP Xtreme boards are supported at this time.
> +* The model below boots to x64 windows 10 home edition and Ubuntu
> 18.04
> +  * Xtreme Intel(R) Core(TM) i3-8145UE CPU @ 2.20GHz with 8GB RAM
> +  * Xtreme Intel(R) Core(TM) i7-8565U CPU @ 1.80GHz with 16GB RAM
> +* Intel(R) Celeron(R) CPU 4305UE @ 2.00GHz with 4GB RAM hangs
> during memory initialization
> 
>  ### **Package Builds**
> 
> @@ -293,5 +308,5 @@ If you would like to help but are not sure where to
> start some areas currently i
>   * Adding board ports for more motherboards and systems
>   * Adding Clang support
> 
> -Please feel free to contact Michael Kubacki (michael.a.kubacki at intel.com)
> and Isaac Oram (isaac.w.oram at intel.com)
> +Please feel free to contact Isaac Oram (isaac.w.oram at intel.com)
>  if you would like to discuss contribution ideas.
> diff --git
> a/Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkg.dsc
> b/Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkg.dsc
> new file mode 100644
> index 00..2ab9cb03ea
> --- /dev/null
> +++
> 

[edk2-devel] [edk2-platforms] [PATCH v2 4/4] WhiskeylakeOpenBoardPkg/UpXtreme: Add DSC and build files

2020-02-13 Thread Agyeman, Prince
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2191

Adds the DSC and build files necessary to build the
UpXtreme board instance.

Key files
=
* build_config.cfg - Board-specific build configuration file.
* OpenBoardPkg.dsc - The UpXtreme board description file.
* OpenBoardPkgPcd.dsc - Used for other PCD customization.
* OpenBoardPkg.fdf - The UpXtreme board flash file.
* OpenBoardPkgBuildOption.dsc - Sets build options Based
  on PCD values.

Co-authored-by: Michael Kubacki 
Cc: Chasel Chiu 
Cc: Nate DeSimone 
Signed-off-by: Prince Agyeman 
---
 Platform/Intel/Readme.md  |  19 +-
 .../UpXtreme/OpenBoardPkg.dsc | 448 +++
 .../UpXtreme/OpenBoardPkg.fdf | 708 ++
 .../UpXtreme/OpenBoardPkgBuildOption.dsc  | 154 
 .../UpXtreme/OpenBoardPkgPcd.dsc  | 409 ++
 .../UpXtreme/build_config.cfg |  35 +
 Platform/Intel/build.cfg  |   1 +
 7 files changed, 1772 insertions(+), 2 deletions(-)
 create mode 100644 
Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkg.dsc
 create mode 100644 
Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkg.fdf
 create mode 100644 
Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkgBuildOption.dsc
 create mode 100644 
Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkgPcd.dsc
 create mode 100644 
Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/build_config.cfg

diff --git a/Platform/Intel/Readme.md b/Platform/Intel/Readme.md
index 02d9517d19..55cf02a8ca 100644
--- a/Platform/Intel/Readme.md
+++ b/Platform/Intel/Readme.md
@@ -59,6 +59,12 @@ A UEFI firmware implementation using MinPlatformPkg is 
constructed using the fol
 
 ### **Supported Hardware**
 
+ AAEON
+
+| Machine Name  | Supported Chipsets   
  | BoardPkg | Board Name |
+||--||
+| UP Xtreme | Whiskey Lake 
  | WhiskeylakeOpenBoardPkg  | UpXtreme   |
+
  Intel
 
 ***Intel Reference and Validation Platform***
@@ -233,6 +239,9 @@ return back to the minimum platform caller.
   |   || build 
settings, environment variables.
   |   ||
   |   ||--WhiskeylakeOpenBoardPkg
+  |   ||   |--UpXtreme
+  |   ||   |---build_config.cfg: UpXtreme 
specific build
+  |   || settings 
environment variables.
   |   ||   |--WhiskeylakeURvp
   |   ||   |---build_config.cfg: 
WhiskeylakeURvp specific build
   |   || settings 
environment variables.
@@ -254,8 +263,14 @@ return back to the minimum platform caller.
 1. This firmware project has only been tested booting to Microsoft Windows 10 
x64 and Ubuntu 17.10 with AHCI mode.
 
 **WhiskeylakeOpenBoardPkg**
-1. This firmware project has only been tested booting to Microsoft Windows 10 
x64 with AHCI mode and Integrated Graphic
+1. This firmware project has mainly been tested booting to Microsoft Windows 
10 x64 with AHCI mode and Integrated Graphic
Device.
+2. UP Xtreme boards might hang during Windows 10 boot.
+3. Not all UP Xtreme boards are supported at this time.
+* The model below boots to x64 windows 10 home edition and Ubuntu 18.04
+  * Xtreme Intel(R) Core(TM) i3-8145UE CPU @ 2.20GHz with 8GB RAM
+  * Xtreme Intel(R) Core(TM) i7-8565U CPU @ 1.80GHz with 16GB RAM
+* Intel(R) Celeron(R) CPU 4305UE @ 2.00GHz with 4GB RAM hangs during 
memory initialization
 
 ### **Package Builds**
 
@@ -293,5 +308,5 @@ If you would like to help but are not sure where to start 
some areas currently i
  * Adding board ports for more motherboards and systems
  * Adding Clang support
 
-Please feel free to contact Michael Kubacki (michael.a.kubacki at intel.com) 
and Isaac Oram (isaac.w.oram at intel.com)
+Please feel free to contact Isaac Oram (isaac.w.oram at intel.com)
 if you would like to discuss contribution ideas.
diff --git a/Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkg.dsc 
b/Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkg.dsc
new file mode 100644
index 00..2ab9cb03ea
--- /dev/null
+++ b/Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkg.dsc
@@ -0,0 +1,448 @@
+## @file
+#  The main build description file for the UpXtreme board.
+#
+#  Copyright (c) 2020, Intel Corporation. All rights reserved.
+#
+#  SPDX-License-Identifier: BSD-2-Clause-Patent
+#
+##
+
+[Defines]
+  DEFINE  PLATFORM_PACKAGE  = MinPlatformPkg
+  DEFINE  PLATFORM_SI_PACKAGE