[edk2-devel] [PATCH] bRefClkFreq UFS card attribute need to be programmed after fDeviceInit
Wu, Hao A
hao.a.wu at intel.com
Mon Mar 28 00:29:34 UTC 2022
Will slightly modify the subject to:
MdeModulePkg/Ufs: bRefClkFreq attribute be programmed after fDeviceInit
Reviewed-by: Hao A Wu <hao.a.wu at intel.com>
Will wait a day before merging to see if comment from other reviewers.
Best Regards,
Hao Wu
> -----Original Message-----
> From: devel at edk2.groups.io <devel at edk2.groups.io> On Behalf Of Bandaru,
> Purna Chandra Rao
> Sent: Friday, March 25, 2022 8:18 PM
> To: devel at edk2.groups.io
> Cc: Bandaru, Purna Chandra Rao <purna.chandra.rao.bandaru at intel.com>; Wu,
> Hao A <hao.a.wu at intel.com>; Albecki, Mateusz <mateusz.albecki at intel.com>;
> Gao, Liming <gaoliming at byosoft.com.cn>; Liu, Zhiguang
> <zhiguang.liu at intel.com>
> Subject: [edk2-devel] [PATCH] bRefClkFreq UFS card attribute need to be
> programmed after fDeviceInit
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3886
>
> bRefClkFreq UFS card attribute need to be read and written after successful
> fDeviceInit and NOP response so that link will be stable.
>
> Cc: Wu Hao A <hao.a.wu at intel.com>
> Cc: Albecki Mateusz <mateusz.albecki at intel.com>
> Cc: Liming Gao <gaoliming at byosoft.com.cn>
> Cc: Zhiguang Liu <zhiguang.liu at intel.com>
>
> Signed-off-by: Purna Chandra Rao Bandaru
> <purna.chandra.rao.bandaru at intel.com>
> ---
> .../Bus/Ufs/UfsPassThruDxe/UfsPassThru.c | 34 +++++++++----------
> 1 file changed, 17 insertions(+), 17 deletions(-)
>
> diff --git a/MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.c
> b/MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.c
> index dc78e09678..ae593ff03a 100644
> --- a/MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.c
> +++ b/MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.c
> @@ -919,6 +919,23 @@ UfsPassThruDriverBindingStart (
> goto Error;
> }
>
> + //
> + // UFS 2.0 spec Section 13.1.3.3:
> + // At the end of the UFS Interconnect Layer initialization on both
> + host and device side, // the host shall send a NOP OUT UPIU to verify that the
> device UTP Layer is ready.
> + //
> + Status = UfsExecNopCmds (Private);
> + if (EFI_ERROR (Status)) {
> + DEBUG ((DEBUG_ERROR, "Ufs Sending NOP IN command Error, Status
> = %r\n", Status));
> + goto Error;
> + }
> +
> + Status = UfsFinishDeviceInitialization (Private); if (EFI_ERROR
> + (Status)) {
> + DEBUG ((DEBUG_ERROR, "Device failed to finish initialization, Status = %r\n",
> Status));
> + goto Error;
> + }
> +
> if ((mUfsHcPlatform != NULL) &&
> ((mUfsHcPlatform->RefClkFreq == EdkiiUfsCardRefClkFreq19p2Mhz) ||
> (mUfsHcPlatform->RefClkFreq == EdkiiUfsCardRefClkFreq26Mhz) || @@ -
> 967,23 +984,6 @@ UfsPassThruDriverBindingStart (
> }
> }
>
> - //
> - // UFS 2.0 spec Section 13.1.3.3:
> - // At the end of the UFS Interconnect Layer initialization on both host and
> device side,
> - // the host shall send a NOP OUT UPIU to verify that the device UTP Layer is
> ready.
> - //
> - Status = UfsExecNopCmds (Private);
> - if (EFI_ERROR (Status)) {
> - DEBUG ((DEBUG_ERROR, "Ufs Sending NOP IN command Error, Status
> = %r\n", Status));
> - goto Error;
> - }
> -
> - Status = UfsFinishDeviceInitialization (Private);
> - if (EFI_ERROR (Status)) {
> - DEBUG ((DEBUG_ERROR, "Device failed to finish initialization, Status = %r\n",
> Status));
> - goto Error;
> - }
> -
> //
> // Check if 8 common luns are active and set corresponding bit mask.
> //
> --
> 2.31.1.windows.1
>
>
>
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#88080): https://edk2.groups.io/g/devel/message/88080
Mute This Topic: https://groups.io/mt/90020932/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list