From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by mx.groups.io with SMTP id smtpd.web12.275.1594230392595084808 for ; Wed, 08 Jul 2020 10:46:32 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=gfrjUBrE; spf=pass (domain: intel.com, ip: 134.134.136.31, mailfrom: maurice.ma@intel.com) IronPort-SDR: i6MYmL0QFBJU945SAgoaq5/gK7ETehnH5Ts0aJTOqAH2FE14asuCp4UHs1FSG9ql5ouro2q6Fl SRpviW5MBFFg== X-IronPort-AV: E=McAfee;i="6000,8403,9676"; a="209405154" X-IronPort-AV: E=Sophos;i="5.75,328,1589266800"; d="scan'208";a="209405154" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 08 Jul 2020 10:46:31 -0700 IronPort-SDR: dRrtG6VzjjAKzqA3WRH0G7lAgEnpyLJViLuNrcCA+ZtUBQd6pXcQOqW+Zzd1mrTBnzod5mcuhf e5Nn0+rb6UTQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.75,328,1589266800"; d="scan'208";a="297788771" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by orsmga002.jf.intel.com with ESMTP; 08 Jul 2020 10:46:31 -0700 Received: from fmsmsx153.amr.corp.intel.com (10.18.125.6) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 8 Jul 2020 10:46:31 -0700 Received: from FMSEDG002.ED.cps.intel.com (10.1.192.134) by FMSMSX153.amr.corp.intel.com (10.18.125.6) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 8 Jul 2020 10:46:30 -0700 Received: from NAM04-CO1-obe.outbound.protection.outlook.com (104.47.45.56) by edgegateway.intel.com (192.55.55.69) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 8 Jul 2020 10:46:28 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=IOnqRgagBMs+vpEiMHuuPMREsIIwnrk+4Ao38ImaZEPKXAnLTodT+wFcEqfDsb/kcTenfWecG5CA22cVKBlxDbTMZKn0lUYmBVr3f7hIdMvw8SvKNNej9dUZqJJnRx3KiSgNQwACLZjSpv3tbmT3/2ApkUbPsTVpduraGg0RvdPwctPwOxqbyYGet5NikagPvDze4rZ9EmHtYNSs7vLuyEMeC4Qfzd10Hy6Ckd5Z5mytc2U7j4NSTw4kqvFgK4nZvF8fnqbgGrGSgd9orCrRJsQCS2AVWunlH0fefbfb/rIbJzK2Qy2Yj+lG6FfNaRQW4Ds8mUMM3dqK8fkI9V3hAQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=XLVLru5z04tHtF0dpeBKJr9mOTJIG59Sg2QeP60qnD4=; b=A5brAuzorr3RYKAjXfxfvSJV58h6X2kYUqvyemerCjPNsC9LmyqID2RIIHfdyUhMaOLJMYwQFGKzqwqIrLT6ZVK8vdfZBTCF89d6gxttGL9rP2zUahlUj8XuFHxhy0dkZbXBJ2ZQLr2IQT6UPnxC4FH2y2eUY6iQMsjnPp5InO7EF1ofCnA98kK7hgNuHVm8Iu3B8e/z/pmSE5rhn/ODFmK2Yuw2z+f/y6Irov25GaeVQYn4MFoAjsYJAE05rIP2fuYoQKi/lxZIoFB4xtdpgTbq7HtdjilATlrTjq3k89EDGvQGyBvx2wQRLlL3x7gCnX7DU1fxRCWEhGAXVf75+Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=XLVLru5z04tHtF0dpeBKJr9mOTJIG59Sg2QeP60qnD4=; b=gfrjUBrEzIOhTH+nw0DaB0wwFUKV4TPiIVdGTjbeb36gaM5shCK7ZM4bZv7rVhZyoxRBU8xgCrSZ5vb82P9bF/Ke07bBNl56G/nhedV9k5YoTz6NnTdJJApXXG4xLGiTI/vFfbEJNcd/IvQDmuy79h2spXoJPlorc5Uw7ccbpJY= Received: from DM6PR11MB2793.namprd11.prod.outlook.com (2603:10b6:5:c0::29) by DM6PR11MB4508.namprd11.prod.outlook.com (2603:10b6:5:200::32) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3153.28; Wed, 8 Jul 2020 17:46:27 +0000 Received: from DM6PR11MB2793.namprd11.prod.outlook.com ([fe80::d43:5665:5cee:b3c5]) by DM6PR11MB2793.namprd11.prod.outlook.com ([fe80::d43:5665:5cee:b3c5%6]) with mapi id 15.20.3174.021; Wed, 8 Jul 2020 17:46:26 +0000 From: "Ma, Maurice" To: Marcello Sylvester Bauer CC: "devel@edk2.groups.io" , Patrick Rudolph , Christian Walter , "Desimone, Nathaniel L" , "Zeng, Star" Subject: Re: [PATCH v1 1/1] UefiPayloadPkg: Remove x86 legacy UART defaults Thread-Topic: [PATCH v1 1/1] UefiPayloadPkg: Remove x86 legacy UART defaults Thread-Index: AQHWVTFiSfZAERfCTEmv5FCCsCq6Zqj9y37AgAAn1YA= Date: Wed, 8 Jul 2020 17:46:26 +0000 Message-ID: References: <20200708140900.1305-1-marcello.bauer@9elements.com> <20200708140900.1305-2-marcello.bauer@9elements.com> In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action authentication-results: 9elements.com; dkim=none (message not signed) header.d=none;9elements.com; dmarc=none action=none header.from=intel.com; x-originating-ip: [192.55.52.211] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 60455b2f-6418-4203-da3d-08d82366d6c4 x-ms-traffictypediagnostic: DM6PR11MB4508: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:7219; x-forefront-prvs: 04583CED1A x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: h4Ep/vZAxWlr66koNcz1e5DJLNw3uqqkv8fRxKw6CdCsWmx5h8Xu5aTdDXUmJ5+vsh4Aga+CAS+Al1vPuzNsDo0ayFw5IHtY+t/9a+jAQz1tIvqqffyYw9PD6x1N+pDERAqhdGh8wyBp71VRsLqxVQDOvy0cvn84mgOwh8M26JS4+ev5Kz6Lxe8f6EVgB19XYx4iWd8qHRoc8EjQ1Mv/feIdJWGlvGWKBb7kAFPu6POB0n9yjSTbKE8YW2yjrE86hw0Uj+pXCjCx1M35CwE3HcTrSQRp44fuRgZN9nmMsFM9jCJAQNMqdGcPbgw/twmqep1UMfwgQjDZ92GhWdI1YQ== x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DM6PR11MB2793.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(4636009)(39860400002)(136003)(376002)(396003)(366004)(346002)(6916009)(83380400001)(316002)(66946007)(186003)(9686003)(4326008)(2906002)(2940100002)(26005)(76116006)(33656002)(86362001)(107886003)(8936002)(53546011)(8676002)(7696005)(71200400001)(5660300002)(6506007)(52536014)(66476007)(19627235002)(66446008)(64756008)(478600001)(66556008)(54906003)(55016002);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata: Vs0r7c0ZoouQxmz//a9BuYdvbABGxvKxuAnb4c0RizutYXKoWbZkRg/Yhlz72peQ+saZFPlqAWZ/CNFzuEKkfnyOt0D9W0AXb9hlnjKXBBRBzsfJSWOCc4Ksvh2HRVlXBRNau+lBkkFpILnR+wuZYwLfAPzc9knOx3s7yekixfDu6w3jRbDcYU74rJQKVLKF9FGiCYavQcBGLYvqYnMZwNue73EcRdBkRFKqU4ZAAk30pwC6dyGKL8+29pk/f1un1D28PEWGB3inOqwCkCeLIWPNaeJRhkVTfsxedIPZJysolHKFBH0B9zUrDx+Ee53d85DdEau9qnt7dWDpLGQ3194kDkgVCMh2JpnE41eSGlhluDuA1GHuWlWcaWm+ZE2T0BmFaSWDwFqDdD73aPOMcJHAFddKh83G3SQr6LwTMEo0We4BN6TvDHUucrp20TcO6rjJOVK/KsmzrIc2CkAex/laj1g5V3x8FcKxZb37xkpfbWP8CzfxM3JPnfGyhIDI MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB2793.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 60455b2f-6418-4203-da3d-08d82366d6c4 X-MS-Exchange-CrossTenant-originalarrivaltime: 08 Jul 2020 17:46:26.8507 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: pQcsIKSTjbbgeugY8qHmrzRqU+TqLRJelLjNK4iJ6eC8QEY6TKdam/6Kv3YGQNCu94EPXSRHVBASkChHc0uFPA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB4508 Return-Path: maurice.ma@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable BTW, I just noticed that not all package maintainers are on the CC list. = =20 For changed under UefiPayloadPkg, please ensure you CC all UefiPayloadPkg = maintainers listed in the Maintainers.txt under the root of the EDK2 direct= ory. =20 Thanks Maurice > -----Original Message----- > From: Ma, Maurice > Sent: Wednesday, July 8, 2020 8:25 > To: 'Marcello Sylvester Bauer' ; > devel@edk2.groups.io > Cc: Patrick Rudolph ; Christian Walter > ; Desimone, Nathaniel L > ; Zeng, Star > Subject: RE: [PATCH v1 1/1] UefiPayloadPkg: Remove x86 legacy UART defaul= ts >=20 > Could you please add some commit message to explain what issue you are tr= ying > to address ? Why is it required ? >=20 > Thanks > Maurice > > -----Original Message----- > > From: Marcello Sylvester Bauer > > Sent: Wednesday, July 8, 2020 7:09 > > To: devel@edk2.groups.io > > Cc: Patrick Rudolph ; Christian Walter > > ; Ma, Maurice ; > > Desimone, Nathaniel L ; Zeng, Star > > > > Subject: [PATCH v1 1/1] UefiPayloadPkg: Remove x86 legacy UART > > defaults > > > > From: Patrick Rudolph > > > > Signed-off-by: Patrick Rudolph > > Signed-off-by: Marcello Sylvester Bauer > > Cc: Patrick Rudolph > > Cc: Christian Walter > > Cc: Maurice Ma > > Cc: Nate DeSimone > > Cc: Star Zeng > > --- > > UefiPayloadPkg/UefiPayloadPkgIa32.dsc | 2 +- > > UefiPayloadPkg/UefiPayloadPkgIa32X64.dsc | 2 +- > > 2 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/UefiPayloadPkg/UefiPayloadPkgIa32.dsc > > b/UefiPayloadPkg/UefiPayloadPkgIa32.dsc > > index 0efcfa8fb2f4..0d8854810700 100644 > > --- a/UefiPayloadPkg/UefiPayloadPkgIa32.dsc > > +++ b/UefiPayloadPkg/UefiPayloadPkgIa32.dsc > > @@ -306,7 +306,7 @@ [PcdsPatchableInModule.common] > > # The following parameters are set by Library/PlatformHookLib > > > > # > > > > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialUseMmio|FALSE > > > > - gEfiMdeModulePkgTokenSpaceGuid.PcdSerialRegisterBase|0x3f8 > > > > + gEfiMdeModulePkgTokenSpaceGuid.PcdSerialRegisterBase|0 > > > > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialBaudRate|$(BAUD_RATE) > > > > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialRegisterStride|1 > > > > > > > > diff --git a/UefiPayloadPkg/UefiPayloadPkgIa32X64.dsc > > b/UefiPayloadPkg/UefiPayloadPkgIa32X64.dsc > > index d645d233cd08..34640b1fc1da 100644 > > --- a/UefiPayloadPkg/UefiPayloadPkgIa32X64.dsc > > +++ b/UefiPayloadPkg/UefiPayloadPkgIa32X64.dsc > > @@ -307,7 +307,7 @@ [PcdsPatchableInModule.common] > > # The following parameters are set by Library/PlatformHookLib > > > > # > > > > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialUseMmio|FALSE > > > > - gEfiMdeModulePkgTokenSpaceGuid.PcdSerialRegisterBase|0x3f8 > > > > + gEfiMdeModulePkgTokenSpaceGuid.PcdSerialRegisterBase|0 > > > > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialBaudRate|$(BAUD_RATE) > > > > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialRegisterStride|1 > > > > > > > > -- > > 2.27.0