From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751922AbbH1Jmf (ORCPT ); Fri, 28 Aug 2015 05:42:35 -0400 Received: from mailout2.samsung.com ([203.254.224.25]:40978 "EHLO mailout2.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751485AbbH1Jmd (ORCPT ); Fri, 28 Aug 2015 05:42:33 -0400 X-AuditID: cbfee690-f796f6d000005054-5a-55e02d077b0c Message-id: <55E02AED.8060209@samsung.com> Date: Fri, 28 Aug 2015 15:03:33 +0530 From: Alim Akhtar User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0 MIME-version: 1.0 To: amit daniel kachhap Cc: linux-scsi@vger.kernel.org, "linux-kernel@vger.kernel.org" , vinholikatti@gmail.com, JBottomley@odin.com, Seungwon Jeon Subject: Re: [PATCH 02/10] scsi: ufs: add quirk to contain unconformable utrd field References: <1440149281-26336-1-git-send-email-alim.akhtar@samsung.com> <1440149281-26336-3-git-send-email-alim.akhtar@samsung.com> In-reply-to: Content-type: text/plain; charset=UTF-8; format=flowed Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFprGIsWRmVeSWpSXmKPExsWyRsSkVpdd90GowZI/EhYNV0Ms/q+/zWJx edccNovu6zvYLD7cv8hssWNhlQObx85Zd9k9Dv/4wezRt2UVo8fnTXIBLFFcNimpOZllqUX6 dglcGW9efWQqmKJZsXexUQPjXJkuRk4OCQETiZVL/7JB2GISF+6tB7K5OIQEVjBKfDu0gwWm aMGebewQiVmMEhOefWCCcB4wSsw/vJIJpIpXQEvifP9zZhCbRUBVYlX7SzCbTUBb4u70LUA1 HByiAhESjy8IQZQLSvyYfI8FJCwiYCgx67s4yEhmgW2MEuu/nAC7SFggVKJr3l1WiF1nGCU6 VtwHu4hTIFji5a9J7CA2s4CZxKOWdcwQtrzE5jVvmSGuPsYu8eqLOsQ9AhLfJh8CWyYhICux 6QBUiaTEwRU3WCYwis1CctIsJFNnIZm6gJF5FaNoakFyQXFSepGJXnFibnFpXrpecn7uJkZg dJ3+92zCDsZ7B6wPMQpwMCrx8FpsuB8qxJpYVlyZe4jRFOiKicxSosn5wBjOK4k3NDYzsjA1 MTU2Mrc0UxLnfS31M1hIID2xJDU7NbUgtSi+qDQntfgQIxMHp1QDY9n3VZtF129ZvL89QrY7 qrZ02o7HYdd4vmrkp7DWixRxfFKYzKbuONXi4+f40kVJGlOfLd8mzPuE5wrny+nSk/XuJHMU BbzKnOH5cKfb2qWnj77wlzrq7czflWqobhJzR2TJqcm+v+7Iqndy33q/pO79nfelxaZTTRpl d80vlGKqWfor0CExSomlOCPRUIu5qDgRAJGn/NupAgAA X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrCIsWRmVeSWpSXmKPExsVy+t9jQV123QehBk17hSwaroZY/F9/m8Xi 8q45bBbd13ewWXy4f5HZYsfCKgc2j52z7rJ7HP7xg9mjb8sqRo/Pm+QCWKIaGG0yUhNTUosU UvOS81My89JtlbyD453jTc0MDHUNLS3MlRTyEnNTbZVcfAJ03TJzgHYrKZQl5pQChQISi4uV 9O0wTQgNcdO1gGmM0PUNCYLrMTJAAwlrGDPevPrIVDBFs2LvYqMGxrkyXYycHBICJhIL9mxj h7DFJC7cW8/WxcjFISQwi1FiwrMPTBDOA0aJ+YdXMoFU8QpoSZzvf84MYrMIqEqsan8JZrMJ aEvcnb4FqIaDQ1QgQuLxBSGIckGJH5PvsYCERQQMJWZ9FwcZySywjVFi/ZcTbCA1wgKhEl3z 7rJC7DrDKNGx4j4LSIJTIFji5a9JYNcxC5hJPGpZxwxhy0tsXvOWeQIj0JkIO2YhKZuFpGwB I/MqRonUguSC4qT0XMO81HK94sTc4tK8dL3k/NxNjOAYfia1g/HgLvdDjAIcjEo8vBYb7ocK sSaWFVfmHmKU4GBWEuENEXoQKsSbklhZlVqUH19UmpNafIjRFBgIE5mlRJPzgeklryTe0NjE 3NTY1NLEwsTMUkmcV3bD5lAhgfTEktTs1NSC1CKYPiYOTqkGRsYFH+JY96bbtb09cfPpws9L r1dbvss9emvy2mddveUvRNmbujz1Tpx93/xCYWZO2z09dgNV76OSG5+0RLx+wnj76Nsr2Su7 FBeq68wNqSs86P7v1Z2vWdqC/w3nHZ7ruC6/UTP0xeT1PwSVXt90O+bEdnCRYdSOJ3xNvL8D +3at8ZJLK+Kyt1JiKc5INNRiLipOBABikNA49wIAAA== DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Amit, Thanks for your review comments. On 08/26/2015 11:43 AM, amit daniel kachhap wrote: > Few minor comments below, > > On Fri, Aug 21, 2015 at 2:57 PM, Alim Akhtar wrote: >> From: Seungwon Jeon >> >> UTRD(UTP Transfer Request Descriptor)'s field such as offset/length, >> especially response's has DWORD expression. This quirk can be specified >> for host controller not to conform standard. >> >> Signed-off-by: Seungwon Jeon >> Signed-off-by: Alim Akhtar >> --- >> drivers/scsi/ufs/ufshcd.c | 28 +++++++++++++++++++++------- >> drivers/scsi/ufs/ufshcd.h | 7 +++++++ >> 2 files changed, 28 insertions(+), 7 deletions(-) >> >> diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c >> index b0ade73..f882bf0 100644 >> --- a/drivers/scsi/ufs/ufshcd.c >> +++ b/drivers/scsi/ufs/ufshcd.c >> @@ -1009,7 +1009,7 @@ ufshcd_send_uic_cmd(struct ufs_hba *hba, struct uic_command *uic_cmd) >> * >> * Returns 0 in case of success, non-zero value in case of failure >> */ >> -static int ufshcd_map_sg(struct ufshcd_lrb *lrbp) >> +static int ufshcd_map_sg(struct ufs_hba *hba, struct ufshcd_lrb *lrbp) >> { >> struct ufshcd_sg_entry *prd_table; >> struct scatterlist *sg; >> @@ -1023,8 +1023,13 @@ static int ufshcd_map_sg(struct ufshcd_lrb *lrbp) >> return sg_segments; >> >> if (sg_segments) { >> - lrbp->utr_descriptor_ptr->prd_table_length = >> - cpu_to_le16((u16) (sg_segments)); >> + if (hba->quirks & UFSHCI_QUIRK_BROKEN_UTRD) >> + lrbp->utr_descriptor_ptr->prd_table_length = >> + cpu_to_le16((u16)(sg_segments * >> + sizeof(struct ufshcd_sg_entry))); >> + else >> + lrbp->utr_descriptor_ptr->prd_table_length = >> + cpu_to_le16((u16) (sg_segments)); >> >> prd_table = (struct ufshcd_sg_entry *)lrbp->ucd_prdt_ptr; >> >> @@ -1347,7 +1352,7 @@ static int ufshcd_queuecommand(struct Scsi_Host *host, struct scsi_cmnd *cmd) >> >> /* form UPIU before issuing the command */ >> ufshcd_compose_upiu(hba, lrbp); >> - err = ufshcd_map_sg(lrbp); >> + err = ufshcd_map_sg(hba, lrbp); >> if (err) { >> lrbp->cmd = NULL; >> clear_bit_unlock(tag, &hba->lrb_in_use); >> @@ -2035,12 +2040,21 @@ static void ufshcd_host_memory_configure(struct ufs_hba *hba) >> cpu_to_le32(upper_32_bits(cmd_desc_element_addr)); >> >> /* Response upiu and prdt offset should be in double words */ > This comment can be moved below for the else case. ok >> - utrdlp[i].response_upiu_offset = >> + if (hba->quirks & UFSHCI_QUIRK_BROKEN_UTRD) { >> + utrdlp[i].response_upiu_offset = >> + cpu_to_le16(response_offset); >> + utrdlp[i].prd_table_offset = >> + cpu_to_le16(prdt_offset); >> + utrdlp[i].response_upiu_length = >> + cpu_to_le16(ALIGNED_UPIU_SIZE); >> + } else { >> + utrdlp[i].response_upiu_offset = >> cpu_to_le16((response_offset >> 2)); >> - utrdlp[i].prd_table_offset = >> + utrdlp[i].prd_table_offset = >> cpu_to_le16((prdt_offset >> 2)); >> - utrdlp[i].response_upiu_length = >> + utrdlp[i].response_upiu_length = >> cpu_to_le16(ALIGNED_UPIU_SIZE >> 2); >> + } >> >> hba->lrb[i].utr_descriptor_ptr = (utrdlp + i); >> hba->lrb[i].ucd_req_ptr = >> diff --git a/drivers/scsi/ufs/ufshcd.h b/drivers/scsi/ufs/ufshcd.h >> index c40a0e7..1fa5ac1 100644 >> --- a/drivers/scsi/ufs/ufshcd.h >> +++ b/drivers/scsi/ufs/ufshcd.h >> @@ -459,6 +459,13 @@ struct ufs_hba { >> */ >> #define UFSHCD_QUIRK_BROKEN_UFS_HCI_VERSION UFS_BIT(5) >> >> + /* >> + * This quirk needs to be enabled if host controller doesn't conform >> + * with UTRD. Some fields such as offset/length might not be in double word, >> + * but in byte. >> + */ >> + #define UFSHCI_QUIRK_BROKEN_UTRD UFS_BIT(6) > This macro name may be given more meaningful name such as > UFSHCI_QUIRK_BYTE_ALIGN_UTRD or something similar. ok, will change >> + >> unsigned int quirks; /* Deviations from standard UFSHCI spec. */ >> >> wait_queue_head_t tm_wq; >> -- >> 1.7.10.4 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> Please read the FAQ at http://www.tux.org/lkml/ >