From nobody Sat May 18 23:44:23 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+87310+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+87310+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=intel.com ARC-Seal: i=1; a=rsa-sha256; t=1646653668; cv=none; d=zohomail.com; s=zohoarc; b=aer2IAM/1DZCysgRfOZk084arftU4bN5UxfRLdXpQHSezEfKwEENMHpk3kZp29g4v4R8zIhBYLVy+yRm7sfud4eZGkxNJPCJXDw0hhF6KToKbKIrQqNUsG5uSipyN/ZAMVNHlKj1h1X3RjuhCTLt66dLqMdHMYsIHRL5WKw9tMk= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1646653668; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:List-Subscribe:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:Sender:Subject:To; bh=qUTXZ+J3+7Xnju0rzT/Cxl3BHkUrB5ri7dPdtfCE5Dg=; b=lFBkLWfEBZf+xF0Uu82v4thz1AmVS1nmRgoEKeAual5BS7+bGBkE3L5xUYII8giHjITP0xZlbB91Sbco53r4xZMKcSld31ChKhANuNtW4VljCJolEQFpswonAivJE7fY6o3XbcDZhUhiIt2A9z7CTrNT38bun4YgkBA6u1JSRQc= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+87310+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1646653668203887.2545228447644; Mon, 7 Mar 2022 03:47:48 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id r4eoYY1788612xQNjPkxY3o2; Mon, 07 Mar 2022 03:47:47 -0800 X-Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by mx.groups.io with SMTP id smtpd.web09.25470.1646653666362833324 for ; Mon, 07 Mar 2022 03:47:46 -0800 X-IronPort-AV: E=McAfee;i="6200,9189,10278"; a="315085620" X-IronPort-AV: E=Sophos;i="5.90,162,1643702400"; d="scan'208";a="315085620" X-Received: from orsmga006.jf.intel.com ([10.7.209.51]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Mar 2022 03:47:45 -0800 X-IronPort-AV: E=Sophos;i="5.90,162,1643702400"; d="scan'208";a="512652750" X-Received: from pbandaru-mobl1.gar.corp.intel.com ([10.252.86.6]) by orsmga006-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Mar 2022 03:47:43 -0800 From: "Bandaru, Purna Chandra Rao" To: devel@edk2.groups.io Cc: Purna Chandra Rao Bandaru , Wu Hao A , Albecki Mateusz , Liming Gao , Zhiguang Liu Subject: [edk2-devel] [PATCH] [PATCH v2] MdeModulePkg: Add bRefClkFreq card attribute programming support Date: Mon, 7 Mar 2022 17:17:26 +0530 Message-Id: <1828e4b1d46a6ae53d275d374707909deba007d6.1646653492.git.purna.chandra.rao.bandaru@intel.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,purna.chandra.rao.bandaru@intel.com X-Gm-Message-State: uuEaoatvVp3ppBM1cFn8PrzEx1787277AA= Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1646653667; bh=Pc0pZoaq3nV3gT+Rl91chVkEfVG0oLTTuDxhETJtbbc=; h=Cc:Content-Type:Date:From:Reply-To:Subject:To; b=JSyPxeT4tvV9HxsZ9FCDI5XTcIOWNuYZ+b71jiT5+MM3eHagG1uLtGKrxoNkoVJTu6J ymccrB1ltJ+cMkTnZ6yYnXr10hENGeReyQl5UGetoxKim+rIigbwHNU8hTuVqSMNgxog7 cNKrbLCjeJ6vcWzxgV0gVkM5HhttKbVEKlQ= X-ZohoMail-DKIM: pass (identity @groups.io) X-ZM-MESSAGEID: 1646653669782100001 REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D3851 When the UFS card comes out of Manufacturer, bRefClkFreq attribute is set to 1h on the UFS card as per the "MDV=E2=80=9D (Manufacturer Default Value) specified by the spec JESD220*. However, depends on the UFS host system environment, it need to be set to correct value. Reference Clock Frequency value 0h:19.2 MHz 1h: 26 MHz 2h: 38.4 MHz 3h: Obsolete Others: Reserved Cc: Wu Hao A Cc: Albecki Mateusz Cc: Liming Gao Cc: Zhiguang Liu Signed-off-by: Purna Chandra Rao Bandaru --- .../Bus/Ufs/UfsPassThruDxe/UfsPassThru.c | 50 ++++++++++++++++++- .../Bus/Ufs/UfsPassThruDxe/UfsPassThruHci.c | 10 +--- .../Protocol/UfsHostControllerPlatform.h | 15 +++++- 3 files changed, 63 insertions(+), 12 deletions(-) diff --git a/MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.c b/MdeModuleP= kg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.c index 4c2d6ae27f..638b14b0c1 100644 --- a/MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.c +++ b/MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThru.c @@ -1,6 +1,6 @@ /** @file =20 - Copyright (c) 2014 - 2021, Intel Corporation. All rights reserved.
+ Copyright (c) 2014 - 2022, Intel Corporation. All rights reserved.
Copyright (c) Microsoft Corporation.
SPDX-License-Identifier: BSD-2-Clause-Patent =20 @@ -843,6 +843,8 @@ UfsPassThruDriverBindingStart ( UFS_DEV_DESC DeviceDescriptor; UINT32 UnitDescriptorSize; UINT32 DeviceDescriptorSize; + EDKII_UFS_CARD_REF_CLK_FREQ_ATTRIBUTE Attributes; + UINT8 RefClkAttr; =20 Status =3D EFI_SUCCESS; UfsHc =3D NULL; @@ -916,6 +918,52 @@ UfsPassThruDriverBindingStart ( DEBUG ((DEBUG_ERROR, "Ufs Host Controller Initialization Error, Status= =3D %r\n", Status)); goto Error; } + if ((mUfsHcPlatform !=3D NULL) && + ((mUfsHcPlatform->RefClkFreq =3D=3D EdkiiUfsCardRefClkFreq19p2Mhz) || + (mUfsHcPlatform->RefClkFreq =3D=3D EdkiiUfsCardRefClkFreq26Mhz) || + (mUfsHcPlatform->RefClkFreq =3D=3D EdkiiUfsCardRefClkFreq38p4Mhz)))= { + RefClkAttr =3D UfsAttrRefClkFreq; + Attributes =3D EdkiiUfsCardRefClkFreqObsolete; + Status =3D UfsRwAttributes (Private, TRUE, RefClkAttr, 0, 0, (UINT32 *= ) &Attributes); + if (!EFI_ERROR (Status)) { + if (Attributes !=3D mUfsHcPlatform->RefClkFreq) { + Attributes =3D mUfsHcPlatform->RefClkFreq; + DEBUG ( + (DEBUG_INFO, + "Setting bRefClkFreq attribute(%x) to %x\n 0 -> 19.2 Mhz\n 1 -= > 26 Mhz\n 2 -> 38.4 Mhz\n 3 -> Obsolete\n", + RefClkAttr, + Attributes) + ); + Status =3D UfsRwAttributes (Private, FALSE, RefClkAttr, 0, 0, (UIN= T32 *) &Attributes); + if (EFI_ERROR (Status)) { + DEBUG ( + (DEBUG_ERROR, + "Failed to Change Reference Clock Attribute to %d, Status =3D = %r \n", + mUfsHcPlatform->RefClkFreq, + Status) + ); + } + } + } else { + DEBUG ( + (DEBUG_ERROR, + "Failed to Read Reference Clock Attribute, Status =3D %r \n", + Status) + ); + } + } + + if ((mUfsHcPlatform !=3D NULL) && (mUfsHcPlatform->Callback !=3D NULL)) { + Status =3D mUfsHcPlatform->Callback (Private->Handle, EdkiiUfsHcPostLi= nkStartup, &Private->UfsHcDriverInterface); + if (EFI_ERROR (Status)) { + DEBUG ( + (DEBUG_ERROR, + "Failure from platform driver during EdkiiUfsHcPostLinkStartup, St= atus =3D %r\n", + Status) + ); + return Status; + } + } =20 // // UFS 2.0 spec Section 13.1.3.3: diff --git a/MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThruHci.c b/MdeModu= lePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThruHci.c index eba35cc669..4a9fa01e7d 100644 --- a/MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThruHci.c +++ b/MdeModulePkg/Bus/Ufs/UfsPassThruDxe/UfsPassThruHci.c @@ -2,7 +2,7 @@ UfsPassThruDxe driver is used to produce EFI_EXT_SCSI_PASS_THRU protocol= interface for upper layer application to execute UFS-supported SCSI cmds. =20 - Copyright (c) 2014 - 2019, Intel Corporation. All rights reserved.
+ Copyright (c) 2014 - 2022, Intel Corporation. All rights reserved.
Copyright (c) Microsoft Corporation.
SPDX-License-Identifier: BSD-2-Clause-Patent =20 @@ -1970,14 +1970,6 @@ UfsDeviceDetection ( return EFI_DEVICE_ERROR; } } else { - if ((mUfsHcPlatform !=3D NULL) && (mUfsHcPlatform->Callback !=3D NUL= L)) { - Status =3D mUfsHcPlatform->Callback (Private->Handle, EdkiiUfsHcPo= stLinkStartup, &Private->UfsHcDriverInterface); - if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "Failure from platform driver during EdkiiU= fsHcPostLinkStartup, Status =3D %r\n", Status)); - return Status; - } - } - return EFI_SUCCESS; } } diff --git a/MdeModulePkg/Include/Protocol/UfsHostControllerPlatform.h b/Md= eModulePkg/Include/Protocol/UfsHostControllerPlatform.h index faa82d0c4e..32e9f6488c 100644 --- a/MdeModulePkg/Include/Protocol/UfsHostControllerPlatform.h +++ b/MdeModulePkg/Include/Protocol/UfsHostControllerPlatform.h @@ -1,7 +1,7 @@ /** @file EDKII_UFS_HC_PLATFORM_PROTOCOL definition. =20 -Copyright (c) 2019, Intel Corporation. All rights reserved.
+Copyright (c) 2019 - 2022, Intel Corporation. All rights reserved.
SPDX-License-Identifier: BSD-2-Clause-Patent =20 **/ @@ -11,7 +11,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent =20 #include =20 -#define EDKII_UFS_HC_PLATFORM_PROTOCOL_VERSION 1 +#define EDKII_UFS_HC_PLATFORM_PROTOCOL_VERSION 2 =20 extern EFI_GUID gEdkiiUfsHcPlatformProtocolGuid; =20 @@ -83,6 +83,13 @@ typedef enum { EdkiiUfsHcPostLinkStartup } EDKII_UFS_HC_PLATFORM_CALLBACK_PHASE; =20 +typedef enum { + EdkiiUfsCardRefClkFreq19p2Mhz, + EdkiiUfsCardRefClkFreq26Mhz, + EdkiiUfsCardRefClkFreq38p4Mhz, + EdkiiUfsCardRefClkFreqObsolete +} EDKII_UFS_CARD_REF_CLK_FREQ_ATTRIBUTE; + /** Callback function for platform driver. =20 @@ -118,6 +125,10 @@ struct _EDKII_UFS_HC_PLATFORM_PROTOCOL { /// for host controller. /// EDKII_UFS_HC_PLATFORM_CALLBACK Callback; + /// + /// Reference Clock Frequency Ufs Card Attribute that need to be set in = this Ufs Host Environment. + /// + EDKII_UFS_CARD_REF_CLK_FREQ_ATTRIBUTE RefClkFreq; }; =20 #endif --=20 2.31.1.windows.1 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#87310): https://edk2.groups.io/g/devel/message/87310 Mute This Topic: https://groups.io/mt/89609901/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-