Re: [edk2-devel] [Patch v2] IntelSiliconPkg/Include/Library:Add ConfigBlockLib.h

2019-12-24 Thread Ethan Tsao
Hi Ray, ConfigBlockLib consume the data structure from ConfigBlock.h. Which is located in KabylakeSiliconPkg and CoffeelakeSiliconPkg. Should we copy them to IntelSiliconPkg as well and delete? Best Regards, Ethan > -Original Message- > From: Ni, Ray > Sent: Tuesday, December 24, 2019

Re: [edk2-devel] [Patch v2] IntelSiliconPkg/Include/Library:Add ConfigBlockLib.h

2019-12-23 Thread Ni, Ray
Reviewed-by: Ray Ni > -Original Message- > From: Tsao, Ethan > Sent: Monday, December 23, 2019 4:25 PM > To: devel@edk2.groups.io > Cc: Ni, Ray ; Chaganty, Rangasai V > > Subject: [Patch v2] IntelSiliconPkg/Include/Library:Add ConfigBlockLib.h > > REF:https://bugzilla.tianocore.org/sho

[edk2-devel] [Patch v2] IntelSiliconPkg/Include/Library:Add ConfigBlockLib.h

2019-12-23 Thread Ethan Tsao
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2318 Create header for BaseConfigBlockLib in InstelSiliconPkg and remove all other ConfigBlock.h Signed-off-by: Ethan Tsao Cc: Ray Ni Cc: Rangasai V Chaganty --- Silicon/Intel/CoffeelakeSiliconPkg/CoffeelakeSiliconPkg.dsc