Resending to group. From: Wu, Hao A Sent: Monday, June 10, 2019 5:18 AM To: Albecki, Mateusz Subject: RE: [PATCH 0/2] Add GetOperatingParam notify phase to SdMmcOverride protocol Hello Mateusz, Thanks for the patch series and sorry for the delayed response. One question first, I did not see the patches being sent to the mailing list 'devel@edk2.groups.io'. We have recently switched to this new mailing list back to early April, the old one 'edk2-devel@lists.01.org' is no longer being used. In case that you have not registered the new list, you can sign up at: https://edk2.groups.io/g/devel or email: devel+subscribe@edk2.groups.io to get registered in the new mailing list. Or you can send me the patch files and I will help to send them to the mailing list. Below are some general level feedbacks (Other patch-specific feedbacks will be replied in other mails): * Could you help to add the below Bugzilla tracker reference in the commit log message? https://bugzilla.tianocore.org/show_bug.cgi?id=1882 * Could you help to run the script 'BaseTools/Scripts/PatchCheck.py' and resolve its report on the commit log format issues? Checking git commit: 6e3f64aeb4 The commit message format is not valid: * Contributed-under! (Note: this must be removed by the code contributor!) * Add quotes (") around name with a comma: Albecki, Mateusz * Email format is invalid: hao.a.wu@intel.com * Line 15 of commit message is too long. https://github.com/tianocore/tianocore.github.io/wiki/Commit-Message-Format The code passed all checks. Checking git commit: 91595ae911 The commit message format is not valid: * Contributed-under! (Note: this must be removed by the code contributor!) * Add quotes (") around name with a comma: Albecki, Mateusz * Email format is invalid: hao.a.wu@intel.com https://github.com/tianocore/tianocore.github.io/wiki/Commit-Message-Format The code passed all checks. * Could you help to resolve the typo 'Prefered' to 'Preferred' in the naming of some variables? * Could you help to use lowercase 'static' (instead of using the uppercase one) for the new functions? Also, I only verified the build of the patch series. Please grant me some time to verify on the HW on my side. Maybe your updated patches arrive first and I will then verify the update ones. Best Regards, Hao Wu > -----Original Message----- > From: Albecki, Mateusz > Sent: Monday, June 03, 2019 7:34 PM > Cc: Albecki, Mateusz; Wu, Hao A > Subject: [PATCH 0/2] Add GetOperatingParam notify phase to > SdMmcOverride protocol > > To allow platform greater control over the bus settings for SD card and > eMMC card > we have added a new notify phase to SdMmcOverrideProtocol called > GetOperatingParam. > This phase is signaled before SD card/eMMC initialization and allows platform > to tweak > the values in new structure called > EDKII_SD_MMC_OPERATING_PARAMETERS which allows to configure > bus width, clock frequency and driver strength. Other bus parameters can be > configured by > overriding host controller capabilities. > > Tests: > - OS boot from eMMC without SdMmcOverride protocol installed > - OS boot from eMMC with SdMmcOverride installed and clock frequency > lowered to 100MHz in HS200 > - OS boot from eMMC with driver strength changed to Type1 > - SD card enumeration in UEFI shell on default speed and high speed(non > UHS-I) with SdMmcOverride installed and UHS-I disabled in capability > > > Cc: hao.a.wu@intel.com > > Albecki, Mateusz (2): > MdeModulePkg/SdMmcOverride: Add GetOperatingParam notify phase > MdeModulePkg/SdMmcHcDxe: Implement revision 3 of > SdMmcOverrideProtocol > > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/EmmcDevice.c | 522 > +++++++++++++++------ > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdDevice.c | 381 > ++++++++++++--- > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHcDxe.c | 52 +- > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHcDxe.h | 18 +- > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHci.c | 8 +- > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHci.h | 8 +- > MdeModulePkg/Include/Protocol/SdMmcOverride.h | 58 ++- > 7 files changed, 802 insertions(+), 245 deletions(-) > > -- > 2.14.1.windows.1 -------------------------------------------------------------------- Intel Technology Poland sp. z o.o. ul. Slowackiego 173 | 80-298 Gdansk | Sad Rejonowy Gdansk Polnoc | VII Wydzial Gospodarczy Krajowego Rejestru Sadowego - KRS 101882 | NIP 957-07-52-316 | Kapital zakladowy 200.000 PLN. Ta wiadomosc wraz z zalacznikami jest przeznaczona dla okreslonego adresata i moze zawierac informacje poufne. W razie przypadkowego otrzymania tej wiadomosci, prosimy o powiadomienie nadawcy oraz trwale jej usuniecie; jakiekolwiek przegladanie lub rozpowszechnianie jest zabronione. This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is strictly prohibited.