MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.c | 25 ++- MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.h | 44 +++++- .../Bus/Ufs/UfsPassThruDxe/UfsPassThruDxe.inf | 3 +- .../Bus/Ufs/UfsPassThruDxe/UfsPassThruHci.c | 172 ++++++++++++++++----- .../Include/Protocol/UfsHostControllerPlatform.h | 124 +++++++++++++++ MdeModulePkg/MdeModulePkg.dec | 3 + 6 files changed, 324 insertions(+), 47 deletions(-) create mode 100644 MdeModulePkg/Include/Protocol/UfsHostControllerPlatform.h
To cover additional host controller programming mentioned in the UFS specification we have added an additional protocol that allows the UEFI driver to give control to platform driver. This allows the platform to perform any additional steps needed for the stable operation. Test coverage: Tested on platform with UFS 2.1 host controller with Samsung UFS2.0 part with 3 LUs enabled All LUs have been enumerated in boot manager. Tested that enumeration works without platform protocol installed(on host controller that can support it) Tested that enumeration works with platform protocol installed and with additional programming steps after link startup(power mode change to GEAR2). Cc: Hao A Wu <hao.a.wu@intel.com> Mateusz Albecki (4): MdeModulePkg: Add definition of the EDKII_UFS_HC_PLATFORM_PROTOCOL MdeModulePkg/UfsPassThruDxe: Refactor UfsExecUicCommand function MdeModulePkg/UfsPassThruDxe: Refactor private data to use EDKII_UFS_HC_INFO MdeModulePkg/UfsPassThruDxe: Implement EDKII_UFS_HC_PLATFORM_PROTOCOL MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.c | 25 ++- MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.h | 44 +++++- .../Bus/Ufs/UfsPassThruDxe/UfsPassThruDxe.inf | 3 +- .../Bus/Ufs/UfsPassThruDxe/UfsPassThruHci.c | 172 ++++++++++++++++----- .../Include/Protocol/UfsHostControllerPlatform.h | 124 +++++++++++++++ MdeModulePkg/MdeModulePkg.dec | 3 + 6 files changed, 324 insertions(+), 47 deletions(-) create mode 100644 MdeModulePkg/Include/Protocol/UfsHostControllerPlatform.h -- 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. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#45022): https://edk2.groups.io/g/devel/message/45022 Mute This Topic: https://groups.io/mt/32784353/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
Dear stewards and Liming, Sorry for top-posting. I plan to add the below new feature which is targeting for the upcoming edk2-stable201908 tag: * Add EDKII_UFS_HC_PLATFORM_PROTOCOL to support platform specific programming of UFS host controllers The BZ tracker for the feature request is at: https://bugzilla.tianocore.org/show_bug.cgi?id=1343 The 1st version of the patch series has been proposed at: https://edk2.groups.io/g/devel/message/45022 And for the series, I think the changes are good in general and I plan to give my RB tag with some minor modifications. If you have any concern with regard to adding this feature in the upcoming stable tag, please help to raise. Thanks in advance. Best Regards, Hao Wu > -----Original Message----- > From: Albecki, Mateusz > Sent: Thursday, August 08, 2019 12:51 AM > To: devel@edk2.groups.io > Cc: Albecki, Mateusz; Wu, Hao A > Subject: [PATCH 0/4] Add EDKII_UFS_HC_PLATFORM_PROTOCOL to support > platform specific programming of UFS host controllers > > To cover additional host controller programming mentioned in the UFS > specification > we have added an additional protocol that allows the UEFI driver to give > control > to platform driver. This allows the platform to perform any additional steps > needed for the stable operation. > > Test coverage: > Tested on platform with UFS 2.1 host controller with Samsung UFS2.0 part > with 3 LUs enabled > All LUs have been enumerated in boot manager. > Tested that enumeration works without platform protocol installed(on host > controller that can support it) > Tested that enumeration works with platform protocol installed and with > additional programming steps after > link startup(power mode change to GEAR2). > > Cc: Hao A Wu <hao.a.wu@intel.com> > > Mateusz Albecki (4): > MdeModulePkg: Add definition of the > EDKII_UFS_HC_PLATFORM_PROTOCOL > MdeModulePkg/UfsPassThruDxe: Refactor UfsExecUicCommand function > MdeModulePkg/UfsPassThruDxe: Refactor private data to use > EDKII_UFS_HC_INFO > MdeModulePkg/UfsPassThruDxe: Implement > EDKII_UFS_HC_PLATFORM_PROTOCOL > > MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.c | 25 ++- > MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.h | 44 +++++- > .../Bus/Ufs/UfsPassThruDxe/UfsPassThruDxe.inf | 3 +- > .../Bus/Ufs/UfsPassThruDxe/UfsPassThruHci.c | 172 > ++++++++++++++++----- > .../Include/Protocol/UfsHostControllerPlatform.h | 124 +++++++++++++++ > MdeModulePkg/MdeModulePkg.dec | 3 + > 6 files changed, 324 insertions(+), 47 deletions(-) > create mode 100644 > MdeModulePkg/Include/Protocol/UfsHostControllerPlatform.h > > -- > 2.14.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#45110): https://edk2.groups.io/g/devel/message/45110 Mute This Topic: https://groups.io/mt/32784353/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
On 08/08/19 04:37, Wu, Hao A wrote: > Dear stewards and Liming, > > Sorry for top-posting. > > I plan to add the below new feature which is targeting for the upcoming > edk2-stable201908 tag: > > * Add EDKII_UFS_HC_PLATFORM_PROTOCOL to support platform specific programming of UFS host controllers > > The BZ tracker for the feature request is at: > https://bugzilla.tianocore.org/show_bug.cgi?id=1343 > > The 1st version of the patch series has been proposed at: > https://edk2.groups.io/g/devel/message/45022 > And for the series, I think the changes are good in general and I plan to give > my RB tag with some minor modifications. > > If you have any concern with regard to adding this feature in the upcoming > stable tag, please help to raise. Thanks in advance. No concerns on my end. Thanks Laszlo > Best Regards, > Hao Wu > > >> -----Original Message----- >> From: Albecki, Mateusz >> Sent: Thursday, August 08, 2019 12:51 AM >> To: devel@edk2.groups.io >> Cc: Albecki, Mateusz; Wu, Hao A >> Subject: [PATCH 0/4] Add EDKII_UFS_HC_PLATFORM_PROTOCOL to support >> platform specific programming of UFS host controllers >> >> To cover additional host controller programming mentioned in the UFS >> specification >> we have added an additional protocol that allows the UEFI driver to give >> control >> to platform driver. This allows the platform to perform any additional steps >> needed for the stable operation. >> >> Test coverage: >> Tested on platform with UFS 2.1 host controller with Samsung UFS2.0 part >> with 3 LUs enabled >> All LUs have been enumerated in boot manager. >> Tested that enumeration works without platform protocol installed(on host >> controller that can support it) >> Tested that enumeration works with platform protocol installed and with >> additional programming steps after >> link startup(power mode change to GEAR2). >> >> Cc: Hao A Wu <hao.a.wu@intel.com> >> >> Mateusz Albecki (4): >> MdeModulePkg: Add definition of the >> EDKII_UFS_HC_PLATFORM_PROTOCOL >> MdeModulePkg/UfsPassThruDxe: Refactor UfsExecUicCommand function >> MdeModulePkg/UfsPassThruDxe: Refactor private data to use >> EDKII_UFS_HC_INFO >> MdeModulePkg/UfsPassThruDxe: Implement >> EDKII_UFS_HC_PLATFORM_PROTOCOL >> >> MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.c | 25 ++- >> MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.h | 44 +++++- >> .../Bus/Ufs/UfsPassThruDxe/UfsPassThruDxe.inf | 3 +- >> .../Bus/Ufs/UfsPassThruDxe/UfsPassThruHci.c | 172 >> ++++++++++++++++----- >> .../Include/Protocol/UfsHostControllerPlatform.h | 124 +++++++++++++++ >> MdeModulePkg/MdeModulePkg.dec | 3 + >> 6 files changed, 324 insertions(+), 47 deletions(-) >> create mode 100644 >> MdeModulePkg/Include/Protocol/UfsHostControllerPlatform.h >> >> -- >> 2.14.1.windows.1 > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#45217): https://edk2.groups.io/g/devel/message/45217 Mute This Topic: https://groups.io/mt/32784353/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
© 2016 - 2024 Red Hat, Inc.