From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.88, mailfrom: hao.a.wu@intel.com) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by groups.io with SMTP; Wed, 07 Aug 2019 19:37:48 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 07 Aug 2019 19:37:48 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,358,1559545200"; d="scan'208";a="168835492" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by orsmga008.jf.intel.com with ESMTP; 07 Aug 2019 19:37:47 -0700 Received: from fmsmsx120.amr.corp.intel.com (10.18.124.208) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 7 Aug 2019 19:37:47 -0700 Received: from shsmsx106.ccr.corp.intel.com (10.239.4.159) by fmsmsx120.amr.corp.intel.com (10.18.124.208) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 7 Aug 2019 19:37:47 -0700 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.112]) by SHSMSX106.ccr.corp.intel.com ([169.254.10.204]) with mapi id 14.03.0439.000; Thu, 8 Aug 2019 10:37:44 +0800 From: "Wu, Hao A" To: Andrew Fish , Laszlo Ersek , "Leif Lindholm" , "Kinney, Michael D" , "Gao, Liming" CC: "Albecki, Mateusz" , "devel@edk2.groups.io" Subject: Re: [PATCH 0/4] Add EDKII_UFS_HC_PLATFORM_PROTOCOL to support platform specific programming of UFS host controllers Thread-Topic: [PATCH 0/4] Add EDKII_UFS_HC_PLATFORM_PROTOCOL to support platform specific programming of UFS host controllers Thread-Index: AQHVTUBWsRXo7l7GTUmRBXHdZSDqU6bweV8w Date: Thu, 8 Aug 2019 02:37:44 +0000 Message-ID: References: <20190807165107.688-1-mateusz.albecki@intel.com> In-Reply-To: <20190807165107.688-1-mateusz.albecki@intel.com> Accept-Language: zh-CN, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: hao.a.wu@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable 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 programmi= ng of UFS host controllers The BZ tracker for the feature request is at: https://bugzilla.tianocore.org/show_bug.cgi?id=3D1343 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 g= ive 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 >=20 > 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 st= eps > needed for the stable operation. >=20 > 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). >=20 > Cc: Hao A Wu >=20 > 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 >=20 > 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 >=20 > -- > 2.14.1.windows.1