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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id F0AA6ECDFA1 for ; Tue, 25 Oct 2022 23:31:18 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=X8mXq49SQSI14pCUYYsz0VUUN+/v/VJ1ZHcafD+l/c0=; b=CIhkc2fSkqNNjj Cu4MIHkpbX87eCkgyNfT90UMUxODiD9CE51ZvK8AppHjU8dXVJSzXi9a0YTrHBQ2hvbghjbRo1mX/ F53AjwnsaTinxY4sZJroN18fhmSQHmvosEhypskjbFBkrhwtRRwXbIvZw/ZF7HdWo8ZRlV3wMgRL1 0Qkujcjr2RCJRGHwrjcnLPdZroOtfpmkghvbRPomYKUVAIGNp+oJJX6Av03l/hCXS2MG6YfH8i5GJ qKZIxcFGi8zKlxpVCDYQA5gq9u3/a8Yaqjw2iVmRIDgfPHfdQd+LzpdqM0BC/G/TnWQQwtn5lTCxo zjhVr43tfC/nXJpJ9crA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1onTMt-007Zj2-H5; Tue, 25 Oct 2022 23:29:59 +0000 Received: from mail-qt1-x82a.google.com ([2607:f8b0:4864:20::82a]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1onTMo-007Zh2-AH for linux-arm-kernel@lists.infradead.org; Tue, 25 Oct 2022 23:29:57 +0000 Received: by mail-qt1-x82a.google.com with SMTP id a24so8766725qto.10 for ; Tue, 25 Oct 2022 16:29:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=ZmnlBbMANFDxVmmG4z+BiB60oz+lx1MeSzu+Mh+tjyQ=; b=nUyj7ZnvQ9LHW4vTV7lmPRRrdGHhe6ln2fXTnK1s6swZRQmrfNlU9+QZvi7jdrKNmX wy0xt3YRu94KTLBocNygIbrkWWhKZ29vpUdSb53SA3XvWrix8+ODYQLOgBPz3nFk2zJS ZyrzXLpvPaAocNJlCFnKc5JNQWuy2s2QzAokyOqAUgqG0V6TjAD8hML2oXRG1EqS0NkM 7pNJG1xiuPAtfLgME0t1zNm5YzzK+f0rioN5cvk4xaJdjlfj7bNf7AR3uXbUk4RuQcRs Hgm6matm5icxiYds3Uv8JaGAuQokIvKcagkuGp+TY+UFyc0UVT6b0gvvatZKX2r9aM1s LALQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=ZmnlBbMANFDxVmmG4z+BiB60oz+lx1MeSzu+Mh+tjyQ=; b=5Rk8rEid+WpPvvfUoBK5l1zD0nTiQL6ysxUF1+kWvYkmQxLccG6zYW8avSFmN1MHHb BOtv/iRIvMUxxMEceEV/jFdKa1IVszvzV08A04bmp/WvHyLKVQP52NRTX/5BjIbwGUux o53MylhrdgV9Z3WipKFPielQX5vs4S2aiIubqLqgjEPx2zkqSqnpBWWKjcvYCH6ekcj+ wRGtb6f8OPPjgcsp4T65PTnaa0BizaVR7T5XlbxiI0ZmlKPbQw+EDqExZinFfVQHo7Ao owoWbK73PeHE+j/sXeNJ+EbCRYq8PZhgLg5K0zXigfuqjVWI5oVtm8919TokdDMq/KHW Gzjg== X-Gm-Message-State: ACrzQf3XGoA6QS8fkqMkVF2UvB+JGcF9Anbj9UCIr/3FhXSX9XTzAH0P HRScskQQ8KmXkt3zlzXJIw4= X-Google-Smtp-Source: AMsMyM53DmDnwTSU5qFLh6JkI1r7iCnhjT2EsyqdkZFqKggzPIkKwfOhQzYNptSTj8hQGRy0yXfs4Q== X-Received: by 2002:a05:622a:1806:b0:39a:68be:5c with SMTP id t6-20020a05622a180600b0039a68be005cmr33907401qtc.327.1666740590017; Tue, 25 Oct 2022 16:29:50 -0700 (PDT) Received: from [10.67.48.245] ([192.19.223.252]) by smtp.googlemail.com with ESMTPSA id dm10-20020a05620a1d4a00b006af0ce13499sm2977247qkb.115.2022.10.25.16.29.41 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 25 Oct 2022 16:29:49 -0700 (PDT) Message-ID: Date: Tue, 25 Oct 2022 16:29:38 -0700 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.2.2 Subject: Re: [PATCH v3 6/7] mmc: sdhci_am654: Fix SDHCI_RESET_ALL for CQHCI Content-Language: en-US To: Brian Norris , Florian Fainelli Cc: Adrian Hunter , Ulf Hansson , Shawn Lin , linux-mmc@vger.kernel.org, Al Cooper , Bjorn Andersson , Sowjanya Komatineni , Broadcom internal kernel review list , Sascha Hauer , Konrad Dybcio , NXP Linux Team , Thierry Reding , Fabio Estevam , Michal Simek , linux-kernel@vger.kernel.org, Shawn Guo , Pengutronix Kernel Team , linux-arm-msm@vger.kernel.org, Haibo Chen , Andy Gross , linux-arm-kernel@lists.infradead.org, Faiz Abbas , Jonathan Hunter References: <20221024175501.2265400-1-briannorris@chromium.org> <20221024105229.v3.6.I35ca9d6220ba48304438b992a76647ca8e5b126f@changeid> <5b91c0eb-52aa-8431-c286-81b7feae84ce@intel.com> <6268199c-78ca-8f55-0377-c14bb0299443@gmail.com> From: Florian Fainelli In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221025_162954_402033_70EF3914 X-CRM114-Status: GOOD ( 25.79 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 10/25/22 15:26, Brian Norris wrote: > On Tue, Oct 25, 2022 at 02:53:46PM -0700, Florian Fainelli wrote: >> On 10/25/22 14:45, Brian Norris wrote: >>> On Tue, Oct 25, 2022 at 04:10:44PM +0300, Adrian Hunter wrote: >>>> On 24/10/22 20:55, Brian Norris wrote: >>>>> diff --git a/drivers/mmc/host/sdhci_am654.c b/drivers/mmc/host/sdhci_am654.c >>>>> index 8f1023480e12..6a282c7a221e 100644 >>>>> --- a/drivers/mmc/host/sdhci_am654.c >>>>> +++ b/drivers/mmc/host/sdhci_am654.c >>> >>>>> @@ -378,7 +379,7 @@ static void sdhci_am654_reset(struct sdhci_host *host, u8 mask) >>>>> struct sdhci_pltfm_host *pltfm_host = sdhci_priv(host); >>>>> struct sdhci_am654_data *sdhci_am654 = sdhci_pltfm_priv(pltfm_host); >>>>> - sdhci_reset(host, mask); >>>>> + sdhci_and_cqhci_reset(host, mask); >>>>> if (sdhci_am654->quirks & SDHCI_AM654_QUIRK_FORCE_CDTEST) { >>>>> ctrl = sdhci_readb(host, SDHCI_HOST_CONTROL); >>>> >>>> What about sdhci_reset in sdhci_am654_ops ? >>> >>> Oops, I think you caught a big fallacy in some of my patches: I assumed >>> there was a single reset() implementation in a given driver (an unwise >>> assumption, I realize). I see at least sdhci-brcmstb.c also has several >>> variant ops that call sdhci_reset(), and I should probably convert them >>> too. >> >> You got it right for sdhci-brcmstb.c because "supports-cqe" which gates the >> enabling of CQE can only be found with the "brcm,bcm7216-sdhci" compatible >> which implies using brcmstb_reset(). > > I don't see any in-tree device trees for these chips (which is OK), and > that's not what the Documentation/ says, and AFAICT nothing in the > driver is limiting other variants from specifying the "supports-cqe" > flag in their (out-of-tree) device tree. The closest thing I see is that > an *example* in brcm,sdhci-brcmstb.yaml shows "supports-cqe" only on > brcm,bcm7216-sdhci -- but an example is not a binding agreement. Am I > missing something? > > Now of course, you probably know behind the scenes that there are no > other sdhci-brcmstb-relevant controllers that "support cqe", but AFAICT > I have no way of knowing that a priori. The driver and bindings give > (too much?) flexibility. Yes that is fair enough, I will amend the binding document to make it clearer that 'supports-cqe' only applies in case of "brcm,bcm7216-sdhci" and not for other compatibles. -- Florian _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel