Re: [edk2-devel] [PATCH v6 06/52] IntelSiliconPkg: Add PCH SPI PPI

2021-10-29 Thread Michael Kubacki
Hi Nate, I will send a separate patch series that removes the v1 PPI and Protocol. Thanks, Michael On 10/29/2021 6:22 PM, Nate DeSimone wrote: Hi Michael, Looking at your new patch series, now that we have the new PCH_SPI2_PPI, there is no longer any need for the existence of IntelSiliconPkg

Re: [edk2-devel] [PATCH v6 06/52] IntelSiliconPkg: Add PCH SPI PPI

2021-10-29 Thread Nate DeSimone
Hi Michael, Looking at your new patch series, now that we have the new PCH_SPI2_PPI, there is no longer any need for the existence of IntelSiliconPkg/Include/Ppi/Spi.h. The PCH_SPI_PPI will always be a silicon generation specific protocol. There is no need to promote that protocol to the multi-

[edk2-devel] [PATCH v6 06/52] IntelSiliconPkg: Add PCH SPI PPI

2021-10-28 Thread Michael Kubacki
From: Michael Kubacki REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3307 This SPI PPI definition is intended to serve as the single definition for Intel platform and silicon packages. Cc: Ray Ni Cc: Rangasai V Chaganty Signed-off-by: Michael Kubacki Reviewed-by: Nate DeSimone Reviewed-