Loop in Ard and Marcin.
Hello Ard and Marcin,
We are proposing to make an extension to the SDMMC Override protocol to
more customization for platforms during the SD/eMMC device/controller
initialization process.
For convenience, the series is available at:
https://github.com/hwu25/edk2/tree/sdmmc_override_extension_v1
I only verified the change will not break the build for platforms
(in edk2-platform repo). Could you help provide feedbacks for this series?
Thanks in advance.
Best Regards,
Hao Wu
From: Albecki,
Mateusz
Sent: Thursday, June 13, 2019 9:44 PM
To: devel@edk2.groups.io
Cc: Wu, Hao A
Subject: RE: [PATCH 0/2] Add GetOperatingParam notify phase to
SdMmcOverride protocol
Resending to group.
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