* [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning
@ 2023-03-09 16:05 Ashraf Ali S
2023-03-09 17:06 ` Chiu, Chasel
` (3 more replies)
0 siblings, 4 replies; 6+ messages in thread
From: Ashraf Ali S @ 2023-03-09 16:05 UTC (permalink / raw)
To: devel; +Cc: Ashraf Ali S, Chasel Chiu, Nate DeSimone, Sai Chaganty, Star Zeng
Function defination should match with declaration.
[-Wlto-type-mismatch]
Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Signed-off-by: Ashraf Ali S <ashraf.ali.s@intel.com>
---
IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c b/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
index 795bb28c0f..a5a51c804c 100644
--- a/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
+++ b/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
@@ -296,6 +296,7 @@ FspTempRamExitDone2 (
**/
VOID
+EFIAPI
FspWaitForNotify (
VOID
)
--
2.38.1.windows.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning
2023-03-09 16:05 [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning Ashraf Ali S
@ 2023-03-09 17:06 ` Chiu, Chasel
2023-03-10 0:21 ` Nate DeSimone
` (2 subsequent siblings)
3 siblings, 0 replies; 6+ messages in thread
From: Chiu, Chasel @ 2023-03-09 17:06 UTC (permalink / raw)
To: S, Ashraf Ali, devel@edk2.groups.io
Cc: Desimone, Nathaniel L, Chaganty, Rangasai V, Zeng, Star
Thanks for fixing this bug!
Reviewed-by: Chasel Chiu <chasel.chiu@intel.com>
> -----Original Message-----
> From: S, Ashraf Ali <ashraf.ali.s@intel.com>
> Sent: Thursday, March 9, 2023 8:06 AM
> To: devel@edk2.groups.io
> Cc: S, Ashraf Ali <ashraf.ali.s@intel.com>; Chiu, Chasel <chasel.chiu@intel.com>;
> Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Chaganty, Rangasai
> V <rangasai.v.chaganty@intel.com>; Zeng, Star <star.zeng@intel.com>
> Subject: [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning
>
> Function defination should match with declaration.
> [-Wlto-type-mismatch]
>
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
>
> Signed-off-by: Ashraf Ali S <ashraf.ali.s@intel.com>
> ---
> IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
> b/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
> index 795bb28c0f..a5a51c804c 100644
> --- a/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
> +++ b/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
> @@ -296,6 +296,7 @@ FspTempRamExitDone2 (
>
> **/
> VOID
> +EFIAPI
> FspWaitForNotify (
> VOID
> )
> --
> 2.38.1.windows.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning
2023-03-09 16:05 [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning Ashraf Ali S
2023-03-09 17:06 ` Chiu, Chasel
@ 2023-03-10 0:21 ` Nate DeSimone
2023-03-10 0:27 ` Chaganty, Rangasai V
2023-03-10 3:20 ` Chiu, Chasel
3 siblings, 0 replies; 6+ messages in thread
From: Nate DeSimone @ 2023-03-10 0:21 UTC (permalink / raw)
To: S, Ashraf Ali, devel@edk2.groups.io
Cc: Chiu, Chasel, Chaganty, Rangasai V, Zeng, Star
Reviewed-by: Nate DeSimone <nathaniel.l.desimone@intel.com>
-----Original Message-----
From: S, Ashraf Ali <ashraf.ali.s@intel.com>
Sent: Thursday, March 9, 2023 8:06 AM
To: devel@edk2.groups.io
Cc: S, Ashraf Ali <ashraf.ali.s@intel.com>; Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>; Zeng, Star <star.zeng@intel.com>
Subject: [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning
Function defination should match with declaration.
[-Wlto-type-mismatch]
Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Signed-off-by: Ashraf Ali S <ashraf.ali.s@intel.com>
---
IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c b/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
index 795bb28c0f..a5a51c804c 100644
--- a/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
+++ b/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
@@ -296,6 +296,7 @@ FspTempRamExitDone2 (
**/
VOID
+EFIAPI
FspWaitForNotify (
VOID
)
--
2.38.1.windows.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning
2023-03-09 16:05 [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning Ashraf Ali S
2023-03-09 17:06 ` Chiu, Chasel
2023-03-10 0:21 ` Nate DeSimone
@ 2023-03-10 0:27 ` Chaganty, Rangasai V
2023-03-10 1:30 ` Zeng, Star
2023-03-10 3:20 ` Chiu, Chasel
3 siblings, 1 reply; 6+ messages in thread
From: Chaganty, Rangasai V @ 2023-03-10 0:27 UTC (permalink / raw)
To: S, Ashraf Ali, devel@edk2.groups.io
Cc: Chiu, Chasel, Desimone, Nathaniel L, Zeng, Star
Reviewed-by: Sai Chaganty <rangasai.v.chaganty@intel.com>
-----Original Message-----
From: S, Ashraf Ali <ashraf.ali.s@intel.com>
Sent: Thursday, March 09, 2023 8:06 AM
To: devel@edk2.groups.io
Cc: S, Ashraf Ali <ashraf.ali.s@intel.com>; Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>; Zeng, Star <star.zeng@intel.com>
Subject: [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning
Function defination should match with declaration.
[-Wlto-type-mismatch]
Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Signed-off-by: Ashraf Ali S <ashraf.ali.s@intel.com>
---
IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c b/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
index 795bb28c0f..a5a51c804c 100644
--- a/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
+++ b/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
@@ -296,6 +296,7 @@ FspTempRamExitDone2 (
**/
VOID
+EFIAPI
FspWaitForNotify (
VOID
)
--
2.38.1.windows.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning
2023-03-10 0:27 ` Chaganty, Rangasai V
@ 2023-03-10 1:30 ` Zeng, Star
0 siblings, 0 replies; 6+ messages in thread
From: Zeng, Star @ 2023-03-10 1:30 UTC (permalink / raw)
To: Chaganty, Rangasai V, S, Ashraf Ali, devel@edk2.groups.io
Cc: Chiu, Chasel, Desimone, Nathaniel L, Zeng, Star
Reviewed-by: Star Zeng <star.zeng@intel.com> with typo defination -> definition fixed in commit message.
-----Original Message-----
From: Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>
Sent: Friday, March 10, 2023 8:27 AM
To: S, Ashraf Ali <ashraf.ali.s@intel.com>; devel@edk2.groups.io
Cc: Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>
Subject: RE: [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning
Reviewed-by: Sai Chaganty <rangasai.v.chaganty@intel.com>
-----Original Message-----
From: S, Ashraf Ali <ashraf.ali.s@intel.com>
Sent: Thursday, March 09, 2023 8:06 AM
To: devel@edk2.groups.io
Cc: S, Ashraf Ali <ashraf.ali.s@intel.com>; Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>; Zeng, Star <star.zeng@intel.com>
Subject: [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning
Function defination should match with declaration.
[-Wlto-type-mismatch]
Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Signed-off-by: Ashraf Ali S <ashraf.ali.s@intel.com>
---
IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c b/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
index 795bb28c0f..a5a51c804c 100644
--- a/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
+++ b/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
@@ -296,6 +296,7 @@ FspTempRamExitDone2 (
**/
VOID
+EFIAPI
FspWaitForNotify (
VOID
)
--
2.38.1.windows.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning
2023-03-09 16:05 [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning Ashraf Ali S
` (2 preceding siblings ...)
2023-03-10 0:27 ` Chaganty, Rangasai V
@ 2023-03-10 3:20 ` Chiu, Chasel
3 siblings, 0 replies; 6+ messages in thread
From: Chiu, Chasel @ 2023-03-10 3:20 UTC (permalink / raw)
To: S, Ashraf Ali, devel@edk2.groups.io
Cc: Desimone, Nathaniel L, Chaganty, Rangasai V, Zeng, Star
Fix patch has been pushed with commit message typo and format correction: https://github.com/tianocore/edk2/commit/8820767fb3bad09eeedecc3030d75c9e0cd4cab7
Thanks,
Chasel
> -----Original Message-----
> From: S, Ashraf Ali <ashraf.ali.s@intel.com>
> Sent: Thursday, March 9, 2023 8:06 AM
> To: devel@edk2.groups.io
> Cc: S, Ashraf Ali <ashraf.ali.s@intel.com>; Chiu, Chasel <chasel.chiu@intel.com>;
> Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Chaganty, Rangasai
> V <rangasai.v.chaganty@intel.com>; Zeng, Star <star.zeng@intel.com>
> Subject: [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning
>
> Function defination should match with declaration.
> [-Wlto-type-mismatch]
>
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
>
> Signed-off-by: Ashraf Ali S <ashraf.ali.s@intel.com>
> ---
> IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
> b/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
> index 795bb28c0f..a5a51c804c 100644
> --- a/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
> +++ b/IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c
> @@ -296,6 +296,7 @@ FspTempRamExitDone2 (
>
> **/
> VOID
> +EFIAPI
> FspWaitForNotify (
> VOID
> )
> --
> 2.38.1.windows.1
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-03-10 3:20 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-03-09 16:05 [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning Ashraf Ali S
2023-03-09 17:06 ` Chiu, Chasel
2023-03-10 0:21 ` Nate DeSimone
2023-03-10 0:27 ` Chaganty, Rangasai V
2023-03-10 1:30 ` Zeng, Star
2023-03-10 3:20 ` Chiu, Chasel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox