Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp968827pxj; Fri, 21 May 2021 03:27:51 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwhl+n3yb3NemNqAUb6HVhnVXZzX9/uPTziWPkmHEGepbkBga/Z+F0mfbqUVCAcQJZnXm0Y X-Received: by 2002:a02:b718:: with SMTP id g24mr3470288jam.16.1621592871765; Fri, 21 May 2021 03:27:51 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1621592871; cv=none; d=google.com; s=arc-20160816; b=VYK5/kIbRN0Y0GYBTxJFOdDzbPd8F0cH+hOKPfE28UsKeW2T1+A/5ma9cp8MAeeUKi pclLCIjd0M+YEufQtyc02ysTunHa/AT0VewNLjr1rm3Qn77lzVcfTK5Apcxol9VT4CNw UEljUqeJgkIw7GM2GLYPzizK1Al3ixU/RzfjcQQxArP+uu5/MYilOGtOJwF95gYOaHf1 huMrv/oFgtXWmQvm/kcxusnyEvBYIFzqLILb1fNNn5JqtQdRjRow1l0lh8VSoRoGOpYL C4H0QuyGAQM95e8uCDvMvXi5yOB2strf2t7230mJ8sPYc8baW7aLttLCFofp5brOdPrx dAFA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:references:dlp-filter:cms-type :content-transfer-encoding:date:message-id:in-reply-to:to:from :sender:reply-to:subject:mime-version:dkim-signature:dkim-filter; bh=oLATn9iM4wIMKYl3x4+8ERAYHMUVpCqKFCJByrenAcY=; b=gGH8MfH3QsYBqt2QVKSrMXwDmhps5/R1o4+lQh2z4YkBgLvHzViNcNLZjhNBChrLVM sqTKZpejBoSI/PVY+ph7uMuUkFwXDtXx7aRrWS8Sykn9pOJif1C6heuvctavdjiGXC2b XpysaXos1AG7SGlqHANEZmq9GLI1SQbMrvb0snG2VObz83aLvoEOIkt1RKNs74lT3ex1 Dq4N9g1ERuQuxOv2JgvBDZQ/bBmy4wCzlrqUCn/54tXgQfZOSkg77cmG2jcL5abWPRMm q7dIfwdzqo5PWYsqADbrdpTAyNuZGdbum/uwlkJBI6vIyr4nltffTg0+z6HOL0/+JFBA zIeA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@samsung.com header.s=mail20170921 header.b=oD2WlRN5; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=samsung.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id b18si5902813ile.24.2021.05.21.03.27.39; Fri, 21 May 2021 03:27:51 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@samsung.com header.s=mail20170921 header.b=oD2WlRN5; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=samsung.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230267AbhEUIOd (ORCPT + 99 others); Fri, 21 May 2021 04:14:33 -0400 Received: from mailout2.samsung.com ([203.254.224.25]:17224 "EHLO mailout2.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229629AbhEUIOb (ORCPT ); Fri, 21 May 2021 04:14:31 -0400 Received: from epcas2p1.samsung.com (unknown [182.195.41.53]) by mailout2.samsung.com (KnoxPortal) with ESMTP id 20210521081307epoutp027a5e91012215cdbad5212697be26dbac~BBsXEMaPB3086830868epoutp02X for ; Fri, 21 May 2021 08:13:07 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 mailout2.samsung.com 20210521081307epoutp027a5e91012215cdbad5212697be26dbac~BBsXEMaPB3086830868epoutp02X DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1621584787; bh=oLATn9iM4wIMKYl3x4+8ERAYHMUVpCqKFCJByrenAcY=; h=Subject:Reply-To:From:To:In-Reply-To:Date:References:From; b=oD2WlRN5mz4Xdi8pdGNM6N1ye723kF+j+TZR+ep99FyyPpoFmFlYKbagjEme9YUdL ljAQ1mYVlJ90fOu53OhTVO4dhKoJvKvZm37p+2JCh4TMX2dd+a01HiHaypz2+eFsAX NtYP0tqqLEZuxxin4vWMkhfrz5IEeBWZ1jL22PHs= Received: from epsnrtp4.localdomain (unknown [182.195.42.165]) by epcas2p1.samsung.com (KnoxPortal) with ESMTP id 20210521081306epcas2p180a06718eec1fe8610b2245c3798f0d9~BBsWBx2DN0947909479epcas2p1D; Fri, 21 May 2021 08:13:06 +0000 (GMT) Received: from epsmges2p3.samsung.com (unknown [182.195.40.187]) by epsnrtp4.localdomain (Postfix) with ESMTP id 4FmfVm6lWRz4x9Q6; Fri, 21 May 2021 08:13:04 +0000 (GMT) X-AuditID: b6c32a47-f4bff700000024d9-bc-60a76b8f716a Received: from epcas2p3.samsung.com ( [182.195.41.55]) by epsmges2p3.samsung.com (Symantec Messaging Gateway) with SMTP id E8.9F.09433.F8B67A06; Fri, 21 May 2021 17:13:03 +0900 (KST) Mime-Version: 1.0 Subject: RE: RE: [PATCH v34 4/4] scsi: ufs: Add HPB 2.0 support Reply-To: daejun7.park@samsung.com Sender: Daejun Park From: Daejun Park To: Avri Altman , Daejun Park , "gregkh@linuxfoundation.org" , "jejb@linux.ibm.com" , "martin.petersen@oracle.com" , "asutoshd@codeaurora.org" , "stanley.chu@mediatek.com" , "cang@codeaurora.org" , "bvanassche@acm.org" , "huobean@gmail.com" , ALIM AKHTAR , "linux-scsi@vger.kernel.org" , "linux-kernel@vger.kernel.org" , JinHwan Park , Javier Gonzalez , Sung-Jun Park , Jinyoung CHOI , Dukhyun Kwon , Keoseong Park , Jaemyung Lee , Jieon Seol X-Priority: 3 X-Content-Kind-Code: NORMAL In-Reply-To: X-CPGS-Detection: blocking_info_exchange X-Drm-Type: N,general X-Msg-Generator: Mail X-Msg-Type: PERSONAL X-Reply-Demand: N Message-ID: <20210521081302epcms2p45e2a734f7f14046d6caab2ab16a2d94c@epcms2p4> Date: Fri, 21 May 2021 17:13:02 +0900 X-CMS-MailID: 20210521081302epcms2p45e2a734f7f14046d6caab2ab16a2d94c Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="utf-8" X-Sendblock-Type: AUTO_CONFIDENTIAL X-CPGSPASS: Y X-CPGSPASS: Y CMS-TYPE: 102P X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrGJsWRmVeSWpSXmKPExsWy7bCmuW5/9vIEgy/LlSwezNvGZrG37QS7 xcufV9kspn34yWzxaf0yVouXhzQtdh08yGax6kG4RfPi9WwWc842MFn09m9ls9h8cAOzxeM7 n9ktFt3YxmTR/6+dxWLbZ0GL4yffMVpc3jWHzaL7+g42i+XH/zFZLN16k9Gic/oaFgcxj8tX vD0u9/UyeeycdZfdY8KiA4we++euYfdoObmfxePj01ssHn1bVjF6fN4k59F+oJspgCsqxyYj NTEltUghNS85PyUzL91WyTs43jne1MzAUNfQ0sJcSSEvMTfVVsnFJ0DXLTMH6E0lhbLEnFKg UEBicbGSvp1NUX5pSapCRn5xia1SakFKToGhYYFecWJucWleul5yfq6VoYGBkSlQZUJOxox9 LxkLvrNWnOufxNjAOIGli5GTQ0LARGLa1qNsILaQwA5Gic2HA7oYOTh4BQQl/u4QBgkLC9hL 7O0/yAxRoiSx/uIsdoi4nsSth2sYQWw2AR2J6SfuA8W5OEQEHrJL9M9uYYOYzysxo/0p1C5p ie3Lt4I1cArESsx5vAIqriHxY1kvM4QtKnFz9Vt2GPv9sfmMELaIROu9s1A1ghIPfu6GiktK HNv9gQnCrpfYeucXI8gREgI9jBKHd95ihUjoS1zr2Ai2jFfAV2L+m3Vgx7EIqEpsP/8M6ggX iRdfPoINZRaQl9j+dg4zKCCYBTQl1u/SBzElBJQljtxigXmrYeNvdnQ2swCfRMfhv3DxHfOe QJ2mJrHu53omiDEyErfmMU5gVJqFCOhZSNbOQli7gJF5FaNYakFxbnpqsVGBMXLUbmIEJ3gt 9x2MM95+0DvEyMTBeIhRgoNZSYSX23F5ghBvSmJlVWpRfnxRaU5q8SFGU6CHJzJLiSbnA3NM Xkm8oamRmZmBpamFqZmRhZI478/UugQhgfTEktTs1NSC1CKYPiYOTqkGpr2WKpvc1naY79nl w/+w8MBVJjuHiyYvzz+d4jVry/+2qrn2d9ZVMfgFnv98TUTA6cLTz0bdTJpb6pXK1n0JLf+g 93D3ZuU7oiriU7UqHq0JCJ3EY5h2ZK3J0TP/uw9uEg975rW16/+rKRELtW4rfJFVrOtcLO/1 xWaJ576DubO/hl90W/VoEucKufTniXd9zzoGTF7R9e70v/WZUp2Rh/79qlS3v75zeXL9pR0H GnboNTczcqwVZ/n0Yu4X88UZBcvDX7benHRUJGrvvJunJMqmz5a1Nj5aU/TfhWOeD0cUd62g oEZk6HOzxYITd1hKzT3Z2+Skmrcy1LlLP/KxylSp/blXOI3ee7K37N3R9EOJpTgj0VCLuag4 EQAkHyjNeQQAAA== DLP-Filter: Pass X-CFilter-Loop: Reflected X-CMS-RootMailID: 20210428232257epcms2p8602b29d63529fca8a06010a21157d5cb References: <20210428232257epcms2p8602b29d63529fca8a06010a21157d5cb@epcms2p8> <20210428232534epcms2p4830e22a86cc78c3319075059fa223540@epcms2p4> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Avri, >> + /* for pre_req */ >> + hpb->pre_req_min_tr_len = hpb_dev_info->max_hpb_single_cmd + 1; >> + >> + if (ufshpb_is_legacy(hba)) >> + hpb->pre_req_max_tr_len = HPB_LEGACY_CHUNK_HIGH; >> + else >> + hpb->pre_req_max_tr_len = max(HPB_MULTI_CHUNK_HIGH, >> + hpb->pre_req_min_tr_len); >I think this should only be >else > hpb->pre_req_max_tr_len = HPB_MULTI_CHUNK_HIGH; > >where HPB_MULTI_CHUNK_HIGH should fit into a single byte, >regardless of bMAX_ DATA_SIZE_FOR_HPB_SINGLE_CMD, >which being an attribute (u32) can be significantly larger. I will fix it as suggested. Thank you. > >Thanks, >Avri > > > > >