Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
IronPort-SDR: hGpCbq1j3xPCOVsqmeXkHSrrxQ3t2WpqhMK/aEU2d53gjlnoiKdKG+prZXbPWdobYG0Bu0XAa8
 hdwwVEPSHVwv+u9EiULSpOefAGkpo/TnTbdnIGor7Po8tylAUVKeOvr6qWFBhOoPqF4hqWg8Wg
 eKiICgSNVSoQgcY0eF9O0lmGA/2yOJ5MPhmQCmFIFE7GsCiM3xUynlMoHYyoJE90hWRGLbRCai
 CJMF5lzgG17acBoz42VhaFkED9IVHmkJLnS/s81vHxAZZiinyAgXsCxzb2lpyIN2NXEzOMEqCK
 L8U=
From:   Avri Altman <Avri.Altman@wdc.com>
To:     Stanley Chu <stanley.chu@mediatek.com>,
        "linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
        "martin.petersen@oracle.com" <martin.petersen@oracle.com>,
        "alim.akhtar@samsung.com" <alim.akhtar@samsung.com>,
        "jejb@linux.ibm.com" <jejb@linux.ibm.com>
CC:     "beanhuo@micron.com" <beanhuo@micron.com>,
        "asutoshd@codeaurora.org" <asutoshd@codeaurora.org>,
        "cang@codeaurora.org" <cang@codeaurora.org>,
        "matthias.bgg@gmail.com" <matthias.bgg@gmail.com>,
        "bvanassche@acm.org" <bvanassche@acm.org>,
        "linux-mediatek@lists.infradead.org" 
        <linux-mediatek@lists.infradead.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "kuohong.wang@mediatek.com" <kuohong.wang@mediatek.com>,
        "peter.wang@mediatek.com" <peter.wang@mediatek.com>,
        "chun-hung.wu@mediatek.com" <chun-hung.wu@mediatek.com>,
        "andy.teng@mediatek.com" <andy.teng@mediatek.com>
Subject: RE: [PATCH v4 0/8] scsi: ufs: some cleanups and make the delay for
 host enabling customizable
Thread-Topic: [PATCH v4 0/8] scsi: ufs: some cleanups and make the delay for
 host enabling customizable
Thread-Index: AQHV+RX9jRrM0tPTvUyCj9SJPGT/n6hH2KzA
Date:   Sat, 14 Mar 2020 09:47:01 +0000
Message-ID: <SN6PR04MB4640CB68C9FBD3702A30CE60FCFB0@SN6PR04MB4640.namprd04.prod.outlook.com>
References: <20200313090103.15390-1-stanley.chu@mediatek.com>
In-Reply-To: <20200313090103.15390-1-stanley.chu@mediatek.com>
Accept-Language: en-US
Content-Language: en-US
wdcipoutbound: EOP-TRUE
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: cf60fd2d-018c-4d7c-0053-08d7c7fca588
X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Mar 2020 09:47:01.7525
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: ndJJzruuu5o/qYz3VTOcgtcNnsPgE9pENeLhaLCShyxuULgasTCz0o1ckGdgPcDpYmq7G+RtW4MiFbp7AIPdGw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN6PR04MB3888
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Looks good to me.
Thanks,
Avri

=20
>=20
> Hi,
>=20
> This patchset applies some driver cleanups and performance improvement
> in ufs host drivers by making the delay for host enabling customizable
> according to vendors' requirements.
>=20
> v3 -> v4
>         - Collect review tags in v2
>         - In patch #8, fix incorrect condition of customized delay for ho=
st
> enabling
>=20
> v2 -> v3
>         - Remove /arch/arm64/configs/defconfig chnage because it is for l=
ocal
> test only
>=20
> v1 -> v2
>         - Add patch #1 "scsi: ufs: fix uninitialized tx_lanes in
> ufshcd_disable_tx_lcc"
>         - Remove struct ufs_init_prefetch in patch #2 "scsi: ufs: remove
> init_prefetch_data in struct ufs_hba"
>         - Introduce common delay function in patch #4
>         - Replace all delay places by common delay function in ufs-mediat=
ek in
> patch #5
>         - Use common delay function instead for host enabling delay in pa=
tch #6
>         - Add patch #7 "scsi: ufs: make HCE polling more compact to impro=
ve
> initializatoin latency"
>         - In patch #8, customize the delay in ufs_mtk_hce_enable_notify
> callback instead of ufs_mtk_init (Avri)
>=20
>  drivers/scsi/ufs/ufs-mediatek.c | 64 ++++++++++++++++-----------
>  drivers/scsi/ufs/ufs-mediatek.h |  1 +
>  drivers/scsi/ufs/ufshcd.c       | 47 +++++++++++---------
>  drivers/scsi/ufs/ufshcd.h       | 78 ++++++++++++++++-----------------
>  4 files changed, 106 insertions(+), 84 deletions(-)
>=20
> --
> 2.18.0