From patchwork Fri Jun 16 08:30:36 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Linus Walleij X-Patchwork-Id: 13282203 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 09C6DEB64D8 for ; Fri, 16 Jun 2023 08:30:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S244209AbjFPIaq (ORCPT ); Fri, 16 Jun 2023 04:30:46 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34150 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S245279AbjFPIal (ORCPT ); Fri, 16 Jun 2023 04:30:41 -0400 Received: from mail-lf1-x12f.google.com (mail-lf1-x12f.google.com [IPv6:2a00:1450:4864:20::12f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C60852D72 for ; Fri, 16 Jun 2023 01:30:40 -0700 (PDT) Received: by mail-lf1-x12f.google.com with SMTP id 2adb3069b0e04-4f74cda5f1dso486371e87.3 for ; Fri, 16 Jun 2023 01:30:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1686904239; x=1689496239; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=Uv8R+uMF6WnTLu76PBd8GfNWo7ysNcJe7YfGwPaXEWU=; b=QoYP48KAf3RwPJaEbejINyEVXQe7cKnpoIbr3A/hnMCCMoYsiPPzr34sjjP0neYCwH yztC+OkJcv7XglNtZ3vuKiPF4UgP8lDqbkbXK4ZXJPfp8QobefMk88MHkDnYJ+kfhULA SildRRCvlMmq57C/kKm5soyFYzcy8M6kBIe5AO0JOZo0+JHncK+y0Y2l7qr3Cd+cfyEg cJdWwxPxeqMiee1sK7nX1Dte8f7s+cF2zLr9uCDTVQRpquRBiUJ72/1czfr7Qenjylhs ommaDHYiNQRb/GBQ4m2llhGrveYQJebrKctJcgQAr1TJsNBvW1HN536U81CvPH9D0FSr mBDQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1686904239; x=1689496239; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=Uv8R+uMF6WnTLu76PBd8GfNWo7ysNcJe7YfGwPaXEWU=; b=RwS7mywn9xkMgOPMq2KYzmmShVfBQzLiVNxLJqgvpCc/pjqIEuWdvi9404QnppADyR 7uT0PqwKx6vLai6y2/rzqahJz2psNMNcdRnEZGVoSB4tnIAPriOPXWFhWNzaiStdtflg EhLkHM+jjphOPJ5BVvrhzgU2ytOEyaRBIswQYjg943oALcvoR1k527elmugmlLl9lHtu pEyadUlJMa94Stu9VajEpLNMZzZ58QMexdmPhDYpJEerLaPq2+1z+wj2yhwu0Iz/pLAp IlQeWTi7Wh/NkFR+oqEVpaGLNF5ZdxwWM0uStzLDmTay4ry9gUclJLSyiBabezM0cWiq C9gw== X-Gm-Message-State: AC+VfDxsriJhhnrVVvVe6Vewn9YxzCngA3whKx9cAcI4U6ooXwemi9t8 IQaarTcJuZCPqH7LxVUQyuwd8Q== X-Google-Smtp-Source: ACHHUZ5WHQjxkZY8dwst0quP+F94G7KA9eK2YAo0+wbrOxLBa2XdWWcG+cXXHZRKkCqcPoOzO+fZXg== X-Received: by 2002:a19:4f1c:0:b0:4f7:6454:655d with SMTP id d28-20020a194f1c000000b004f76454655dmr690211lfb.0.1686904239036; Fri, 16 Jun 2023 01:30:39 -0700 (PDT) Received: from [192.168.1.2] (c-05d8225c.014-348-6c756e10.bbcust.telenor.se. [92.34.216.5]) by smtp.gmail.com with ESMTPSA id j8-20020ac25508000000b004eb44c2ab6bsm2918106lfk.294.2023.06.16.01.30.38 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 16 Jun 2023 01:30:38 -0700 (PDT) From: Linus Walleij Date: Fri, 16 Jun 2023 10:30:36 +0200 Subject: [PATCH v6 4/9] mmc: mmci: Break out error check in busy detect MIME-Version: 1.0 Message-Id: <20230405-pl180-busydetect-fix-v6-4-b850ec8019f3@linaro.org> References: <20230405-pl180-busydetect-fix-v6-0-b850ec8019f3@linaro.org> In-Reply-To: <20230405-pl180-busydetect-fix-v6-0-b850ec8019f3@linaro.org> To: Yann Gautier , Stefan Hansson , Ulf Hansson , Maxime Coquelin , Alexandre Torgue Cc: linux-mmc@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, Linus Walleij X-Mailer: b4 0.12.2 Precedence: bulk List-ID: X-Mailing-List: linux-mmc@vger.kernel.org The busy detect callback for Ux500 checks for an error in the status in the first if() clause. The only practical reason is that if an error occurs, the if()-clause is not executed, and the code falls through to the last if()-clause if (host->busy_status) which will clear and disable the irq. Make this explicit instead: it is easier to read. Signed-off-by: Linus Walleij --- ChangeLog v4->v6: - No changes. ChangeLog v3->v4: - No changes. ChangeLog v2->v3: - Rebased. ChangeLog v1->v2: - No changes --- drivers/mmc/host/mmci.c | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/drivers/mmc/host/mmci.c b/drivers/mmc/host/mmci.c index 47b306e45f78..d632658d9d20 100644 --- a/drivers/mmc/host/mmci.c +++ b/drivers/mmc/host/mmci.c @@ -665,6 +665,15 @@ static bool ux500_busy_complete(struct mmci_host *host, u32 status, u32 err_msk) { void __iomem *base = host->base; + if (status & err_msk) { + /* Stop any ongoing busy detection if an error occurs */ + writel(host->variant->busy_detect_mask, base + MMCICLEAR); + writel(readl(base + MMCIMASK0) & + ~host->variant->busy_detect_mask, base + MMCIMASK0); + host->busy_status = 0; + return true; + } + /* * Before unmasking for the busy end IRQ, confirm that the * command was sent successfully. To keep track of having a @@ -678,7 +687,7 @@ static bool ux500_busy_complete(struct mmci_host *host, u32 status, u32 err_msk) * while, to allow it to be set, but tests indicates that it * isn't needed. */ - if (!host->busy_status && !(status & err_msk)) { + if (!host->busy_status) { status = readl(base + MMCISTATUS); if (status & host->variant->busy_detect_flag) { writel(readl(base + MMCIMASK0) |