From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by mx.groups.io with SMTP id smtpd.web08.1964.1666659097628709887 for ; Mon, 24 Oct 2022 17:51:37 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="unable to parse pub key" header.i=@intel.com header.s=intel header.b=l6TK/4pk; spf=pass (domain: intel.com, ip: 192.55.52.115, mailfrom: nathaniel.l.desimone@intel.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1666659097; x=1698195097; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=ns2PvaxxNnOejTz6IVGiueIB48vb6GPfnHyA/vc3Wug=; b=l6TK/4pkRCRdgK8qYBVlZCl4CdlKFlDon8N8LIOuhc1ZOcLiBOQi8DNu jlStF4YSFOcrvF4U12i5JzLuc3jaNrqp0Dloap0DA2D2+a+t0ibhZqQ5y R+/Zg7WiZNlLwxirToX6KRt7hd9VYxl70sUdY0vL8p+qkCsG9pMpjydD4 f+6AI8WuEOfvRvqHRclTjw7nJ4CfhCtM0uxBzXAtKNIa7op4csmvX0vkw ZOXc6+OMzyYtoERKO+uhNHeoAyh15yRBhOx4rs0VkZSCMyILESNEH2ZuP 65+uN9WC/QUoFAbectX7MkkAWyMlNw/CGucFsCjmZUMBScLNOqR97bDM/ A==; X-IronPort-AV: E=McAfee;i="6500,9779,10510"; a="307548907" X-IronPort-AV: E=Sophos;i="5.95,210,1661842800"; d="scan'208";a="307548907" Received: from orsmga007.jf.intel.com ([10.7.209.58]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 24 Oct 2022 17:51:37 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6500,9779,10510"; a="626249685" X-IronPort-AV: E=Sophos;i="5.95,210,1661842800"; d="scan'208";a="626249685" Received: from orsmsx601.amr.corp.intel.com ([10.22.229.14]) by orsmga007.jf.intel.com with ESMTP; 24 Oct 2022 17:51:37 -0700 Received: from orsmsx601.amr.corp.intel.com (10.22.229.14) by ORSMSX601.amr.corp.intel.com (10.22.229.14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.31; Mon, 24 Oct 2022 17:51:36 -0700 Received: from ORSEDG602.ED.cps.intel.com (10.7.248.7) by orsmsx601.amr.corp.intel.com (10.22.229.14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.31 via Frontend Transport; Mon, 24 Oct 2022 17:51:36 -0700 Received: from NAM11-CO1-obe.outbound.protection.outlook.com (104.47.56.176) by edgegateway.intel.com (134.134.137.103) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2375.31; Mon, 24 Oct 2022 17:51:36 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=YlQ2I3YyJdC/kWMXgQxgwKNwYSCMCk8MHaLxBDFwqonvHkH50GSj2wlRZcc/SJMD+14rToGQBuWWp3ZEpea6NAWgso8UxJ3jsRo9y5WHVaY5ejLVE8yTiC1h9G/Kp75sp9xjSIXv6Fp7xgSrPh538Qm8K+5fe80j6o+PbZSPMG6wttlwMsYPBnWCFJByeOZh5SrMgk0RijWBFoxBs4Np4hup05YuyTTyoswR6u0ZF4HD3ecP0bx/Y2y7pVuZ/5a78c0uez0EUrpyqoNLgqNGpOJ401XZq7z+ZE5ojWvk5K0CB5tZD9w3TRNxzzZwwCE1Swy+R92nCUsVrcywVe+YlQ== 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=SX4V/GbnJ5xLjKmoEN9ib7JXLodu5Cfh3OiginCtNss=; b=DpaZCn0/JJj+lv0TKb7y+weHwmyuFapAyiuFel9LfVg+M57oa4o/VLuRwUoBgWdUaEF4bOY61km3HQr2J0DYztw08R9DKgLea+mbvXCblnn3Zel2HqydO62UrxKwBxMUXXMS9DucfnWCVPup2Raa6GMdmN3gIU/48d8174HSc37SC2VZW1/OLtjIHk9VkTE44OPDObGX0sdwtOu05xLgg5nhQzWdkO0dJheJGAscnYqopmzQKQ+TsDKM4gPTTwL9nlX8SepmCSPGGuAF65DL7ZEC34t/Ozz2MlptPwRdqBZQTqMPXm8NbxMF5HKuirIljm4jgwT3vyNo9Xh2sWf5cA== 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 Received: from MW4PR11MB5821.namprd11.prod.outlook.com (2603:10b6:303:184::5) by PH7PR11MB6498.namprd11.prod.outlook.com (2603:10b6:510:1f1::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5746.21; Tue, 25 Oct 2022 00:51:35 +0000 Received: from MW4PR11MB5821.namprd11.prod.outlook.com ([fe80::a748:40a8:78d3:38e6]) by MW4PR11MB5821.namprd11.prod.outlook.com ([fe80::a748:40a8:78d3:38e6%3]) with mapi id 15.20.5723.033; Tue, 25 Oct 2022 00:51:35 +0000 From: "Nate DeSimone" To: "mikuback@linux.microsoft.com" , "devel@edk2.groups.io" CC: "Chiu, Chasel" , "Luo, Heng" , "Chaganty, Rangasai V" Subject: Re: [edk2-platforms][PATCH v2 0/3] Silicon/Intel: Fix invalid DEBUG() macros Thread-Topic: [edk2-platforms][PATCH v2 0/3] Silicon/Intel: Fix invalid DEBUG() macros Thread-Index: AQHY3kVB74lM+T7HKUCOUNnyutKL4a4eWyLQ Date: Tue, 25 Oct 2022 00:51:35 +0000 Message-ID: References: <20221012141610.668-1-mikuback@linux.microsoft.com> In-Reply-To: <20221012141610.668-1-mikuback@linux.microsoft.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: MW4PR11MB5821:EE_|PH7PR11MB6498:EE_ x-ms-office365-filtering-correlation-id: 14916001-1374-430d-39fa-08dab623110c x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: J1DRuLa06mMFlOMv9/JOpgADmX+XVWPlO4jOEmu/lSb4XnsTzEB8/gc/vAVpGc0h65g99ubNo3nJza6owT45hpEpfNg3eAvVcDvMHv5gG9iZ92sychvXUtvW0ezfR6Px3z+ZgNvDPMpXtkRjA9gcBkf26MBLmCur4lBsoIDc7IkAdOr00npUpQcEi0EjnEHpj6Cqpm648TUmUaE2NrqgtCVuqGhukzSi4ybDbGOoM92r9rKTAidC+FEuPlaESa9NFFvSnZQZn3oItaIlXz6NVqr0hsT5xzFKAA9wLWmGo+ZTLfjgpVF04nnp54508cUIopBghQdYxMNzqaalSBLuqAUyyBtHzxjkpP4v2pjYmUBiqdj2ANcxcB6bi7p4G7MP8aVV0b6xKzmm8M1CC2GFv4BLmvny5dPiYdCIqj4aoiSczpJm+aHIip7WHp0IC2dukYGIB4Qdc6v3MtdWESlZcO755M4lV7tv0dneBsQ1ahSdvVCZm730OVZoefbfRIwuigkHGJgIE1yrsQRbYD2IoqSJzkLyt06TLDF1N02bWiRjjXlqyfYkHnZgqIFKQfLH8B+6MEdUu5fuFa2aEAvkT2rh6iVX1w19ufPm0uKf/wWI80Iga8FN3FKKrVh9CpxBTdXhDarc5WoFkydvDbjL6PtNX1qBRbzUPLR4QtHGhsCuECnBJopfoZim+b1bSN3zdlCZ5mWcszCpz9JSHhAZyXaoQ8YmMW3aj76/iher58xKToke0Dj50KN7SxheY58+XxPYZF2vsPAO6qZf4NMNXAqhpfBoxybfN704gI5gKNY= x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:MW4PR11MB5821.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230022)(346002)(396003)(39860400002)(366004)(136003)(376002)(451199015)(64756008)(86362001)(76116006)(8676002)(4326008)(186003)(7696005)(66476007)(66556008)(478600001)(66446008)(66946007)(38100700002)(41300700001)(83380400001)(2906002)(6506007)(55016003)(33656002)(53546011)(82960400001)(110136005)(54906003)(107886003)(26005)(38070700005)(316002)(9686003)(71200400001)(52536014)(5660300002)(122000001)(8936002);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?q1ymJrnmdWuEYU1JqMkKVYUmUL9DK6/2a+wJxW1JrYYKg8YCOH9cTv8dlv6A?= =?us-ascii?Q?Jzg9xJDOMVzb1+VWc6YAscgleT0fLpU7OFZqI1KZCuCpR4rOAu8NHbWFo70E?= =?us-ascii?Q?axr5yJsJOMJj7xEZIr55JiM1HHZvJ7XiZNyo/z27hViBpZh5lJIDQCqwNjXM?= =?us-ascii?Q?B5V6ha1aYXz6Y7IG6mF75uh/Wun1VAAmr6E3k+RqnFHotUhrRimFFUG33+Gi?= =?us-ascii?Q?/r1qErU3RT0d2nDrekFU/HrUhTbcGh6zdHwJbKnPQ5ixJhDUbmg9fHFO8qBZ?= =?us-ascii?Q?kjYgMU7qGKl2gma0N8w9ZP5Ctf9iYPBSf4EyylwNbV494D+H5OzV6M2YraAQ?= =?us-ascii?Q?goB0XFlCChcitEprTJRTvuFc84W7TnCSJ3Cdmn+5q4MalirsRL+TFm4UF1Ro?= =?us-ascii?Q?wDik0YbcpYVLoRobYBayeQJ9hJLh+TMNLDDC0kSCLnzuzHq7izkaAT5XByu0?= =?us-ascii?Q?rpubiO1L54+GKy0DFOSNoTZPuDDbt9Cg5C7q83u+zJX1/TcLEhnEVd2ffF9d?= =?us-ascii?Q?UNDNve3OegXLmT7D2wN7WbAPGy3BMBSnZs9o51iEn/NDN/Wgg+yhqP63mlE2?= =?us-ascii?Q?wmgzVUs/E5prZoXp3EzIpDV9Z47K8lDE+QQGNZwvxPSFjJZ2RPGLK+jobbeF?= =?us-ascii?Q?Z1ndTqYR6RPoTwpTqmVKI051Cn0uRplUCiuXF4c4PZykFqgSNUZUQCWK8gLB?= =?us-ascii?Q?hXIWPsU+eyTK/4UWjgFpZO2kdyA+Vs40ziat7pc38D/EwEI/ktRXC42M2AC8?= =?us-ascii?Q?ajuixRmi3XFKGV9xAYbE5TTzepiDGWqQBcTPTHRpy9q7CIDmtn0VsFwuSxRQ?= =?us-ascii?Q?lmtIZqv4irWnuvN5iBXQKj/MwgabIjpQ6ySDlGKwGx0qFlO4LuV6XXBpRul6?= =?us-ascii?Q?n5PCDmSiRIHlviLBdruXuhk6zb3TX2Ebtsna+zMXklCLwtho/fjo1heiv03o?= =?us-ascii?Q?sxJOOFKvtJhUJfpBMGEk8iyA3TgvDmCTsEqTRRsxZ/lCyqPIB8vj/hRllet9?= =?us-ascii?Q?zge6xn/tHn7LYFGIekyWSjVjOKJea9B+5Xou7gLOZQnVWprtbX25bI6JA74J?= =?us-ascii?Q?hO7kJK+gnfI645fGvwf/5kYlRfsEBZx5/SwgHlBqxC99f14uXUyeGyXTg3Tp?= =?us-ascii?Q?H+qSnI+0TSx9VOkEg5RKboiTPgFqqW9j2J5oxrn0hjSDulBK976Qs++t2VDs?= =?us-ascii?Q?OOZDeLN2FG1q9yOYFufZRxeWUisu2u6uCN2owrtwQs51Ev5gD+HK6MQK+e6n?= =?us-ascii?Q?WL/luy2cxEejfFeFRTNcr63B73sZ3kEAExaAkwh3acHIVAsGoVXXp9SJEUmv?= =?us-ascii?Q?FFMTMu7XA7lJ21i99md7JjgM8bHWIGx/3F40dPgPdnE7nxFVI5gwhlJbR0NQ?= =?us-ascii?Q?1Ru+Zmp2TcV+baOnxeu3ZgoDzV2vUJDwPqi14ckTXCF16RTL7b0eWIQ7MK1E?= =?us-ascii?Q?YG6KabtebSrQr9zPTf8sW7TSzV6l/55VY+bAdEQpgedgND3sfGipftM+TJJ0?= =?us-ascii?Q?BX8fBsZZRkon44K49v1jzwXwgWrk/y3TE2myMRkOeTU9XKR/NCsOAolrkaV8?= =?us-ascii?Q?KIeOrkDZPRirCIDT8UrYiPW94G9DyM6Wn9WfD2/c6w8eIWdNjFEtSQsOhdKb?= =?us-ascii?Q?7A=3D=3D?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: MW4PR11MB5821.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 14916001-1374-430d-39fa-08dab623110c X-MS-Exchange-CrossTenant-originalarrivaltime: 25 Oct 2022 00:51:35.1989 (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: lb8+qBQjTFaxONN4oZ/88DJ7clWnesPJV0UO9vzZltKAPnQOkZumLoQQ5SZ0hyqlM05eqeIS06Knuqg9MV4vtHHBRMI/aWN/ylgQdI1w1fo= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR11MB6498 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 ba4318a~..8ea6ec3 Thanks, Nate -----Original Message----- From: mikuback@linux.microsoft.com =20 Sent: Wednesday, October 12, 2022 7:16 AM To: devel@edk2.groups.io Cc: Chiu, Chasel ; Luo, Heng ; D= esimone, Nathaniel L ; Chaganty, Rangasai V= Subject: [edk2-platforms][PATCH v2 0/3] Silicon/Intel: Fix invalid DEBUG() = macros From: Michael Kubacki REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D4095 Fixes several debug macros in Silicon/Intel that have a mismatched number o= f print specifiers to arguments. The original author's intention is not always 100% obvious. Though, this se= ries is relatively straightforward. v2 changes: - Applied debug macro suggestions in Nate's feedback Cc: Chasel Chiu Cc: Heng Luo Cc: Nate DeSimone Cc: Sai Chaganty Signed-off-by: Michael Kubacki Michael Kubacki (3): CoffeelakeSiliconPkg: Fix invalid debug macros KabylakeSiliconPkg: Fix invalid debug macros TigerlakeSiliconPkg: Fix invalid debug macros Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/CpuPrintPol= icy.c | 2 +- Silicon/Intel/CoffeelakeSiliconPkg/Pch/Library/PeiDxeSmmGbeMdiLib/GbeMdiLi= b.c | 2 +- Silicon/Intel/CoffeelakeSiliconPkg/Pch/Library/PeiOcWdtLib/PeiOcWdtLib.c = | 4 ++-- Silicon/Intel/CoffeelakeSiliconPkg/Pch/Library/Private/PeiDxeSmmPchPciExpr= essHelpersLib/PchPciExpressHelpersLibrary.c | 2 +- Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/CpuPrintPolic= y.c | 19 +++++++++++-------- Silicon/Intel/KabylakeSiliconPkg/Pch/Library/PeiOcWdtLib/PeiOcWdtLib.c = | 4 ++-- Silicon/Intel/TigerlakeSiliconPkg/IpBlock/Gbe/LibraryPrivate/PeiDxeSmmGbeM= diLib/GbeMdiLib.c | 2 +- Silicon/Intel/TigerlakeSiliconPkg/IpBlock/PcieRp/LibraryPrivate/PciExpress= HelpersLibrary/PciExpressHelpersLibrary.c | 2 +- 8 files changed, 20 insertions(+), 17 deletions(-) -- 2.28.0.windows.1