Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2023-03-21 Thread Ni, Ray
. Thanks, Ray From: Sean Rhodes Sent: Monday, March 20, 2023 8:47 PM To: Ni, Ray Cc: Tan, Lean Sheng ; devel@edk2.groups.io; Kinney, Michael D ; Gao, Zhichao ; Wang, Jian J ; Gao, Liming Subject: Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo Hi

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2023-03-20 Thread Sean Rhodes
gt; > > Perhaps a single PCD that is a enum of logo locations. Default 0x00 can > be EDK II default that is centered in the display. 0x01 can be BGRT. > Leaves from for more if there are additional public standard logo locations. > > > > Mike > > > > > > *Fr

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2023-03-20 Thread Ni, Ray
0, 2023 4:12 PM To: devel@edk2.groups.io; Ni, Ray ; Kinney, Michael D Cc: Rhodes, Sean ; Gao, Zhichao ; Wang, Jian J ; Gao, Liming Subject: Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo Hi Ray, Any feedback per Mic feedback? On 15. Mar 2023, at 1

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2023-03-20 Thread Sheng Lean Tan
tems> Cc: Gao, Zhichao <zhichao@intel.com>; Wang, Jian J <jian.j.w...@intel.com>; Gao, Liming <gaolim...@byosoft.com.cn> Subject: Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo   Are you suggesting that the exiting logic be updated

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2023-03-15 Thread Michael D Kinney
From: Ni, Ray Sent: Wednesday, March 15, 2023 2:24 AM To: devel@edk2.groups.io; Rhodes, Sean Cc: Kinney, Michael D ; Gao, Zhichao ; Wang, Jian J ; Gao, Liming Subject: RE: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo What’s the meaning of “have both

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2023-03-15 Thread Ni, Ray
.groups.io; Kinney, Michael D ; Gao, Zhichao ; Wang, Jian J ; Gao, Liming Subject: Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo Hi Ray > You can return a carefully-calculated X/Y value to make the > logo at MS preferred position.

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2023-03-15 Thread Sheng Lean Tan
devel@edk2.groups.io; Rhodes, Sean Cc: Gao, Zhichao <zhichao@intel.com>; Wang, Jian J <jian.j.w...@intel.com>; Gao, Liming <gaolim...@byosoft.com.cn> Subject: Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo   Are you sugge

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2023-03-13 Thread Sheng Lean Tan
evel@edk2.groups.io * On Behalf Of *Ni, >> Ray >> *Sent:* Wednesday, October 26, 2022 10:32 AM >> *To:* Kinney, Michael D ; >> devel@edk2.groups.io; Rhodes, Sean >> *Cc:* Gao, Zhichao ; Wang, Jian J < >> jian.j.w...@intel.com>; Gao, Liming >> *Sub

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2023-03-10 Thread Sean Rhodes
*Ni, > Ray > *Sent:* Wednesday, October 26, 2022 10:32 AM > *To:* Kinney, Michael D ; devel@edk2.groups.io; > Rhodes, Sean > *Cc:* Gao, Zhichao ; Wang, Jian J < > jian.j.w...@intel.com>; Gao, Liming > *Subject:* Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2023-03-08 Thread Ni, Ray
Sent: Wednesday, October 26, 2022 10:32 AM To: Kinney, Michael D ; devel@edk2.groups.io; Rhodes, Sean Cc: Gao, Zhichao ; Wang, Jian J ; Gao, Liming Subject: Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo Are you suggesting that the exiting logic

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2022-10-26 Thread Michael D Kinney
1:30 PM To: Kinney, Michael D Cc: Ni, Ray ; devel@edk2.groups.io; Gao, Zhichao ; Wang, Jian J ; Gao, Liming Subject: Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo Hi Mike This patch is being upstreamed from the coreboot fork; about 70

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2022-10-26 Thread Sean Rhodes
nney, Michael D ; devel@edk2.groups.io; > Rhodes, Sean > *Cc:* Gao, Zhichao ; Wang, Jian J < > jian.j.w...@intel.com>; Gao, Liming > *Subject:* RE: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to > control the position of the Logo > > > > Are you suggestin

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2022-10-25 Thread Michael D Kinney
: Gao, Zhichao ; Wang, Jian J ; Gao, Liming Subject: RE: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo Are you suggesting that the exiting logic be updated for this use case without adding a new enum? * yes. From: Kinney, Michael D

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2022-10-25 Thread Ni, Ray
Subject: RE: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo Ray, Are you suggesting that the exiting logic be updated for this use case without adding a new enum? Sean, can you provide a revised patch that does this? Thanks, Mike From: devel@edk2

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2022-10-25 Thread Sean Rhodes
Hi Mike/Ray Thanks - so you mean something like https://github.com/tianocore/edk2/pull/3528? ( https://github.com/tianocore/edk2/pull/3528 ) (Just for example) If not, I'm not sure how to control it without the PCD? Sean -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2022-10-25 Thread Michael D Kinney
, Sean Cc: Gao, Zhichao ; Wang, Jian J ; Gao, Liming Subject: Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo I need a reason of adding EdkiiPlatformLogoDisplayAttributeMicrosoftRecommended. In my opinion, without adding this new enum value, it’s

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2022-10-25 Thread Ni, Ray
: devel@edk2.groups.io; Gao, Zhichao ; Wang, Jian J ; Gao, Liming Subject: Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo Hi Ray Where would you suggest this code goes? edk2 should support both Microsoft recommended and "normal". The orig

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2022-10-25 Thread Sean Rhodes
Hi Ray Where would you suggest this code goes? edk2 should support both Microsoft recommended and "normal". The original patch handled this well. Thanks Sean On Mon, 10 Oct 2022 at 10:25, Ni, Ray wrote: > The logic I shared below is from the LogoDxe driver which produces >

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2022-10-10 Thread Sean Rhodes
+ Matt @Matt DeVillier Does Ray's code work for you? On Mon, 10 Oct 2022 at 10:25, Ni, Ray wrote: > The logic I shared below is from the LogoDxe driver which produces > EDKII_PLATFORM_LOGO_PROTOCOL. > > This driver should know the image size and it can account for the image > size. > > > >

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2022-10-10 Thread Ni, Ray
The logic I shared below is from the LogoDxe driver which produces EDKII_PLATFORM_LOGO_PROTOCOL. This driver should know the image size and it can account for the image size. Thanks, Ray From: Sean Rhodes Sent: Monday, October 10, 2022 4:51 PM To: Ni, Ray Cc: devel@edk2.groups.io; Gao,

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2022-10-10 Thread Sean Rhodes
Hi Ray Thank you, it does, and I think it will work for most splash images. However, the way it's written in my patch accounts for the Image size. This will handle splash images that are equal to, or larger than the resolution of the display. Thanks Sean On Sat, 8 Oct 2022 at 03:02, Ni, Ray

Re: [edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2022-10-07 Thread Ni, Ray
Sean, I remember that I evaluated the BGRT requirement when designing the PlatformLogo protocol. So, I went back to got the code I wrote long time ago as below. I didn't try to understand them now. Does it make sense to you? Status = gBS->HandleProtocol (gST->ConsoleOutHandle, , (VOID **)

[edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2022-09-26 Thread Sean Rhodes
When set to true, the Logo is positioned according to the BGRT specification, 38.2% from the top of the screen. When set to false, no behaviour is changed and the logo is positioned centrally. Cc: Zhichao Gao Cc: Ray Ni Cc: Jian J Wang Cc: Liming Gao Signed-off-by: Sean Rhodes ---

[edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2022-08-05 Thread Sean Rhodes
When set to true, the Logo is positioned according to the BGRT specification, 38.2% from the top of the screen. When set to false, no behaviour is changed and the logo is positioned centrally. Cc: Zhichao Gao Cc: Ray Ni Cc: Jian J Wang Cc: Liming Gao Signed-off-by: Sean Rhodes ---

[edk2-devel] [PATCH 2/3] MdeModulePkg/Logo: Add a PCD to control the position of the Logo

2022-07-26 Thread Sean Rhodes
When set to true, the Logo is positioned according to the BGRT specification, 38.2% from the top of the screen. When set to false, no behaviour is changed and the logo is positioned centrally. Cc: Zhichao Gao Cc: Ray Ni Cc: Jian J Wang Cc: Liming Gao Signed-off-by: Sean Rhodes ---