From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by mx.groups.io with SMTP id smtpd.web11.9314.1586527166348310420 for ; Fri, 10 Apr 2020 06:59:26 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.136, mailfrom: liming.gao@intel.com) IronPort-SDR: nNZwdAVWWB8XrZcqcrQN2LCUh5TstrLsIzcRqGuZy+mNigN+1uND76h54tgJqebHXXgYdTPYEb 8zncYAuvHbMw== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Apr 2020 06:59:25 -0700 IronPort-SDR: 3hmX7DE7yAowUBxG7gxvodbgVH26ppvVGQwJIA9WVBfog+snKrJBH8ad8gBvzk6rmiMsbQzbWd 0AWlHe4bgcgw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.72,367,1580803200"; d="scan'208";a="331184521" Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by orsmga001.jf.intel.com with ESMTP; 10 Apr 2020 06:59:12 -0700 Received: from shsmsx604.ccr.corp.intel.com (10.109.6.214) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.439.0; Fri, 10 Apr 2020 06:59:12 -0700 Received: from shsmsx604.ccr.corp.intel.com (10.109.6.214) by SHSMSX604.ccr.corp.intel.com (10.109.6.214) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Fri, 10 Apr 2020 21:59:10 +0800 Received: from shsmsx604.ccr.corp.intel.com ([10.109.6.214]) by SHSMSX604.ccr.corp.intel.com ([10.109.6.214]) with mapi id 15.01.1713.004; Fri, 10 Apr 2020 21:59:10 +0800 From: "Liming Gao" To: "devel@edk2.groups.io" , "ard.biesheuvel@arm.com" CC: "Wang, Jian J" , "Wu, Hao A" , "Bi, Dandan" Subject: Re: [edk2-devel] [PATCH 0/2] MdeModulePkg/DxeCore: clean up image loading functions Thread-Topic: [edk2-devel] [PATCH 0/2] MdeModulePkg/DxeCore: clean up image loading functions Thread-Index: AQHWDolRazDNPd5kOESqfD4yDidtcqhyYzWg Date: Fri, 10 Apr 2020 13:59:10 +0000 Message-ID: References: <20200409160948.23427-1-ard.biesheuvel@arm.com> In-Reply-To: <20200409160948.23427-1-ard.biesheuvel@arm.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-version: 11.2.0.6 dlp-product: dlpe-windows dlp-reaction: no-action x-originating-ip: [10.239.127.36] MIME-Version: 1.0 Return-Path: liming.gao@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Ard: What's purpose for this change? Only code clean up? Is there any other b= enefit? > -----Original Message----- > From: devel@edk2.groups.io On Behalf Of Ard Biesh= euvel > Sent: Friday, April 10, 2020 12:10 AM > To: devel@edk2.groups.io > Cc: Wang, Jian J ; Wu, Hao A = ; Ard Biesheuvel > Subject: [edk2-devel] [PATCH 0/2] MdeModulePkg/DxeCore: clean up image l= oading functions >=20 > A pair of cleanup patches for the LoadImage/StartImage implementations > in DxeCore. >=20 > https://github.com/ardbiesheuvel/edk2/pull/new/dxecore-loadimage-cleanup >=20 > Ard Biesheuvel (2): > MdeModulePkg/DxeCore/Image: make local functions STATIC > MdeModulePkg/DxeCore/Image: remove unused function arguments >=20 > MdeModulePkg/Core/Dxe/Image/Image.c | 91 ++++---------------- > 1 file changed, 17 insertions(+), 74 deletions(-) >=20 > -- > 2.17.1 >=20 >=20 >=20