From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by ml01.01.org (Postfix) with ESMTP id E24AA1A1DF1 for ; Wed, 10 Aug 2016 23:05:38 -0700 (PDT) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga101.jf.intel.com with ESMTP; 10 Aug 2016 23:05:38 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.28,503,1464678000"; d="scan'208";a="1012363513" Received: from dmangels-mobl1.amr.corp.intel.com (HELO localhost) ([10.252.128.101]) by orsmga001.jf.intel.com with ESMTP; 10 Aug 2016 23:05:38 -0700 MIME-Version: 1.0 To: "Ni, Ruiyu" , "edk2-devel@lists.01.org" Message-ID: <147089553827.3499.1443547713216811039@jljusten-ivb> From: Jordan Justen In-Reply-To: <734D49CCEBEEF84792F5B80ED585239D1ADB687A@SHSMSX103.ccr.corp.intel.com> Cc: "Tian, Feng" , "Chan, Amy" References: <20160810055611.266232-1-ruiyu.ni@intel.com> <20160810055611.266232-12-ruiyu.ni@intel.com> <147085038216.19680.17963102819835257407@jljusten-ivb> <734D49CCEBEEF84792F5B80ED585239D1ADB687A@SHSMSX103.ccr.corp.intel.com> User-Agent: alot/0.3.7 Date: Wed, 10 Aug 2016 23:05:38 -0700 Subject: Re: [PATCH 11/11] MdeModulePkg/ResetSystemRuntimeDxe: Support EfiResetPlatformSpecific 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, 11 Aug 2016 06:05:39 -0000 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable On 2016-08-10 19:43:03, Ni, Ruiyu wrote: > = > > -----Original Message----- > > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of > > Jordan Justen > > = > > On 2016-08-09 22:56:11, Ruiyu Ni wrote: > > > + indicate the reason for the system r= eset. ResetData is only > > > + valid if ResetStatus is something ot= her than EFI_SUCCESS > > > + unless the ResetType is EfiResetPlat= formSpecific > > > + where a minimum amount of ResetData = is always > > required. > > = > > Most of the patches in this series have lines that are longer than 80 c= olumns. > > Can you fix that? > = > Do we have rule to limit the line to 80 columns? I saw many files have lo= ng > lines. > Or I can change OvmfPkg changes to follow the 80 rule. What do you think? Yes, it is in the code style document. You should update all patches for it. It looks like most of them will just be copy/paste. > = > > = > > I think you should move patch 11 before patch 09 "OvmfPkg: Use > > MdeModulePkg/ResetSystemRuntimeDxe". I think this should allow reset to > > continue working through the entire series for OVMF. (right?) > = > Reset works using my current order. > But I agree moving #11 before #9 can move ResetPlatformSpecific support > earilier, and group the ResetPlatformSpecific patches together. I will do= that. Oh, I see. I guess this patch updates the ResetPlatformSpecific type, so it probably won't be too important for OVMF. I think it might be a little better earlier, but you can decide if you want to move it or not. -Jordan