From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by mx.groups.io with SMTP id smtpd.web09.220.1618005004128909006 for ; Fri, 09 Apr 2021 14:50:04 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=By8SXhNv; spf=pass (domain: intel.com, ip: 134.134.136.65, mailfrom: nathaniel.l.desimone@intel.com) IronPort-SDR: 3JzKJmBIXxKTYWn2v/AD95bVY4KIz0UUPqrcsLmCg8q28oysb8T7Ib5UBhjz0e+MuFqT0XgtmI 9JwdydtjcDEA== X-IronPort-AV: E=McAfee;i="6000,8403,9949"; a="193883541" X-IronPort-AV: E=Sophos;i="5.82,210,1613462400"; d="scan'208";a="193883541" Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 09 Apr 2021 14:50:03 -0700 IronPort-SDR: PSy/ZcbpdfACTDaMu/H7C2mlITau7w7tfvicbNHJs/CHqjZLynOC5KuOSJICCK4Hgkm7wcjyGi jhUGnJTCXaZw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.82,210,1613462400"; d="scan'208";a="520413868" Received: from orsmsx605.amr.corp.intel.com ([10.22.229.18]) by fmsmga001.fm.intel.com with ESMTP; 09 Apr 2021 14:50:02 -0700 Received: from orsmsx610.amr.corp.intel.com (10.22.229.23) by ORSMSX605.amr.corp.intel.com (10.22.229.18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2106.2; Fri, 9 Apr 2021 14:50:02 -0700 Received: from ORSEDG601.ED.cps.intel.com (10.7.248.6) by orsmsx610.amr.corp.intel.com (10.22.229.23) 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, 9 Apr 2021 14:50:02 -0700 Received: from NAM02-SN1-obe.outbound.protection.outlook.com (104.47.36.57) 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, 9 Apr 2021 14:50:01 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=b+ugxDuuUycFFSojy5UTTjU0Nu9lAO0J/TdIG/Or8iKaGjJotOACIWsVvgNANXZd/tasQPpeoeV3Dooaud/HtGqGid2gM9ROlhTZ/WI/nn26dWp0Oqxj8ITWKPk4+NbZsHfAzp4GENP4PghM49C+TrWylgQrr8Vq6Il6gflQFpDKr7nSPqoVYoNvEfFRSrAT0okb0JJ7pkj1e94e5DBgY2uhJH6Nxzw45ybBCNab6ZgzjL5Q1HaosjEH4SE9ycCLJwj2kCaqUQ99MZUU0O6Pc8HaOr6fIIWvPc2KO8PamVHgJHZ9xr9teLl+AvY5hNsKqg1vQv2zAspvyE83+yZ5hg== 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=hpCkWg8N6VFqm2ElSoZwx2hPo+vRq1J1jNqkhaHczlE=; b=S27ZkQxP5IARbQQsbHG+oHRa+ifrSiviDvmwqmcd52Ig2jYTdiED2PZ92fHuRLVhmnau0NG58Mj7juVeTHm5bLhGjMLVrRHcaOYrR9Fa6CYpkwBkqWFKnm3z3wLdzHgSMVkLQzlytHxKSSlMBSle02GW46brydUuNc2RwB5gktBug7iLlkyLBTozOGW5bO65uRTrRwjIig9ImtsrpgSuvVg1/LNyPxasqGe/DIiOuvkzqVAPnq2W9geG5qEn7jjqf/Kiq2g+GIUExtKaa/jN2oW/25aTo9902ZENYQGaUYJTHqbdq6T93VgJtuzGUYipu4tFGsLk+YuPD2TqPDtDMQ== 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=hpCkWg8N6VFqm2ElSoZwx2hPo+vRq1J1jNqkhaHczlE=; b=By8SXhNvpvGL5na5rKNCR/O0QxT9nLOISwYTZowCj8D1TWw6NGnm2n+vglmqFtLToRQoXsfKBqzQgFknnArRjE6MxIFqJiwceBhs55flD3kRBUCTl73rLwCpNC8mx7saxqr8r1/F6VHiG1BlhsJHvXDDK9iXkGe3/GsD+YtJaZc= Received: from MWHPR1101MB2160.namprd11.prod.outlook.com (2603:10b6:301:5b::12) by MWHPR11MB1341.namprd11.prod.outlook.com (2603:10b6:300:2b::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3999.32; Fri, 9 Apr 2021 21:50:01 +0000 Received: from MWHPR1101MB2160.namprd11.prod.outlook.com ([fe80::6540:bca7:645a:6788]) by MWHPR1101MB2160.namprd11.prod.outlook.com ([fe80::6540:bca7:645a:6788%5]) with mapi id 15.20.4020.021; Fri, 9 Apr 2021 21:50:00 +0000 From: "Nate DeSimone" To: "devel@edk2.groups.io" , "mikuback@linux.microsoft.com" CC: "Chiu, Chasel" , Liming Gao , "Dong, Eric" Subject: Re: [edk2-devel] [edk2-platforms][PATCH v2 0/2] MinPlatformPkg: Improve NULL ACPI table lib maintainability Thread-Topic: [edk2-devel] [edk2-platforms][PATCH v2 0/2] MinPlatformPkg: Improve NULL ACPI table lib maintainability Thread-Index: AQHXLL2qEya+AojJgU24UM5kagb0saqsutFA Date: Fri, 9 Apr 2021 21:50:00 +0000 Message-ID: References: <20210408212433.632-1-mikuback@linux.microsoft.com> In-Reply-To: <20210408212433.632-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: edk2.groups.io; dkim=none (message not signed) header.d=none;edk2.groups.io; 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: 9aaf1338-f80a-476e-ad68-08d8fba16cfb x-ms-traffictypediagnostic: MWHPR11MB1341: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:6108; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 0AXWxYoZWMvHXk1Vy9zTID4nPVJKpSKaPfUZn2uUknwpSSpME6mFWYxyStkOzI74y27hxjCCylrbadLnuZsylRtG+vdH1QQOg87dK8dWRIKk4ZMWvT+63Hj1h+Vikwh6rLbmUhItMAsCYualsfOxW4jgi8MsI7GIzsZFM5fMBAu5Vg+09TvWNH5Mh0LoMxoDOLhR4oK8XYFsTnE/EJDcmTkeR1iyzzxMmedqcHU3ouWeO4PiWwxkc9UVF8e8GWF8BKSDYWM200mLtG8pV41w2ONVPOcL7tvTCbR8iJkdNd/SMNwnO6dPrcvrfeWRiXYwksKlwq6vTjrxWfEeXPNyiVTfTuJemU3ZyMy2T8eNEuVZqASM5+tjRXnpavhume9eWk8o5f+FC1y8SO+Tv9QfcJ6c2VJL712ncE8m4ah0HldiPy1q1jUVt0RZarruU3tt/Ta00DETb3Bmec5gFYoXPklVG9NcNApeCGn9be+RjalOBCTnT5TD/2hyRpR2z5rgYicSQ5O3WOd34ZGwoGpQsxFkHMk8m4W5Bi0wIaNUJv4Yf3BJumcnFj0ciyzDtrohdbbB4+T7BD9JGMmiVXVxIsnCqkWjv4gYc7ZpT6lp+IQnopQAkhQoVYXlcUK51zzdwP9xAnAhCzLuMziI8VkmM4dLdBGXwUW1NArP27NZakK+kDhIpDnYXZCNhY7sqVAabL7bt7Jt80IPU0BeVzYJk1e8iXnxA1iz7kU0Vmw94gU= x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:MWHPR1101MB2160.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(39860400002)(346002)(376002)(136003)(396003)(366004)(110136005)(71200400001)(316002)(8676002)(186003)(4326008)(54906003)(38100700001)(2906002)(26005)(478600001)(9686003)(7696005)(66446008)(55016002)(966005)(8936002)(5660300002)(86362001)(33656002)(64756008)(76116006)(66946007)(66556008)(83380400001)(66476007)(107886003)(6506007)(53546011)(52536014);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata: =?us-ascii?Q?Cnzj84XoNGzB/1RboiRPdliuOc/BtQOhbWLYLuHqWPlUKHo/GLzyStd0vTK3?= =?us-ascii?Q?DUc0/KKjsK9miHeyqHudyOt8quD+R2Yq8ojRTTR3/81u3cOxzswen43zkCfw?= =?us-ascii?Q?vjL1a4ZyEcC0SFZgk08C/8Aw7d24vhxL7fMDbn8B5Jg8S/kDEa0vN2g5Shyh?= =?us-ascii?Q?xEZmgxYjq2bSLNbX2YcdcAlva3kBo/fGqZB6nUcJDrsSmLGe7eQEqbu/XrhG?= =?us-ascii?Q?tUiiSRMDH9L5IpzjosmZdBRks6Vsdf4zsV9FEKnaHBcypDqTr8FUKqH/nqQc?= =?us-ascii?Q?wvb5yqFjAOv8N1JJLU8p1MqfOhtvQop5FCgQAZkcgZ8MJg43Uw5eCPYTX9+h?= =?us-ascii?Q?SS3GEBrKruiitnijIR60C7IIGI4k4zmWx1S+6z4ocWhosV7afKKGUyDGH0hh?= =?us-ascii?Q?VjWNPmbhc7EuP2oVgn+iPg8SybPqPjtTOSwdJmtAqfaAzayL7XXPA3kHOeYj?= =?us-ascii?Q?FR8f4/JWp72TCIXYNPiM/w1cGpxKDLJm3WNf4xjiyJCjPcnYqcco+ZgNjSDZ?= =?us-ascii?Q?OA8fwm2xvdoXwXOPARRB36hI/QoTtlIUGUrmUUoD3WmzSOKtAIayOIrgqrmc?= =?us-ascii?Q?I81XwzW9uLTvdjM3ApJdj9ff4Uwwahch2H22W7jvHpcCXaKehNT7VFNKqkbW?= =?us-ascii?Q?MVzNMUAM5waPQRCBOI6Hc6sxlL+kp/JsTKFD4Jt67DnIYabpjy9pan7b4UbH?= =?us-ascii?Q?vtxyuLFt5oOJuIXwbkYyNs1+QpOB7KqYGZ+eIGYU4fS53rH0EZQ/zB3Yu2CS?= =?us-ascii?Q?4/rLH2v18oYKUYwDzF0q1oWlcbTehXjOUR+SAQ2xc091LdQMDEL4U2/i43TD?= =?us-ascii?Q?g8dThZ6f9B6NmIY/qQN/AZHFD4HY2tgCSFhf2tgGxc++mcNJWaAvVXU+Eg/P?= =?us-ascii?Q?kcoc5PV9UecIFc9/C1JdLSJC671DsZMYeUDUnEPwrcQQTK+EI0oY0wL7nVOn?= =?us-ascii?Q?9jOvDsPGELECe4cgyB/atytHD3ZEVyIV90/Kqcg7NVaT8T5rsBkmknmSVxAg?= =?us-ascii?Q?vbh9l78G3krZGh90OAeWEoLld6PnzPK/mw99kXV+4pDbomMXGfcOLuVurvlv?= =?us-ascii?Q?1SX6ulKJ2uvXG5YUAFIYXujMY5OJFgcXc1gBJB6vlmtssk7o0GlYglo2WaXE?= =?us-ascii?Q?1KI9Z/ocxmPl7DgydX8WbSWeNg00/rHbKh8pGPCMNYPn1f7fOQTSdfw2F0LX?= =?us-ascii?Q?qsAmnsBJ2YBtY1WOO52cZq93WY5RuHIUCQJgVwJZo2+kbejTX8KtHi14meRg?= =?us-ascii?Q?roQZ2Y6G5P/m3bYjkbe/bAKAjdPPThJyolg9NajbFfYNnrqz+xYlVIx4xILr?= =?us-ascii?Q?rl14yO7wGkpbAxwTjyod6xP+?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: MWHPR1101MB2160.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 9aaf1338-f80a-476e-ad68-08d8fba16cfb X-MS-Exchange-CrossTenant-originalarrivaltime: 09 Apr 2021 21:50:00.7775 (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: 0VUrVZpVeWuRy95yIRD+U6lXYJJvQF3n4j0RHXWr4xGu7UefKYV58EpVZ3dG4mQHYgwB5Tk/F9WRB+pr3OQHw06pMmzaLJl+GwvO6V5JJS4= X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR11MB1341 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 The series has been pushed as 3204cbfe~..ade2b32 Thanks, Nate -----Original Message----- From: devel@edk2.groups.io On Behalf Of Michael Kuba= cki Sent: Thursday, April 8, 2021 2:25 PM To: devel@edk2.groups.io Cc: Chiu, Chasel ; Desimone, Nathaniel L ; Liming Gao ; Dong, Eric Subject: [edk2-devel] [edk2-platforms][PATCH v2 0/2] MinPlatformPkg: Improv= e NULL ACPI table lib maintainability From: Michael Kubacki This series makes some simple changes to the organization of BoardAcpiEnabl= eLibNull and BoardAcpiTableLibNull to better align the library instances wi= th patterns typically used so they are easier to integrate into platforms. The following issues are resolved: 1. Sharing of a directory with another unrelated library instance. 2. The directory name "BoardAcpiLibNull" is not directly related to either library instance name in the directory. 3. The library instances have unnecessary dependencies. 4. The BASE_NAME does not indicate the library instance is the NULL instance. 5. The C source file names do not match the INF file name making finding the C source by search more cumbersome than needed. V2 changes: 1. Remove Base.h from BoardAcpiEnableLibNull.c and BoardAcpiTableLibNull.c Cc: Chasel Chiu Cc: Nate DeSimone Cc: Liming Gao Cc: Eric Dong Signed-off-by: Michael Kubacki Michael Kubacki (2): MinPlatformPkg/BoardAcpiEnableLibNull: Improve maintainability MinPlatformPkg/BoardAcpiTableLibNull: Improve maintainability Platform/Intel/MinPlatformPkg/Acpi/Library/{BoardAcpiLibNull/BoardAcpiEnab= leLib.c =3D> BoardAcpiEnableLibNull/BoardAcpiEnableLibNull.c} | 3 --- Platform/Intel/MinPlatformPkg/Acpi/Library/{BoardAcpiLibNull/BoardAcpiTabl= eLib.c =3D> BoardAcpiTableLibNull/BoardAcpiTableLibNull.c} | 3 --- Platform/Intel/MinPlatformPkg/Acpi/Library/{BoardAcpiLibNull =3D> BoardAcp= iEnableLibNull}/BoardAcpiEnableLibNull.inf | 12 ++++----= ---- Platform/Intel/MinPlatformPkg/Acpi/Library/{BoardAcpiLibNull =3D> BoardAcp= iTableLibNull}/BoardAcpiTableLibNull.inf | 12 ++++----= ---- Platform/Intel/MinPlatformPkg/MinPlatformPkg.dsc = | 8 ++++---- 5 files changed, 12 insertions(+), 26 deletions(-) rename Platform/Intel/= MinPlatformPkg/Acpi/Library/{BoardAcpiLibNull/BoardAcpiEnableLib.c =3D> Boa= rdAcpiEnableLibNull/BoardAcpiEnableLibNull.c} (73%) rename Platform/Intel/= MinPlatformPkg/Acpi/Library/{BoardAcpiLibNull/BoardAcpiTableLib.c =3D> Boar= dAcpiTableLibNull/BoardAcpiTableLibNull.c} (73%) rename Platform/Intel/Min= PlatformPkg/Acpi/Library/{BoardAcpiLibNull =3D> BoardAcpiEnableLibNull}/Boa= rdAcpiEnableLibNull.inf (67%) rename Platform/Intel/MinPlatformPkg/Acpi/Li= brary/{BoardAcpiLibNull =3D> BoardAcpiTableLibNull}/BoardAcpiTableLibNull.i= nf (67%) -- 2.28.0.windows.1 -=3D-=3D-=3D-=3D-=3D-=3D Groups.io Links: You receive all messages sent to this group. View/Reply Online (#73880): https://edk2.groups.io/g/devel/message/73880 Mute This Topic: https://groups.io/mt/81953909/1767664 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [nathaniel.l.desimone@int= el.com] -=3D-=3D-=3D-=3D-=3D-=3D