Re: [edk2-devel] [PATCH 11/24] IntelFsp2WrapperPkg: Remove duplicated words

2022-09-01 Thread Nate DeSimone
Pushed: https://github.com/tianocore/edk2/commit/6edd2578

> -Original Message-
> From: devel@edk2.groups.io  On Behalf Of
> PierreGondois
> Sent: Tuesday, August 23, 2022 9:00 AM
> To: devel@edk2.groups.io
> Cc: Pierre Gondois ; Chiu, Chasel
> ; Desimone, Nathaniel L
> 
> Subject: [edk2-devel] [PATCH 11/24] IntelFsp2WrapperPkg: Remove
> duplicated words
> 
> From: Pierre Gondois 
> 
> In an effort to clean the documentation of the above package, remove
> duplicated words.
> 
> Cc: Chasel Chiu 
> Cc: Nate DeSimone 
> Signed-off-by: Pierre Gondois 
> ---
>  IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
> b/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
> index c43b0c2267a7..08af6d814ad1 100644
> --- a/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
> +++ b/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
> @@ -87,7 +87,7 @@ [PcdsFixedAtBuild, PcdsPatchableInModule]
># We need a flag (PCD) to indicate if there is need to do such FSP
> measurement or NOT.
># 2) The FSP binary includes FSP code and FSP UPD region. The UPD region
> is considered
># as configuration block, and it may be updated by OEM by design.
> -  # This flag (PCD) is to indicate if we need isolate the the UPD region from
> the FSP code region.
> +  # This flag (PCD) is to indicate if we need isolate the UPD region from the
> FSP code region.
># BIT0: Need measure FSP. (for FSP1.x) - reserved in FSP2.
># BIT1: Need measure FSPT. (for FSP 2.x)
># BIT2: Need measure FSPM. (for FSP 2.x)
> --
> 2.25.1
> 
> 
> 
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#92693): https://edk2.groups.io/g/devel/message/92693
> Mute This Topic: https://groups.io/mt/93207394/1767664
> Group Owner: devel+ow...@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub
> [nathaniel.l.desim...@intel.com]
> -=-=-=-=-=-=
> 



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




Re: [edk2-devel] [PATCH 11/24] IntelFsp2WrapperPkg: Remove duplicated words

2022-09-01 Thread Nate DeSimone
Good catch!
Reviewed-by: Nate DeSimone 

> -Original Message-
> From: devel@edk2.groups.io  On Behalf Of
> PierreGondois
> Sent: Tuesday, August 23, 2022 9:00 AM
> To: devel@edk2.groups.io
> Cc: Pierre Gondois ; Chiu, Chasel
> ; Desimone, Nathaniel L
> 
> Subject: [edk2-devel] [PATCH 11/24] IntelFsp2WrapperPkg: Remove
> duplicated words
> 
> From: Pierre Gondois 
> 
> In an effort to clean the documentation of the above package, remove
> duplicated words.
> 
> Cc: Chasel Chiu 
> Cc: Nate DeSimone 
> Signed-off-by: Pierre Gondois 
> ---
>  IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
> b/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
> index c43b0c2267a7..08af6d814ad1 100644
> --- a/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
> +++ b/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
> @@ -87,7 +87,7 @@ [PcdsFixedAtBuild, PcdsPatchableInModule]
># We need a flag (PCD) to indicate if there is need to do such FSP
> measurement or NOT.
># 2) The FSP binary includes FSP code and FSP UPD region. The UPD region
> is considered
># as configuration block, and it may be updated by OEM by design.
> -  # This flag (PCD) is to indicate if we need isolate the the UPD region from
> the FSP code region.
> +  # This flag (PCD) is to indicate if we need isolate the UPD region from the
> FSP code region.
># BIT0: Need measure FSP. (for FSP1.x) - reserved in FSP2.
># BIT1: Need measure FSPT. (for FSP 2.x)
># BIT2: Need measure FSPM. (for FSP 2.x)
> --
> 2.25.1
> 
> 
> 
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#92693): https://edk2.groups.io/g/devel/message/92693
> Mute This Topic: https://groups.io/mt/93207394/1767664
> Group Owner: devel+ow...@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub
> [nathaniel.l.desim...@intel.com]
> -=-=-=-=-=-=
> 



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




Re: [edk2-devel] [PATCH 11/24] IntelFsp2WrapperPkg: Remove duplicated words

2022-08-23 Thread Chiu, Chasel


Thanks for fixing the typo!
Reviewed-by: Chasel Chiu 


> -Original Message-
> From: pierre.gond...@arm.com 
> Sent: Tuesday, August 23, 2022 9:00 AM
> To: devel@edk2.groups.io
> Cc: Pierre Gondois ; Chiu, Chasel
> ; Desimone, Nathaniel L
> 
> Subject: [PATCH 11/24] IntelFsp2WrapperPkg: Remove duplicated words
> 
> From: Pierre Gondois 
> 
> In an effort to clean the documentation of the above package, remove
> duplicated words.
> 
> Cc: Chasel Chiu 
> Cc: Nate DeSimone 
> Signed-off-by: Pierre Gondois 
> ---
>  IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
> b/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
> index c43b0c2267a7..08af6d814ad1 100644
> --- a/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
> +++ b/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
> @@ -87,7 +87,7 @@ [PcdsFixedAtBuild, PcdsPatchableInModule]
># We need a flag (PCD) to indicate if there is need to do such FSP
> measurement or NOT.
># 2) The FSP binary includes FSP code and FSP UPD region. The UPD region
> is considered
># as configuration block, and it may be updated by OEM by design.
> -  # This flag (PCD) is to indicate if we need isolate the the UPD region from
> the FSP code region.
> +  # This flag (PCD) is to indicate if we need isolate the UPD region from the
> FSP code region.
># BIT0: Need measure FSP. (for FSP1.x) - reserved in FSP2.
># BIT1: Need measure FSPT. (for FSP 2.x)
># BIT2: Need measure FSPM. (for FSP 2.x)
> --
> 2.25.1



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




[edk2-devel] [PATCH 11/24] IntelFsp2WrapperPkg: Remove duplicated words

2022-08-23 Thread PierreGondois
From: Pierre Gondois 

In an effort to clean the documentation of the above
package, remove duplicated words.

Cc: Chasel Chiu 
Cc: Nate DeSimone 
Signed-off-by: Pierre Gondois 
---
 IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec 
b/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
index c43b0c2267a7..08af6d814ad1 100644
--- a/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
+++ b/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec
@@ -87,7 +87,7 @@ [PcdsFixedAtBuild, PcdsPatchableInModule]
   # We need a flag (PCD) to indicate if there is need to do such FSP 
measurement or NOT.
   # 2) The FSP binary includes FSP code and FSP UPD region. The UPD region is 
considered
   # as configuration block, and it may be updated by OEM by design.
-  # This flag (PCD) is to indicate if we need isolate the the UPD region from 
the FSP code region.
+  # This flag (PCD) is to indicate if we need isolate the UPD region from the 
FSP code region.
   # BIT0: Need measure FSP. (for FSP1.x) - reserved in FSP2.
   # BIT1: Need measure FSPT. (for FSP 2.x)
   # BIT2: Need measure FSPM. (for FSP 2.x)
-- 
2.25.1



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