<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Reviewed-by: Alexei Fedorov <Alexei.Fedorov@arm.com><br>
</div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div id="Signature">
<div id="divtagdefaultwrapper" dir="ltr" style="font-size:12pt; color:#000000; font-family:Calibri,Helvetica,sans-serif">
<p style="margin-top: 0px; margin-bottom: 0px;margin-top:0px; margin-bottom:0px">
Alexei<br>
</p>
</div>
</div>
<div id="appendonsend"></div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Sami Mujawar <sami.mujawar@arm.com><br>
<b>Sent:</b> 23 August 2019 11:55<br>
<b>To:</b> devel@edk2.groups.io <devel@edk2.groups.io><br>
<b>Cc:</b> Sami Mujawar <Sami.Mujawar@arm.com>; Alexei Fedorov <Alexei.Fedorov@arm.com>; leif.lindholm@linaro.org <leif.lindholm@linaro.org>; Matteo Carlini <Matteo.Carlini@arm.com>; nd <nd@arm.com><br>
<b>Subject:</b> [PATCH v1 01/19] DynamicTablesPkg: Fix entry point param definition</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">VS2017 reports 'warning C4028: formal parameter 2 different<br>
from declaration' for the library constructor and destructor<br>
interfaces for the Generator modules. VS2017 compiler also<br>
reports similar warnings for the DXE entry points.<br>
<br>
Remove the CONST qualifier for the SystemTable pointer (the<br>
second parameter to the constructor/destructor/DXE Entry<br>
point) to make it compatible with the formal declaration.<br>
<br>
Signed-off-by: Sami Mujawar <sami.mujawar@arm.com><br>
---<br>
DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/DynamicTableFactoryDxe.c | 6 +++---<br>
DynamicTablesPkg/Drivers/DynamicTableManagerDxe/DynamicTableManagerDxe.c | 4 ++--<br>
DynamicTablesPkg/Library/Acpi/Arm/AcpiDbg2LibArm/Dbg2Generator.c | 8 ++++----<br>
DynamicTablesPkg/Library/Acpi/Arm/AcpiFadtLibArm/FadtGenerator.c | 8 ++++----<br>
DynamicTablesPkg/Library/Acpi/Arm/AcpiGtdtLibArm/GtdtGenerator.c | 8 ++++----<br>
DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c | 8 ++++----<br>
DynamicTablesPkg/Library/Acpi/Arm/AcpiMadtLibArm/MadtGenerator.c | 8 ++++----<br>
DynamicTablesPkg/Library/Acpi/Arm/AcpiMcfgLibArm/McfgGenerator.c | 8 ++++----<br>
DynamicTablesPkg/Library/Acpi/Arm/AcpiPpttLibArm/PpttGenerator.c | 8 ++++----<br>
DynamicTablesPkg/Library/Acpi/Arm/AcpiRawLibArm/RawGenerator.c | 8 ++++----<br>
DynamicTablesPkg/Library/Acpi/Arm/AcpiSpcrLibArm/SpcrGenerator.c | 8 ++++----<br>
11 files changed, 41 insertions(+), 41 deletions(-)<br>
<br>
diff --git a/DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/DynamicTableFactoryDxe.c b/DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/DynamicTableFactoryDxe.c<br>
index d87ea7809eae09cdb5ff06290cff8c2d7e7fa904..d1432348f05143bed8588cea54d7b9b6722b9b6d 100644<br>
--- a/DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/DynamicTableFactoryDxe.c<br>
+++ b/DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/DynamicTableFactoryDxe.c<br>
@@ -1,7 +1,7 @@<br>
/** @file<br>
Dynamic Table Factory Dxe<br>
<br>
- Copyright (c) 2017 - 2018, ARM Limited. All rights reserved.<br>
+ Copyright (c) 2017 - 2019, ARM Limited. All rights reserved.<br>
<br>
SPDX-License-Identifier: BSD-2-Clause-Patent<br>
<br>
@@ -60,8 +60,8 @@ EDKII_DYNAMIC_TABLE_FACTORY_PROTOCOL DynamicTableFactoryProtocol = {<br>
EFI_STATUS<br>
EFIAPI<br>
DynamicTableFactoryDxeInitialize (<br>
- IN EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
diff --git a/DynamicTablesPkg/Drivers/DynamicTableManagerDxe/DynamicTableManagerDxe.c b/DynamicTablesPkg/Drivers/DynamicTableManagerDxe/DynamicTableManagerDxe.c<br>
index 39b75996d9922d2daf953bb28809aa285f7d214c..e27dcaf37470d811c4d6218d4f40d2260f550c02 100644<br>
--- a/DynamicTablesPkg/Drivers/DynamicTableManagerDxe/DynamicTableManagerDxe.c<br>
+++ b/DynamicTablesPkg/Drivers/DynamicTableManagerDxe/DynamicTableManagerDxe.c<br>
@@ -655,8 +655,8 @@ ProcessAcpiTables (<br>
EFI_STATUS<br>
EFIAPI<br>
DynamicTableManagerDxeInitialize (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiDbg2LibArm/Dbg2Generator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiDbg2LibArm/Dbg2Generator.c<br>
index 0b45e15d2d1c2c1e35e801ca719d68d9ff88a65e..346ab5b22f5402bf87c385558f68f080d1b454ed 100644<br>
--- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiDbg2LibArm/Dbg2Generator.c<br>
+++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiDbg2LibArm/Dbg2Generator.c<br>
@@ -418,8 +418,8 @@ ACPI_TABLE_GENERATOR Dbg2Generator = {<br>
EFI_STATUS<br>
EFIAPI<br>
AcpiDbg2LibConstructor (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
@@ -442,8 +442,8 @@ AcpiDbg2LibConstructor (<br>
EFI_STATUS<br>
EFIAPI<br>
AcpiDbg2LibDestructor (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiFadtLibArm/FadtGenerator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiFadtLibArm/FadtGenerator.c<br>
index e9e4991dc127abb97f859436df0c4ceafdb3f5ec..6cb60e898f697ce4347ad3adeb5bca1cc1f30a0c 100644<br>
--- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiFadtLibArm/FadtGenerator.c<br>
+++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiFadtLibArm/FadtGenerator.c<br>
@@ -648,8 +648,8 @@ ACPI_TABLE_GENERATOR FadtGenerator = {<br>
EFI_STATUS<br>
EFIAPI<br>
AcpiFadtLibConstructor (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
@@ -671,8 +671,8 @@ AcpiFadtLibConstructor (<br>
EFI_STATUS<br>
EFIAPI<br>
AcpiFadtLibDestructor (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiGtdtLibArm/GtdtGenerator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiGtdtLibArm/GtdtGenerator.c<br>
index a30870806d7f223fde7311e3609cca92f47731cc..7e86c30649bd36ecebac75c7e3a86a0c25cd590b 100644<br>
--- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiGtdtLibArm/GtdtGenerator.c<br>
+++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiGtdtLibArm/GtdtGenerator.c<br>
@@ -738,8 +738,8 @@ ACPI_TABLE_GENERATOR GtdtGenerator = {<br>
EFI_STATUS<br>
EFIAPI<br>
AcpiGtdtLibConstructor (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
@@ -761,8 +761,8 @@ AcpiGtdtLibConstructor (<br>
EFI_STATUS<br>
EFIAPI<br>
AcpiGtdtLibDestructor (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c<br>
index 05e360c899c7563aed04c071921865929524fdc4..f303e9f29cc700634d236c08505cab91b9d62cb8 100644<br>
--- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c<br>
+++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c<br>
@@ -2040,8 +2040,8 @@ ACPI_IORT_GENERATOR IortGenerator = {<br>
EFI_STATUS<br>
EFIAPI<br>
AcpiIortLibConstructor (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
@@ -2063,8 +2063,8 @@ AcpiIortLibConstructor (<br>
EFI_STATUS<br>
EFIAPI<br>
AcpiIortLibDestructor (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiMadtLibArm/MadtGenerator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiMadtLibArm/MadtGenerator.c<br>
index dc5238078376746847e7ae14258bbe35ca07ca2c..2651ca15414694c494678bcec655fc882ee2cc0c 100644<br>
--- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiMadtLibArm/MadtGenerator.c<br>
+++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiMadtLibArm/MadtGenerator.c<br>
@@ -795,8 +795,8 @@ ACPI_TABLE_GENERATOR MadtGenerator = {<br>
EFI_STATUS<br>
EFIAPI<br>
AcpiMadtLibConstructor (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
@@ -818,8 +818,8 @@ AcpiMadtLibConstructor (<br>
EFI_STATUS<br>
EFIAPI<br>
AcpiMadtLibDestructor (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiMcfgLibArm/McfgGenerator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiMcfgLibArm/McfgGenerator.c<br>
index d219e2f04ad6602b8157b048ae840a3094492680..1293ac8e43c8e0ed641b20ba42519172bc541b54 100644<br>
--- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiMcfgLibArm/McfgGenerator.c<br>
+++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiMcfgLibArm/McfgGenerator.c<br>
@@ -329,8 +329,8 @@ ACPI_TABLE_GENERATOR McfgGenerator = {<br>
EFI_STATUS<br>
EFIAPI<br>
AcpiMcfgLibConstructor (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
@@ -352,8 +352,8 @@ AcpiMcfgLibConstructor (<br>
EFI_STATUS<br>
EFIAPI<br>
AcpiMcfgLibDestructor (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiPpttLibArm/PpttGenerator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiPpttLibArm/PpttGenerator.c<br>
index c8713dec625cc18b0806659ffcdc3d9e70411e11..9e42eee9b75bb330833b0f56c98947563d9eb821 100644<br>
--- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiPpttLibArm/PpttGenerator.c<br>
+++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiPpttLibArm/PpttGenerator.c<br>
@@ -1492,8 +1492,8 @@ ACPI_PPTT_GENERATOR PpttGenerator = {<br>
EFI_STATUS<br>
EFIAPI<br>
AcpiPpttLibConstructor (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
@@ -1516,8 +1516,8 @@ AcpiPpttLibConstructor (<br>
EFI_STATUS<br>
EFIAPI<br>
AcpiPpttLibDestructor (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiRawLibArm/RawGenerator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiRawLibArm/RawGenerator.c<br>
index 544ea06f0bcb3442b78cf6409846f70ae5360664..2facfaa048115575829c9787084876efbe60834a 100644<br>
--- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiRawLibArm/RawGenerator.c<br>
+++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiRawLibArm/RawGenerator.c<br>
@@ -109,8 +109,8 @@ ACPI_TABLE_GENERATOR RawGenerator = {<br>
EFI_STATUS<br>
EFIAPI<br>
AcpiRawLibConstructor (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
@@ -132,8 +132,8 @@ AcpiRawLibConstructor (<br>
EFI_STATUS<br>
EFIAPI<br>
AcpiRawLibDestructor (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiSpcrLibArm/SpcrGenerator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiSpcrLibArm/SpcrGenerator.c<br>
index 7b70ba2c2c55a9a15a569680d1a57b359db4e73d..1404279f828d5c06bb7605cb2fe6d864ef7a080e 100644<br>
--- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiSpcrLibArm/SpcrGenerator.c<br>
+++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiSpcrLibArm/SpcrGenerator.c<br>
@@ -308,8 +308,8 @@ ACPI_TABLE_GENERATOR SpcrGenerator = {<br>
EFI_STATUS<br>
EFIAPI<br>
AcpiSpcrLibConstructor (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
@@ -331,8 +331,8 @@ AcpiSpcrLibConstructor (<br>
EFI_STATUS<br>
EFIAPI<br>
AcpiSpcrLibDestructor (<br>
- IN CONST EFI_HANDLE ImageHandle,<br>
- IN EFI_SYSTEM_TABLE * CONST SystemTable<br>
+ IN EFI_HANDLE ImageHandle,<br>
+ IN EFI_SYSTEM_TABLE * SystemTable<br>
)<br>
{<br>
EFI_STATUS Status;<br>
-- <br>
'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'<br>
<br>
<br>
</div>
</span></font></div>
</body>
</html>
<div width="1" style="color:white;clear:both">_._,_._,_</div>
<hr>
Groups.io Links:<p>
You receive all messages sent to this group.
<p>
<a target="_blank" href="https://edk2.groups.io/g/devel/message/46282">View/Reply Online (#46282)</a> |
|
<a target="_blank" href="https://groups.io/mt/32999778/1813853">Mute This Topic</a>
| <a href="https://edk2.groups.io/g/devel/post">New Topic</a><br>
<br>
<a href="https://edk2.groups.io/g/devel/editsub/1813853">Your Subscription</a> |
<a href="mailto:devel+owner@edk2.groups.io">Contact Group Owner</a> |
<a href="https://edk2.groups.io/g/devel/unsub">Unsubscribe</a>
[edk2-devel-archive@redhat.com]<br>
<div width="1" style="color:white;clear:both">_._,_._,_</div>