Re: [edk2-devel] [edk2platforms][PATCH v3 1/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-12-05 Thread Ni, Ray
Better to remove the signature. -Original Message- From: Czajkowski, Maciej Sent: Saturday, December 4, 2021 12:15 AM To: Ni, Ray ; devel@edk2.groups.io Cc: Chaganty, Rangasai V Subject: RE: [edk2platforms][PATCH v3 1/1] IntelSiliconPkg: Add IntelDieInfoProtocol Hi Ray, Thanks for

Re: [edk2-devel] [edk2platforms][PATCH v3 1/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-12-03 Thread Maciej Czajkowski
Hi Ray, Thanks for the feedback. This field has been added since a couple of protocols are still using that, and we thought it is used. If you think that it is not necessary and it is not commonly used, it can be deleted in the next patch. Regards, Maciej -Original Message- From: Ni,

Re: [edk2-devel] [edk2platforms][PATCH v3 1/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-11-30 Thread Ni, Ray
Is there a reason to keep "Signature" field? Do you expect consumers check the Signature before using? It's not a common practice in EDKII today. Thanks, ray > -Original Message- > From: Czajkowski, Maciej > Sent: Thursday, November 18, 2021 11:16 PM > To: devel@edk2.groups.io > Cc:

[edk2-devel] [edk2platforms][PATCH v3 1/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-11-18 Thread Maciej Czajkowski
Added IntelDieInfo header into IntelSiliconPkg tree. The purpose is to have generic and unified interface for getting information about dies installed in the system. It will be implemented by silicon code. Cc: Ray Ni Cc: Rangasai V Chaganty Signed-off-by: Maciej Czajkowski --- Notes: v3: