Re: [edk2-devel] [PATCH] IntelFsp2WrapperPkg: Add header for PlatformMultiPhaseLib.

2022-10-04 Thread Nate DeSimone
Reviewed-by: Nate DeSimone 

-Original Message-
From: Chiu, Chasel  
Sent: Tuesday, October 4, 2022 1:34 PM
To: devel@edk2.groups.io
Cc: Chiu, Chasel ; Desimone, Nathaniel L 
; Zeng, Star 
Subject: [PATCH] IntelFsp2WrapperPkg: Add header for PlatformMultiPhaseLib.

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4092

To comply with coding style rule each library class should have its own header 
even if it is just a private child library instance consumed by parent public 
library in the same package.

Cc: Nate DeSimone 
Cc: Star Zeng 
Signed-off-by: Chasel Chiu 
---
 
IntelFsp2WrapperPkg/Library/FspWrapperMultiPhaseProcessLib/PeiFspWrapperMultiPhaseProcessLib.c
 |  2 +-
 IntelFsp2WrapperPkg/Include/Library/FspWrapperMultiPhaseProcessLib.h   
| 18 --
 IntelFsp2WrapperPkg/Include/Library/FspWrapperPlatformMultiPhaseLib.h  
| 30 ++
 IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
|  2 +-
 4 files changed, 32 insertions(+), 20 deletions(-)

diff --git 
a/IntelFsp2WrapperPkg/Library/FspWrapperMultiPhaseProcessLib/PeiFspWrapperMultiPhaseProcessLib.c
 
b/IntelFsp2WrapperPkg/Library/FspWrapperMultiPhaseProcessLib/PeiFspWrapperMultiPhaseProcessLib.c
index 3ddc07690a..1248accf87 100644
--- 
a/IntelFsp2WrapperPkg/Library/FspWrapperMultiPhaseProcessLib/PeiFspWrapperMultiPhaseProcessLib.c
+++ b/IntelFsp2WrapperPkg/Library/FspWrapperMultiPhaseProcessLib/PeiFspW
+++ rapperMultiPhaseProcessLib.c
@@ -16,7 +16,7 @@
 #include  #include  #include 
-#include 
+#include 
  /**   Execute 32-bit FSP API entry 
code.diff --git 
a/IntelFsp2WrapperPkg/Include/Library/FspWrapperMultiPhaseProcessLib.h 
b/IntelFsp2WrapperPkg/Include/Library/FspWrapperMultiPhaseProcessLib.h
index 65a5db233f..aadacc9b50 100644
--- a/IntelFsp2WrapperPkg/Include/Library/FspWrapperMultiPhaseProcessLib.h
+++ b/IntelFsp2WrapperPkg/Include/Library/FspWrapperMultiPhaseProcessLib
+++ .h
@@ -9,24 +9,6 @@
 #ifndef __FSP_WRAPPER_MULTI_PHASE_PROCESS_LIB_H__ #define 
__FSP_WRAPPER_MULTI_PHASE_PROCESS_LIB_H__ -/**-  FSP Wrapper Platform 
MultiPhase Handler--  @param[in] FspHobListPtr- Pointer to FSP HobList 
(valid after FSP-M completed)-  @param[in] ComponentIndex   - FSP Component 
which executing MultiPhase initialization.-  @param[in] PhaseIndex   - 
Indicates current execution phase of FSP MultiPhase initialization.--  @retval 
EFI_STATUSAlways return 
EFI_SUCCESS--**/-VOID-EFIAPI-FspWrapperPlatformMultiPhaseHandler (-  IN OUT 
VOID  **FspHobListPtr,-  IN UINT8 ComponentIndex,-  IN UINT32
PhaseIndex-  );- /**   FSP Wrapper Variable Request Handler diff --git 
a/IntelFsp2WrapperPkg/Include/Library/FspWrapperPlatformMultiPhaseLib.h 
b/IntelFsp2WrapperPkg/Include/Library/FspWrapperPlatformMultiPhaseLib.h
new file mode 100644
index 00..79b5ba0ec3
--- /dev/null
+++ b/IntelFsp2WrapperPkg/Include/Library/FspWrapperPlatformMultiPhaseLi
+++ b.h
@@ -0,0 +1,30 @@
+/** @file+  Provide FSP wrapper Platform MultiPhase handling functions.++  
Copyright (c) 2022, Intel Corporation. All rights reserved.+  
SPDX-License-Identifier: BSD-2-Clause-Patent++**/++#ifndef 
__FSP_WRAPPER_PLATFORM_MULTI_PHASE_PROCESS_LIB_H__+#define 
__FSP_WRAPPER_PLATFORM_MULTI_PHASE_PROCESS_LIB_H__++/**+  FSP Wrapper Platform 
MultiPhase Handler++  @param[in] FspHobListPtr- Pointer to FSP HobList 
(valid after FSP-M completed)+  @param[in] ComponentIndex   - FSP Component 
which executing MultiPhase initialization.+  @param[in] PhaseIndex   - 
Indicates current execution phase of FSP MultiPhase initialization.++  @retval 
EFI_STATUSAlways return 
EFI_SUCCESS++**/+VOID+EFIAPI+FspWrapperPlatformMultiPhaseHandler (+  IN OUT 
VOID  **FspHobListPtr,+  IN UINT8 ComponentIndex,+  IN UINT32
PhaseIndex+  );++#endifdiff --git a/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec 
b/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
index 95ada0f7a1..922ccc063f 100644
--- a/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
+++ b/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
@@ -33,7 +33,7 @@
   
FspWrapperMultiPhaseProcessLib|Include/Library/FspWrapperMultiPhaseProcessLib.h 
   ##  @libraryclass  Provide MultiPhase platform actions related functions.-  
FspWrapperPlatformMultiPhaseLib|Include/Library/FspWrapperMultiPhaseProcessLib.h+
  
FspWrapperPlatformMultiPhaseLib|Include/Library/FspWrapperPlatformMultiPhaseLib.h
   [Guids]-- 
2.35.0.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94723): https://edk2.groups.io/g/devel/message/94723
Mute This Topic: https://groups.io/mt/94122844/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




[edk2-devel] [PATCH] IntelFsp2WrapperPkg: Add header for PlatformMultiPhaseLib.

2022-10-04 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4092

To comply with coding style rule each library class
should have its own header even if it is just a private child library
instance consumed by parent public library in the same package.

Cc: Nate DeSimone 
Cc: Star Zeng 
Signed-off-by: Chasel Chiu 
---
 
IntelFsp2WrapperPkg/Library/FspWrapperMultiPhaseProcessLib/PeiFspWrapperMultiPhaseProcessLib.c
 |  2 +-
 IntelFsp2WrapperPkg/Include/Library/FspWrapperMultiPhaseProcessLib.h   
| 18 --
 IntelFsp2WrapperPkg/Include/Library/FspWrapperPlatformMultiPhaseLib.h  
| 30 ++
 IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
|  2 +-
 4 files changed, 32 insertions(+), 20 deletions(-)

diff --git 
a/IntelFsp2WrapperPkg/Library/FspWrapperMultiPhaseProcessLib/PeiFspWrapperMultiPhaseProcessLib.c
 
b/IntelFsp2WrapperPkg/Library/FspWrapperMultiPhaseProcessLib/PeiFspWrapperMultiPhaseProcessLib.c
index 3ddc07690a..1248accf87 100644
--- 
a/IntelFsp2WrapperPkg/Library/FspWrapperMultiPhaseProcessLib/PeiFspWrapperMultiPhaseProcessLib.c
+++ 
b/IntelFsp2WrapperPkg/Library/FspWrapperMultiPhaseProcessLib/PeiFspWrapperMultiPhaseProcessLib.c
@@ -16,7 +16,7 @@
 #include 
 #include 
 #include 
-#include 
+#include 
 
 /**
   Execute 32-bit FSP API entry code.
diff --git 
a/IntelFsp2WrapperPkg/Include/Library/FspWrapperMultiPhaseProcessLib.h 
b/IntelFsp2WrapperPkg/Include/Library/FspWrapperMultiPhaseProcessLib.h
index 65a5db233f..aadacc9b50 100644
--- a/IntelFsp2WrapperPkg/Include/Library/FspWrapperMultiPhaseProcessLib.h
+++ b/IntelFsp2WrapperPkg/Include/Library/FspWrapperMultiPhaseProcessLib.h
@@ -9,24 +9,6 @@
 #ifndef __FSP_WRAPPER_MULTI_PHASE_PROCESS_LIB_H__
 #define __FSP_WRAPPER_MULTI_PHASE_PROCESS_LIB_H__
 
-/**
-  FSP Wrapper Platform MultiPhase Handler
-
-  @param[in] FspHobListPtr- Pointer to FSP HobList (valid after FSP-M 
completed)
-  @param[in] ComponentIndex   - FSP Component which executing MultiPhase 
initialization.
-  @param[in] PhaseIndex   - Indicates current execution phase of FSP 
MultiPhase initialization.
-
-  @retval EFI_STATUSAlways return EFI_SUCCESS
-
-**/
-VOID
-EFIAPI
-FspWrapperPlatformMultiPhaseHandler (
-  IN OUT VOID  **FspHobListPtr,
-  IN UINT8 ComponentIndex,
-  IN UINT32PhaseIndex
-  );
-
 /**
   FSP Wrapper Variable Request Handler
 
diff --git 
a/IntelFsp2WrapperPkg/Include/Library/FspWrapperPlatformMultiPhaseLib.h 
b/IntelFsp2WrapperPkg/Include/Library/FspWrapperPlatformMultiPhaseLib.h
new file mode 100644
index 00..79b5ba0ec3
--- /dev/null
+++ b/IntelFsp2WrapperPkg/Include/Library/FspWrapperPlatformMultiPhaseLib.h
@@ -0,0 +1,30 @@
+/** @file
+  Provide FSP wrapper Platform MultiPhase handling functions.
+
+  Copyright (c) 2022, Intel Corporation. All rights reserved.
+  SPDX-License-Identifier: BSD-2-Clause-Patent
+
+**/
+
+#ifndef __FSP_WRAPPER_PLATFORM_MULTI_PHASE_PROCESS_LIB_H__
+#define __FSP_WRAPPER_PLATFORM_MULTI_PHASE_PROCESS_LIB_H__
+
+/**
+  FSP Wrapper Platform MultiPhase Handler
+
+  @param[in] FspHobListPtr- Pointer to FSP HobList (valid after FSP-M 
completed)
+  @param[in] ComponentIndex   - FSP Component which executing MultiPhase 
initialization.
+  @param[in] PhaseIndex   - Indicates current execution phase of FSP 
MultiPhase initialization.
+
+  @retval EFI_STATUSAlways return EFI_SUCCESS
+
+**/
+VOID
+EFIAPI
+FspWrapperPlatformMultiPhaseHandler (
+  IN OUT VOID  **FspHobListPtr,
+  IN UINT8 ComponentIndex,
+  IN UINT32PhaseIndex
+  );
+
+#endif
diff --git a/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec 
b/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
index 95ada0f7a1..922ccc063f 100644
--- a/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
+++ b/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
@@ -33,7 +33,7 @@
   
FspWrapperMultiPhaseProcessLib|Include/Library/FspWrapperMultiPhaseProcessLib.h
 
   ##  @libraryclass  Provide MultiPhase platform actions related functions.
-  
FspWrapperPlatformMultiPhaseLib|Include/Library/FspWrapperMultiPhaseProcessLib.h
+  
FspWrapperPlatformMultiPhaseLib|Include/Library/FspWrapperPlatformMultiPhaseLib.h
 
 
 [Guids]
-- 
2.35.0.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94722): https://edk2.groups.io/g/devel/message/94722
Mute This Topic: https://groups.io/mt/94122844/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-