Re: [edk2] [PATCH v2 00/16] UefiCpuPkg: Add CPU SMM and SecCore

2015-10-16 Thread Laszlo Ersek
On 10/15/15 01:26, Michael Kinney wrote: > Public branch: I diffed this against my own CRLF-ization of your v1 patches, plus two patches applied ("UefiCpuPkg: CpuDxe: Fix ASSERT() when only 1 CPU detected", "UefiCpuPkg: PiSmmCpuDxeSmm:

Re: [edk2] [PATCH v2 00/16] UefiCpuPkg: Add CPU SMM and SecCore

2015-10-16 Thread Fan, Jeff
It's good! Reviewed-by: Jeff Fan <jeff@intel.com> -Original Message- From: Kinney, Michael D Sent: Friday, October 16, 2015 10:54 AM To: Kinney, Michael D; edk2-devel@lists.01.org; Fan, Jeff Cc: Paolo Bonzini; Laszlo Ersek Subject: RE: [edk2] [PATCH v2 00/16] UefiCpuPkg: Add C

Re: [edk2] [PATCH v2 00/16] UefiCpuPkg: Add CPU SMM and SecCore

2015-10-15 Thread Kinney, Michael D
Hi Jeff, I have addressed the feedback you sent on code style issues. 1) Fix function header comment blocks. 2) Added missing .h files to PisSmmCpuDxeSmm INF file. I have posted a V3 public branch with your feedback incorporated at: