Abner: The change is good to me. Reviewed-by: Liming Gao <gaolim...@byosoft.com.cn>
Thanks Liming > -----邮件原件----- > 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Chang, Abner > via groups.io > 发送时间: 2024年4月4日 17:25 > 收件人: devel@edk2.groups.io > 抄送: Michael D Kinney <michael.d.kin...@intel.com>; Liming Gao > <gaolim...@byosoft.com.cn>; Zhiguang Liu <zhiguang....@intel.com>; Abdul > Lateef Attar <abdat...@amd.com>; Hao A Wu <hao.a...@intel.com>; Ray Ni > <ray...@intel.com>; Jian J Wang <jian.j.w...@intel.com>; Brit Chesley > <brit.ches...@amd.com> > 主题: [edk2-devel] [PATCH V5 0/6] SPI NOR Flash Driver for SFDP flash device > > From: Abner Chang <abner.ch...@amd.com> > > Sending V4 for review as SPI related ECR is approved by > PIWG in PI spec 1.8A > > In V5: Correct patch 6/6, as some source files are fixed in this patch. > In RESEND V4: Add Cc in patch 6/6 > In V4: Enhance timeout algorithm > In V3: Relocate GUIDs/PCDs for SPI SFDP drivers to > MdeModulePkg.dec. > In V2: Add maintainer entry for Bus/Spi. > > This patch fixes the missing SPI defintions per to PI spec > 1.8 errata A. > This patch also provides the implemtation of SPI NOR flash > driver that supports JEDEC SFDP specification. > This driver support the flash device that has the GUID > gEdk2JedecSfdpSpiDxeDriverGuid or gEdk2JedecSfdpSpiSmmDriverGuid > declared in EFI_SPI_PERIPHERAL SpiPeripheralDriverGuid structure. > > The driver has been verified with Macronix MX25U6435F flash device. > > Signed-off-by: Abner Chang <abner.ch...@amd.com> > Cc: Michael D Kinney <michael.d.kin...@intel.com> > Cc: Liming Gao <gaolim...@byosoft.com.cn> > Cc: Zhiguang Liu <zhiguang....@intel.com> > Cc: Abdul Lateef Attar <abdat...@amd.com> > Cc: Hao A Wu <hao.a...@intel.com> > Cc: Ray Ni <ray...@intel.com> > Cc: Jian J Wang <jian.j.w...@intel.com> > Cc: Liming Gao <gaolim...@byosoft.com.cn> > Cc: Brit Chesley <brit.ches...@amd.com> > > Abner Chang (2): > MdeModulePkg: Add definitions in DEC for SPI NOR Flash SFDP driver > Maintainers: AMD as SPI driver stack maintainer > > abnchang (4): > MdePkg/Include: Update definitions of SPI related header files > MdePkg/Include: Add SPI NOR Flash JEDEC SFDP header file > MdeModulePkg/SpiNorFlashJedecSfdp: SPI NOR Flash JEDEC SFDP > MdeModulePkg: Add SPI NOR FLash SFDP drivers to DSC > > MdeModulePkg/MdeModulePkg.dec | 28 +- > MdeModulePkg/MdeModulePkg.dsc | 4 +- > .../SpiNorFlashJedecSfdpDxe.inf | 64 + > .../SpiNorFlashJedecSfdpSmm.inf | 64 + > .../Spi/SpiNorFlashJedecSfdp/SpiNorFlash.h | 286 +++ > .../SpiNorFlashJedecSfdpInternal.h | 299 +++ > .../IndustryStandard/SpiNorFlashJedecSfdp.h | 324 +++ > MdePkg/Include/Protocol/SpiConfiguration.h | 10 +- > MdePkg/Include/Protocol/SpiHc.h | 16 +- > MdePkg/Include/Protocol/SpiIo.h | 12 +- > .../Spi/SpiNorFlashJedecSfdp/SpiNorFlash.c | 1141 +++++++++++ > .../SpiNorFlashJedecSfdp.c | 1780 > +++++++++++++++++ > .../SpiNorFlashJedecSfdpDxe.c | 261 +++ > .../SpiNorFlashJedecSfdpSmm.c | 234 +++ > Maintainers.txt | 11 + > .../SpiNorFlashJedecSfdpDxe.uni | 13 + > .../SpiNorFlashJedecSfdpExtra.uni | 11 + > .../SpiNorFlashJedecSfdpSmm.uni | 13 + > 18 files changed, 4566 insertions(+), 5 deletions(-) > create mode 100644 > MdeModulePkg/Bus/Spi/SpiNorFlashJedecSfdp/SpiNorFlashJedecSfdpDxe.inf > create mode 100644 > MdeModulePkg/Bus/Spi/SpiNorFlashJedecSfdp/SpiNorFlashJedecSfdpSmm.in > f > create mode 100644 > MdeModulePkg/Bus/Spi/SpiNorFlashJedecSfdp/SpiNorFlash.h > create mode 100644 > MdeModulePkg/Bus/Spi/SpiNorFlashJedecSfdp/SpiNorFlashJedecSfdpInternal > .h > create mode 100644 > MdePkg/Include/IndustryStandard/SpiNorFlashJedecSfdp.h > create mode 100644 > MdeModulePkg/Bus/Spi/SpiNorFlashJedecSfdp/SpiNorFlash.c > create mode 100644 > MdeModulePkg/Bus/Spi/SpiNorFlashJedecSfdp/SpiNorFlashJedecSfdp.c > create mode 100644 > MdeModulePkg/Bus/Spi/SpiNorFlashJedecSfdp/SpiNorFlashJedecSfdpDxe.c > create mode 100644 > MdeModulePkg/Bus/Spi/SpiNorFlashJedecSfdp/SpiNorFlashJedecSfdpSmm.c > create mode 100644 > MdeModulePkg/Bus/Spi/SpiNorFlashJedecSfdp/SpiNorFlashJedecSfdpDxe.uni > create mode 100644 > MdeModulePkg/Bus/Spi/SpiNorFlashJedecSfdp/SpiNorFlashJedecSfdpExtra.u > ni > create mode 100644 > MdeModulePkg/Bus/Spi/SpiNorFlashJedecSfdp/SpiNorFlashJedecSfdpSmm.u > ni > > -- > 2.37.1.windows.1 > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#117531): https://edk2.groups.io/g/devel/message/117531 Mute This Topic: https://groups.io/mt/105417086/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-