Re: [patch 1/5] MdePkg: Correct the string expression of UTF8 vendor device path

["Ni, Ruiyu" <ruiyu.ni@Intel.com>]

On 10/8/2018 11:31 AM, Dandan Bi wrote:
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1225
> 
> According to UEFI spec, the string expression of UTF8 vendor
> device node should be displayed as: VenUtf8(). Current code
> display it as: VenUft8() by mistake when convert device
> path node to text.
> 
> This commit is to fix this bug.
> 
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Dandan Bi <dandan.bi@intel.com>
> ---
>   MdePkg/Library/UefiDevicePathLib/DevicePathToText.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/MdePkg/Library/UefiDevicePathLib/DevicePathToText.c b/MdePkg/Library/UefiDevicePathLib/DevicePathToText.c
> index 7d8d304f6f..85f5e97131 100644
> --- a/MdePkg/Library/UefiDevicePathLib/DevicePathToText.c
> +++ b/MdePkg/Library/UefiDevicePathLib/DevicePathToText.c
> @@ -193,11 +193,11 @@ DevPathToTextVendor (
>           return ;
>         } else if (CompareGuid (&Vendor->Guid, &gEfiVT100PlusGuid)) {
>           UefiDevicePathLibCatPrint (Str, L"VenVt100Plus()");
>           return ;
>         } else if (CompareGuid (&Vendor->Guid, &gEfiVTUTF8Guid)) {
> -        UefiDevicePathLibCatPrint (Str, L"VenUft8()");
> +        UefiDevicePathLibCatPrint (Str, L"VenUtf8()");
>           return ;
>         } else if (CompareGuid (&Vendor->Guid, &gEfiUartDevicePathGuid)) {
>           FlowControlMap = (((UART_FLOW_CONTROL_DEVICE_PATH *) Vendor)->FlowControlMap);
>           switch (FlowControlMap & 0x00000003) {
>           case 0:
> 
Reviewed-by: Ruiyu Ni <ruiyu.ni@intel.com>

-- 
Thanks,
Ray