Re: [edk2] [PATCH] IntelFsp2WrapperPkg: Add a PCD to control if signaling first event.

2016-10-17 Thread Mudusuru, Giri P
2016 3:53 PM > To: edk2-devel@lists.01.org > Cc: Yao, Jiewen ; Mudusuru, Giri P > ; Dong, Guo > Subject: [edk2] [PATCH] IntelFsp2WrapperPkg: Add a PCD to control if signaling > first event. > > PciEnumerationComplete might be signaled to FSP in Coreboot. Add a > PCD P

Re: [edk2] [PATCH] IntelFsp2WrapperPkg: Add a PCD to control if signaling first event.

2016-10-17 Thread Yao, Jiewen
a...@intel.com>> Subject: RE: [edk2] [PATCH] IntelFsp2WrapperPkg: Add a PCD to control if signaling first event. Hello Guo, FSP wrapper and Coreboot are bootloaders consuming FSP binary. FSP must get called Post PCI bus enumeration to do the required silicon initialization. Why do we

Re: [edk2] [PATCH] IntelFsp2WrapperPkg: Add a PCD to control if signaling first event.

2016-10-21 Thread Yao, Jiewen
ewen From: Dong, Guo Sent: Tuesday, October 18, 2016 1:11 PM To: Yao, Jiewen ; Mudusuru, Giri P ; edk2-devel@lists.01.org Cc: Ma, Maurice ; Yarlagadda, Satya P Subject: RE: [edk2] [PATCH] IntelFsp2WrapperPkg: Add a PCD to control if signaling first event. Hi Jiewen, For Coreboot boot loader,

Re: [edk2] [PATCH] IntelFsp2WrapperPkg: Add a PCD to control if signaling first event.

2016-10-21 Thread Dong, Guo
: Ma, Maurice mailto:maurice...@intel.com>>; Yarlagadda, Satya P mailto:satya.p.yarlaga...@intel.com>> Subject: RE: [edk2] [PATCH] IntelFsp2WrapperPkg: Add a PCD to control if signaling first event. Hi Jiewen, For Coreboot boot loader, PCI device enumeration is done in c

Re: [edk2] [PATCH] IntelFsp2WrapperPkg: Add a PCD to control if signaling first event.

2016-10-21 Thread Yao, Jiewen
combination in the future. From: Dong, Guo Sent: Saturday, October 22, 2016 7:52 AM To: Yao, Jiewen ; Mudusuru, Giri P ; edk2-devel@lists.01.org Cc: Ma, Maurice ; Yarlagadda, Satya P Subject: RE: [edk2] [PATCH] IntelFsp2WrapperPkg: Add a PCD to control if signaling first event. I am OK with bit

Re: [edk2] [PATCH] IntelFsp2WrapperPkg: Add a PCD to control if signaling first event.

2016-10-23 Thread Mudusuru, Giri P
Satya P Subject: RE: [edk2] [PATCH] IntelFsp2WrapperPkg: Add a PCD to control if signaling first event. My thinking is to define mask for ALL APIs, not 3. We have 5 APIs, and 3 sub API for notify phase. How about this : # BIT0~15 is for function: # BIT0 - Mask TempRamInit # BIT1 - Mask Memor

Re: [edk2] [PATCH] IntelFsp2WrapperPkg: Add a PCD to control if signaling first event.

2016-10-24 Thread Dong, Guo
1.org> Cc: Ma, Maurice mailto:maurice...@intel.com>>; Yarlagadda, Satya P mailto:satya.p.yarlaga...@intel.com>> Subject: RE: [edk2] [PATCH] IntelFsp2WrapperPkg: Add a PCD to control if signaling first event. I am OK with bit mask PCD. Now only 3 APIs in this driver. Do we really need