From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lj1-f174.google.com (mail-lj1-f174.google.com [209.85.208.174]) by mx.groups.io with SMTP id smtpd.web09.17129.1664788957326743917 for ; Mon, 03 Oct 2022 02:22:37 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@9elements.com header.s=google header.b=VqXw0iU1; spf=pass (domain: 9elements.com, ip: 209.85.208.174, mailfrom: sheng.tan@9elements.com) Received: by mail-lj1-f174.google.com with SMTP id p5so11048589ljc.13 for ; Mon, 03 Oct 2022 02:22:36 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=9elements.com; s=google; h=mime-version:content-language:accept-language:in-reply-to :references:message-id:date:thread-index:thread-topic:subject:cc:to :from:from:to:cc:subject:date; bh=kqCvVe5piaCh4upGAjWbakxXQqGi8vm/qH1BuUdsNpY=; b=VqXw0iU1WQMqtzDzjCHRYOk9eKuUrNXEIgSgmPsIkbvqIR9hsDZwEkjpXCnTOhSRGA tX0mMwZnpfCO/nxn379OQ9hB6z1kg4aVUslbJRvXgWWVscF6/SBCgcDdVMCBjU6zDmX5 364uQWad7rQltWdAjI2/1ofkjtah8QEdBW152ukhjk5QxLVFl2PYJ5dAsd+Xl9WVWKaZ xuhF4744TNeSPblgBGK4hxaIFCzP0OFcDm2PJ89IjV1c02N43OjucphKjgbM81kXsKiH x2w15qEpuRapmYpJ1fQtoOw+UcYDz2qHyPdZlmGeTY1i/h9Dy3KiO8csIzuMMsX0wIcl nyEw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=mime-version:content-language:accept-language:in-reply-to :references:message-id:date:thread-index:thread-topic:subject:cc:to :from:x-gm-message-state:from:to:cc:subject:date; bh=kqCvVe5piaCh4upGAjWbakxXQqGi8vm/qH1BuUdsNpY=; b=H5GGd8Qn+BCxfFtZLaqgh4MXC8cjVwRN03gn22ATlad/XPuUbegoaDvkAWTyXEZrGw 2D7TpFlInoETq5oDXcYjmndmpiZ5+kxD0m1/LjThzqGLBZA8jprFnS1+cjyES638OUER RF9KAX6AC1bUbPbjWQ0X+55f+8ir/UIbqGMuu0vrE+eEEqTWCOTbOd2hDfwiClTEA+LE 5ZWso0IgRD1vZm4q8tTR0ko3HxaEYxj+M+Geuj8x+r0XkRQtJP9Mb+gj8GwwtNUpc8Lw YtFg1qpOlf/QnHci6Ek8QF/30hQEpchXDcwV7QXe8rWSpfbSVfU4BrzDzyPzW/iK4teD g7/w== X-Gm-Message-State: ACrzQf3NuT7Ja+tWCFksQNCXu8bJH2Fr6N+8Vo29sTqc99ba2Wg8XUrg IJ+KlReLfMVUyvJ1LVLjpaRaRf9zu4frZheh X-Google-Smtp-Source: AMsMyM4tOuGcJ+4Hv2uYTJsl5prgEfI2w5R+4d1/oIIYhnCdW02DgmE0FX/9B1beZ/fYYK7E9ld/pQ== X-Received: by 2002:a2e:bf06:0:b0:26d:cec9:3920 with SMTP id c6-20020a2ebf06000000b0026dcec93920mr2421619ljr.262.1664788954971; Mon, 03 Oct 2022 02:22:34 -0700 (PDT) Return-Path: Received: from GV2P195MB2090.EURP195.PROD.OUTLOOK.COM ([2603:1026:c11:c37::5]) by smtp.gmail.com with ESMTPSA id h19-20020a056512221300b004a054535826sm1385420lfu.24.2022.10.03.02.22.34 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Mon, 03 Oct 2022 02:22:34 -0700 (PDT) From: "Sheng Lean Tan" To: "devel@edk2.groups.io" , "sean@starlabs.systems" CC: Sean Rhodes , Zhichao Gao , Ray Ni , Jian J Wang , Liming Gao , "Dong, Guo" , "Guo, Gua" Subject: Re: [edk2-devel] [PATCH 1/3] MdeModulePkg/BootLogoLib: Add option to follow Microsoft Recommendations Thread-Topic: [edk2-devel] [PATCH 1/3] MdeModulePkg/BootLogoLib: Add option to follow Microsoft Recommendations Thread-Index: AW45OTcwbalRE96pGc2GiiM1dVLf4tU7n2dP X-MS-Exchange-MessageSentRepresentingType: 1 Date: Mon, 3 Oct 2022 09:22:32 +0000 Message-ID: References: <9c3d2f03a7f021c5044301f7b4d869b70e8990e0.1664179796.git.sean@starlabs.systems> In-Reply-To: <9c3d2f03a7f021c5044301f7b4d869b70e8990e0.1664179796.git.sean@starlabs.systems> Accept-Language: en-GB, en-US X-MS-Has-Attach: X-MS-Exchange-Organization-SCL: -1 X-MS-TNEF-Correlator: X-MS-Exchange-Organization-RecordReviewCfmType: 0 MIME-Version: 1.0 Content-Language: en-GB Content-Type: multipart/alternative; boundary="_000_GV2P195MB2090CAB83A29E9F86BB0FBD8FC5B9GV2P195MB2090EURP_" --_000_GV2P195MB2090CAB83A29E9F86BB0FBD8FC5B9GV2P195MB2090EURP_ Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Hi Liming, Mind to help to take a look at these 2 patches? Sean has updated the initia= l patches per your recommendation: https://edk2.groups.io/g/devel/message/94318?p=3D%2C%2C%2C20%2C0%2C0%2C0%3A= %3ACreated%2C%2Cboot+logo+sean%2C20%2C2%2C0%2C93922542 https://edk2.groups.io/g/devel/message/94320?p=3D%2C%2C%2C20%2C0%2C0%2C0%3A= %3ACreated%2C%2Cboot+logo+sean%2C20%2C2%2C0%2C93922544 Previous feedback: https://edk2.groups.io/g/devel/message/92385?p=3D%2C%2C%2C20%2C0%2C0%2C0%3A= %3ACreated%2C%2Cboot+logo+sean%2C20%2C2%2C0%2C92974182 https://edk2.groups.io/g/devel/message/92146?p=3D%2C%2C%2C20%2C0%2C0%2C0%3A= %3ACreated%2C%2Cboot+logo+sean%2C20%2C2%2C0%2C92830014 Thanks, Sheng From: devel@edk2.groups.io on behalf of Sean Rhodes = Date: Monday, 26. September 2022 at 10:10 To: devel@edk2.groups.io Cc: Sean Rhodes , Zhichao Gao , Ray Ni , Jian J Wang , Liming G= ao Subject: [edk2-devel] [PATCH 1/3] MdeModulePkg/BootLogoLib: Add option to f= ollow Microsoft Recommendations Add an option to position the logo 38.2% from the top of the screen, which follows the recommendations from Microsoft. These can be found here: https://docs.microsoft.com/en-us/windows-hardware/drivers/bringup/boot-scre= en-components Cc: Zhichao Gao Cc: Ray Ni Cc: Jian J Wang Cc: Liming Gao Signed-off-by: Sean Rhodes --- MdeModulePkg/Include/Protocol/PlatformLogo.h | 3 ++- MdeModulePkg/Library/BootLogoLib/BootLogoLib.c | 7 ++++--- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/MdeModulePkg/Include/Protocol/PlatformLogo.h b/MdeModulePkg/In= clude/Protocol/PlatformLogo.h index 08e1dc35a4..b24d7d5b79 100644 --- a/MdeModulePkg/Include/Protocol/PlatformLogo.h +++ b/MdeModulePkg/Include/Protocol/PlatformLogo.h @@ -29,7 +29,8 @@ typedef enum { EdkiiPlatformLogoDisplayAttributeCenterBottom, EdkiiPlatformLogoDisplayAttributeLeftBottom, EdkiiPlatformLogoDisplayAttributeCenterLeft, - EdkiiPlatformLogoDisplayAttributeCenter + EdkiiPlatformLogoDisplayAttributeCenter, + EdkiiPlatformLogoDisplayAttributeMicrosoftRecommended } EDKII_PLATFORM_LOGO_DISPLAY_ATTRIBUTE; /** diff --git a/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c b/MdeModulePkg/= Library/BootLogoLib/BootLogoLib.c index 478ec2d40e..9065e5281b 100644 --- a/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c +++ b/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c @@ -169,7 +169,6 @@ BootLogoEnableLogo ( DestX =3D SizeOfX - Image.Width; DestY =3D 0; break; - case EdkiiPlatformLogoDisplayAttributeCenterLeft: DestX =3D 0; DestY =3D (SizeOfY - Image.Height) / 2; @@ -182,7 +181,6 @@ BootLogoEnableLogo ( DestX =3D SizeOfX - Image.Width; DestY =3D (SizeOfY - Image.Height) / 2; break; - case EdkiiPlatformLogoDisplayAttributeLeftBottom: DestX =3D 0; DestY =3D SizeOfY - Image.Height; @@ -195,7 +193,10 @@ BootLogoEnableLogo ( DestX =3D SizeOfX - Image.Width; DestY =3D SizeOfY - Image.Height; break; - + case EdkiiPlatformLogoDisplayAttributeMicrosoftRecommended: + DestX =3D (SizeOfX - Image.Width) / 2; + DestY =3D (SizeOfY * 382) / 1000 - Image.Height / 2; + break; default: ASSERT (FALSE); continue; -- 2.34.1 -=3D-=3D-=3D-=3D-=3D-=3D Groups.io Links: You receive all messages sent to this group. View/Reply Online (#94318): https://edk2.groups.io/g/devel/message/94318 Mute This Topic: https://groups.io/mt/93922542/6757431 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [sheng.tan@9elements.com] -=3D-=3D-=3D-=3D-=3D-=3D --_000_GV2P195MB2090CAB83A29E9F86BB0FBD8FC5B9GV2P195MB2090EURP_ Content-Type: text/html; charset="us-ascii" Content-Transfer-Encoding: quoted-printable

Hi L= iming,

Mind= to help to take a look at these 2 patches? Sean has updated the initial pa= tches per your recommendation:

https://edk2= .groups.io/g/devel/message/94318?p=3D%2C%2C%2C20%2C0%2C0%2C0%3A%3ACreated%2= C%2Cboot+logo+sean%2C20%2C2%2C0%2C93922542

 

https://edk2= .groups.io/g/devel/message/94320?p=3D%2C%2C%2C20%2C0%2C0%2C0%3A%3ACreated%2= C%2Cboot+logo+sean%2C20%2C2%2C0%2C93922544

 

Prev= ious feedback:

https://edk2= .groups.io/g/devel/message/92385?p=3D%2C%2C%2C20%2C0%2C0%2C0%3A%3ACreated%2= C%2Cboot+logo+sean%2C20%2C2%2C0%2C92974182

 

https://edk2= .groups.io/g/devel/message/92146?p=3D%2C%2C%2C20%2C0%2C0%2C0%3A%3ACreated%2= C%2Cboot+logo+sean%2C20%2C2%2C0%2C92830014

 

Than= ks,

Shen= g

 

From: devel@edk2.groups.i= o <devel@edk2.groups.io> on behalf of Sean Rhodes <sean@starlabs.s= ystems>
Date: Monday, 26. September 2022 at 10:10
To: devel@edk2.groups.io <devel@edk2.groups.io>
Cc: Sean Rhodes <sean@starlabs.systems>, Zhichao Gao <zhich= ao.gao@intel.com>, Ray Ni <ray.ni@intel.com>, Jian J Wang <jian= .j.wang@intel.com>, Liming Gao <gaoliming@byosoft.com.cn>
Subject: [edk2-devel] [PATCH 1/3] MdeModulePkg/BootLogoLib: Add opti= on to follow Microsoft Recommendations

Add an option to position the logo 38.2% from the top of the sc= reen,
which follows the recommendations from Microsoft. These can be found
here:
https://docs.microsoft.com/en-us/windows-hardware= /drivers/bringup/boot-screen-components

Cc: Zhichao Gao <zhichao.gao@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Signed-off-by: Sean Rhodes <sean@starlabs.systems>
---
 MdeModulePkg/Include/Protocol/PlatformLogo.h   | 3 ++-
 MdeModulePkg/Library/BootLogoLib/BootLogoLib.c | 7 ++++---
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/MdeModulePkg/Include/Protocol/PlatformLogo.h b/MdeModulePkg/In= clude/Protocol/PlatformLogo.h
index 08e1dc35a4..b24d7d5b79 100644
--- a/MdeModulePkg/Include/Protocol/PlatformLogo.h
+++ b/MdeModulePkg/Include/Protocol/PlatformLogo.h
@@ -29,7 +29,8 @@ typedef enum {
   EdkiiPlatformLogoDisplayAttributeCenterBottom,

   EdkiiPlatformLogoDisplayAttributeLeftBottom,

   EdkiiPlatformLogoDisplayAttributeCenterLeft,

-  EdkiiPlatformLogoDisplayAttributeCenter

+  EdkiiPlatformLogoDisplayAttributeCenter,

+  EdkiiPlatformLogoDisplayAttributeMicrosoftRecommended

 } EDKII_PLATFORM_LOGO_DISPLAY_ATTRIBUTE;

 

 /**

diff --git a/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c b/MdeModulePkg/= Library/BootLogoLib/BootLogoLib.c
index 478ec2d40e..9065e5281b 100644
--- a/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c
+++ b/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c
@@ -169,7 +169,6 @@ BootLogoEnableLogo (
         DestX =3D SizeOfX - Image.= Width;

         DestY =3D 0;

         break;

-

       case EdkiiPlatformLogoDisplayAttribute= CenterLeft:

         DestX =3D 0;

         DestY =3D (SizeOfY - Image= .Height) / 2;

@@ -182,7 +181,6 @@ BootLogoEnableLogo (
         DestX =3D SizeOfX - Image.= Width;

         DestY =3D (SizeOfY - Image= .Height) / 2;

         break;

-

       case EdkiiPlatformLogoDisplayAttribute= LeftBottom:

         DestX =3D 0;

         DestY =3D SizeOfY - Image.= Height;

@@ -195,7 +193,10 @@ BootLogoEnableLogo (
         DestX =3D SizeOfX - Image.= Width;

         DestY =3D SizeOfY - Image.= Height;

         break;

-

+      case EdkiiPlatformLogoDisplayAttributeMicro= softRecommended:

+        DestX =3D (SizeOfX - Image.Widt= h) / 2;

+        DestY =3D (SizeOfY * 382) / 100= 0 - Image.Height / 2;

+        break;

       default:

         ASSERT (FALSE);

         continue;

--
2.34.1



-=3D-=3D-=3D-=3D-=3D-=3D
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94318): https://edk2.groups.io/g/devel/message/94318
Mute This Topic: https://= groups.io/mt/93922542/6757431
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.= groups.io/g/devel/unsub [sheng.tan@9elements.com]
-=3D-=3D-=3D-=3D-=3D-=3D

--_000_GV2P195MB2090CAB83A29E9F86BB0FBD8FC5B9GV2P195MB2090EURP_--