From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 7B93C8195A for ; Wed, 4 Jan 2017 17:03:17 -0800 (PST) Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by orsmga103.jf.intel.com with ESMTP; 04 Jan 2017 17:03:17 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,318,1477983600"; d="scan'208,217";a="209767642" Received: from fmsmsx108.amr.corp.intel.com ([10.18.124.206]) by fmsmga004.fm.intel.com with ESMTP; 04 Jan 2017 17:03:16 -0800 Received: from fmsmsx123.amr.corp.intel.com (10.18.125.38) by FMSMSX108.amr.corp.intel.com (10.18.124.206) with Microsoft SMTP Server (TLS) id 14.3.248.2; Wed, 4 Jan 2017 17:03:17 -0800 Received: from shsmsx152.ccr.corp.intel.com (10.239.6.52) by fmsmsx123.amr.corp.intel.com (10.18.125.38) with Microsoft SMTP Server (TLS) id 14.3.248.2; Wed, 4 Jan 2017 17:03:16 -0800 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.88]) by SHSMSX152.ccr.corp.intel.com ([169.254.6.132]) with mapi id 14.03.0248.002; Thu, 5 Jan 2017 09:03:14 +0800 From: "Yao, Jiewen" To: "Justen, Jordan L" , Laszlo Ersek , edk2-devel-01 Thread-Topic: [edk2] [PATCH] OvmfPkg/SmmControl2Dxe: correct PCI_CONFIG_READ_WRITE in S3 boot script Thread-Index: AQHSZioVqFHUxmJepEqLnhObd/ZqR6EnplkAgACzhoCAALgKkA== Date: Thu, 5 Jan 2017 01:03:14 +0000 Message-ID: <74D8A39837DF1E4DA445A8C0B3885C503A8DB4FE@shsmsx102.ccr.corp.intel.com> References: <20161202104844.6093-1-lersek@redhat.com> <148349340944.16413.4237312013765520247@jljusten-ivb> <401681ee-93af-e490-88aa-28291bf41c1a@redhat.com> <148356731339.32514.12895184140738530358@jljusten-ivb> In-Reply-To: <148356731339.32514.12895184140738530358@jljusten-ivb> Accept-Language: zh-CN, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 X-Content-Filtered-By: Mailman/MimeDel 2.1.21 Subject: Re: [PATCH] OvmfPkg/SmmControl2Dxe: correct PCI_CONFIG_READ_WRITE in S3 boot script X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 05 Jan 2017 01:03:17 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Hi I agree we do use EFI_ prefix here. But using _EFI as suffix is also odd. :) Do we have better name? Thank you Yao Jiewen From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Jord= an Justen Sent: Thursday, January 5, 2017 6:02 AM To: Laszlo Ersek ; edk2-devel-01 Subject: Re: [edk2] [PATCH] OvmfPkg/SmmControl2Dxe: correct PCI_CONFIG_READ= _WRITE in S3 boot script On 2017-01-04 03:19:20, Laszlo Ersek wrote: > On 01/04/17 02:30, Jordan Justen wrote: > > On 2016-12-02 02:48:44, Laszlo Ersek wrote: > >> @@ -362,7 +390,9 @@ OnS3SaveStateInstalled ( > >> S3SaveState, > >> EFI_BOOT_SCRIPT_PCI_CONFIG_READ_WRITE_OPCOD= E, > >> EfiBootScriptWidthUint16, > >> - (UINT64)POWER_MGMT_REGISTER_Q35 (ICH9_GEN_P= MCON_1), > >> + ConvertPciLibToEfiPciAddress ( > >> + POWER_MGMT_REGISTER_Q35 (ICH9_GEN_PMCON_1= ) > > > > I think we should just add a EFI_POWER_MGMT_REGISTER_Q35 macro. > > I thought of that, but I didn't want to use the EFI_ prefix for a macro > that has nothing to do with the UEFI / PI specs. Can you suggest an > alternative name? Perhaps POWER_MGMT_REGISTER_Q35_EFI? Good point. Yeah, that name seems fine to me. -Jordan _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel