* [edk2-platforms] [patch V3 01/35] Drivers/ASIX: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
@ 2021-03-26 8:38 ` Dandan Bi
2021-03-26 8:38 ` [edk2-platforms] [patch V3 02/35] Drivers/DisplayLink: " Dandan Bi
` (33 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:38 UTC (permalink / raw)
To: devel; +Cc: Samer El-Haj-Mahmoud, Ray Ni
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Samer El-Haj-Mahmoud <samer.el-haj-mahmoud@arm.com>
Cc: Ray Ni <ray.ni@intel.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Drivers/ASIX/Asix.dsc | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Drivers/ASIX/Asix.dsc b/Drivers/ASIX/Asix.dsc
index 5e02e11760..36ec2d587f 100644
--- a/Drivers/ASIX/Asix.dsc
+++ b/Drivers/ASIX/Asix.dsc
@@ -24,10 +24,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this Platform.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
DebugLib|MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.inf
DebugPrintErrorLevelLib|MdePkg/Library/BaseDebugPrintErrorLevelLib/BaseDebugPrintErrorLevelLib.inf
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
BaseMemoryLib|MdePkg/Library/BaseMemoryLib/BaseMemoryLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 02/35] Drivers/DisplayLink: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
2021-03-26 8:38 ` [edk2-platforms] [patch V3 01/35] Drivers/ASIX: " Dandan Bi
@ 2021-03-26 8:38 ` Dandan Bi
2021-03-26 8:38 ` [edk2-platforms] [patch V3 03/35] Drivers/OptionRomPkg: " Dandan Bi
` (32 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:38 UTC (permalink / raw)
To: devel; +Cc: Leif Lindholm, Ard Biesheuvel, Andy Hayes
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Andy Hayes <andy.hayes@displaylink.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkPkg.dsc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkPkg.dsc b/Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkPkg.dsc
index 955331ba60..d2dffecd9e 100644
--- a/Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkPkg.dsc
+++ b/Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkPkg.dsc
@@ -14,10 +14,12 @@ [Defines]
OUTPUT_DIRECTORY = Build/DisplayLink
SUPPORTED_ARCHITECTURES = X64|IA32|AARCH64|ARM
BUILD_TARGETS = DEBUG|RELEASE|NOOPT
SKUID_IDENTIFIER = DEFAULT
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
BaseMemoryLib|MdePkg/Library/BaseMemoryLib/BaseMemoryLib.inf
DebugLib|MdePkg/Library/UefiDebugLibConOut/UefiDebugLibConOut.inf
DebugPrintErrorLevelLib|MdePkg/Library/BaseDebugPrintErrorLevelLib/BaseDebugPrintErrorLevelLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 03/35] Drivers/OptionRomPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
2021-03-26 8:38 ` [edk2-platforms] [patch V3 01/35] Drivers/ASIX: " Dandan Bi
2021-03-26 8:38 ` [edk2-platforms] [patch V3 02/35] Drivers/DisplayLink: " Dandan Bi
@ 2021-03-26 8:38 ` Dandan Bi
2021-03-26 8:38 ` [edk2-platforms] [patch V3 04/35] Features/Debugging: " Dandan Bi
` (31 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:38 UTC (permalink / raw)
To: devel; +Cc: Ray Ni
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Ray Ni <ray.ni@intel.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Drivers/OptionRomPkg/OptionRomPkg.dsc | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/Drivers/OptionRomPkg/OptionRomPkg.dsc b/Drivers/OptionRomPkg/OptionRomPkg.dsc
index 8a13cc54e6..bb166d11c3 100644
--- a/Drivers/OptionRomPkg/OptionRomPkg.dsc
+++ b/Drivers/OptionRomPkg/OptionRomPkg.dsc
@@ -4,11 +4,11 @@
# This package is designed to interoperate with the EDK II open source project
# at http://www.tianocore.org, and this package is required to build PCI compliant
# Option ROM image for all CPU architectures, including EBC target.
# A single driver can support mixes of EFI 1.1, UEFI 2.0 and UEFI 2.1.
#
-# Copyright (c) 2007 - 2018, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2007 - 2021, Intel Corporation. All rights reserved.<BR>
# Copyright (c) 2016, Linaro Ltd. All rights reserved.<BR>
# Copyright (c) 2020, ARM Limited. All rights reserved.
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
@@ -36,10 +36,12 @@ [Defines]
#
################################################################################
[SkuIds]
0|DEFAULT # The entry: 0|DEFAULT is reserved and always required.
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
DebugLib|MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.inf
DebugPrintErrorLevelLib|MdePkg/Library/BaseDebugPrintErrorLevelLib/BaseDebugPrintErrorLevelLib.inf
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
BaseMemoryLib|MdePkg/Library/BaseMemoryLib/BaseMemoryLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 04/35] Features/Debugging: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (2 preceding siblings ...)
2021-03-26 8:38 ` [edk2-platforms] [patch V3 03/35] Drivers/OptionRomPkg: " Dandan Bi
@ 2021-03-26 8:38 ` Dandan Bi
2021-03-26 8:38 ` [edk2-platforms] [patch V3 05/35] Features/Network: " Dandan Bi
` (30 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:38 UTC (permalink / raw)
To: devel; +Cc: Eric Dong, Liming Gao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Eric Dong <eric.dong@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
.../AcpiDebugFeaturePkg/Include/AcpiDebugFeature.dsc | 5 ++++-
.../BeepDebugFeaturePkg/Include/BeepDebugFeature.dsc | 5 ++++-
.../PostCodeDebugFeaturePkg/Include/PostCodeDebugFeature.dsc | 5 ++++-
.../Usb3DebugFeaturePkg/Include/Usb3DebugFeature.dsc | 5 ++++-
4 files changed, 16 insertions(+), 4 deletions(-)
diff --git a/Features/Intel/Debugging/AcpiDebugFeaturePkg/Include/AcpiDebugFeature.dsc b/Features/Intel/Debugging/AcpiDebugFeaturePkg/Include/AcpiDebugFeature.dsc
index 3e6b1f69c2..856222a53d 100644
--- a/Features/Intel/Debugging/AcpiDebugFeaturePkg/Include/AcpiDebugFeature.dsc
+++ b/Features/Intel/Debugging/AcpiDebugFeaturePkg/Include/AcpiDebugFeature.dsc
@@ -4,11 +4,11 @@
#
# The DEC files are used by the utilities that parse DSC and
# INF files to generate AutoGen.c and AutoGen.h files
# for the build infrastructure.
#
-# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2019 - 2021, Intel Corporation. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -28,10 +28,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this feature.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
#######################################
# Edk2 Packages
#######################################
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
diff --git a/Features/Intel/Debugging/BeepDebugFeaturePkg/Include/BeepDebugFeature.dsc b/Features/Intel/Debugging/BeepDebugFeaturePkg/Include/BeepDebugFeature.dsc
index 65e00b5979..b24b3e458f 100644
--- a/Features/Intel/Debugging/BeepDebugFeaturePkg/Include/BeepDebugFeature.dsc
+++ b/Features/Intel/Debugging/BeepDebugFeaturePkg/Include/BeepDebugFeature.dsc
@@ -4,11 +4,11 @@
#
# The DEC files are used by the utilities that parse DSC and
# INF files to generate AutoGen.c and AutoGen.h files
# for the build infrastructure.
#
-# Copyright (c) 2020, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2020 - 2021, Intel Corporation. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -38,10 +38,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this feature.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
#######################################
# Edk2 Packages
#######################################
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
diff --git a/Features/Intel/Debugging/PostCodeDebugFeaturePkg/Include/PostCodeDebugFeature.dsc b/Features/Intel/Debugging/PostCodeDebugFeaturePkg/Include/PostCodeDebugFeature.dsc
index 2852b9cf7c..3d886fff38 100644
--- a/Features/Intel/Debugging/PostCodeDebugFeaturePkg/Include/PostCodeDebugFeature.dsc
+++ b/Features/Intel/Debugging/PostCodeDebugFeaturePkg/Include/PostCodeDebugFeature.dsc
@@ -4,11 +4,11 @@
#
# The DEC files are used by the utilities that parse DSC and
# INF files to generate AutoGen.c and AutoGen.h files
# for the build infrastructure.
#
-# Copyright (c) 2020, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2020 - 2021, Intel Corporation. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -38,10 +38,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this feature.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
#######################################
# Edk2 Packages
#######################################
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
diff --git a/Features/Intel/Debugging/Usb3DebugFeaturePkg/Include/Usb3DebugFeature.dsc b/Features/Intel/Debugging/Usb3DebugFeaturePkg/Include/Usb3DebugFeature.dsc
index 95adb01a74..1e3aaecd5d 100644
--- a/Features/Intel/Debugging/Usb3DebugFeaturePkg/Include/Usb3DebugFeature.dsc
+++ b/Features/Intel/Debugging/Usb3DebugFeaturePkg/Include/Usb3DebugFeature.dsc
@@ -4,11 +4,11 @@
#
# The DEC files are used by the utilities that parse DSC and
# INF files to generate AutoGen.c and AutoGen.h files
# for the build infrastructure.
#
-# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2019 - 2021, Intel Corporation. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -28,10 +28,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this feature.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
#######################################
# Edk2 Packages
#######################################
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 05/35] Features/Network: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (3 preceding siblings ...)
2021-03-26 8:38 ` [edk2-platforms] [patch V3 04/35] Features/Debugging: " Dandan Bi
@ 2021-03-26 8:38 ` Dandan Bi
2021-03-26 8:38 ` [edk2-platforms] [patch V3 06/35] Features/OutOfBandManagement: " Dandan Bi
` (29 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:38 UTC (permalink / raw)
To: devel; +Cc: Liming Gao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
.../Network/NetworkFeaturePkg/Include/NetworkFeature.dsc | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/Features/Intel/Network/NetworkFeaturePkg/Include/NetworkFeature.dsc b/Features/Intel/Network/NetworkFeaturePkg/Include/NetworkFeature.dsc
index c59c21ee6c..bbc91ecbb1 100644
--- a/Features/Intel/Network/NetworkFeaturePkg/Include/NetworkFeature.dsc
+++ b/Features/Intel/Network/NetworkFeaturePkg/Include/NetworkFeature.dsc
@@ -4,11 +4,11 @@
#
# The DEC files are used by the utilities that parse DSC and
# INF files to generate AutoGen.c and AutoGen.h files
# for the build infrastructure.
#
-# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2019 - 2021, Intel Corporation. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -38,10 +38,13 @@ [PcdsFixedAtBuild]
################################################################################
#
# Library Class section - list of all Library Classes needed by this feature.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
!include NetworkPkg/NetworkLibs.dsc.inc
#######################################
# Edk2 Packages
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 06/35] Features/OutOfBandManagement: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (4 preceding siblings ...)
2021-03-26 8:38 ` [edk2-platforms] [patch V3 05/35] Features/Network: " Dandan Bi
@ 2021-03-26 8:38 ` Dandan Bi
2021-03-26 8:38 ` [edk2-platforms] [patch V3 07/35] Features/PowerManagement: " Dandan Bi
` (28 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:38 UTC (permalink / raw)
To: devel; +Cc: Liming Gao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
.../IpmiFeaturePkg/Include/IpmiFeature.dsc | 5 ++++-
.../SpcrFeaturePkg/Include/SpcrFeature.dsc | 5 ++++-
2 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Include/IpmiFeature.dsc b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Include/IpmiFeature.dsc
index a82f18a68c..dd36b8c697 100644
--- a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Include/IpmiFeature.dsc
+++ b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Include/IpmiFeature.dsc
@@ -4,11 +4,11 @@
#
# The DEC files are used by the utilities that parse DSC and
# INF files to generate AutoGen.c and AutoGen.h files
# for the build infrastructure.
#
-# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2019 - 2021, Intel Corporation. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -28,10 +28,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this feature.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
#######################################
# Edk2 Packages
#######################################
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
diff --git a/Features/Intel/OutOfBandManagement/SpcrFeaturePkg/Include/SpcrFeature.dsc b/Features/Intel/OutOfBandManagement/SpcrFeaturePkg/Include/SpcrFeature.dsc
index 89ef648f31..c51e552df5 100644
--- a/Features/Intel/OutOfBandManagement/SpcrFeaturePkg/Include/SpcrFeature.dsc
+++ b/Features/Intel/OutOfBandManagement/SpcrFeaturePkg/Include/SpcrFeature.dsc
@@ -4,11 +4,11 @@
#
# The DEC files are used by the utilities that parse DSC and
# INF files to generate AutoGen.c and AutoGen.h files
# for the build infrastructure.
#
-# Copyright (c) 2020, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2020 - 2021, Intel Corporation. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -28,10 +28,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this feature.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
#######################################
# Edk2 Packages
#######################################
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 07/35] Features/PowerManagement: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (5 preceding siblings ...)
2021-03-26 8:38 ` [edk2-platforms] [patch V3 06/35] Features/OutOfBandManagement: " Dandan Bi
@ 2021-03-26 8:38 ` Dandan Bi
2021-03-26 8:38 ` [edk2-platforms] [patch V3 08/35] Features/SystemInformation: " Dandan Bi
` (27 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:38 UTC (permalink / raw)
To: devel; +Cc: Liming Gao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
.../Intel/PowerManagement/S3FeaturePkg/Include/S3Feature.dsc | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/Features/Intel/PowerManagement/S3FeaturePkg/Include/S3Feature.dsc b/Features/Intel/PowerManagement/S3FeaturePkg/Include/S3Feature.dsc
index 647dc3f93a..312870981e 100644
--- a/Features/Intel/PowerManagement/S3FeaturePkg/Include/S3Feature.dsc
+++ b/Features/Intel/PowerManagement/S3FeaturePkg/Include/S3Feature.dsc
@@ -4,11 +4,11 @@
#
# The DEC files are used by the utilities that parse DSC and
# INF files to generate AutoGen.c and AutoGen.h files
# for the build infrastructure.
#
-# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2019 - 2021, Intel Corporation. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -28,10 +28,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this feature.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
#######################################
# Edk2 Packages
#######################################
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 08/35] Features/SystemInformation: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (6 preceding siblings ...)
2021-03-26 8:38 ` [edk2-platforms] [patch V3 07/35] Features/PowerManagement: " Dandan Bi
@ 2021-03-26 8:38 ` Dandan Bi
2021-03-26 8:38 ` [edk2-platforms] [patch V3 09/35] Features/UserInterface: " Dandan Bi
` (26 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:38 UTC (permalink / raw)
To: devel; +Cc: Liming Gao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
.../SmbiosFeaturePkg/Include/SmbiosFeature.dsc | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/Features/Intel/SystemInformation/SmbiosFeaturePkg/Include/SmbiosFeature.dsc b/Features/Intel/SystemInformation/SmbiosFeaturePkg/Include/SmbiosFeature.dsc
index a59fdf3702..31d44b68b3 100644
--- a/Features/Intel/SystemInformation/SmbiosFeaturePkg/Include/SmbiosFeature.dsc
+++ b/Features/Intel/SystemInformation/SmbiosFeaturePkg/Include/SmbiosFeature.dsc
@@ -4,11 +4,11 @@
#
# The DEC files are used by the utilities that parse DSC and
# INF files to generate AutoGen.c and AutoGen.h files
# for the build infrastructure.
#
-# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2019 - 2021, Intel Corporation. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -28,10 +28,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this feature.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
#######################################
# Edk2 Packages
#######################################
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 09/35] Features/UserInterface: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (7 preceding siblings ...)
2021-03-26 8:38 ` [edk2-platforms] [patch V3 08/35] Features/SystemInformation: " Dandan Bi
@ 2021-03-26 8:38 ` Dandan Bi
2021-03-26 8:38 ` [edk2-platforms] [patch V3 10/35] Platform/AMD: " Dandan Bi
` (25 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:38 UTC (permalink / raw)
To: devel; +Cc: Liming Gao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
.../UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc | 5 ++++-
.../UserAuthFeaturePkg/Include/UserAuthFeature.dsc | 5 ++++-
.../Include/VirtualKeyboardFeature.dsc | 5 ++++-
3 files changed, 12 insertions(+), 3 deletions(-)
diff --git a/Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc b/Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc
index d2dcdeb36a..1898be12ed 100644
--- a/Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc
+++ b/Features/Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc
@@ -4,11 +4,11 @@
#
# The DEC files are used by the utilities that parse DSC and
# INF files to generate AutoGen.c and AutoGen.h files
# for the build infrastructure.
#
-# Copyright (c) 2020, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2020 - 2021, Intel Corporation. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -28,10 +28,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this feature.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
#######################################
# Edk2 Packages
#######################################
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
diff --git a/Features/Intel/UserInterface/UserAuthFeaturePkg/Include/UserAuthFeature.dsc b/Features/Intel/UserInterface/UserAuthFeaturePkg/Include/UserAuthFeature.dsc
index 7d784f4dff..d4f7a5ed23 100644
--- a/Features/Intel/UserInterface/UserAuthFeaturePkg/Include/UserAuthFeature.dsc
+++ b/Features/Intel/UserInterface/UserAuthFeaturePkg/Include/UserAuthFeature.dsc
@@ -4,11 +4,11 @@
#
# The DEC files are used by the utilities that parse DSC and
# INF files to generate AutoGen.c and AutoGen.h files
# for the build infrastructure.
#
-# Copyright (c) 2020, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2020 - 2021, Intel Corporation. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -28,10 +28,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this feature.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
#######################################
# Edk2 Packages
#######################################
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
diff --git a/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/Include/VirtualKeyboardFeature.dsc b/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/Include/VirtualKeyboardFeature.dsc
index c10fb2d567..a8c3819480 100644
--- a/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/Include/VirtualKeyboardFeature.dsc
+++ b/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/Include/VirtualKeyboardFeature.dsc
@@ -4,11 +4,11 @@
#
# The DEC files are used by the utilities that parse DSC and
# INF files to generate AutoGen.c and AutoGen.h files
# for the build infrastructure.
#
-# Copyright (c) 2020, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2020 - 2021, Intel Corporation. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -28,10 +28,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this feature.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
#######################################
# Edk2 Packages
#######################################
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 10/35] Platform/AMD: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (8 preceding siblings ...)
2021-03-26 8:38 ` [edk2-platforms] [patch V3 09/35] Features/UserInterface: " Dandan Bi
@ 2021-03-26 8:38 ` Dandan Bi
2021-03-26 8:38 ` [edk2-platforms] [patch V3 11/35] Platform/ARM: " Dandan Bi
` (24 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:38 UTC (permalink / raw)
To: devel; +Cc: Ard Biesheuvel, Leif Lindholm
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Leif Lindholm <leif@nuviainc.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Platform/AMD/OverdriveBoard/OverdriveBoard.dsc | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc b/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
index 05d1b69c83..bc8d8e37d6 100644
--- a/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
+++ b/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
@@ -27,10 +27,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this Platform.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
!if $(TARGET) == RELEASE
DebugLib|MdePkg/Library/BaseDebugLibNull/BaseDebugLibNull.inf
!else
DebugLib|MdePkg/Library/BaseDebugLibSerialPort/BaseDebugLibSerialPort.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 11/35] Platform/ARM: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (9 preceding siblings ...)
2021-03-26 8:38 ` [edk2-platforms] [patch V3 10/35] Platform/AMD: " Dandan Bi
@ 2021-03-26 8:38 ` Dandan Bi
2021-03-26 8:38 ` [edk2-platforms] [patch V3 12/35] Platform/BeagleBoard: " Dandan Bi
` (23 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:38 UTC (permalink / raw)
To: devel; +Cc: Ard Biesheuvel, Thomas Abraham, Sami Mujawar
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Thomas Abraham <thomas.abraham@arm.com>
Cc: Sami Mujawar <sami.mujawar@arm.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Platform/ARM/SgiPkg/PlatformStandaloneMm.dsc | 3 +++
Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc | 2 ++
2 files changed, 5 insertions(+)
diff --git a/Platform/ARM/SgiPkg/PlatformStandaloneMm.dsc b/Platform/ARM/SgiPkg/PlatformStandaloneMm.dsc
index 38bf6020ba..e281d54909 100644
--- a/Platform/ARM/SgiPkg/PlatformStandaloneMm.dsc
+++ b/Platform/ARM/SgiPkg/PlatformStandaloneMm.dsc
@@ -27,10 +27,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this Platform.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
#
# Basic
#
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
diff --git a/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc b/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc
index fee7cfcc2d..01a7677c85 100644
--- a/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc
+++ b/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc
@@ -25,10 +25,12 @@ [BuildOptions.common.EDKII.DXE_CORE,BuildOptions.common.EDKII.DXE_DRIVER,BuildOp
[BuildOptions.common.EDKII.DXE_RUNTIME_DRIVER]
GCC:*_*_ARM_DLINK_FLAGS = -z common-page-size=0x1000
GCC:*_*_AARCH64_DLINK_FLAGS = -z common-page-size=0x10000
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
!if $(TARGET) == RELEASE
DebugLib|MdePkg/Library/BaseDebugLibNull/BaseDebugLibNull.inf
!else
DebugLib|MdePkg/Library/BaseDebugLibSerialPort/BaseDebugLibSerialPort.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 12/35] Platform/BeagleBoard: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (10 preceding siblings ...)
2021-03-26 8:38 ` [edk2-platforms] [patch V3 11/35] Platform/ARM: " Dandan Bi
@ 2021-03-26 8:38 ` Dandan Bi
2021-03-26 8:39 ` [edk2-platforms] [patch V3 13/35] Platform/BoardModulePkg: " Dandan Bi
` (22 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:38 UTC (permalink / raw)
To: devel; +Cc: Ard Biesheuvel, Leif Lindholm
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Leif Lindholm <leif@nuviainc.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.dsc | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.dsc b/Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.dsc
index c5fc978c5f..8a4b0ad885 100644
--- a/Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.dsc
+++ b/Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.dsc
@@ -1,10 +1,10 @@
#/** @file
# Beagle board package.
#
# Copyright (c) 2009 - 2010, Apple Inc. All rights reserved.<BR>
-# Copyright (c) 2015 - 2019, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2015 - 2021, Intel Corporation. All rights reserved.<BR>
# Copyright (c) 2016, Linaro Ltd. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
#**/
@@ -23,10 +23,11 @@ [Defines]
SUPPORTED_ARCHITECTURES = ARM
BUILD_TARGETS = DEBUG|RELEASE|NOOPT
SKUID_IDENTIFIER = DEFAULT
FLASH_DEFINITION = Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.fdf
+!include MdePkg/MdeLibs.dsc.inc
[LibraryClasses.common]
ArmLib|ArmPkg/Library/ArmLib/ArmBaseLib.inf
ArmMmuLib|ArmPkg/Library/ArmMmuLib/ArmMmuBaseLib.inf
ArmPlatformLib|Platform/BeagleBoard/BeagleBoardPkg/Library/BeagleBoardLib/BeagleBoardLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 13/35] Platform/BoardModulePkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (11 preceding siblings ...)
2021-03-26 8:38 ` [edk2-platforms] [patch V3 12/35] Platform/BeagleBoard: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 8:39 ` [edk2-platforms] [patch V3 14/35] Platform/MinPlatformPkg: " Dandan Bi
` (21 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Eric Dong, Liming Gao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Eric Dong <eric.dong@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Platform/Intel/BoardModulePkg/BoardModulePkg.dsc | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/Platform/Intel/BoardModulePkg/BoardModulePkg.dsc b/Platform/Intel/BoardModulePkg/BoardModulePkg.dsc
index bcc2dfbfeb..9f00592a19 100644
--- a/Platform/Intel/BoardModulePkg/BoardModulePkg.dsc
+++ b/Platform/Intel/BoardModulePkg/BoardModulePkg.dsc
@@ -4,11 +4,11 @@
#
# The DEC files are used by the utilities that parse DSC and
# INF files to generate AutoGen.c and AutoGen.h files
# for the build infrastructure.
#
-# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2019 - 2021, Intel Corporation. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -20,10 +20,12 @@ [Defines]
OUTPUT_DIRECTORY = Build/BoardModulePkg
SUPPORTED_ARCHITECTURES = IA32|X64
BUILD_TARGETS = DEBUG|RELEASE|NOOPT
SKUID_IDENTIFIER = DEFAULT
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
BaseMemoryLib|MdePkg/Library/BaseMemoryLib/BaseMemoryLib.inf
PrintLib|MdePkg/Library/BasePrintLib/BasePrintLib.inf
PeiServicesLib|MdePkg/Library/PeiServicesLib/PeiServicesLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 14/35] Platform/MinPlatformPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (12 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 13/35] Platform/BoardModulePkg: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 8:39 ` [edk2-platforms] [patch V3 15/35] Platform/QuarkPlatformPkg: " Dandan Bi
` (20 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Chasel Chiu, Nate DeSimone, Liming Gao, Eric Dong
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
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: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc b/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc
index bcabb797e9..cf2940cf02 100644
--- a/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc
+++ b/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc
@@ -1,18 +1,20 @@
## @file
# Platform description.
#
-# Copyright (c) 2017 - 2018, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2017 - 2021, Intel Corporation. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
#
# Generic EDKII Lib
#
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
#
# Entry point
#
PeiCoreEntryPoint|MdePkg/Library/PeiCoreEntryPoint/PeiCoreEntryPoint.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 15/35] Platform/QuarkPlatformPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (13 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 14/35] Platform/MinPlatformPkg: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 8:39 ` [edk2-platforms] [patch V3 16/35] Platform/Vlv2TbltDevicePkg: " Dandan Bi
` (19 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Michael D Kinney, Kelly Steele
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Kelly Steele <kelly.steele@intel.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Platform/Intel/QuarkPlatformPkg/Quark.dsc | 3 +++
Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc | 3 +++
2 files changed, 6 insertions(+)
diff --git a/Platform/Intel/QuarkPlatformPkg/Quark.dsc b/Platform/Intel/QuarkPlatformPkg/Quark.dsc
index c58da58348..cc1eba4df4 100644
--- a/Platform/Intel/QuarkPlatformPkg/Quark.dsc
+++ b/Platform/Intel/QuarkPlatformPkg/Quark.dsc
@@ -67,10 +67,13 @@ [SkuIds]
################################################################################
#
# Library Class section - list of all Library Classes needed by this Platform.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
#
# Entry point
#
PeiCoreEntryPoint|MdePkg/Library/PeiCoreEntryPoint/PeiCoreEntryPoint.inf
diff --git a/Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc b/Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc
index 6fdfba9a58..993d698eec 100644
--- a/Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc
+++ b/Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc
@@ -55,10 +55,13 @@ [SkuIds]
################################################################################
#
# Library Class section - list of all Library Classes needed by this Platform.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
#
# Entry point
#
PeiCoreEntryPoint|MdePkg/Library/PeiCoreEntryPoint/PeiCoreEntryPoint.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 16/35] Platform/Vlv2TbltDevicePkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (14 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 15/35] Platform/QuarkPlatformPkg: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 8:39 ` [edk2-platforms] [patch V3 17/35] Platform/LeMaker: " Dandan Bi
` (18 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Michael D Kinney, Zailiang Sun, Yi Qian
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Zailiang Sun <zailiang.sun@intel.com>
Cc: Yi Qian <yi.qian@intel.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc | 5 ++++-
Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc | 5 ++++-
2 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc b/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc
index 409f31c982..bdf69b7e9a 100644
--- a/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc
+++ b/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc
@@ -1,9 +1,9 @@
#/** @file
# Platform description.
#
-# Copyright (c) 2012 - 2020, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2012 - 2021, Intel Corporation. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
#
#**/
@@ -61,10 +61,13 @@ [DefaultStores]
################################################################################
#
# Library Class section - list of all Library Classes needed by this Platform.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
#
# Entry point
#
PeiCoreEntryPoint|MdePkg/Library/PeiCoreEntryPoint/PeiCoreEntryPoint.inf
diff --git a/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc b/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc
index 38bd825c8b..5f26bc02dd 100644
--- a/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc
+++ b/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc
@@ -1,9 +1,9 @@
#/** @file
# Platform description.
#
-# Copyright (c) 2012 - 2020, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2012 - 2021, Intel Corporation. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
#
#**/
@@ -63,10 +63,13 @@ [DefaultStores]
################################################################################
#
# Library Class section - list of all Library Classes needed by this Platform.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
#
# Entry point
#
PeiCoreEntryPoint|MdePkg/Library/PeiCoreEntryPoint/PeiCoreEntryPoint.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 17/35] Platform/LeMaker: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (15 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 16/35] Platform/Vlv2TbltDevicePkg: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 8:39 ` [edk2-platforms] [patch V3 18/35] Platform/Qemu: " Dandan Bi
` (17 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Ard Biesheuvel, Leif Lindholm
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Leif Lindholm <leif@nuviainc.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Platform/LeMaker/CelloBoard/CelloBoard.dsc | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Platform/LeMaker/CelloBoard/CelloBoard.dsc b/Platform/LeMaker/CelloBoard/CelloBoard.dsc
index 26bbbb2497..cf2754a509 100644
--- a/Platform/LeMaker/CelloBoard/CelloBoard.dsc
+++ b/Platform/LeMaker/CelloBoard/CelloBoard.dsc
@@ -33,10 +33,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this Platform.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
!if $(TARGET) == RELEASE
DebugLib|MdePkg/Library/BaseDebugLibNull/BaseDebugLibNull.inf
!else
DebugLib|MdePkg/Library/BaseDebugLibSerialPort/BaseDebugLibSerialPort.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 18/35] Platform/Qemu: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (16 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 17/35] Platform/LeMaker: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 8:39 ` [edk2-platforms] [patch V3 19/35] Platform/RaspberryPi: " Dandan Bi
` (16 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Ard Biesheuvel, Leif Lindholm, Graeme Gregory, Radoslaw Biernacki
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Graeme Gregory <graeme@nuviainc.com>
Cc: Radoslaw Biernacki <rad@semihalf.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Platform/Qemu/SbsaQemu/SbsaQemu.dsc | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Platform/Qemu/SbsaQemu/SbsaQemu.dsc b/Platform/Qemu/SbsaQemu/SbsaQemu.dsc
index c1f8a46965..11ce361cdb 100644
--- a/Platform/Qemu/SbsaQemu/SbsaQemu.dsc
+++ b/Platform/Qemu/SbsaQemu/SbsaQemu.dsc
@@ -39,10 +39,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this Platform.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
!if $(TARGET) == RELEASE
DebugLib|MdePkg/Library/BaseDebugLibNull/BaseDebugLibNull.inf
!else
DebugLib|MdePkg/Library/BaseDebugLibSerialPort/BaseDebugLibSerialPort.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 19/35] Platform/RaspberryPi: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (17 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 18/35] Platform/Qemu: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 8:39 ` [edk2-platforms] [patch V3 20/35] Platform/RISC-V: " Dandan Bi
` (15 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Ard Biesheuvel, Leif Lindholm, Pete Batard
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Pete Batard <pete@akeo.ie>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
Reviewed-by: Pete Batard <pete@akeo.ie>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Platform/RaspberryPi/RPi3/RPi3.dsc | 5 ++++-
Platform/RaspberryPi/RPi4/RPi4.dsc | 5 ++++-
2 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/Platform/RaspberryPi/RPi3/RPi3.dsc b/Platform/RaspberryPi/RPi3/RPi3.dsc
index 107cbda297..0961133ae9 100644
--- a/Platform/RaspberryPi/RPi3/RPi3.dsc
+++ b/Platform/RaspberryPi/RPi3/RPi3.dsc
@@ -1,10 +1,10 @@
# @file
#
# Copyright (c) 2011 - 2020, ARM Limited. All rights reserved.
# Copyright (c) 2014, Linaro Limited. All rights reserved.
-# Copyright (c) 2015 - 2016, Intel Corporation. All rights reserved.
+# Copyright (c) 2015 - 2021, Intel Corporation. All rights reserved.
# Copyright (c) 2017 - 2018, Andrei Warkentin <andrey.warkentin@gmail.com>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -52,10 +52,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this Platform.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
!if $(TARGET) == RELEASE
DebugLib|MdePkg/Library/BaseDebugLibNull/BaseDebugLibNull.inf
!else
DebugLib|MdePkg/Library/BaseDebugLibSerialPort/BaseDebugLibSerialPort.inf
diff --git a/Platform/RaspberryPi/RPi4/RPi4.dsc b/Platform/RaspberryPi/RPi4/RPi4.dsc
index e0fad6f744..2c05c31118 100644
--- a/Platform/RaspberryPi/RPi4/RPi4.dsc
+++ b/Platform/RaspberryPi/RPi4/RPi4.dsc
@@ -1,10 +1,10 @@
# @file
#
# Copyright (c) 2011 - 2020, ARM Limited. All rights reserved.
# Copyright (c) 2017 - 2018, Andrei Warkentin <andrey.warkentin@gmail.com>
-# Copyright (c) 2015 - 2016, Intel Corporation. All rights reserved.
+# Copyright (c) 2015 - 2021, Intel Corporation. All rights reserved.
# Copyright (c) 2014, Linaro Limited. All rights reserved.
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -50,10 +50,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this Platform.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
!if $(TARGET) == RELEASE
DebugLib|MdePkg/Library/BaseDebugLibNull/BaseDebugLibNull.inf
!else
DebugLib|MdePkg/Library/BaseDebugLibSerialPort/BaseDebugLibSerialPort.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 20/35] Platform/RISC-V: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (18 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 19/35] Platform/RaspberryPi: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-29 1:58 ` Abner Chang
2021-03-26 8:39 ` [edk2-platforms] [patch V3 21/35] Platform/SiFive: " Dandan Bi
` (14 subsequent siblings)
34 siblings, 1 reply; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Abner Chang, Daniel Schaefer, Gilbert Chen
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Abner Chang <abner.chang@hpe.com>
Cc: Daniel Schaefer <daniel.schaefer@hpe.com>
Cc: Gilbert Chen <gilbert.chen@hpe.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
Reviewed-by: Abner Chang <abner.chang@hpe.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Platform/RISC-V/PlatformPkg/RiscVPlatformPkg.dsc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Platform/RISC-V/PlatformPkg/RiscVPlatformPkg.dsc b/Platform/RISC-V/PlatformPkg/RiscVPlatformPkg.dsc
index 092717d2ef..5d9674a965 100644
--- a/Platform/RISC-V/PlatformPkg/RiscVPlatformPkg.dsc
+++ b/Platform/RISC-V/PlatformPkg/RiscVPlatformPkg.dsc
@@ -34,10 +34,12 @@ [BuildOptions]
#
################################################################################
[SkuIds]
0|DEFAULT
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
RiscVOpensbiPlatformLib|Platform/RISC-V/PlatformPkg/Library/OpensbiPlatformLibNull/OpensbiPlatformLibNull.inf
RiscVCpuLib|Silicon/RISC-V/ProcessorPkg/Library/RiscVCpuLib/RiscVCpuLib.inf
RiscVEdk2SbiLib|Silicon/RISC-V/ProcessorPkg/Library/RiscVEdk2SbiLib/RiscVEdk2SbiLib.inf
RiscVOpensbiLib|Silicon/RISC-V/ProcessorPkg/Library/RiscVOpensbiLib/RiscVOpensbiLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* Re: [edk2-platforms] [patch V3 20/35] Platform/RISC-V: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:39 ` [edk2-platforms] [patch V3 20/35] Platform/RISC-V: " Dandan Bi
@ 2021-03-29 1:58 ` Abner Chang
0 siblings, 0 replies; 50+ messages in thread
From: Abner Chang @ 2021-03-29 1:58 UTC (permalink / raw)
To: Dandan Bi, devel@edk2.groups.io; +Cc: Schaefer, Daniel, Chen, Gilbert
Reviewed-by: Abner Chang <abner.chang@hpe.com>
> -----Original Message-----
> From: Dandan Bi [mailto:dandan.bi@intel.com]
> Sent: Friday, March 26, 2021 4:39 PM
> To: devel@edk2.groups.io
> Cc: Chang, Abner (HPS SW/FW Technologist) <abner.chang@hpe.com>;
> Schaefer, Daniel <daniel.schaefer@hpe.com>; Chen, Gilbert
> <gilbert.chen@hpe.com>
> Subject: [edk2-platforms] [patch V3 20/35] Platform/RISC-V: Consume
> MdeLibs.dsc.inc for RegisterFilterLib
>
> REF: INVALID URI REMOVED
> 3A__bugzilla.tianocore.org_show-5Fbug.cgi-3Fid-
> 3D3246&d=DwIBAg&c=C5b8zRQO1miGmBeVZ2LFWg&r=_SN6FZBN4Vgi4Ulks
> kz6qU3NYRO03nHp9P7Z5q59A3E&m=3Ut4hhEyZDbul2hIaDzUdz-
> eIVkbh4MI_N9NkTGHQRA&s=ysb22szNtvQEtzB-0NYy-
> UJ577b0hqVcww9rqwibN4k&e=
>
> MdeLibs.dsc.inc was added for some basic/default library
> instances provided by MdePkg and RegisterFilterLibNull Library
> was also added into it as the first version of MdeLibs.dsc.inc.
>
> So update platform dsc to consume MdeLibs.dsc.inc for
> RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
>
> Cc: Abner Chang <abner.chang@hpe.com>
> Cc: Daniel Schaefer <daniel.schaefer@hpe.com>
> Cc: Gilbert Chen <gilbert.chen@hpe.com>
> Signed-off-by: Dandan Bi <dandan.bi@intel.com>
> Reviewed-by: Abner Chang <abner.chang@hpe.com>
> ---
> V3:
> Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
> [LibraryClasses] section since we can specify different [LibraryClasses]
> section for libary instances in MdeLibs.dsc.inc.
> ---
> Platform/RISC-V/PlatformPkg/RiscVPlatformPkg.dsc | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/Platform/RISC-V/PlatformPkg/RiscVPlatformPkg.dsc
> b/Platform/RISC-V/PlatformPkg/RiscVPlatformPkg.dsc
> index 092717d2ef..5d9674a965 100644
> --- a/Platform/RISC-V/PlatformPkg/RiscVPlatformPkg.dsc
> +++ b/Platform/RISC-V/PlatformPkg/RiscVPlatformPkg.dsc
> @@ -34,10 +34,12 @@ [BuildOptions]
> #
>
> ##########################################################
> ######################
> [SkuIds]
> 0|DEFAULT
>
> +!include MdePkg/MdeLibs.dsc.inc
> +
> [LibraryClasses.common]
> RiscVOpensbiPlatformLib|Platform/RISC-
> V/PlatformPkg/Library/OpensbiPlatformLibNull/OpensbiPlatformLibNull.inf
> RiscVCpuLib|Silicon/RISC-
> V/ProcessorPkg/Library/RiscVCpuLib/RiscVCpuLib.inf
> RiscVEdk2SbiLib|Silicon/RISC-
> V/ProcessorPkg/Library/RiscVEdk2SbiLib/RiscVEdk2SbiLib.inf
> RiscVOpensbiLib|Silicon/RISC-
> V/ProcessorPkg/Library/RiscVOpensbiLib/RiscVOpensbiLib.inf
> --
> 2.18.0.windows.1
^ permalink raw reply [flat|nested] 50+ messages in thread
* [edk2-platforms] [patch V3 21/35] Platform/SiFive: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (19 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 20/35] Platform/RISC-V: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-29 1:58 ` Abner Chang
2021-03-26 8:39 ` [edk2-platforms] [patch V3 22/35] Platform/Socionext: " Dandan Bi
` (13 subsequent siblings)
34 siblings, 1 reply; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Abner Chang, Daniel Schaefer, Gilbert Chen
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Abner Chang <abner.chang@hpe.com>
Cc: Daniel Schaefer <daniel.schaefer@hpe.com>
Cc: Gilbert Chen <gilbert.chen@hpe.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
Reviewed-by: Abner Chang <abner.chang@hpe.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc | 3 +++
.../U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc | 3 +++
2 files changed, 6 insertions(+)
diff --git a/Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc b/Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc
index 2d652d215f..b91823ceeb 100644
--- a/Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc
+++ b/Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc
@@ -58,10 +58,13 @@ [SkuIds]
################################################################################
#
# Library Class section - list of all Library Classes needed by this Platform.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
PcdLib|MdePkg/Library/BasePcdLibNull/BasePcdLibNull.inf
PrintLib|MdePkg/Library/BasePrintLib/BasePrintLib.inf
BaseMemoryLib|MdePkg/Library/BaseMemoryLib/BaseMemoryLib.inf
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
diff --git a/Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc b/Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc
index 9f60475189..0eafe29880 100644
--- a/Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc
+++ b/Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc
@@ -58,10 +58,13 @@ [SkuIds]
################################################################################
#
# Library Class section - list of all Library Classes needed by this Platform.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
PcdLib|MdePkg/Library/BasePcdLibNull/BasePcdLibNull.inf
PrintLib|MdePkg/Library/BasePrintLib/BasePrintLib.inf
BaseMemoryLib|MdePkg/Library/BaseMemoryLib/BaseMemoryLib.inf
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* Re: [edk2-platforms] [patch V3 21/35] Platform/SiFive: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:39 ` [edk2-platforms] [patch V3 21/35] Platform/SiFive: " Dandan Bi
@ 2021-03-29 1:58 ` Abner Chang
0 siblings, 0 replies; 50+ messages in thread
From: Abner Chang @ 2021-03-29 1:58 UTC (permalink / raw)
To: Dandan Bi, devel@edk2.groups.io; +Cc: Schaefer, Daniel, Chen, Gilbert
Reviewed-by: Abner Chang <abner.chang@hpe.com>
> -----Original Message-----
> From: Dandan Bi [mailto:dandan.bi@intel.com]
> Sent: Friday, March 26, 2021 4:39 PM
> To: devel@edk2.groups.io
> Cc: Chang, Abner (HPS SW/FW Technologist) <abner.chang@hpe.com>;
> Schaefer, Daniel <daniel.schaefer@hpe.com>; Chen, Gilbert
> <gilbert.chen@hpe.com>
> Subject: [edk2-platforms] [patch V3 21/35] Platform/SiFive: Consume
> MdeLibs.dsc.inc for RegisterFilterLib
>
> REF: INVALID URI REMOVED
> 3A__bugzilla.tianocore.org_show-5Fbug.cgi-3Fid-
> 3D3246&d=DwIBAg&c=C5b8zRQO1miGmBeVZ2LFWg&r=_SN6FZBN4Vgi4Ulks
> kz6qU3NYRO03nHp9P7Z5q59A3E&m=1gS4Zk0pxQK3vF4oqFkAoL9G7NrZVlIgj
> b4OcSPatDw&s=H54pTOE25ZxggxmX9Xs1-qH5EAngZoRCVWt6iT_h2Yk&e=
>
> MdeLibs.dsc.inc was added for some basic/default library
> instances provided by MdePkg and RegisterFilterLibNull Library
> was also added into it as the first version of MdeLibs.dsc.inc.
>
> So update platform dsc to consume MdeLibs.dsc.inc for
> RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
>
> Cc: Abner Chang <abner.chang@hpe.com>
> Cc: Daniel Schaefer <daniel.schaefer@hpe.com>
> Cc: Gilbert Chen <gilbert.chen@hpe.com>
> Signed-off-by: Dandan Bi <dandan.bi@intel.com>
> Reviewed-by: Abner Chang <abner.chang@hpe.com>
> ---
> V3:
> Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
> [LibraryClasses] section since we can specify different [LibraryClasses]
> section for libary instances in MdeLibs.dsc.inc.
> ---
> Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc | 3 +++
> .../U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc | 3 +++
> 2 files changed, 6 insertions(+)
>
> diff --git a/Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc
> b/Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc
> index 2d652d215f..b91823ceeb 100644
> --- a/Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc
> +++ b/Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc
> @@ -58,10 +58,13 @@ [SkuIds]
>
> ##########################################################
> ######################
> #
> # Library Class section - list of all Library Classes needed by this Platform.
> #
>
> ##########################################################
> ######################
> +
> +!include MdePkg/MdeLibs.dsc.inc
> +
> [LibraryClasses]
> PcdLib|MdePkg/Library/BasePcdLibNull/BasePcdLibNull.inf
> PrintLib|MdePkg/Library/BasePrintLib/BasePrintLib.inf
> BaseMemoryLib|MdePkg/Library/BaseMemoryLib/BaseMemoryLib.inf
> BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
> diff --git
> a/Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.d
> sc
> b/Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.
> dsc
> index 9f60475189..0eafe29880 100644
> ---
> a/Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.d
> sc
> +++
> b/Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.
> dsc
> @@ -58,10 +58,13 @@ [SkuIds]
>
> ##########################################################
> ######################
> #
> # Library Class section - list of all Library Classes needed by this Platform.
> #
>
> ##########################################################
> ######################
> +
> +!include MdePkg/MdeLibs.dsc.inc
> +
> [LibraryClasses]
> PcdLib|MdePkg/Library/BasePcdLibNull/BasePcdLibNull.inf
> PrintLib|MdePkg/Library/BasePrintLib/BasePrintLib.inf
> BaseMemoryLib|MdePkg/Library/BaseMemoryLib/BaseMemoryLib.inf
> BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
> --
> 2.18.0.windows.1
^ permalink raw reply [flat|nested] 50+ messages in thread
* [edk2-platforms] [patch V3 22/35] Platform/Socionext: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (20 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 21/35] Platform/SiFive: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 16:47 ` [edk2-devel] " Michael D Kinney
2021-03-26 8:39 ` [edk2-platforms] [patch V3 23/35] Platform/SoftIron: " Dandan Bi
` (12 subsequent siblings)
34 siblings, 1 reply; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Ard Biesheuvel, Leif Lindholm
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Leif Lindholm <leif@nuviainc.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc | 2 ++
Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc | 2 ++
2 files changed, 4 insertions(+)
diff --git a/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc b/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc
index 0a364bc457..dfd562f53b 100644
--- a/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc
+++ b/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc
@@ -21,10 +21,12 @@ [BuildOptions.common.EDKII.DXE_CORE,BuildOptions.common.EDKII.DXE_DRIVER,BuildOp
[BuildOptions.common.EDKII.DXE_RUNTIME_DRIVER]
GCC:*_*_ARM_DLINK_FLAGS = -z common-page-size=0x1000
GCC:*_*_AARCH64_DLINK_FLAGS = -z common-page-size=0x10000
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
ArmPlatformLib|Silicon/Socionext/SynQuacer/Library/SynQuacerLib/SynQuacerLib.inf
ArmHvcLib|ArmPkg/Library/ArmHvcLib/ArmHvcLib.inf
ArmSmcLib|ArmPkg/Library/ArmSmcLib/ArmSmcLib.inf
diff --git a/Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc b/Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc
index 18accbfd5d..e531589762 100644
--- a/Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc
+++ b/Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc
@@ -37,10 +37,12 @@ [BuildOptions.common.EDKII.DXE_CORE,BuildOptions.common.EDKII.DXE_DRIVER,BuildOp
[BuildOptions.common.EDKII.DXE_RUNTIME_DRIVER]
GCC:*_*_ARM_DLINK_FLAGS = -z common-page-size=0x1000
GCC:*_*_AARCH64_DLINK_FLAGS = -z common-page-size=0x10000
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
ArmPlatformLib|Silicon/Socionext/SynQuacer/Library/SynQuacerLib/SynQuacerLib.inf
ArmHvcLib|ArmPkg/Library/ArmHvcLib/ArmHvcLib.inf
ArmSmcLib|ArmPkg/Library/ArmSmcLib/ArmSmcLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* Re: [edk2-devel] [edk2-platforms] [patch V3 22/35] Platform/Socionext: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:39 ` [edk2-platforms] [patch V3 22/35] Platform/Socionext: " Dandan Bi
@ 2021-03-26 16:47 ` Michael D Kinney
0 siblings, 0 replies; 50+ messages in thread
From: Michael D Kinney @ 2021-03-26 16:47 UTC (permalink / raw)
To: devel@edk2.groups.io, Bi, Dandan, Kinney, Michael D
Cc: Ard Biesheuvel, Leif Lindholm
Hi Dandan,
One more patch I think can be dropped to remove dsc.inc including dsc.inc
from another package.
Mike
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Dandan Bi
> Sent: Friday, March 26, 2021 1:39 AM
> To: devel@edk2.groups.io
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>; Leif Lindholm <leif@nuviainc.com>
> Subject: [edk2-devel] [edk2-platforms] [patch V3 22/35] Platform/Socionext: Consume MdeLibs.dsc.inc for RegisterFilterLib
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
>
> MdeLibs.dsc.inc was added for some basic/default library
> instances provided by MdePkg and RegisterFilterLibNull Library
> was also added into it as the first version of MdeLibs.dsc.inc.
>
> So update platform dsc to consume MdeLibs.dsc.inc for
> RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
>
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Signed-off-by: Dandan Bi <dandan.bi@intel.com>
> ---
> V3:
> Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
> [LibraryClasses] section since we can specify different [LibraryClasses]
> section for libary instances in MdeLibs.dsc.inc.
> ---
> Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc | 2 ++
> Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc | 2 ++
> 2 files changed, 4 insertions(+)
>
> diff --git a/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc b/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc
> index 0a364bc457..dfd562f53b 100644
> --- a/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc
> +++ b/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc
> @@ -21,10 +21,12 @@ [BuildOptions.common.EDKII.DXE_CORE,BuildOptions.common.EDKII.DXE_DRIVER,BuildOp
>
> [BuildOptions.common.EDKII.DXE_RUNTIME_DRIVER]
> GCC:*_*_ARM_DLINK_FLAGS = -z common-page-size=0x1000
> GCC:*_*_AARCH64_DLINK_FLAGS = -z common-page-size=0x10000
>
> +!include MdePkg/MdeLibs.dsc.inc
> +
> [LibraryClasses]
> ArmPlatformLib|Silicon/Socionext/SynQuacer/Library/SynQuacerLib/SynQuacerLib.inf
> ArmHvcLib|ArmPkg/Library/ArmHvcLib/ArmHvcLib.inf
> ArmSmcLib|ArmPkg/Library/ArmSmcLib/ArmSmcLib.inf
>
> diff --git a/Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc
> b/Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc
> index 18accbfd5d..e531589762 100644
> --- a/Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc
> +++ b/Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc
> @@ -37,10 +37,12 @@ [BuildOptions.common.EDKII.DXE_CORE,BuildOptions.common.EDKII.DXE_DRIVER,BuildOp
>
> [BuildOptions.common.EDKII.DXE_RUNTIME_DRIVER]
> GCC:*_*_ARM_DLINK_FLAGS = -z common-page-size=0x1000
> GCC:*_*_AARCH64_DLINK_FLAGS = -z common-page-size=0x10000
>
> +!include MdePkg/MdeLibs.dsc.inc
> +
> [LibraryClasses.common]
> ArmPlatformLib|Silicon/Socionext/SynQuacer/Library/SynQuacerLib/SynQuacerLib.inf
> ArmHvcLib|ArmPkg/Library/ArmHvcLib/ArmHvcLib.inf
> ArmSmcLib|ArmPkg/Library/ArmSmcLib/ArmSmcLib.inf
>
> --
> 2.18.0.windows.1
>
>
>
>
>
^ permalink raw reply [flat|nested] 50+ messages in thread
* [edk2-platforms] [patch V3 23/35] Platform/SoftIron: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (21 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 22/35] Platform/Socionext: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 8:39 ` [edk2-platforms] [patch V3 24/35] Silicon/Hisilicon: " Dandan Bi
` (11 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Ard Biesheuvel, Leif Lindholm
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Leif Lindholm <leif@nuviainc.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc b/Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc
index dd45b4142c..38620b3e4d 100644
--- a/Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc
+++ b/Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc
@@ -34,10 +34,13 @@ [Defines]
################################################################################
#
# Library Class section - list of all Library Classes needed by this Platform.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
!if $(TARGET) == RELEASE
DebugLib|MdePkg/Library/BaseDebugLibNull/BaseDebugLibNull.inf
!else
DebugLib|MdePkg/Library/BaseDebugLibSerialPort/BaseDebugLibSerialPort.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 24/35] Silicon/Hisilicon: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (22 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 23/35] Platform/SoftIron: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 16:43 ` [edk2-devel] " Michael D Kinney
2021-03-26 8:39 ` [edk2-platforms] [patch V3 25/35] Silicon/CoffeelakeSiliconPkg: " Dandan Bi
` (10 subsequent siblings)
34 siblings, 1 reply; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Leif Lindholm, Ard Biesheuvel, Wenyi Xie
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Wenyi Xie <xiewenyi2@huawei.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Silicon/Hisilicon/Hisilicon.dsc.inc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Silicon/Hisilicon/Hisilicon.dsc.inc b/Silicon/Hisilicon/Hisilicon.dsc.inc
index 6bacd9cafd..a0db833bef 100644
--- a/Silicon/Hisilicon/Hisilicon.dsc.inc
+++ b/Silicon/Hisilicon/Hisilicon.dsc.inc
@@ -5,10 +5,12 @@
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
#
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
!if $(TARGET) == RELEASE
DebugLib|MdePkg/Library/BaseDebugLibSerialPort/BaseDebugLibSerialPort.inf
!else
DebugLib|MdePkg/Library/BaseDebugLibSerialPort/BaseDebugLibSerialPort.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* Re: [edk2-devel] [edk2-platforms] [patch V3 24/35] Silicon/Hisilicon: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:39 ` [edk2-platforms] [patch V3 24/35] Silicon/Hisilicon: " Dandan Bi
@ 2021-03-26 16:43 ` Michael D Kinney
0 siblings, 0 replies; 50+ messages in thread
From: Michael D Kinney @ 2021-03-26 16:43 UTC (permalink / raw)
To: devel@edk2.groups.io, Bi, Dandan, Kinney, Michael D
Cc: Leif Lindholm, Ard Biesheuvel, Wenyi Xie
Hi Dandan,
Here is another example of adding an include of a dsc.inc from another dsc.inc.
I do not think we should do this.
The NetworkPkg and UnitTestFranmeworkPkg have examples where it is ok
for one dsc.inc to include another dsc.inc, and that is if the included
dsc.inc is in the same package.
I would like to refine the suggestion to say that the dsc.inc from one
package should not include a dsc.inc from a different package.
Best regards,
Mike
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Dandan Bi
> Sent: Friday, March 26, 2021 1:39 AM
> To: devel@edk2.groups.io
> Cc: Leif Lindholm <leif@nuviainc.com>; Ard Biesheuvel <ardb+tianocore@kernel.org>; Wenyi Xie <xiewenyi2@huawei.com>
> Subject: [edk2-devel] [edk2-platforms] [patch V3 24/35] Silicon/Hisilicon: Consume MdeLibs.dsc.inc for RegisterFilterLib
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
>
> MdeLibs.dsc.inc was added for some basic/default library
> instances provided by MdePkg and RegisterFilterLibNull Library
> was also added into it as the first version of MdeLibs.dsc.inc.
>
> So update platform dsc to consume MdeLibs.dsc.inc for
> RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> Cc: Wenyi Xie <xiewenyi2@huawei.com>
> Signed-off-by: Dandan Bi <dandan.bi@intel.com>
> ---
> V3:
> Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
> [LibraryClasses] section since we can specify different [LibraryClasses]
> section for libary instances in MdeLibs.dsc.inc.
> ---
> Silicon/Hisilicon/Hisilicon.dsc.inc | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/Silicon/Hisilicon/Hisilicon.dsc.inc b/Silicon/Hisilicon/Hisilicon.dsc.inc
> index 6bacd9cafd..a0db833bef 100644
> --- a/Silicon/Hisilicon/Hisilicon.dsc.inc
> +++ b/Silicon/Hisilicon/Hisilicon.dsc.inc
> @@ -5,10 +5,12 @@
> #
> # SPDX-License-Identifier: BSD-2-Clause-Patent
> #
> #
>
> +!include MdePkg/MdeLibs.dsc.inc
> +
> [LibraryClasses.common]
> !if $(TARGET) == RELEASE
> DebugLib|MdePkg/Library/BaseDebugLibSerialPort/BaseDebugLibSerialPort.inf
> !else
> DebugLib|MdePkg/Library/BaseDebugLibSerialPort/BaseDebugLibSerialPort.inf
> --
> 2.18.0.windows.1
>
>
>
>
>
^ permalink raw reply [flat|nested] 50+ messages in thread
* [edk2-platforms] [patch V3 25/35] Silicon/CoffeelakeSiliconPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (23 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 24/35] Silicon/Hisilicon: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 22:15 ` Chaganty, Rangasai V
2021-03-26 8:39 ` [edk2-platforms] [patch V3 26/35] Silicon/IntelSiliconPkg: " Dandan Bi
` (9 subsequent siblings)
34 siblings, 1 reply; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Chasel Chiu, Sai Chaganty
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Silicon/Intel/CoffeelakeSiliconPkg/CoffeelakeSiliconPkg.dsc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Silicon/Intel/CoffeelakeSiliconPkg/CoffeelakeSiliconPkg.dsc b/Silicon/Intel/CoffeelakeSiliconPkg/CoffeelakeSiliconPkg.dsc
index 2f25bdb35e..c5f10492ed 100644
--- a/Silicon/Intel/CoffeelakeSiliconPkg/CoffeelakeSiliconPkg.dsc
+++ b/Silicon/Intel/CoffeelakeSiliconPkg/CoffeelakeSiliconPkg.dsc
@@ -77,10 +77,12 @@ [Defines]
#
# Definition for Build Flag
#
!include $(PLATFORM_SI_PACKAGE)/SiPkgBuildOption.dsc
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
#
# Entry point
#
PeiCoreEntryPoint|MdePkg/Library/PeiCoreEntryPoint/PeiCoreEntryPoint.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* Re: [edk2-platforms] [patch V3 25/35] Silicon/CoffeelakeSiliconPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:39 ` [edk2-platforms] [patch V3 25/35] Silicon/CoffeelakeSiliconPkg: " Dandan Bi
@ 2021-03-26 22:15 ` Chaganty, Rangasai V
0 siblings, 0 replies; 50+ messages in thread
From: Chaganty, Rangasai V @ 2021-03-26 22:15 UTC (permalink / raw)
To: Bi, Dandan, devel@edk2.groups.io; +Cc: Chiu, Chasel
Reviewed-by: Sai Chaganty <rangasai.v.chaganty@intel.com>
-----Original Message-----
From: Bi, Dandan <dandan.bi@intel.com>
Sent: Friday, March 26, 2021 1:39 AM
To: devel@edk2.groups.io
Cc: Chiu, Chasel <chasel.chiu@intel.com>; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>
Subject: [edk2-platforms] [patch V3 25/35] Silicon/CoffeelakeSiliconPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library instances provided by MdePkg and RegisterFilterLibNull Library was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common [LibraryClasses] section since we can specify different [LibraryClasses] section for libary instances in MdeLibs.dsc.inc.
---
Silicon/Intel/CoffeelakeSiliconPkg/CoffeelakeSiliconPkg.dsc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Silicon/Intel/CoffeelakeSiliconPkg/CoffeelakeSiliconPkg.dsc b/Silicon/Intel/CoffeelakeSiliconPkg/CoffeelakeSiliconPkg.dsc
index 2f25bdb35e..c5f10492ed 100644
--- a/Silicon/Intel/CoffeelakeSiliconPkg/CoffeelakeSiliconPkg.dsc
+++ b/Silicon/Intel/CoffeelakeSiliconPkg/CoffeelakeSiliconPkg.dsc
@@ -77,10 +77,12 @@ [Defines]
#
# Definition for Build Flag
#
!include $(PLATFORM_SI_PACKAGE)/SiPkgBuildOption.dsc
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
#
# Entry point
#
PeiCoreEntryPoint|MdePkg/Library/PeiCoreEntryPoint/PeiCoreEntryPoint.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread
* [edk2-platforms] [patch V3 26/35] Silicon/IntelSiliconPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (24 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 25/35] Silicon/CoffeelakeSiliconPkg: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 22:12 ` Chaganty, Rangasai V
2021-03-26 8:39 ` [edk2-platforms] [patch V3 27/35] Silicon/KabylakeSiliconPkg: " Dandan Bi
` (8 subsequent siblings)
34 siblings, 1 reply; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Ray Ni, Rangasai V Chaganty
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Ray Ni <ray.ni@intel.com>
Cc: Rangasai V Chaganty <rangasai.v.chaganty@intel.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dsc | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dsc b/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dsc
index 6dff68f681..edc79c9b9c 100644
--- a/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dsc
+++ b/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dsc
@@ -1,9 +1,9 @@
## @file
# This package provides common open source Intel silicon modules.
#
-# Copyright (c) 2017 - 2020, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2017 - 2021, Intel Corporation. All rights reserved.<BR>
# Copyright (c) Microsoft Corporation.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -16,10 +16,12 @@ [Defines]
OUTPUT_DIRECTORY = Build/IntelSiliconPkg
SUPPORTED_ARCHITECTURES = IA32|X64
BUILD_TARGETS = DEBUG|RELEASE|NOOPT
SKUID_IDENTIFIER = DEFAULT
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
BaseMemoryLib|MdePkg/Library/BaseMemoryLib/BaseMemoryLib.inf
PrintLib|MdePkg/Library/BasePrintLib/BasePrintLib.inf
DebugLib|MdePkg/Library/BaseDebugLibNull/BaseDebugLibNull.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* Re: [edk2-platforms] [patch V3 26/35] Silicon/IntelSiliconPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:39 ` [edk2-platforms] [patch V3 26/35] Silicon/IntelSiliconPkg: " Dandan Bi
@ 2021-03-26 22:12 ` Chaganty, Rangasai V
0 siblings, 0 replies; 50+ messages in thread
From: Chaganty, Rangasai V @ 2021-03-26 22:12 UTC (permalink / raw)
To: Bi, Dandan, devel@edk2.groups.io; +Cc: Ni, Ray
Reviewed-by: Sai Chaganty <rangasai.v.chaganty@intel.com>
-----Original Message-----
From: Bi, Dandan <dandan.bi@intel.com>
Sent: Friday, March 26, 2021 1:39 AM
To: devel@edk2.groups.io
Cc: Ni, Ray <ray.ni@intel.com>; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>
Subject: [edk2-platforms] [patch V3 26/35] Silicon/IntelSiliconPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library instances provided by MdePkg and RegisterFilterLibNull Library was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Ray Ni <ray.ni@intel.com>
Cc: Rangasai V Chaganty <rangasai.v.chaganty@intel.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common [LibraryClasses] section since we can specify different [LibraryClasses] section for libary instances in MdeLibs.dsc.inc.
---
Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dsc | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dsc b/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dsc
index 6dff68f681..edc79c9b9c 100644
--- a/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dsc
+++ b/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dsc
@@ -1,9 +1,9 @@
## @file
# This package provides common open source Intel silicon modules.
#
-# Copyright (c) 2017 - 2020, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2017 - 2021, Intel Corporation. All rights
+reserved.<BR>
# Copyright (c) Microsoft Corporation.<BR> #
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -16,10 +16,12 @@ [Defines]
OUTPUT_DIRECTORY = Build/IntelSiliconPkg
SUPPORTED_ARCHITECTURES = IA32|X64
BUILD_TARGETS = DEBUG|RELEASE|NOOPT
SKUID_IDENTIFIER = DEFAULT
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
BaseMemoryLib|MdePkg/Library/BaseMemoryLib/BaseMemoryLib.inf
PrintLib|MdePkg/Library/BasePrintLib/BasePrintLib.inf
DebugLib|MdePkg/Library/BaseDebugLibNull/BaseDebugLibNull.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread
* [edk2-platforms] [patch V3 27/35] Silicon/KabylakeSiliconPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (25 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 26/35] Silicon/IntelSiliconPkg: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 22:11 ` Chaganty, Rangasai V
2021-03-26 8:39 ` [edk2-platforms] [patch V3 28/35] Silicon/QuarkSocPkg: " Dandan Bi
` (7 subsequent siblings)
34 siblings, 1 reply; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Chasel Chiu, Sai Chaganty
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Silicon/Intel/KabylakeSiliconPkg/KabylakeSiliconPkg.dsc | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/Silicon/Intel/KabylakeSiliconPkg/KabylakeSiliconPkg.dsc b/Silicon/Intel/KabylakeSiliconPkg/KabylakeSiliconPkg.dsc
index aa481d0307..d980a2c536 100644
--- a/Silicon/Intel/KabylakeSiliconPkg/KabylakeSiliconPkg.dsc
+++ b/Silicon/Intel/KabylakeSiliconPkg/KabylakeSiliconPkg.dsc
@@ -1,9 +1,9 @@
## @file
# Component description file for the SkyLake SiPkg DSC file.
#
-# Copyright (c) 2017 - 2019, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2017 - 2021, Intel Corporation. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -68,10 +68,12 @@ [Defines]
#
# Definition for Build Flag
#
!include $(PLATFORM_SI_PACKAGE)/SiPkgBuildOption.dsc
+ !include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
#
# Entry point
#
PeiCoreEntryPoint|MdePkg/Library/PeiCoreEntryPoint/PeiCoreEntryPoint.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* Re: [edk2-platforms] [patch V3 27/35] Silicon/KabylakeSiliconPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:39 ` [edk2-platforms] [patch V3 27/35] Silicon/KabylakeSiliconPkg: " Dandan Bi
@ 2021-03-26 22:11 ` Chaganty, Rangasai V
0 siblings, 0 replies; 50+ messages in thread
From: Chaganty, Rangasai V @ 2021-03-26 22:11 UTC (permalink / raw)
To: Bi, Dandan, devel@edk2.groups.io; +Cc: Chiu, Chasel
Reviewed-by: Sai Chaganty <rangasai.v.chaganty@intel.com>
-----Original Message-----
From: Bi, Dandan <dandan.bi@intel.com>
Sent: Friday, March 26, 2021 1:39 AM
To: devel@edk2.groups.io
Cc: Chiu, Chasel <chasel.chiu@intel.com>; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>
Subject: [edk2-platforms] [patch V3 27/35] Silicon/KabylakeSiliconPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library instances provided by MdePkg and RegisterFilterLibNull Library was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common [LibraryClasses] section since we can specify different [LibraryClasses] section for libary instances in MdeLibs.dsc.inc.
---
Silicon/Intel/KabylakeSiliconPkg/KabylakeSiliconPkg.dsc | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/Silicon/Intel/KabylakeSiliconPkg/KabylakeSiliconPkg.dsc b/Silicon/Intel/KabylakeSiliconPkg/KabylakeSiliconPkg.dsc
index aa481d0307..d980a2c536 100644
--- a/Silicon/Intel/KabylakeSiliconPkg/KabylakeSiliconPkg.dsc
+++ b/Silicon/Intel/KabylakeSiliconPkg/KabylakeSiliconPkg.dsc
@@ -1,9 +1,9 @@
## @file
# Component description file for the SkyLake SiPkg DSC file.
#
-# Copyright (c) 2017 - 2019, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2017 - 2021, Intel Corporation. All rights
+reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent # ##
@@ -68,10 +68,12 @@ [Defines]
#
# Definition for Build Flag
#
!include $(PLATFORM_SI_PACKAGE)/SiPkgBuildOption.dsc
+ !include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
#
# Entry point
#
PeiCoreEntryPoint|MdePkg/Library/PeiCoreEntryPoint/PeiCoreEntryPoint.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread
* [edk2-platforms] [patch V3 28/35] Silicon/QuarkSocPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (26 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 27/35] Silicon/KabylakeSiliconPkg: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 8:39 ` [edk2-platforms] [patch V3 29/35] Silicon/TigerlakeSiliconPkg: " Dandan Bi
` (6 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Michael D Kinney, Kelly Steele
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Kelly Steele <kelly.steele@intel.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Silicon/Intel/QuarkSocPkg/QuarkSocPkg.dsc | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/Silicon/Intel/QuarkSocPkg/QuarkSocPkg.dsc b/Silicon/Intel/QuarkSocPkg/QuarkSocPkg.dsc
index e743a5e272..80e37e26a9 100644
--- a/Silicon/Intel/QuarkSocPkg/QuarkSocPkg.dsc
+++ b/Silicon/Intel/QuarkSocPkg/QuarkSocPkg.dsc
@@ -2,11 +2,11 @@
# INTEL Quark SoC Module Package Reference Implementations
#
# This DSC file is used for Package Level build.
#
# This Module provides FRAMEWORK reference implementation for INTEL Quark SoC.
-# Copyright (c) 2013-2016 Intel Corporation.
+# Copyright (c) 2013-2021 Intel Corporation.
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
##
@@ -37,10 +37,13 @@ [SkuIds]
################################################################################
#
# Library Class section - list of all Library Classes needed by this Platform.
#
################################################################################
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
#
# Entry point
#
PeimEntryPoint|MdePkg/Library/PeimEntryPoint/PeimEntryPoint.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 29/35] Silicon/TigerlakeSiliconPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (27 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 28/35] Silicon/QuarkSocPkg: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 16:22 ` Chaganty, Rangasai V
2021-03-26 22:10 ` Chaganty, Rangasai V
2021-03-26 8:39 ` [edk2-platforms] [patch V3 30/35] Silicon/Marvell: " Dandan Bi
` (5 subsequent siblings)
34 siblings, 2 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Sai Chaganty, Nate DeSimone, Heng Luo
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Heng Luo <heng.luo@intel.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc b/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc
index 73a2594887..36df41f09f 100644
--- a/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc
+++ b/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc
@@ -90,10 +90,12 @@ [Defines]
#
# Definition for Build Flag
#
!include $(PLATFORM_SI_PACKAGE)/SiPkgBuildOption.dsc
+ !include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
#
# Entry point
#
PeiCoreEntryPoint|MdePkg/Library/PeiCoreEntryPoint/PeiCoreEntryPoint.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* Re: [edk2-platforms] [patch V3 29/35] Silicon/TigerlakeSiliconPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:39 ` [edk2-platforms] [patch V3 29/35] Silicon/TigerlakeSiliconPkg: " Dandan Bi
@ 2021-03-26 16:22 ` Chaganty, Rangasai V
2021-03-26 18:07 ` [edk2-devel] " Michael D Kinney
2021-03-26 22:10 ` Chaganty, Rangasai V
1 sibling, 1 reply; 50+ messages in thread
From: Chaganty, Rangasai V @ 2021-03-26 16:22 UTC (permalink / raw)
To: Bi, Dandan, devel@edk2.groups.io; +Cc: Desimone, Nathaniel L, Luo, Heng
Since this .dsc include is building some basic library instances from MdePkg, please move the include under [LibraryClasses.common] section, preferably under the #Basic category.
Thanks,
Sai
-----Original Message-----
From: Bi, Dandan <dandan.bi@intel.com>
Sent: Friday, March 26, 2021 1:39 AM
To: devel@edk2.groups.io
Cc: Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>; Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Luo, Heng <heng.luo@intel.com>
Subject: [edk2-platforms] [patch V3 29/35] Silicon/TigerlakeSiliconPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library instances provided by MdePkg and RegisterFilterLibNull Library was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Heng Luo <heng.luo@intel.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common [LibraryClasses] section since we can specify different [LibraryClasses] section for libary instances in MdeLibs.dsc.inc.
---
Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc b/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc
index 73a2594887..36df41f09f 100644
--- a/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc
+++ b/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc
@@ -90,10 +90,12 @@ [Defines]
#
# Definition for Build Flag
#
!include $(PLATFORM_SI_PACKAGE)/SiPkgBuildOption.dsc
+ !include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
#
# Entry point
#
PeiCoreEntryPoint|MdePkg/Library/PeiCoreEntryPoint/PeiCoreEntryPoint.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread
* Re: [edk2-devel] [edk2-platforms] [patch V3 29/35] Silicon/TigerlakeSiliconPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 16:22 ` Chaganty, Rangasai V
@ 2021-03-26 18:07 ` Michael D Kinney
2021-03-26 22:06 ` Chaganty, Rangasai V
0 siblings, 1 reply; 50+ messages in thread
From: Michael D Kinney @ 2021-03-26 18:07 UTC (permalink / raw)
To: devel@edk2.groups.io, Chaganty, Rangasai V, Kinney, Michael D
Hi Sai,
The latest patch series adds [LibraryClasses] statement to the MdeLibs.dsc.inc
on purpose so MdeLibs.dsc.inc can be extended to support multiple library classes
sections for different CPU archs and different modules types as needed in the
future.
Putting the !include outside the [LibraryClasses.common] section for Tigerlake is
the correct way to consume MdeLibs.dsc.inc.
Thanks,
Mike
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Chaganty, Rangasai V
> Sent: Friday, March 26, 2021 9:22 AM
> To: Bi, Dandan <dandan.bi@intel.com>; devel@edk2.groups.io
> Cc: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Luo, Heng <heng.luo@intel.com>
> Subject: Re: [edk2-devel] [edk2-platforms] [patch V3 29/35] Silicon/TigerlakeSiliconPkg: Consume MdeLibs.dsc.inc for
> RegisterFilterLib
>
> Since this .dsc include is building some basic library instances from MdePkg, please move the include under
> [LibraryClasses.common] section, preferably under the #Basic category.
>
> Thanks,
> Sai
>
> -----Original Message-----
> From: Bi, Dandan <dandan.bi@intel.com>
> Sent: Friday, March 26, 2021 1:39 AM
> To: devel@edk2.groups.io
> Cc: Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>; Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Luo,
> Heng <heng.luo@intel.com>
> Subject: [edk2-platforms] [patch V3 29/35] Silicon/TigerlakeSiliconPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
>
> MdeLibs.dsc.inc was added for some basic/default library instances provided by MdePkg and RegisterFilterLibNull Library
> was also added into it as the first version of MdeLibs.dsc.inc.
>
> So update platform dsc to consume MdeLibs.dsc.inc for RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
>
> Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Heng Luo <heng.luo@intel.com>
> Signed-off-by: Dandan Bi <dandan.bi@intel.com>
> ---
> V3:
> Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common [LibraryClasses] section since we can specify
> different [LibraryClasses] section for libary instances in MdeLibs.dsc.inc.
> ---
> Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc
> b/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc
> index 73a2594887..36df41f09f 100644
> --- a/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc
> +++ b/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc
> @@ -90,10 +90,12 @@ [Defines]
> #
> # Definition for Build Flag
> #
> !include $(PLATFORM_SI_PACKAGE)/SiPkgBuildOption.dsc
>
> + !include MdePkg/MdeLibs.dsc.inc
> +
> [LibraryClasses.common]
> #
> # Entry point
> #
> PeiCoreEntryPoint|MdePkg/Library/PeiCoreEntryPoint/PeiCoreEntryPoint.inf
> --
> 2.18.0.windows.1
>
>
>
>
>
^ permalink raw reply [flat|nested] 50+ messages in thread
* Re: [edk2-devel] [edk2-platforms] [patch V3 29/35] Silicon/TigerlakeSiliconPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 18:07 ` [edk2-devel] " Michael D Kinney
@ 2021-03-26 22:06 ` Chaganty, Rangasai V
0 siblings, 0 replies; 50+ messages in thread
From: Chaganty, Rangasai V @ 2021-03-26 22:06 UTC (permalink / raw)
To: Kinney, Michael D, devel@edk2.groups.io
I see. That makes sense. Thanks for the clarification.
Regards,
Sai
-----Original Message-----
From: Kinney, Michael D <michael.d.kinney@intel.com>
Sent: Friday, March 26, 2021 11:07 AM
To: devel@edk2.groups.io; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>
Subject: RE: [edk2-devel] [edk2-platforms] [patch V3 29/35] Silicon/TigerlakeSiliconPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
Hi Sai,
The latest patch series adds [LibraryClasses] statement to the MdeLibs.dsc.inc on purpose so MdeLibs.dsc.inc can be extended to support multiple library classes sections for different CPU archs and different modules types as needed in the future.
Putting the !include outside the [LibraryClasses.common] section for Tigerlake is the correct way to consume MdeLibs.dsc.inc.
Thanks,
Mike
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> Chaganty, Rangasai V
> Sent: Friday, March 26, 2021 9:22 AM
> To: Bi, Dandan <dandan.bi@intel.com>; devel@edk2.groups.io
> Cc: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Luo, Heng
> <heng.luo@intel.com>
> Subject: Re: [edk2-devel] [edk2-platforms] [patch V3 29/35]
> Silicon/TigerlakeSiliconPkg: Consume MdeLibs.dsc.inc for
> RegisterFilterLib
>
> Since this .dsc include is building some basic library instances from
> MdePkg, please move the include under [LibraryClasses.common] section, preferably under the #Basic category.
>
> Thanks,
> Sai
>
> -----Original Message-----
> From: Bi, Dandan <dandan.bi@intel.com>
> Sent: Friday, March 26, 2021 1:39 AM
> To: devel@edk2.groups.io
> Cc: Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>; Desimone,
> Nathaniel L <nathaniel.l.desimone@intel.com>; Luo, Heng
> <heng.luo@intel.com>
> Subject: [edk2-platforms] [patch V3 29/35]
> Silicon/TigerlakeSiliconPkg: Consume MdeLibs.dsc.inc for
> RegisterFilterLib
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
>
> MdeLibs.dsc.inc was added for some basic/default library instances
> provided by MdePkg and RegisterFilterLibNull Library was also added into it as the first version of MdeLibs.dsc.inc.
>
> So update platform dsc to consume MdeLibs.dsc.inc for RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
>
> Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Heng Luo <heng.luo@intel.com>
> Signed-off-by: Dandan Bi <dandan.bi@intel.com>
> ---
> V3:
> Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
> [LibraryClasses] section since we can specify different [LibraryClasses] section for libary instances in MdeLibs.dsc.inc.
> ---
> Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc
> b/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc
> index 73a2594887..36df41f09f 100644
> --- a/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc
> +++ b/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc
> @@ -90,10 +90,12 @@ [Defines]
> #
> # Definition for Build Flag
> #
> !include $(PLATFORM_SI_PACKAGE)/SiPkgBuildOption.dsc
>
> + !include MdePkg/MdeLibs.dsc.inc
> +
> [LibraryClasses.common]
> #
> # Entry point
> #
>
> PeiCoreEntryPoint|MdePkg/Library/PeiCoreEntryPoint/PeiCoreEntryPoint.i
> nf
> --
> 2.18.0.windows.1
>
>
>
>
>
^ permalink raw reply [flat|nested] 50+ messages in thread
* Re: [edk2-platforms] [patch V3 29/35] Silicon/TigerlakeSiliconPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:39 ` [edk2-platforms] [patch V3 29/35] Silicon/TigerlakeSiliconPkg: " Dandan Bi
2021-03-26 16:22 ` Chaganty, Rangasai V
@ 2021-03-26 22:10 ` Chaganty, Rangasai V
1 sibling, 0 replies; 50+ messages in thread
From: Chaganty, Rangasai V @ 2021-03-26 22:10 UTC (permalink / raw)
To: Bi, Dandan, devel@edk2.groups.io; +Cc: Desimone, Nathaniel L, Luo, Heng
Reviewed-by: Sai Chaganty <rangasai.v.chaganty@intel.com>
-----Original Message-----
From: Bi, Dandan <dandan.bi@intel.com>
Sent: Friday, March 26, 2021 1:39 AM
To: devel@edk2.groups.io
Cc: Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>; Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Luo, Heng <heng.luo@intel.com>
Subject: [edk2-platforms] [patch V3 29/35] Silicon/TigerlakeSiliconPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library instances provided by MdePkg and RegisterFilterLibNull Library was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Heng Luo <heng.luo@intel.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common [LibraryClasses] section since we can specify different [LibraryClasses] section for libary instances in MdeLibs.dsc.inc.
---
Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc b/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc
index 73a2594887..36df41f09f 100644
--- a/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc
+++ b/Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc
@@ -90,10 +90,12 @@ [Defines]
#
# Definition for Build Flag
#
!include $(PLATFORM_SI_PACKAGE)/SiPkgBuildOption.dsc
+ !include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
#
# Entry point
#
PeiCoreEntryPoint|MdePkg/Library/PeiCoreEntryPoint/PeiCoreEntryPoint.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread
* [edk2-platforms] [patch V3 30/35] Silicon/Marvell: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (28 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 29/35] Silicon/TigerlakeSiliconPkg: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 16:46 ` [edk2-devel] " Michael D Kinney
2021-03-26 8:39 ` [edk2-platforms] [patch V3 31/35] Silicon/NXP: " Dandan Bi
` (4 subsequent siblings)
34 siblings, 1 reply; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Marcin Wojtas, Leif Lindholm
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Marcin Wojtas <mw@semihalf.com>
Cc: Leif Lindholm <leif@nuviainc.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc b/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
index 4cdafe8b1f..9802da98c4 100644
--- a/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
+++ b/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
@@ -2,10 +2,13 @@
#Copyright (c) 2011-2012, ARM Limited. All rights reserved.
#Copyright (C) 2016 Marvell International Ltd.
#
#SPDX-License-Identifier: BSD-2-Clause-Patent
#
+
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
ArmadaIcuLib|Silicon/Marvell/Library/IcuLib/IcuLib.inf
ArmadaSoCDescLib|Silicon/Marvell/Armada7k8k/Library/Armada7k8kSoCDescLib/Armada7k8kSoCDescLib.inf
ArmPlatformLib|Silicon/Marvell/Armada7k8k/Library/Armada7k8kLib/Armada7k8kLib.inf
ComPhyLib|Silicon/Marvell/Library/ComPhyLib/ComPhyLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* Re: [edk2-devel] [edk2-platforms] [patch V3 30/35] Silicon/Marvell: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:39 ` [edk2-platforms] [patch V3 30/35] Silicon/Marvell: " Dandan Bi
@ 2021-03-26 16:46 ` Michael D Kinney
0 siblings, 0 replies; 50+ messages in thread
From: Michael D Kinney @ 2021-03-26 16:46 UTC (permalink / raw)
To: devel@edk2.groups.io, Bi, Dandan, Kinney, Michael D
Cc: Marcin Wojtas, Leif Lindholm
Hi Dandan,
Another patch I think can be dropped to remove dsc.inc including dsc.inc
from another package.
Mike
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Dandan Bi
> Sent: Friday, March 26, 2021 1:39 AM
> To: devel@edk2.groups.io
> Cc: Marcin Wojtas <mw@semihalf.com>; Leif Lindholm <leif@nuviainc.com>
> Subject: [edk2-devel] [edk2-platforms] [patch V3 30/35] Silicon/Marvell: Consume MdeLibs.dsc.inc for RegisterFilterLib
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
>
> MdeLibs.dsc.inc was added for some basic/default library
> instances provided by MdePkg and RegisterFilterLibNull Library
> was also added into it as the first version of MdeLibs.dsc.inc.
>
> So update platform dsc to consume MdeLibs.dsc.inc for
> RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
>
> Cc: Marcin Wojtas <mw@semihalf.com>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Signed-off-by: Dandan Bi <dandan.bi@intel.com>
> ---
> V3:
> Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
> [LibraryClasses] section since we can specify different [LibraryClasses]
> section for libary instances in MdeLibs.dsc.inc.
> ---
> Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc b/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
> index 4cdafe8b1f..9802da98c4 100644
> --- a/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
> +++ b/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
> @@ -2,10 +2,13 @@
> #Copyright (c) 2011-2012, ARM Limited. All rights reserved.
> #Copyright (C) 2016 Marvell International Ltd.
> #
> #SPDX-License-Identifier: BSD-2-Clause-Patent
> #
> +
> +!include MdePkg/MdeLibs.dsc.inc
> +
> [LibraryClasses.common]
> ArmadaIcuLib|Silicon/Marvell/Library/IcuLib/IcuLib.inf
> ArmadaSoCDescLib|Silicon/Marvell/Armada7k8k/Library/Armada7k8kSoCDescLib/Armada7k8kSoCDescLib.inf
> ArmPlatformLib|Silicon/Marvell/Armada7k8k/Library/Armada7k8kLib/Armada7k8kLib.inf
> ComPhyLib|Silicon/Marvell/Library/ComPhyLib/ComPhyLib.inf
> --
> 2.18.0.windows.1
>
>
>
>
>
^ permalink raw reply [flat|nested] 50+ messages in thread
* [edk2-platforms] [patch V3 31/35] Silicon/NXP: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (29 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 30/35] Silicon/Marvell: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 16:44 ` [edk2-devel] " Michael D Kinney
2021-03-26 8:39 ` [edk2-platforms] [patch V3 32/35] Silicon/Openmoko: " Dandan Bi
` (3 subsequent siblings)
34 siblings, 1 reply; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Leif Lindholm, Meenakshi Aggarwal
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Meenakshi Aggarwal <meenakshi.aggarwal@nxp.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Silicon/NXP/NxpQoriqLs.dsc.inc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Silicon/NXP/NxpQoriqLs.dsc.inc b/Silicon/NXP/NxpQoriqLs.dsc.inc
index 99a317aba7..2535a70035 100644
--- a/Silicon/NXP/NxpQoriqLs.dsc.inc
+++ b/Silicon/NXP/NxpQoriqLs.dsc.inc
@@ -20,10 +20,12 @@ [Defines]
DSC_SPECIFICATION = 0x0001001A
SUPPORTED_ARCHITECTURES = AARCH64
BUILD_TARGETS = DEBUG|RELEASE|NOOPT
SKUID_IDENTIFIER = DEFAULT
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
ArmLib|ArmPkg/Library/ArmLib/ArmBaseLib.inf
ArmMmuLib|ArmPkg/Library/ArmMmuLib/ArmMmuBaseLib.inf
ArmSmcLib|ArmPkg/Library/ArmSmcLib/ArmSmcLib.inf
ArmGenericTimerCounterLib|ArmPkg/Library/ArmGenericTimerPhyCounterLib/ArmGenericTimerPhyCounterLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* Re: [edk2-devel] [edk2-platforms] [patch V3 31/35] Silicon/NXP: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:39 ` [edk2-platforms] [patch V3 31/35] Silicon/NXP: " Dandan Bi
@ 2021-03-26 16:44 ` Michael D Kinney
0 siblings, 0 replies; 50+ messages in thread
From: Michael D Kinney @ 2021-03-26 16:44 UTC (permalink / raw)
To: devel@edk2.groups.io, Bi, Dandan, Kinney, Michael D
Cc: Leif Lindholm, Meenakshi Aggarwal
Hi Dandan,
Another example of adsc.inc including a dsc.in from another package.
I recommend this patch be dropped.
Mike
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Dandan Bi
> Sent: Friday, March 26, 2021 1:39 AM
> To: devel@edk2.groups.io
> Cc: Leif Lindholm <leif@nuviainc.com>; Meenakshi Aggarwal <meenakshi.aggarwal@nxp.com>
> Subject: [edk2-devel] [edk2-platforms] [patch V3 31/35] Silicon/NXP: Consume MdeLibs.dsc.inc for RegisterFilterLib
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
>
> MdeLibs.dsc.inc was added for some basic/default library
> instances provided by MdePkg and RegisterFilterLibNull Library
> was also added into it as the first version of MdeLibs.dsc.inc.
>
> So update platform dsc to consume MdeLibs.dsc.inc for
> RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Meenakshi Aggarwal <meenakshi.aggarwal@nxp.com>
> Signed-off-by: Dandan Bi <dandan.bi@intel.com>
> ---
> V3:
> Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
> [LibraryClasses] section since we can specify different [LibraryClasses]
> section for libary instances in MdeLibs.dsc.inc.
> ---
> Silicon/NXP/NxpQoriqLs.dsc.inc | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/Silicon/NXP/NxpQoriqLs.dsc.inc b/Silicon/NXP/NxpQoriqLs.dsc.inc
> index 99a317aba7..2535a70035 100644
> --- a/Silicon/NXP/NxpQoriqLs.dsc.inc
> +++ b/Silicon/NXP/NxpQoriqLs.dsc.inc
> @@ -20,10 +20,12 @@ [Defines]
> DSC_SPECIFICATION = 0x0001001A
> SUPPORTED_ARCHITECTURES = AARCH64
> BUILD_TARGETS = DEBUG|RELEASE|NOOPT
> SKUID_IDENTIFIER = DEFAULT
>
> +!include MdePkg/MdeLibs.dsc.inc
> +
> [LibraryClasses.common]
> ArmLib|ArmPkg/Library/ArmLib/ArmBaseLib.inf
> ArmMmuLib|ArmPkg/Library/ArmMmuLib/ArmMmuBaseLib.inf
> ArmSmcLib|ArmPkg/Library/ArmSmcLib/ArmSmcLib.inf
> ArmGenericTimerCounterLib|ArmPkg/Library/ArmGenericTimerPhyCounterLib/ArmGenericTimerPhyCounterLib.inf
> --
> 2.18.0.windows.1
>
>
>
>
>
^ permalink raw reply [flat|nested] 50+ messages in thread
* [edk2-platforms] [patch V3 32/35] Silicon/Openmoko: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (30 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 31/35] Silicon/NXP: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 8:39 ` [edk2-platforms] [patch V3 33/35] Silicon/RISC_V: " Dandan Bi
` (2 subsequent siblings)
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Ard Biesheuvel, Leif Lindholm
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Leif Lindholm <leif@nuviainc.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Silicon/Openmoko/Openmoko.dsc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Silicon/Openmoko/Openmoko.dsc b/Silicon/Openmoko/Openmoko.dsc
index 572be3d730..c565d0425b 100644
--- a/Silicon/Openmoko/Openmoko.dsc
+++ b/Silicon/Openmoko/Openmoko.dsc
@@ -13,10 +13,12 @@ [Defines]
OUTPUT_DIRECTORY = Build/Openmoko
SUPPORTED_ARCHITECTURES = AARCH64|ARM|EBC|IA32|IA64|X64
BUILD_TARGETS = DEBUG|RELEASE
SKUID_IDENTIFIER = DEFAULT
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
BaseMemoryLib|MdePkg/Library/BaseMemoryLib/BaseMemoryLib.inf
DebugLib|MdePkg/Library/BaseDebugLibNull/BaseDebugLibNull.inf
DevicePathLib|MdePkg/Library/UefiDevicePathLib/UefiDevicePathLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 33/35] Silicon/RISC_V: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (31 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 32/35] Silicon/Openmoko: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-29 1:59 ` Abner Chang
2021-03-26 8:39 ` [edk2-platforms] [patch V3 34/35] Silicon/Synopsys/DesignWare: " Dandan Bi
2021-03-26 8:39 ` [edk2-platforms] [patch V3 35/35] Silicon/TexasInstruments: " Dandan Bi
34 siblings, 1 reply; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Abner Chang, Daniel Schaefer, Gilbert Chen
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Abner Chang <abner.chang@hpe.com>
Cc: Daniel Schaefer <daniel.schaefer@hpe.com>
Cc: Gilbert Chen <gilbert.chen@hpe.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
Reviewed-by: Abner Chang <abner.chang@hpe.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Silicon/RISC-V/ProcessorPkg/RiscVProcessorPkg.dsc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Silicon/RISC-V/ProcessorPkg/RiscVProcessorPkg.dsc b/Silicon/RISC-V/ProcessorPkg/RiscVProcessorPkg.dsc
index 5f88f5e89f..5c5cfcb525 100644
--- a/Silicon/RISC-V/ProcessorPkg/RiscVProcessorPkg.dsc
+++ b/Silicon/RISC-V/ProcessorPkg/RiscVProcessorPkg.dsc
@@ -34,10 +34,12 @@ [BuildOptions]
#
################################################################################
[SkuIds]
0|DEFAULT
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
CpuExceptionHandlerLib|Silicon/RISC-V/ProcessorPkg/Library/RiscVExceptionLib/CpuExceptionHandlerDxeLib.inf
RiscVCpuLib|Silicon/RISC-V/ProcessorPkg/Library/RiscVCpuLib/RiscVCpuLib.inf
RiscVEdk2SbiLib|Silicon/RISC-V/ProcessorPkg/Library/RiscVEdk2SbiLib/RiscVEdk2SbiLib.inf
RiscVOpensbiLib|Silicon/RISC-V/ProcessorPkg/Library/RiscVOpensbiLib/RiscVOpensbiLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* Re: [edk2-platforms] [patch V3 33/35] Silicon/RISC_V: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:39 ` [edk2-platforms] [patch V3 33/35] Silicon/RISC_V: " Dandan Bi
@ 2021-03-29 1:59 ` Abner Chang
0 siblings, 0 replies; 50+ messages in thread
From: Abner Chang @ 2021-03-29 1:59 UTC (permalink / raw)
To: Dandan Bi, devel@edk2.groups.io; +Cc: Schaefer, Daniel, Chen, Gilbert
Reviewed-by: Abner Chang <abner.chang@hpe.com>
> -----Original Message-----
> From: Dandan Bi [mailto:dandan.bi@intel.com]
> Sent: Friday, March 26, 2021 4:39 PM
> To: devel@edk2.groups.io
> Cc: Chang, Abner (HPS SW/FW Technologist) <abner.chang@hpe.com>;
> Schaefer, Daniel <daniel.schaefer@hpe.com>; Chen, Gilbert
> <gilbert.chen@hpe.com>
> Subject: [edk2-platforms] [patch V3 33/35] Silicon/RISC_V: Consume
> MdeLibs.dsc.inc for RegisterFilterLib
>
> REF: INVALID URI REMOVED
> 3A__bugzilla.tianocore.org_show-5Fbug.cgi-3Fid-
> 3D3246&d=DwIBAg&c=C5b8zRQO1miGmBeVZ2LFWg&r=_SN6FZBN4Vgi4Ulks
> kz6qU3NYRO03nHp9P7Z5q59A3E&m=aVLCntLfyhT68diUb9p_tdmrcU4qoFrXt
> bCXgxq89Yo&s=5FM6QXSoTLmhFsqNnotHztOlDDaCDTqlbFIwMzbvRFk&e=
>
> MdeLibs.dsc.inc was added for some basic/default library
> instances provided by MdePkg and RegisterFilterLibNull Library
> was also added into it as the first version of MdeLibs.dsc.inc.
>
> So update platform dsc to consume MdeLibs.dsc.inc for
> RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
>
> Cc: Abner Chang <abner.chang@hpe.com>
> Cc: Daniel Schaefer <daniel.schaefer@hpe.com>
> Cc: Gilbert Chen <gilbert.chen@hpe.com>
> Signed-off-by: Dandan Bi <dandan.bi@intel.com>
> Reviewed-by: Abner Chang <abner.chang@hpe.com>
> ---
> V3:
> Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
> [LibraryClasses] section since we can specify different [LibraryClasses]
> section for libary instances in MdeLibs.dsc.inc.
> ---
> Silicon/RISC-V/ProcessorPkg/RiscVProcessorPkg.dsc | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/Silicon/RISC-V/ProcessorPkg/RiscVProcessorPkg.dsc
> b/Silicon/RISC-V/ProcessorPkg/RiscVProcessorPkg.dsc
> index 5f88f5e89f..5c5cfcb525 100644
> --- a/Silicon/RISC-V/ProcessorPkg/RiscVProcessorPkg.dsc
> +++ b/Silicon/RISC-V/ProcessorPkg/RiscVProcessorPkg.dsc
> @@ -34,10 +34,12 @@ [BuildOptions]
> #
>
> ##########################################################
> ######################
> [SkuIds]
> 0|DEFAULT
>
> +!include MdePkg/MdeLibs.dsc.inc
> +
> [LibraryClasses.common]
> CpuExceptionHandlerLib|Silicon/RISC-
> V/ProcessorPkg/Library/RiscVExceptionLib/CpuExceptionHandlerDxeLib.inf
> RiscVCpuLib|Silicon/RISC-
> V/ProcessorPkg/Library/RiscVCpuLib/RiscVCpuLib.inf
> RiscVEdk2SbiLib|Silicon/RISC-
> V/ProcessorPkg/Library/RiscVEdk2SbiLib/RiscVEdk2SbiLib.inf
> RiscVOpensbiLib|Silicon/RISC-
> V/ProcessorPkg/Library/RiscVOpensbiLib/RiscVOpensbiLib.inf
> --
> 2.18.0.windows.1
^ permalink raw reply [flat|nested] 50+ messages in thread
* [edk2-platforms] [patch V3 34/35] Silicon/Synopsys/DesignWare: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (32 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 33/35] Silicon/RISC_V: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
2021-03-26 8:39 ` [edk2-platforms] [patch V3 35/35] Silicon/TexasInstruments: " Dandan Bi
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Ard Biesheuvel, Leif Lindholm
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Leif Lindholm <leif@nuviainc.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Silicon/Synopsys/DesignWare/DesignWare.dsc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Silicon/Synopsys/DesignWare/DesignWare.dsc b/Silicon/Synopsys/DesignWare/DesignWare.dsc
index 098bba3f7d..b5a7b38e14 100755
--- a/Silicon/Synopsys/DesignWare/DesignWare.dsc
+++ b/Silicon/Synopsys/DesignWare/DesignWare.dsc
@@ -13,10 +13,12 @@ [Defines]
OUTPUT_DIRECTORY = Build/Synopsys
SUPPORTED_ARCHITECTURES = AARCH64|ARM
BUILD_TARGETS = DEBUG|RELEASE|NOOPT
SKUID_IDENTIFIER = DEFAULT
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses]
ArmGenericTimerCounterLib|ArmPkg/Library/ArmGenericTimerPhyCounterLib/ArmGenericTimerPhyCounterLib.inf
ArmLib|ArmPkg/Library/ArmLib/ArmBaseLib.inf
BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
BaseMemoryLib|MdePkg/Library/BaseMemoryLib/BaseMemoryLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread* [edk2-platforms] [patch V3 35/35] Silicon/TexasInstruments: Consume MdeLibs.dsc.inc for RegisterFilterLib
2021-03-26 8:38 [edk2-platforms] [patch V3 00/35] Consume MdeLibs.dsc.inc for RegisterFilterLib Dandan Bi
` (33 preceding siblings ...)
2021-03-26 8:39 ` [edk2-platforms] [patch V3 34/35] Silicon/Synopsys/DesignWare: " Dandan Bi
@ 2021-03-26 8:39 ` Dandan Bi
34 siblings, 0 replies; 50+ messages in thread
From: Dandan Bi @ 2021-03-26 8:39 UTC (permalink / raw)
To: devel; +Cc: Ard Biesheuvel, Leif Lindholm
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
MdeLibs.dsc.inc was added for some basic/default library
instances provided by MdePkg and RegisterFilterLibNull Library
was also added into it as the first version of MdeLibs.dsc.inc.
So update platform dsc to consume MdeLibs.dsc.inc for
RegisterFilterLibNull which will be consumed by IoLib and BaseLib.
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Leif Lindholm <leif@nuviainc.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
V3:
Update plaftom dsc to consume the MdeLibs.dsc.inc outside the common
[LibraryClasses] section since we can specify different [LibraryClasses]
section for libary instances in MdeLibs.dsc.inc.
---
Silicon/TexasInstruments/Omap35xxPkg/Omap35xxPkg.dsc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Silicon/TexasInstruments/Omap35xxPkg/Omap35xxPkg.dsc b/Silicon/TexasInstruments/Omap35xxPkg/Omap35xxPkg.dsc
index 18ebc4aedf..8bf56c3768 100644
--- a/Silicon/TexasInstruments/Omap35xxPkg/Omap35xxPkg.dsc
+++ b/Silicon/TexasInstruments/Omap35xxPkg/Omap35xxPkg.dsc
@@ -21,10 +21,12 @@ [Defines]
OUTPUT_DIRECTORY = Build/Omap35xxPkg
SUPPORTED_ARCHITECTURES = ARM
BUILD_TARGETS = DEBUG|RELEASE|NOOPT
SKUID_IDENTIFIER = DEFAULT
+!include MdePkg/MdeLibs.dsc.inc
+
[LibraryClasses.common]
DebugLib|MdePkg/Library/BaseDebugLibNull/BaseDebugLibNull.inf
ArmLib|ArmPkg/Library/ArmLib/ArmBaseLib.inf
MemoryAllocationLib|MdePkg/Library/UefiMemoryAllocationLib/UefiMemoryAllocationLib.inf
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 50+ messages in thread