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.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED 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 6256FC12002 for ; Fri, 16 Jul 2021 23:55:13 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 47812611BE for ; Fri, 16 Jul 2021 23:55:13 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229984AbhGPX5v (ORCPT ); Fri, 16 Jul 2021 19:57:51 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57354 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229934AbhGPX5u (ORCPT ); Fri, 16 Jul 2021 19:57:50 -0400 Received: from mail-oi1-x22f.google.com (mail-oi1-x22f.google.com [IPv6:2607:f8b0:4864:20::22f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3A847C061762 for ; Fri, 16 Jul 2021 16:54:55 -0700 (PDT) Received: by mail-oi1-x22f.google.com with SMTP id w194so12916996oie.5 for ; Fri, 16 Jul 2021 16:54:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=VJGdspG2J1Lj8nevb/Lj+7rawdMA9S4WQ8v9/yvZMvo=; b=OkAzUjq+SaObhfByR+nUx5mgIQ3Mj2gMLVbecs/XMcl0FGNc538qnz06xKozYcDCH+ T2G34oXtNn6K04Y3tJ/tzc7PAdF9ohrL/VszWgflvy/59nrcZM586yOU9z2qwdFfSxoz 4rk+f/B2DvrfjOHw/2DRWu8foTSqwez0nexmMgwvUX65nMDUUOljQtPVEw1W9PqI6ixE uznwYPTE3PqGoglVJ3dT1PRSWyK40h86fx7eCXlJDVoUh4MiYQtlU7+rJSkLsB6WuQIX nb89ZRwBzc8ZP5sHAnolgZ4W7sX2x8Gm6XH2a9mu82raDwbjxpywlhcj7DuniAbxeSgJ oaQw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=VJGdspG2J1Lj8nevb/Lj+7rawdMA9S4WQ8v9/yvZMvo=; b=Uyx//GeGqVMUKLom8vWmqpbJhQEFPO5KpjuqZBsYRe8xHTkwirA/fDc72Bmv1KdmfI m3fj+ZaKArtlN9eIIwG6+nXacPBuxY3CI9But8XKiheU2IZTGGOogv9njk7hR9wxqnig 21+e9ZqxVDg3SS5eTiyFsYCZQmttgNiHObVT8kQ25Hha3Qqf0+vcMKs9+f2FVpzPM4hq xqrEEn85Y/fg8WpAsREQiWY89xZUaIMhwcP+nTuVqTZSrFrHaonrNYbXeswsPlOtz502 btc2Dt7paZ2Ja9G4IubjMZ/S3GhQPxqmlB2t3cRA19JXj7hfrSrOVPsov2OvdRIrUK1L z+rA== X-Gm-Message-State: AOAM530HWJvaJnUH/MiazlaEnsKo5zgeqf9x/yeRUNDt4uXEWSxZ8pLt 8L4TsUXgKYjwBHUwUJxLTRBSGQ== X-Google-Smtp-Source: ABdhPJzPb0xrQfYz/QsYG3LsjhsCqhhunDjTyM/p4U47JfwRJ5DQarU6cR+d2fPgJkfLT6b+EUMmLQ== X-Received: by 2002:aca:dc44:: with SMTP id t65mr9803129oig.76.1626479694464; Fri, 16 Jul 2021 16:54:54 -0700 (PDT) Received: from yoga (104-57-184-186.lightspeed.austtx.sbcglobal.net. [104.57.184.186]) by smtp.gmail.com with ESMTPSA id b10sm2434834oiy.4.2021.07.16.16.54.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 16 Jul 2021 16:54:54 -0700 (PDT) Date: Fri, 16 Jul 2021 18:54:51 -0500 From: Bjorn Andersson To: Sarthak Garg Cc: adrian.hunter@intel.com, ulf.hansson@linaro.org, stummala@codeaurora.org, linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, Andy Gross Subject: Re: [PATCH V1 2/2] mmc: sdhci-msm: Use maximum possible data timeout value Message-ID: References: <1626444182-2187-1-git-send-email-sartgarg@codeaurora.org> <1626444182-2187-3-git-send-email-sartgarg@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1626444182-2187-3-git-send-email-sartgarg@codeaurora.org> Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org On Fri 16 Jul 09:03 CDT 2021, Sarthak Garg wrote: > The Qcom SD controller defines the usage of 0xF in data > timeout counter register (0x2E) which is actually a reserved > bit as per specification. This would result in maximum of 21.26 secs > timeout value. > > Some SDcard taking more time than 2.67secs (timeout value corresponding > to 0xE) and with that observed data timeout errors. > So increasing the timeout value to max possible timeout. > > Signed-off-by: Sahitya Tummala The From: of the email says you wrote the patch, but the first person to certify its origin is Sahitya. Did you perhaps write this together? If so you should have a: Co-developed-by: Sahitya If Sahitya wrote the patch, certifies its origin and then you picks it up and certify that you got it from Sahitya, then you should write it as you did - but you should have retained Sahitya as author... Regards, Bjorn > Signed-off-by: Sarthak Garg > --- > drivers/mmc/host/sdhci-msm.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/mmc/host/sdhci-msm.c b/drivers/mmc/host/sdhci-msm.c > index e44b7a6..19e4673 100644 > --- a/drivers/mmc/host/sdhci-msm.c > +++ b/drivers/mmc/host/sdhci-msm.c > @@ -2696,6 +2696,9 @@ static int sdhci_msm_probe(struct platform_device *pdev) > > msm_host->mmc->caps |= MMC_CAP_WAIT_WHILE_BUSY | MMC_CAP_NEED_RSP_BUSY; > > + /* Set the timeout value to max possible */ > + host->max_timeout_count = 0xF; > + > pm_runtime_get_noresume(&pdev->dev); > pm_runtime_set_active(&pdev->dev); > pm_runtime_enable(&pdev->dev); > -- > 2.7.4 >