Hao, czw., 1 lis 2018 o 08:11 Wu, Hao A <hao.a...@intel.com> napisaĆ(a): > > Hi Marcin and Ard, > > > -----Original Message----- > > From: Marcin Wojtas [mailto:m...@semihalf.com] > > Sent: Friday, October 05, 2018 9:25 PM > > To: edk2-devel@lists.01.org > > Cc: Tian, Feng; Kinney, Michael D; Gao, Liming; leif.lindh...@linaro.org; > > Wu, > > Hao A; ard.biesheu...@linaro.org; nad...@marvell.com; > > m...@semihalf.com; j...@semihalf.com; t...@semihalf.com > > Subject: [PATCH v2 0/4] SdMmcOverride extension > > > > Hi, > > > > This is the second version of the patchset. Initial one was > > interleaved with the fixes, which after split got already merged. > > The biggest change is - resigning from the new callbacks > > and extending parameter lists of both NotifyPhase and Capability > > routines. > > > > Patches are available in the github: > > https://github.com/MarvellEmbeddedProcessors/edk2-open- > > platform/commits/sdmmc-override-upstream-r20181005 > > > > Please note that extending SdMmcOverride protocol was impacting > > so far the only user of it (Synquacer controller). In paralel > > edk2-platforms patchset, a patch can be found: > > ("Silicon/SynQuacer/PlatformDxe: adjust to updated SdMmcOverride") > > which immunizes for above and future extensions of the protocol: > > https://github.com/MarvellEmbeddedProcessors/edk2-open- > > platform/commits/xenon-upstream-r20181005 > > > > I'm looking forward to the comments and remarks. > > Since there are protocol service interface changes, maybe a version change > (macro EDKII_SD_MMC_OVERRIDE_PROTOCOL_VERSION) for the SdMmc Override Protocol > is needed. > > Please share your thoughts on this one. Thanks.
IMO such change is justified. I can add it in a separate on top, you can decide later with Ard, if it's good to merge it. Best regards, Marcin > > Best Regards, > Hao Wu > > > > > Best regards, > > Marcin > > > > Changelog: > > v1 -> v2 > > * Rebase onto newest master > > * 1/4 [new patch] - preparation for extending NotifyPhase > > * 2/4 - UhsSignaling as a part of NotifyPhase > > * 3/4 - SwitchClockFreqPost as a part of NotifyPhase > > * 4/4 - Allow updating BaseClkFreq via Capability instead of the > > independent callback. > > > > Marcin Wojtas (2): > > MdeModulePkg/SdMmcPciHcDxe: Add an optional parameter in > > NotifyPhase > > MdeModulePkg/SdMmcPciHcDxe: Allow overriding base clock frequency > > > > Tomasz Michalec (2): > > MdeModulePkg/SdMmcPciHcDxe: Add UhsSignaling to SdMmcOverride > > protocol > > MdeModulePkg/SdMmcPciHcDxe: Add SwitchClockFreqPost to > > SdMmcOverride > > > > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHcDxe.h | 6 + > > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHci.h | 62 +++++- > > MdeModulePkg/Include/Protocol/SdMmcOverride.h | 12 +- > > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/EmmcDevice.c | 215 > > ++++++++++++++------ > > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdDevice.c | 57 +++++- > > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHcDxe.c | 18 +- > > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHci.c | 108 ++++++++- > > - > > 7 files changed, 383 insertions(+), 95 deletions(-) > > > > -- > > 2.7.4 > _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel