On 15 July 2015 at 08:26, Gao, Liming <liming....@intel.com> wrote: > Ard: > From compatible view, I don't prefer to make this change. The driver in > other EDKII package may use this PCD. After this change, they will build > break. > > I suggest to keep this PCD, and update its comment to include new type. Its > comment can highlight this type is not defined in UEFI specification. >
OK, that is also fine, Thanks, Ard. > -----Original Message----- > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Tuesday, July 14, 2015 6:36 PM > To: edk2-devel@lists.sourceforge.net; heyi....@linaro.org; ler...@redhat.com; > Tian, Feng; Gao, Liming > Cc: roy.fr...@linaro.org; Ard Biesheuvel > Subject: [PATCH 2/2] MdePkg: remove PcdDefaultTerminalType > > The PCD gEfiMdePkgTokenSpaceGuid.PcdDefaultTerminalType is not used anywhere > in MdePkg itself, and with the introduction of TTYTERM, it may now assume > values that are not defined in MdePkg either. > > So now that we moved it to MdeModulePkg, remove this one that is now unused. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Ard Biesheuvel <ard.biesheu...@linaro.org> > --- > MdePkg/MdePkg.dec | 9 --------- > 1 file changed, 9 deletions(-) > > diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec index > bda6550ed6c6..026a94987bef 100644 > --- a/MdePkg/MdePkg.dec > +++ b/MdePkg/MdePkg.dec > @@ -2007,15 +2007,6 @@ [PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, > PcdsDynamicEx] > # @ValidRange 0x80000001 | 0 - 3 > gEfiMdePkgTokenSpaceGuid.PcdUartDefaultStopBits|1|UINT8|0x00000023 > > - ## Indicates the usable type of terminal.<BR><BR> > - # 0 - PCANSI<BR> > - # 1 - VT100<BR> > - # 2 - VT100+<BR> > - # 3 - UTF8<BR> > - # @Prompt Default Terminal Type. > - # @ValidRange 0x80000001 | 0 - 3 > - gEfiMdePkgTokenSpaceGuid.PcdDefaultTerminalType|0|UINT8|0x00000024 > - > ## Error level for hardware recorder. > # If value 0, platform does not support feature of hardware error record. > # @Prompt Error Level For Hardware Recorder > -- > 1.9.1 > ------------------------------------------------------------------------------ Don't Limit Your Business. Reach for the Cloud. GigeNET's Cloud Solutions provide you with the tools and support that you need to offload your IT needs and focus on growing your business. Configured For All Businesses. Start Your Cloud Today. https://www.gigenetcloud.com/ _______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel