From mboxrd@z Thu Jan 1 00:00:00 1970 Subject: Re: [edk2-devel] [PATCH V2 2/2] MdeModulePkg/GraphicsConsoleDxe: Do not clean the screen To: Gao, Zhichao ,devel@edk2.groups.io From: "Sean" X-Originating-IP: 131.107.160.225 User-Agent: GROUPS.IO Web Poster MIME-Version: 1.0 Date: Thu, 25 Apr 2019 18:12:28 -0700 References: <3CE959C139B4C44DBEA1810E3AA6F9000B7C4343@SHSMSX101.ccr.corp.intel.com> In-Reply-To: <3CE959C139B4C44DBEA1810E3AA6F9000B7C4343@SHSMSX101.ccr.corp.intel.com> Message-ID: <7170.1556241148246604928@groups.io> Content-Type: multipart/alternative; boundary="ldRw6uTiLyAYd3m3AufV" --ldRw6uTiLyAYd3m3AufV Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Is there a branch where this code change can be reviewed?=C2=A0=C2=A0 The intent of bug 1412 was not to break spec alignment on the uefi defined= protocol and SetMode() function.=C2=A0 We had a proposed change that can b= e seen here.=C2=A0https://github.com/Microsoft/mu_basecore/pull/13/files=C2= = =A0 The idea was for new devices published, only clear the screen if the mode = needs to change.=C2=A0 Our reading of the UEFI spec didn't define how new d= evices needed to be added and didn't require the screen be cleared. --ldRw6uTiLyAYd3m3AufV Content-Type: text/html; charset="utf-8" Content-Transfer-Encoding: quoted-printable Is there a branch where this code change can be reviewed?  
=
The intent of bug 1412 was not to break spec alignment on the uefi de= fined protocol and SetMode() function.  We had a proposed change that = can be seen here. https://github.com/Microsoft/mu_basecore/pull/13/files 
Th= e idea was for new devices published, only clear the screen if the mode nee= ds to change.  Our reading of the UEFI spec didn't define how new devi= ces needed to be added and didn't require the screen be cleared.  = ; --ldRw6uTiLyAYd3m3AufV--