Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Content-Type: text/plain;
        charset=us-ascii
Mime-Version: 1.0 (Mac OS X Mail 12.4 \(3445.104.21\))
Subject: Re: [BUG] mmc: core: adjust polling interval for CMD1
From:   "H. Nikolaus Schaller" <hns@goldelico.com>
In-Reply-To: <20220302082034.GA5723@math.uni-bielefeld.de>
Date:   Wed, 2 Mar 2022 10:40:25 +0100
Cc:     Huijin Park <huijin.park@samsung.com>,
        linux-kernel <linux-kernel@vger.kernel.org>,
        Discussions about the Letux Kernel 
        <letux-kernel@openphoenux.org>,
        Linux-OMAP <linux-omap@vger.kernel.org>,
        linux-mmc@vger.kernel.org, Tony Lindgren <tony@atomide.com>
Content-Transfer-Encoding: quoted-printable
Message-Id: <6715A5BE-CA64-4A3D-8EE5-5BEEB63F268A@goldelico.com>
References: <27DDB061-1235-4F4C-B6A8-F035D77AC9CF@goldelico.com>
 <CAPDyKFrz_2Vp64SUzB8CiHJLTjO8Hx8m3QEhY1VU2ksZhVEx7A@mail.gmail.com>
 <20220302082034.GA5723@math.uni-bielefeld.de>
To:     Jean Rene Dawin <jdawin@math.uni-bielefeld.de>,
        Ulf Hansson <ulf.hansson@linaro.org>
Precedence: bulk

Hi,

> Am 02.03.2022 um 09:20 schrieb Jean Rene Dawin =
<jdawin@math.uni-bielefeld.de>:
>=20
> Ulf Hansson wrote on Tue  1/03/22 14:38:
>> On Thu, 17 Feb 2022 at 21:12, H. Nikolaus Schaller =
<hns@goldelico.com> wrote:
>>>=20
>>=20
>> From: Ulf Hansson <ulf.hansson@linaro.org>
>> Date: Tue, 1 Mar 2022 14:24:21 +0100
>> Subject: [PATCH] mmc: core: Extend timeout to 2s for MMC_SEND_OP_COND
>>=20
>> It looks like the timeout for the MMC_SEND_OP_COND (CMD1) might have =
become
>> a bit too small due to recent changes. Therefore, let's extend it to =
2s,
>> which is probably more inline with its previous value, to fix the =
reported
>> timeout problems.
>>=20
>> While at it, let's add a define for the timeout value, rather than =
using
>> a hard-coded value for it.
>>=20
>> Reported-by: Jean Rene Dawin <jdawin@math.uni-bielefeld.de>
>> Reported-by: H. Nikolaus Schaller <hns@goldelico.com>
>> Cc: Huijin Park <huijin.park@samsung.com>
>> Fixes: 76bfc7ccc2fa ("mmc: core: adjust polling interval for CMD1")
>> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
>> ---
>> drivers/mmc/core/mmc_ops.c | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>=20
>> diff --git a/drivers/mmc/core/mmc_ops.c b/drivers/mmc/core/mmc_ops.c
>> index d63d1c735335..1f57174b3cf3 100644
>> --- a/drivers/mmc/core/mmc_ops.c
>> +++ b/drivers/mmc/core/mmc_ops.c
>> @@ -21,6 +21,7 @@
>>=20
>> #define MMC_BKOPS_TIMEOUT_MS           (120 * 1000) /* 120s */
>> #define MMC_SANITIZE_TIMEOUT_MS                (240 * 1000) /* 240s =
*/
>> +#define MMC_OP_COND_TIMEOUT_MS         2000 /* 2s */
>>=20
>> static const u8 tuning_blk_pattern_4bit[] =3D {
>>        0xff, 0x0f, 0xff, 0x00, 0xff, 0xcc, 0xc3, 0xcc,
>> @@ -232,7 +233,8 @@ int mmc_send_op_cond(struct mmc_host *host, u32
>> ocr, u32 *rocr)
>>        cmd.arg =3D mmc_host_is_spi(host) ? 0 : ocr;
>>        cmd.flags =3D MMC_RSP_SPI_R1 | MMC_RSP_R3 | MMC_CMD_BCR;
>>=20
>> -       err =3D __mmc_poll_for_busy(host, 1000, =
&__mmc_send_op_cond_cb, &cb_data);
>> +       err =3D __mmc_poll_for_busy(host, MMC_OP_COND_TIMEOUT_MS,
>> +                                 &__mmc_send_op_cond_cb, &cb_data);
>>        if (err)
>>                return err;
>>=20
>> --=20
>> 2.25.1
>=20
> Hi,
>=20
> thanks. But testing with this patch still gives the same errors:
>=20
> [   52.259940] mmc1: Card stuck being busy! __mmc_poll_for_busy
> [   52.273380] mmc1: error -110 doing runtime resume
>=20
> and the system gets stuck eventually.

Same result from my tests.

BR and thanks,
Nikolaus