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.web11.1296.1613169166086119223 for ; Fri, 12 Feb 2021 14:32:46 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=JNvPV8Cv; spf=pass (domain: intel.com, ip: 134.134.136.31, mailfrom: nathaniel.l.desimone@intel.com) IronPort-SDR: pWkf7kwXHXx9QcjjRAaszpeLdDqQiEp4pFR6dJz8Nv6viWOUyiY8wzcONrLs0seGMqEkfncLcW qea6IcWXHM+A== X-IronPort-AV: E=McAfee;i="6000,8403,9893"; a="243973612" X-IronPort-AV: E=Sophos;i="5.81,174,1610438400"; d="scan'208";a="243973612" Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 12 Feb 2021 14:32:44 -0800 IronPort-SDR: VVGatbVppK9ibjM4Nr22iXusDeqQkbfHWny0XSEedt8BGBjwFihaQlEVDvA6CydrlJZenoNfw3 mD8YPqNZGq9A== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.81,174,1610438400"; d="scan'208";a="364242718" Received: from orsmsx603.amr.corp.intel.com ([10.22.229.16]) by fmsmga007.fm.intel.com with ESMTP; 12 Feb 2021 14:32:44 -0800 Received: from orsmsx608.amr.corp.intel.com (10.22.229.21) by ORSMSX603.amr.corp.intel.com (10.22.229.16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2106.2; Fri, 12 Feb 2021 14:32:44 -0800 Received: from orsmsx606.amr.corp.intel.com (10.22.229.19) by ORSMSX608.amr.corp.intel.com (10.22.229.21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2106.2; Fri, 12 Feb 2021 14:32:43 -0800 Received: from ORSEDG601.ED.cps.intel.com (10.7.248.6) by orsmsx606.amr.corp.intel.com (10.22.229.19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2106.2 via Frontend Transport; Fri, 12 Feb 2021 14:32:43 -0800 Received: from NAM10-BN7-obe.outbound.protection.outlook.com (104.47.70.103) by edgegateway.intel.com (134.134.137.102) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2106.2; Fri, 12 Feb 2021 14:32:43 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=m+Cr1tj8xFeVJinwlJK20ESyBLr8uAl/iyeDJdAJjndP9Jw1D2w+3qcTTyMOEjmuoSZnvMtIFjA/AzE1SsQPjVPPZiwOqhzQ4uMa+aGWGZiYwAd3YhyC1ck6Obta+BYRhqhynqlSUw4v5HbQOjMQv38rO6mKvfBefA1lRtbaMNPgEOCgNsx68kr1fT+8ymRu/7nn3Y3lOLHA2YA0eI/Fi+LMJnz38Ly+R5qd735xDrg9IpA2ESaEsaPKvX80racMGIVhhxcCVSi8e6uVe4DQMidgAw1ZgigyGSiH2zJsmlJsRrmhCvIB/Xx9EdG7HXRtDetTRJOcBRt84OYFoNc8qg== 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=zPYJQBH49r4mcu//PW9t6EnGaf+W7ZM2CnDS1bcklss=; b=BuLgOJ4vT94nLmS73sRhJatNGY4du69Q8OCOehaofC4jgiDSKi2+JaigBpDETRltG8DxEwVgYNbSIMEVytBLPoorV8zg2R6Gkxuoh3qjBNWLffbUW7HDHSw3ccNHizv5wDFFVAiOe02oAXjjQpcUTvWNI5JIIJm/e1//fteAiDdeWMwDgQTYG2UiQQwz52H18dVtyYzyy3J3eyPLEGGJYvpwq4kSErS1FV1pYLgCyXPdHHEAaclLHfZCG9Hco4KpV7R+DXuM8cUP36xU/71QW21ewhrPpgcgrxaOSRXmB9dlV0jyBBT9W+bCjqbg0jzOtYBDxSsBE1fq9JYHOsiNEg== 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=zPYJQBH49r4mcu//PW9t6EnGaf+W7ZM2CnDS1bcklss=; b=JNvPV8CvWAtX6Ixx7WWJXPYemAKczGWC06SPpbvpPqRHSk2PwYLXchHVvoHVNDz3TqqOZIfUJ6N4L0AdnW9Kwxcr6DzVQfTMAusql7bLZj4uW3TneQxGMVs0O4pO9WQ+j6hvK17UsugMJDhd5Q9+pLGSeF/CqytR/DLKYcvxPXs= Received: from DM5PR1101MB2154.namprd11.prod.outlook.com (2603:10b6:4:4e::20) by DM6PR11MB2570.namprd11.prod.outlook.com (2603:10b6:5:ce::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3825.30; Fri, 12 Feb 2021 22:32:42 +0000 Received: from DM5PR1101MB2154.namprd11.prod.outlook.com ([fe80::257f:9cf5:79fa:c0a0]) by DM5PR1101MB2154.namprd11.prod.outlook.com ([fe80::257f:9cf5:79fa:c0a0%9]) with mapi id 15.20.3846.030; Fri, 12 Feb 2021 22:32:42 +0000 From: "Nate DeSimone" To: "mikuback@linux.microsoft.com" , "devel@edk2.groups.io" CC: "Chiu, Chasel" , Liming Gao , "Dong, Eric" Subject: Re: [edk2-platforms][PATCH v2 0/3] MinPlatformPkg: Add SpiFvbServiceStandaloneMm Thread-Topic: [edk2-platforms][PATCH v2 0/3] MinPlatformPkg: Add SpiFvbServiceStandaloneMm Thread-Index: AQHXALYES4C8huYvGUaDfAoduBp6jqpVHGwA Date: Fri, 12 Feb 2021 22:32:42 +0000 Message-ID: References: <20210211203855.1656-1-mikuback@linux.microsoft.com> In-Reply-To: <20210211203855.1656-1-mikuback@linux.microsoft.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-reaction: no-action dlp-version: 11.5.1.3 dlp-product: dlpe-windows authentication-results: linux.microsoft.com; dkim=none (message not signed) header.d=none;linux.microsoft.com; dmarc=none action=none header.from=intel.com; x-originating-ip: [50.53.190.176] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 5ca0cdde-9153-4312-b4cd-08d8cfa61c7c x-ms-traffictypediagnostic: DM6PR11MB2570: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:8273; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: H1C3NufJwKYLpoSwpyNzT+L8qu7/y+vsugWn5pDcsFmQzMC/95hiyO0hI7SgA/F5hSVyp1V1mWA47WhbCG/uP2isLda7Iy34mMQdEMyoK/vzXvExs84EDWK7NtrutH5t9PDQErQ+K9swPgkKZsxHmEws7et1nAPUBON4L7icZvnd588s/AAOZvslWDu+Qomfs9EsgDKdfr9jVK2kOZNesleQAQeB5XI1b7fiyOBbTtKxJoLWG6o/Gqi8qwjfcjbcjQlh4/+mls3wBOONRGlKYbxffno2AckO1IYNcvFrKzId13NitieDj00BiLN4IOPqCbzmExzYk10y7bpfxAUuw+rsJNVBsW6XwKH2YxNYknRpOaCXJVig40CAykerjY03GKJRZTiiGWad6bBeMwIOMfYPy2hpw8fuC0DNeuLchINfWwyAHkgBGa7m2D5NciuaVVcE5Nwchsn8GN+4PAx5lV6ZY2mxdedhTRpu93K04oIbc/gTBOCTl9EXBqyGwXDHBtI+g/bPPYroo9e6cJyHGQ== x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DM5PR1101MB2154.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(39860400002)(396003)(366004)(346002)(136003)(376002)(66946007)(86362001)(478600001)(66556008)(66476007)(66446008)(55016002)(83380400001)(76116006)(64756008)(2906002)(8936002)(316002)(186003)(8676002)(9686003)(71200400001)(107886003)(26005)(110136005)(7696005)(52536014)(5660300002)(54906003)(33656002)(4326008)(53546011)(6506007);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata: =?us-ascii?Q?2yFf4P+Sco9QMtfgxgE7Xh+nZcmHuo4AatS66Lx4ZqdM9rWveWVP8LczXfeE?= =?us-ascii?Q?TRqSeN1c9z6hgShngUo6MyMIr2BezZ5pPBMny+WP3HT/VKvmzUt2OwsbtbG1?= =?us-ascii?Q?tkIVPLaHU7CcUjR2jwFqw3vGQ0NC3V1SJiUMWn72Z/QVzSkieDkVta42vhji?= =?us-ascii?Q?HvmYNuLmDHypjLbi8Rx/A9lDkA2reQuLd6gv6VnNjvgVaWUUC1lslwnoZqpk?= =?us-ascii?Q?Bn50Ltj3yx8ArknDuOTRoW2leYga9Ro59esoO/ylzJqEfgtYlxk0i44rhoKU?= =?us-ascii?Q?1XlMgvKrPdv2EoZnRO/GS4JX9vitnfoRntNOuVIXHndrTXCYJLCyvNRYe1Vo?= =?us-ascii?Q?YwZ2OqbmxNGWhWXLKt5jeWfurx98YvLHn1tlX939lv0yn5l91wdpDPeQDnNr?= =?us-ascii?Q?yRHjokuney9+9eN3wr4w3nvjOohWnKVFdo48eyd+px9hShKejk+41h3+yTs8?= =?us-ascii?Q?8DoFuUP1Xk0ji4nDjsUh0wEhaHeHX/nVU9XZ3USVQ9ScPIVBQ5VIYoR4y3+U?= =?us-ascii?Q?3hfDKHYI04HMZS3c9TCX525eMaoJWmOUYkHInWk6GOxe7jL8qKnunLp9/YDa?= =?us-ascii?Q?fF5sV2k8IRK1Wpt688lmd6FdpvODGErXHtyfAr8h5rvKVHy2ZolkQJU4HlkK?= =?us-ascii?Q?OBtTi3fGECCHWyitL33rGLHDHdkg6MIzUj2XAUDTwtjA/oFBFA5JQS7letFp?= =?us-ascii?Q?5bZuTvSx+BU82DrGaViTWyPqx58XVHB3QbEs5VSlSXeYMbz8j9fP97Xggnwd?= =?us-ascii?Q?hz7XKJDhY/nadOd8A317I//bMlfZigu1XWbE6/IYhMZiZ2bx3beQcFlzkhXl?= =?us-ascii?Q?Hp7td0NULNG6Nr6zOwF3oc08J67cs9LGs72TlXMH9M3xBeLO6hwiprMgPIDu?= =?us-ascii?Q?WGd//d6YofDCdajuvh6fIsAON4JuX27A66mlqEuw6xXMHIIJOZYhB/z2knF1?= =?us-ascii?Q?9K1Upt5POjC/1jO3OBGzdUBQB/+qzGEe0/gWEG/IllQx5pbM27p1HsatV1Wi?= =?us-ascii?Q?ZJr4qlW+La2r8Px4Qu5KQoQaEfkLWLtng7OxWpHBnZjskKrHl1BIu7kf0AEi?= =?us-ascii?Q?f+9QstHgTysM7bukacuJVdEkLdHgy2m+5ZvF/AQt8jcFaZltThN8QuW1zNpL?= =?us-ascii?Q?FLBvUN40OnAZSOl0nz3wGPouVB2IvqN2RogMrfQxddAyubasl9HNjzVNEhpS?= =?us-ascii?Q?4mOIA5EODeZHNEn6+AJ0DOi3xshK5DCOhZnE6i2FBSdGSXUlFwZoloJNLH4Z?= =?us-ascii?Q?YZmPCEpSHmjs/BlRjrkV0k1RPA97ckKSaffjZp5VA6fLWbCLe7kukNR6C/Ha?= =?us-ascii?Q?g5Waq41aWFkfVXU/wK42ai8I?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM5PR1101MB2154.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 5ca0cdde-9153-4312-b4cd-08d8cfa61c7c X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Feb 2021 22:32:42.1101 (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: 1XOSKCmp43JI5s3YjXcjvjPdEKLVi0H35o9VUVvj9vb2Ps2Dp77Qsj1Zror4DiIi17/2XxA1yCRXaidBNgjGiM4ZztPzdgZIoks8SywzGKo= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB2570 Return-Path: nathaniel.l.desimone@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable For the series... Reviewed-by: Nate DeSimone > -----Original Message----- > From: mikuback@linux.microsoft.com > Sent: Thursday, February 11, 2021 12:39 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Liming Gao > ; Dong, Eric > Subject: [edk2-platforms][PATCH v2 0/3] MinPlatformPkg: Add > SpiFvbServiceStandaloneMm >=20 > From: Michael Kubacki >=20 > Adds a new component called SpiFvbServiceStandaloneMm that serves as a > Standalone MM compatible SPI FVB service driver. >=20 > Note that a MM_STANDALONE version of SpiFlashCommonLib is being > prepared to be sent but for the time being the module can be added to the > MinPlatformPkg build and rely upon a NULL instance in the build. >=20 > V2 changes: > 1. Add Intel copyright to SpiFvbServiceMm.c >=20 > Cc: Chasel Chiu > Cc: Nate DeSimone > Cc: Liming Gao > Cc: Eric Dong > Signed-off-by: Michael Kubacki >=20 > Michael Kubacki (3): > MinPlatformPkg/SpiFlashCommonLibNull: Make MODULE_TYPE BASE > MinPlatformPkg/MinPlatformPkg.dsc: Add basic MM_STANDALONE > libraries > MinPlatformPkg/SpiFvbService: Add Standalone MM support >=20 > Platform/Intel/MinPlatformPkg/Flash/SpiFvbService/{Common =3D> > }/FvbInfo.c | 0 > Platform/Intel/MinPlatformPkg/Flash/SpiFvbService/{Common =3D> > }/SpiFvbServiceCommon.c | 0 >=20 > Platform/Intel/MinPlatformPkg/Flash/SpiFvbService/{Smm/SpiFvbServiceSm > m.c =3D> SpiFvbServiceMm.c} | 35 ++++++----------- >=20 > Platform/Intel/MinPlatformPkg/Flash/SpiFvbService/SpiFvbServiceStandalon > eMm.c | 32 ++++++++++++++++ >=20 > Platform/Intel/MinPlatformPkg/Flash/SpiFvbService/SpiFvbServiceTraditiona > lMm.c | 32 ++++++++++++++++ >=20 > Platform/Intel/MinPlatformPkg/Flash/Library/SpiFlashCommonLibNull/SpiFla > shCommonLibNull.inf | 4 +- > Platform/Intel/MinPlatformPkg/Flash/SpiFvbService/{Common =3D> > }/SpiFvbServiceCommon.h | 4 -- > Platform/Intel/MinPlatformPkg/Flash/SpiFvbService/SpiFvbServiceMm.h > | 22 +++++++++++ > Platform/Intel/MinPlatformPkg/Flash/SpiFvbService/SpiFvbServiceSmm.inf > | 17 +++++---- > Platform/Intel/MinPlatformPkg/Flash/SpiFvbService/{SpiFvbServiceSmm.inf > =3D> SpiFvbServiceStandaloneMm.inf} | 40 ++++++++++---------- > Platform/Intel/MinPlatformPkg/MinPlatformPkg.dsc > | 9 +++++ > 11 files changed, 139 insertions(+), 56 deletions(-) rename > Platform/Intel/MinPlatformPkg/Flash/SpiFvbService/{Common =3D> > }/FvbInfo.c (100%) rename > Platform/Intel/MinPlatformPkg/Flash/SpiFvbService/{Common =3D> > }/SpiFvbServiceCommon.c (100%) rename > Platform/Intel/MinPlatformPkg/Flash/SpiFvbService/{Smm/SpiFvbServiceSm > m.c =3D> SpiFvbServiceMm.c} (89%) create mode 100644 > Platform/Intel/MinPlatformPkg/Flash/SpiFvbService/SpiFvbServiceStandalon > eMm.c > create mode 100644 > Platform/Intel/MinPlatformPkg/Flash/SpiFvbService/SpiFvbServiceTraditiona > lMm.c > rename Platform/Intel/MinPlatformPkg/Flash/SpiFvbService/{Common =3D> > }/SpiFvbServiceCommon.h (94%) create mode 100644 > Platform/Intel/MinPlatformPkg/Flash/SpiFvbService/SpiFvbServiceMm.h > copy > Platform/Intel/MinPlatformPkg/Flash/SpiFvbService/{SpiFvbServiceSmm.inf > =3D> SpiFvbServiceStandaloneMm.inf} (64%) >=20 > -- > 2.28.0.windows.1