* Re: [edk2-devel] [PATCH 1/1] OvmfPkg:Fix Hii form name mismatch with EFI variable
2023-07-14 8:59 ` [PATCH 1/1] OvmfPkg:Fix Hii form name mismatch with EFI variable Wang, Yin
@ 2023-07-17 2:52 ` Wang, Yin
2023-07-17 11:36 ` Gerd Hoffmann
2023-08-07 12:02 ` Sunil V L
2 siblings, 0 replies; 6+ messages in thread
From: Wang, Yin @ 2023-07-17 2:52 UTC (permalink / raw)
To: devel; +Cc: kraxel, ard.biesheuvel, jordan.l.justen, sunilvl
Corrected Cc unsuccessful in git.
Thanks for your reviewing.
On 7/14/2023 4:59 PM, Yin Wang wrote:
> Onemore need change beside fixs:16acacf24c ("OvmfPkg: fix PlatformConfig")
> Find by sct case:ExtractConfigConformance fail.
>
> Signed-off-by: Yin Wang <yin.wang@intel.com>
> ---
> OvmfPkg/PlatformDxe/Platform.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/OvmfPkg/PlatformDxe/Platform.c b/OvmfPkg/PlatformDxe/Platform.c
> index cd3bfd554c98..c32439683d83 100644
> --- a/OvmfPkg/PlatformDxe/Platform.c
> +++ b/OvmfPkg/PlatformDxe/Platform.c
> @@ -279,7 +279,7 @@ ExtractConfig (
> //
> ConfigRequestHdr = HiiConstructConfigHdr (
> &gOvmfPlatformConfigGuid,
> - mVariableName,
> + mHiiFormName,
> mImageHandle
> );
> if (ConfigRequestHdr == NULL) {
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#106955): https://edk2.groups.io/g/devel/message/106955
Mute This Topic: https://groups.io/mt/100142143/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [edk2-devel] [PATCH 1/1] OvmfPkg:Fix Hii form name mismatch with EFI variable
2023-07-14 8:59 ` [PATCH 1/1] OvmfPkg:Fix Hii form name mismatch with EFI variable Wang, Yin
2023-07-17 2:52 ` [edk2-devel] " Wang, Yin
@ 2023-07-17 11:36 ` Gerd Hoffmann
2023-07-20 0:57 ` Wang, Yin
2023-08-07 12:02 ` Sunil V L
2 siblings, 1 reply; 6+ messages in thread
From: Gerd Hoffmann @ 2023-07-17 11:36 UTC (permalink / raw)
To: devel, yin.wang
On Fri, Jul 14, 2023 at 04:59:48PM +0800, Wang, Yin wrote:
> Onemore need change beside fixs:16acacf24c ("OvmfPkg: fix PlatformConfig")
> Find by sct case:ExtractConfigConformance fail.
>
> Signed-off-by: Yin Wang <yin.wang@intel.com>
> ---
> OvmfPkg/PlatformDxe/Platform.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/OvmfPkg/PlatformDxe/Platform.c b/OvmfPkg/PlatformDxe/Platform.c
> index cd3bfd554c98..c32439683d83 100644
> --- a/OvmfPkg/PlatformDxe/Platform.c
> +++ b/OvmfPkg/PlatformDxe/Platform.c
> @@ -279,7 +279,7 @@ ExtractConfig (
> //
> ConfigRequestHdr = HiiConstructConfigHdr (
> &gOvmfPlatformConfigGuid,
> - mVariableName,
> + mHiiFormName,
> mImageHandle
> );
> if (ConfigRequestHdr == NULL) {
Acked-by: Gerd Hoffmann <kraxel@redhat.com>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#106964): https://edk2.groups.io/g/devel/message/106964
Mute This Topic: https://groups.io/mt/100142143/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [edk2-devel] [PATCH 1/1] OvmfPkg:Fix Hii form name mismatch with EFI variable
2023-07-17 11:36 ` Gerd Hoffmann
@ 2023-07-20 0:57 ` Wang, Yin
0 siblings, 0 replies; 6+ messages in thread
From: Wang, Yin @ 2023-07-20 0:57 UTC (permalink / raw)
To: Gerd Hoffmann, devel; +Cc: ard.biesheuvel, jordan.l.justen, sunilvl
Hi, Could you help to review on it?
Thanks in advance!
Yin
On 7/17/2023 7:36 PM, Gerd Hoffmann wrote:
> On Fri, Jul 14, 2023 at 04:59:48PM +0800, Wang, Yin wrote:
>> Onemore need change beside fixs:16acacf24c ("OvmfPkg: fix PlatformConfig")
>> Find by sct case:ExtractConfigConformance fail.
>>
>> Signed-off-by: Yin Wang <yin.wang@intel.com>
>> ---
>> OvmfPkg/PlatformDxe/Platform.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/OvmfPkg/PlatformDxe/Platform.c b/OvmfPkg/PlatformDxe/Platform.c
>> index cd3bfd554c98..c32439683d83 100644
>> --- a/OvmfPkg/PlatformDxe/Platform.c
>> +++ b/OvmfPkg/PlatformDxe/Platform.c
>> @@ -279,7 +279,7 @@ ExtractConfig (
>> //
>> ConfigRequestHdr = HiiConstructConfigHdr (
>> &gOvmfPlatformConfigGuid,
>> - mVariableName,
>> + mHiiFormName,
>> mImageHandle
>> );
>> if (ConfigRequestHdr == NULL) {
> Acked-by: Gerd Hoffmann <kraxel@redhat.com>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#107078): https://edk2.groups.io/g/devel/message/107078
Mute This Topic: https://groups.io/mt/100142143/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [edk2-devel] [PATCH 1/1] OvmfPkg:Fix Hii form name mismatch with EFI variable
2023-07-14 8:59 ` [PATCH 1/1] OvmfPkg:Fix Hii form name mismatch with EFI variable Wang, Yin
2023-07-17 2:52 ` [edk2-devel] " Wang, Yin
2023-07-17 11:36 ` Gerd Hoffmann
@ 2023-08-07 12:02 ` Sunil V L
2023-08-08 1:44 ` Wang, Yin
2 siblings, 1 reply; 6+ messages in thread
From: Sunil V L @ 2023-08-07 12:02 UTC (permalink / raw)
To: devel, yin.wang
Hi Wang,
Thanks for the patch!
It will be good if you add versions to the patch subject. Even if you
go back to v1, better to send as v3 with the change log.
On Fri, Jul 14, 2023 at 04:59:48PM +0800, Wang, Yin wrote:
> Onemore need change beside fixs:16acacf24c ("OvmfPkg: fix PlatformConfig")
fixs -> Fixes? Why don't we need "Fixes:" tag to either aefcc91805fd or
16acacf24c2c?
I think it is better to have commit message with exact error or
information on what is the issue instead of just pointing to another
commit.
> Find by sct case:ExtractConfigConformance fail.
>
> Signed-off-by: Yin Wang <yin.wang@intel.com>
> ---
> OvmfPkg/PlatformDxe/Platform.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/OvmfPkg/PlatformDxe/Platform.c b/OvmfPkg/PlatformDxe/Platform.c
> index cd3bfd554c98..c32439683d83 100644
> --- a/OvmfPkg/PlatformDxe/Platform.c
> +++ b/OvmfPkg/PlatformDxe/Platform.c
> @@ -279,7 +279,7 @@ ExtractConfig (
> //
> ConfigRequestHdr = HiiConstructConfigHdr (
> &gOvmfPlatformConfigGuid,
> - mVariableName,
> + mHiiFormName,
> mImageHandle
The code change LGTM.
Acked-by: Sunil V L <sunilvl@ventanamicro.com>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#107618): https://edk2.groups.io/g/devel/message/107618
Mute This Topic: https://groups.io/mt/100142143/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [edk2-devel] [PATCH 1/1] OvmfPkg:Fix Hii form name mismatch with EFI variable
2023-08-07 12:02 ` Sunil V L
@ 2023-08-08 1:44 ` Wang, Yin
0 siblings, 0 replies; 6+ messages in thread
From: Wang, Yin @ 2023-08-08 1:44 UTC (permalink / raw)
To: Sunil V L, devel@edk2.groups.io
Hi Sunil,
I will be providing a detailed update soon, along with a new version. Thank you for your careful review and comments.
Best regards,
Yin
-----Original Message-----
From: Sunil V L <sunilvl@ventanamicro.com>
Sent: Monday, August 7, 2023 8:02 PM
To: devel@edk2.groups.io; Wang, Yin <yin.wang@intel.com>
Subject: Re: [edk2-devel] [PATCH 1/1] OvmfPkg:Fix Hii form name mismatch with EFI variable
Hi Wang,
Thanks for the patch!
It will be good if you add versions to the patch subject. Even if you go back to v1, better to send as v3 with the change log.
On Fri, Jul 14, 2023 at 04:59:48PM +0800, Wang, Yin wrote:
> Onemore need change beside fixs:16acacf24c ("OvmfPkg: fix
> PlatformConfig")
fixs -> Fixes? Why don't we need "Fixes:" tag to either aefcc91805fd or 16acacf24c2c?
I think it is better to have commit message with exact error or information on what is the issue instead of just pointing to another commit.
> Find by sct case:ExtractConfigConformance fail.
>
> Signed-off-by: Yin Wang <yin.wang@intel.com>
> ---
> OvmfPkg/PlatformDxe/Platform.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/OvmfPkg/PlatformDxe/Platform.c
> b/OvmfPkg/PlatformDxe/Platform.c index cd3bfd554c98..c32439683d83
> 100644
> --- a/OvmfPkg/PlatformDxe/Platform.c
> +++ b/OvmfPkg/PlatformDxe/Platform.c
> @@ -279,7 +279,7 @@ ExtractConfig (
> //
> ConfigRequestHdr = HiiConstructConfigHdr (
> &gOvmfPlatformConfigGuid,
> - mVariableName,
> + mHiiFormName,
> mImageHandle
The code change LGTM.
Acked-by: Sunil V L <sunilvl@ventanamicro.com>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#107628): https://edk2.groups.io/g/devel/message/107628
Mute This Topic: https://groups.io/mt/100142143/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
^ permalink raw reply [flat|nested] 6+ messages in thread