* [PATCH for-edk2-stable201905] Revert "EmulatorPkg: don't display the cpu current speed"
@ 2019-06-04 18:58 Laszlo Ersek
2019-06-05 0:23 ` [edk2-devel] " Liming Gao
0 siblings, 1 reply; 3+ messages in thread
From: Laszlo Ersek @ 2019-06-04 18:58 UTC (permalink / raw)
To: edk2-devel-groups-io
Cc: Ard Biesheuvel, Hao A Wu, Jian J Wang, Liming Gao, Ray Ni,
Star Zeng
This reverts commit 7cea4d71a8a87a93924a07ab32348332f5881ef9.
Said commit was not suitable for pushing during the edk2-stable201905 hard
feature freeze; it was pushed only by mistake. The subject line referenced
EmulatorPkg, but the patch changed MdeModulePkg/UiApp, regressing the
display of the CPU speed from SMBIOS in multiple platforms.
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Hao A Wu <hao.a.wu@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1877
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
---
Notes:
Repo: https://github.com/lersek/edk2.git
Branch: revert_cpu_speed_hide
MdeModulePkg/Application/UiApp/FrontPage.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/MdeModulePkg/Application/UiApp/FrontPage.c b/MdeModulePkg/Application/UiApp/FrontPage.c
index fded7634062a..4b95cccb5cf5 100644
--- a/MdeModulePkg/Application/UiApp/FrontPage.c
+++ b/MdeModulePkg/Application/UiApp/FrontPage.c
@@ -621,6 +621,11 @@ UpdateFrontPageBannerStrings (
HiiSetString (gFrontPagePrivate.HiiHandle, STRING_TOKEN (STR_FRONT_PAGE_CPU_MODEL), NewString, NULL);
FreePool (NewString);
+ ConvertProcessorToString(Type4Record->CurrentSpeed, 6, &NewString);
+ UiCustomizeFrontPageBanner (2, FALSE, &NewString);
+ HiiSetString (gFrontPagePrivate.HiiHandle, STRING_TOKEN (STR_FRONT_PAGE_CPU_SPEED), NewString, NULL);
+ FreePool (NewString);
+
FoundCpu = TRUE;
}
}
--
2.19.1.3.g30247aa5d201
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [edk2-devel] [PATCH for-edk2-stable201905] Revert "EmulatorPkg: don't display the cpu current speed"
2019-06-04 18:58 [PATCH for-edk2-stable201905] Revert "EmulatorPkg: don't display the cpu current speed" Laszlo Ersek
@ 2019-06-05 0:23 ` Liming Gao
2019-06-05 8:12 ` Laszlo Ersek
0 siblings, 1 reply; 3+ messages in thread
From: Liming Gao @ 2019-06-05 0:23 UTC (permalink / raw)
To: devel@edk2.groups.io, lersek@redhat.com
Cc: Ard Biesheuvel, Wu, Hao A, Wang, Jian J, Ni, Ray, Zeng, Star
Reviewed-by: Liming Gao <liming.gao@intel.com>
>-----Original Message-----
>From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
>Laszlo Ersek
>Sent: Wednesday, June 05, 2019 2:59 AM
>To: edk2-devel-groups-io <devel@edk2.groups.io>
>Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>; Wu, Hao A
><hao.a.wu@intel.com>; Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
><liming.gao@intel.com>; Ni, Ray <ray.ni@intel.com>; Zeng, Star
><star.zeng@intel.com>
>Subject: [edk2-devel] [PATCH for-edk2-stable201905] Revert "EmulatorPkg:
>don't display the cpu current speed"
>
>This reverts commit 7cea4d71a8a87a93924a07ab32348332f5881ef9.
>
>Said commit was not suitable for pushing during the edk2-stable201905 hard
>feature freeze; it was pushed only by mistake. The subject line referenced
>EmulatorPkg, but the patch changed MdeModulePkg/UiApp, regressing the
>display of the CPU speed from SMBIOS in multiple platforms.
>
>Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>Cc: Hao A Wu <hao.a.wu@intel.com>
>Cc: Jian J Wang <jian.j.wang@intel.com>
>Cc: Liming Gao <liming.gao@intel.com>
>Cc: Ray Ni <ray.ni@intel.com>
>Cc: Star Zeng <star.zeng@intel.com>
>Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1877
>Signed-off-by: Laszlo Ersek <lersek@redhat.com>
>---
>
>Notes:
> Repo: https://github.com/lersek/edk2.git
> Branch: revert_cpu_speed_hide
>
> MdeModulePkg/Application/UiApp/FrontPage.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
>diff --git a/MdeModulePkg/Application/UiApp/FrontPage.c
>b/MdeModulePkg/Application/UiApp/FrontPage.c
>index fded7634062a..4b95cccb5cf5 100644
>--- a/MdeModulePkg/Application/UiApp/FrontPage.c
>+++ b/MdeModulePkg/Application/UiApp/FrontPage.c
>@@ -621,6 +621,11 @@ UpdateFrontPageBannerStrings (
> HiiSetString (gFrontPagePrivate.HiiHandle, STRING_TOKEN
>(STR_FRONT_PAGE_CPU_MODEL), NewString, NULL);
> FreePool (NewString);
>
>+ ConvertProcessorToString(Type4Record->CurrentSpeed, 6, &NewString);
>+ UiCustomizeFrontPageBanner (2, FALSE, &NewString);
>+ HiiSetString (gFrontPagePrivate.HiiHandle, STRING_TOKEN
>(STR_FRONT_PAGE_CPU_SPEED), NewString, NULL);
>+ FreePool (NewString);
>+
> FoundCpu = TRUE;
> }
> }
>--
>2.19.1.3.g30247aa5d201
>
>
>-=-=-=-=-=-=
>Groups.io Links: You receive all messages sent to this group.
>
>View/Reply Online (#41886): https://edk2.groups.io/g/devel/message/41886
>Mute This Topic: https://groups.io/mt/31929712/1759384
>Group Owner: devel+owner@edk2.groups.io
>Unsubscribe: https://edk2.groups.io/g/devel/unsub [liming.gao@intel.com]
>-=-=-=-=-=-=
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [edk2-devel] [PATCH for-edk2-stable201905] Revert "EmulatorPkg: don't display the cpu current speed"
2019-06-05 0:23 ` [edk2-devel] " Liming Gao
@ 2019-06-05 8:12 ` Laszlo Ersek
0 siblings, 0 replies; 3+ messages in thread
From: Laszlo Ersek @ 2019-06-05 8:12 UTC (permalink / raw)
To: devel, liming.gao
Cc: Ard Biesheuvel, Wu, Hao A, Wang, Jian J, Ni, Ray, Zeng, Star
On 06/05/19 02:23, Liming Gao wrote:
> Reviewed-by: Liming Gao <liming.gao@intel.com>
Thanks, patch pushed as commit be689ecc93e5.
Laszlo
>> -----Original Message-----
>> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
>> Laszlo Ersek
>> Sent: Wednesday, June 05, 2019 2:59 AM
>> To: edk2-devel-groups-io <devel@edk2.groups.io>
>> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>; Wu, Hao A
>> <hao.a.wu@intel.com>; Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
>> <liming.gao@intel.com>; Ni, Ray <ray.ni@intel.com>; Zeng, Star
>> <star.zeng@intel.com>
>> Subject: [edk2-devel] [PATCH for-edk2-stable201905] Revert "EmulatorPkg:
>> don't display the cpu current speed"
>>
>> This reverts commit 7cea4d71a8a87a93924a07ab32348332f5881ef9.
>>
>> Said commit was not suitable for pushing during the edk2-stable201905 hard
>> feature freeze; it was pushed only by mistake. The subject line referenced
>> EmulatorPkg, but the patch changed MdeModulePkg/UiApp, regressing the
>> display of the CPU speed from SMBIOS in multiple platforms.
>>
>> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>> Cc: Hao A Wu <hao.a.wu@intel.com>
>> Cc: Jian J Wang <jian.j.wang@intel.com>
>> Cc: Liming Gao <liming.gao@intel.com>
>> Cc: Ray Ni <ray.ni@intel.com>
>> Cc: Star Zeng <star.zeng@intel.com>
>> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1877
>> Signed-off-by: Laszlo Ersek <lersek@redhat.com>
>> ---
>>
>> Notes:
>> Repo: https://github.com/lersek/edk2.git
>> Branch: revert_cpu_speed_hide
>>
>> MdeModulePkg/Application/UiApp/FrontPage.c | 5 +++++
>> 1 file changed, 5 insertions(+)
>>
>> diff --git a/MdeModulePkg/Application/UiApp/FrontPage.c
>> b/MdeModulePkg/Application/UiApp/FrontPage.c
>> index fded7634062a..4b95cccb5cf5 100644
>> --- a/MdeModulePkg/Application/UiApp/FrontPage.c
>> +++ b/MdeModulePkg/Application/UiApp/FrontPage.c
>> @@ -621,6 +621,11 @@ UpdateFrontPageBannerStrings (
>> HiiSetString (gFrontPagePrivate.HiiHandle, STRING_TOKEN
>> (STR_FRONT_PAGE_CPU_MODEL), NewString, NULL);
>> FreePool (NewString);
>>
>> + ConvertProcessorToString(Type4Record->CurrentSpeed, 6, &NewString);
>> + UiCustomizeFrontPageBanner (2, FALSE, &NewString);
>> + HiiSetString (gFrontPagePrivate.HiiHandle, STRING_TOKEN
>> (STR_FRONT_PAGE_CPU_SPEED), NewString, NULL);
>> + FreePool (NewString);
>> +
>> FoundCpu = TRUE;
>> }
>> }
>> --
>> 2.19.1.3.g30247aa5d201
>>
>>
>> -=-=-=-=-=-=
>> Groups.io Links: You receive all messages sent to this group.
>>
>> View/Reply Online (#41886): https://edk2.groups.io/g/devel/message/41886
>> Mute This Topic: https://groups.io/mt/31929712/1759384
>> Group Owner: devel+owner@edk2.groups.io
>> Unsubscribe: https://edk2.groups.io/g/devel/unsub [liming.gao@intel.com]
>> -=-=-=-=-=-=
>
>
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-06-05 8:13 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-06-04 18:58 [PATCH for-edk2-stable201905] Revert "EmulatorPkg: don't display the cpu current speed" Laszlo Ersek
2019-06-05 0:23 ` [edk2-devel] " Liming Gao
2019-06-05 8:12 ` Laszlo Ersek
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox