From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-15.3 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C33CFC433EF for ; Fri, 24 Sep 2021 05:29:18 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 9D2AA61211 for ; Fri, 24 Sep 2021 05:29:18 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S243369AbhIXFas (ORCPT ); Fri, 24 Sep 2021 01:30:48 -0400 Received: from mga04.intel.com ([192.55.52.120]:1448 "EHLO mga04.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S242549AbhIXFar (ORCPT ); Fri, 24 Sep 2021 01:30:47 -0400 X-IronPort-AV: E=McAfee;i="6200,9189,10116"; a="222117451" X-IronPort-AV: E=Sophos;i="5.85,319,1624345200"; d="scan'208";a="222117451" Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga104.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Sep 2021 22:29:13 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.85,319,1624345200"; d="scan'208";a="558844083" Received: from ahunter-desktop.fi.intel.com (HELO [10.237.72.84]) ([10.237.72.84]) by fmsmga002.fm.intel.com with ESMTP; 23 Sep 2021 22:29:11 -0700 Subject: Re: [PATCH v1 2/2] mmc: sdhci: Use the SW timer when the HW timer cannot meet the timeout value required by the device To: Bean Huo , Ulf Hansson Cc: Bean Huo , linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org References: <20210917172727.26834-1-huobean@gmail.com> <20210917172727.26834-3-huobean@gmail.com> From: Adrian Hunter Organization: Intel Finland Oy, Registered Address: PL 281, 00181 Helsinki, Business Identity Code: 0357606 - 4, Domiciled in Helsinki Message-ID: Date: Fri, 24 Sep 2021 08:29:51 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Firefox/78.0 Thunderbird/78.14.0 MIME-Version: 1.0 In-Reply-To: <20210917172727.26834-3-huobean@gmail.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-mmc@vger.kernel.org On 17/09/21 8:27 pm, Bean Huo wrote: > From: Bean Huo > > If the data transmission timeout value required by the device exceeds > the maximum timeout value of the host HW timer, we still use the HW > timer with the maximum timeout value of the HW timer. This setting is > suitable for most R/W situations. But sometimes, the device will complete > the R/W task within its required timeout value (greater than the HW timer). > In this case, the HW timer for data transmission will time out. > > Currently, in this condition, we disable the HW timer and use the SW > timer only when the SDHCI_QUIRK2_DISABLE_HW_TIMEOUT quirk is set by the > host driver. The patch is to remove this if statement restriction and > allow data transmission to use the SW timer when the hardware timer cannot > meet the required timeout value. The reason it is a quirk is because it does not work for all hardware. For some controllers the timeout cannot really be disabled, only the interrupt is disabled, and then the controller never indicates completion if the timeout is exceeded. > > Signed-off-by: Bean Huo > --- > drivers/mmc/host/sdhci.c | 6 +----- > 1 file changed, 1 insertion(+), 5 deletions(-) > > diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c > index 357b365bf0ec..463517fd9886 100644 > --- a/drivers/mmc/host/sdhci.c > +++ b/drivers/mmc/host/sdhci.c > @@ -969,9 +969,6 @@ static u8 sdhci_calc_timeout(struct sdhci_host *host, struct mmc_command *cmd, > count++; > current_timeout <<= 1; > if (count > host->max_timeout_count) { > - if (!(host->quirks2 & SDHCI_QUIRK2_DISABLE_HW_TIMEOUT)) > - DBG("Too large timeout 0x%x requested for CMD%d!\n", > - count, cmd->opcode); > count = host->max_timeout_count; > *too_big = true; > break; > @@ -1016,8 +1013,7 @@ void __sdhci_set_timeout(struct sdhci_host *host, struct mmc_command *cmd) > bool too_big = false; > u8 count = sdhci_calc_timeout(host, cmd, &too_big); > > - if (too_big && > - host->quirks2 & SDHCI_QUIRK2_DISABLE_HW_TIMEOUT) { > + if (too_big) { > sdhci_calc_sw_timeout(host, cmd); > sdhci_set_data_timeout_irq(host, false); > } else if (!(host->ier & SDHCI_INT_DATA_TIMEOUT)) { >