Hi, wt., 24 gru 2019 o 03:52 Wu, Hao A <hao.a...@intel.com> napisał(a): > > > -----Original Message----- > > From: Albecki, Mateusz > > Sent: Saturday, December 21, 2019 1:13 AM > > To: devel@edk2.groups.io > > Cc: Albecki, Mateusz; Wu, Hao A; Marcin Wojtas; Gao, Zhichao; Gao, Liming > > Subject: [PATCH 0/2] MdeModulePkg/SdMmcPciHcDxe: Send the > > EdkiiSdMmcSwitchClockFreq notification before sending CMD13 > > > > The first patch refactors the SdMmcClockSupply function with a goal > > of sending the EdkiiSdMmcSwitchClockFreq notification before we send the > > CMD13 to check the switch status in eMMC init flow. This is required to > > avoid sending the CMD13 on link that still has not been fixed by platform. > > > > To avoid changing the driver behavior we avoid sending notifications > > when the clock is setup for the first time or when we setup the clock > > after the voltage switch procedure(adressed in second patch). > > > > The second patch in the series optimizes the SD card detection routine > > to stop it from going through the process of internal clock setup > > after switching the voltage. According to SD HC specification there > > is no need to setup internal clock all over again. > > > > Tests performed: > > - Booted eMMC in HS400 mode on platform which required post clock freq > > fixes > > > > I wasn't able to test SD card yet due to the lack of setup with working SD. > > > I performed a quick verification on the eMMC device and SD card on my side. > They work properly after the series. > > So for the series, > Tested-by: Hao A Wu <hao.a...@intel.com> >
I verified eMMC HS200 / HS@50MHz and SD cards on 2 Armada platforms, so: Tested-by: Marcin Wojtas <m...@semihalf.com> Best regards, Marcin > > > > > The patch series is available on github here: > > https://github.com/malbecki/edk2/tree/sdmmc_post_freq_notify > > > Add Ard to the loop to see if there is additional comment. > > Best Regards, > Hao Wu > > > > > > Cc: Hao A Wu <hao.a...@intel.com> > > Cc: Marcin Wojtas <m...@semihalf.com> > > Cc: Zhichao Gao <zhichao....@intel.com> > > Cc: Liming Gao <liming....@intel.com> > > > > Mateusz Albecki (2): > > SdMmcPciHcDxe: Send EdkiiSdMmcSwitchClockFreq after SD clock start > > MdeModulePkg/SdMmcPciHcDxe: Add function to start SD clock > > > > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/EmmcDevice.c | 20 +-- > > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdDevice.c | 25 +--- > > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHcDxe.h | 24 ++++ > > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHci.c | 136 > > +++++++++++---------- > > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHci.h | 45 ++----- > > 5 files changed, 112 insertions(+), 138 deletions(-) > > > > -- > > 2.14.1.windows.1 > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#52820): https://edk2.groups.io/g/devel/message/52820 Mute This Topic: https://groups.io/mt/68852773/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
