Re: [edk2] [patch] UefiCpuPkg/PiSmmCpu: Move RestoreSmmConfigurationInS3 function to PerformPreTasks().

2015-11-24 Thread Kinney, Michael D
Reviewed-by: Michael Kinney > -Original Message- > From: Yao, Jiewen > Sent: Tuesday, November 24, 2015 12:36 AM > To: edk2-de...@ml01.01.org > Cc: Yao, Jiewen ; Fan, Jeff ; > Kinney, Michael D

[edk2] [patch] UefiCpuPkg/PiSmmCpu: Move RestoreSmmConfigurationInS3 function to PerformPreTasks().

2015-11-24 Thread jiewen yao
In this way, we can centralize the silicon configuration in PerformRemainingTasks()/PerformPreTasks() function. If there are more features need to be configured, they can put in PerformRemainingTasks()/PerformPreTasks() only. Contributed-under: TianoCore Contribution Agreement 1.0

Re: [edk2] [patch] UefiCpuPkg/PiSmmCpu: Move RestoreSmmConfigurationInS3 function to PerformPreTasks().

2015-11-24 Thread Laszlo Ersek
On 11/24/15 09:36, jiewen yao wrote: > In this way, we can centralize the silicon configuration in > PerformRemainingTasks()/PerformPreTasks() function. > If there are more features need to be configured, they can put in > PerformRemainingTasks()/PerformPreTasks() only. > > Contributed-under:

Re: [edk2] [patch] UefiCpuPkg/PiSmmCpu: Move RestoreSmmConfigurationInS3 function to PerformPreTasks().

2015-11-24 Thread Yao, Jiewen
Got it. I will remove the two leading space characters. Thanks! -Original Message- From: Laszlo Ersek [mailto:ler...@redhat.com] Sent: Tuesday, November 24, 2015 6:21 PM To: Yao, Jiewen; edk2-de...@ml01.01.org Cc: Fan, Jeff; Kinney, Michael D Subject: Re: [patch] UefiCpuPkg/PiSmmCpu: