From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from pandora.armlinux.org.uk (pandora.armlinux.org.uk [78.32.30.218]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2F135364A1; Mon, 15 Apr 2024 08:23:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=78.32.30.218 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1713169400; cv=none; b=N3tBoLany8wQSWREpoxjFm+BMutqDeeIFd27mfnF3prhyyebXqLD/RVbKmrZ8YNWH0GUHEsdPZRhu6QUfIlxCpG/+RV3mESKYDeMowu1mm5HKCep/baVHGIERYKUlFwv2yObIMLc2ZQyEb97+NIirtcK5b3dxYEB90FfYQeJkI4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1713169400; c=relaxed/simple; bh=k43CipV8ZHSfpduPM45MpqkCIYdzmykelApczgqGy3g=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=czoc/oPlpejsWDvkF6ezcvSRZcsod+MV42DmvtmcWGX+TJMVqL9SFOiODnaN7PTLE2MMMrOD5/wgRXTUXZhuB+11XfX+YgNHALG5ms6SkSRtOkvI/ir6JFCwJQbuq5q78W/cG8AXyBaaz2mGimO/9pkL+qS5TWDeyzVpN/wPpA4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=armlinux.org.uk; spf=none smtp.mailfrom=armlinux.org.uk; dkim=pass (2048-bit key) header.d=armlinux.org.uk header.i=@armlinux.org.uk header.b=txcTHLnK; arc=none smtp.client-ip=78.32.30.218 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=armlinux.org.uk Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=armlinux.org.uk Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=armlinux.org.uk header.i=@armlinux.org.uk header.b="txcTHLnK" DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To: Content-Transfer-Encoding:Content-Type:MIME-Version:References:Message-ID: Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=BSJeJ5B1I88yxnyki1y91ceWqhSeop2Eza3ElbkRR1c=; b=txcTHLnKJi9KZQYV3Spam/Qcd7 nQdqTA/llna3f5QWvsBb500ayPp8XY7VYH7Iyr3IDpVwCZjuo25PNp7h02o+9oRZmHDhOBhm6D7WZ 0ShvMAmTYwtCgoaG1uUi7gwz/PyNAL5Jl58+PauQinJoZ56N82WdgxB96rB1m9Via+Hpymk4bLl40 JmlAl7Q35cel89jG0ONxsvDwFuq9w8Z6QCXDGnK2mfSVOpoNO/QfHy03TZK8knjKLaa6JB/uirHMU y+MUiB3gGx1pLjIQSaxWvqhana82u0ByAnF5IimKbEME8tMNxwXUAb/qcD32orAkVG/CWuzPlaq4s DvxDNO1g==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:50578) by pandora.armlinux.org.uk with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1rwHcN-0005ld-1Z; Mon, 15 Apr 2024 09:23:11 +0100 Received: from linux by shell.armlinux.org.uk with local (Exim 4.94.2) (envelope-from ) id 1rwHcN-00023Y-49; Mon, 15 Apr 2024 09:23:11 +0100 Date: Mon, 15 Apr 2024 09:23:11 +0100 From: "Russell King (Oracle)" To: richard clark Cc: Adrian Hunter , ulf.hansson@linaro.org, linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH] sdhci: Fix SD card detection issue Message-ID: References: <20240415070620.133143-1-richard.xnu.clark@gmail.com> <52c08a01-8357-44dd-b727-a06438ec6c30@intel.com> Precedence: bulk X-Mailing-List: linux-mmc@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Sender: Russell King (Oracle) On Mon, Apr 15, 2024 at 04:17:14PM +0800, richard clark wrote: > On Mon, Apr 15, 2024 at 3:22 PM Russell King (Oracle) > wrote: > > > > On Mon, Apr 15, 2024 at 10:18:39AM +0300, Adrian Hunter wrote: > > > On 15/04/24 10:06, Richard Clark wrote: > > > > The mmc_gpio_get_cd(...) will return 0 called from sdhci_get_cd(...), which means > > > > the card is not present. Actually, the card detection pin is active low by default > > > > according to the SDHCI psec, thus the card detection result is not correct, more > > > > > > SDHCI spec covers the SDHCI lines. GPIO is separate. > > > > ... and the key bit of information that should be mentioned is in the > > case of a GPIO, the GPIO library can be told if a GPIO is active-high > > or active-low in either firmware or via the GPIO lookup data, and this > > should be used instead of drivers inventing their own "quirking". > > > Agree! But unfortunately, it seems I can't find the right place to > handle this from either firmware or via the GPIO lookup data. Will be > appreciated if any suggestion about that?! If you're using DT, then, for example: cd-gpios = <&gpio1 4 GPIO_ACTIVE_LOW>; is all it takes. If you are using firmware then GPIO lookup data isn't what you should be using. I'm afraid I don't know the ACPI bindings for SDHCI. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last! 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 69632C4345F for ; Mon, 15 Apr 2024 08:23:31 +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-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=pJA5eR4+Q0CpxNXWiQAf22FZexZ3n0h2KhIl/5oIymg=; b=1tJ50SfXKmdmWe dy2EJyhmOPuKBzWAZGkMOUrHEhE7VwT5GzDB+AFaAX7/Wo8E0KwJkGhcyQodF7PXK4msYkLusLzeX Fqf1/CLHQ+Yo7K+sqVCwkYLRNJgRQlnkM4tv9x3ddFZNiNPWTtkbKjKM3j5srFY40R+uB8QvnitxL DdV8GzoAOVithyu42OqHXYLhIY+gvpC18Jajbvm+r2Ie1su98PRiutq6vQSUJBwr63wqXW8fBz/x2 6qUI5+ZtrRb6LwU9gvbrO2Ioi3LAkNCyAKms8ml2urEpsSO2uBxrpjDiPakfHysoACyXabi3YrEZO meqjhkpJNeSG5aygMVpQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rwHcW-00000007WsB-39V1; Mon, 15 Apr 2024 08:23:20 +0000 Received: from pandora.armlinux.org.uk ([2001:4d48:ad52:32c8:5054:ff:fe00:142]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rwHcU-00000007WrT-1MBN for linux-arm-kernel@lists.infradead.org; Mon, 15 Apr 2024 08:23:19 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To: Content-Transfer-Encoding:Content-Type:MIME-Version:References:Message-ID: Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=BSJeJ5B1I88yxnyki1y91ceWqhSeop2Eza3ElbkRR1c=; b=txcTHLnKJi9KZQYV3Spam/Qcd7 nQdqTA/llna3f5QWvsBb500ayPp8XY7VYH7Iyr3IDpVwCZjuo25PNp7h02o+9oRZmHDhOBhm6D7WZ 0ShvMAmTYwtCgoaG1uUi7gwz/PyNAL5Jl58+PauQinJoZ56N82WdgxB96rB1m9Via+Hpymk4bLl40 JmlAl7Q35cel89jG0ONxsvDwFuq9w8Z6QCXDGnK2mfSVOpoNO/QfHy03TZK8knjKLaa6JB/uirHMU y+MUiB3gGx1pLjIQSaxWvqhana82u0ByAnF5IimKbEME8tMNxwXUAb/qcD32orAkVG/CWuzPlaq4s DvxDNO1g==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:50578) by pandora.armlinux.org.uk with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1rwHcN-0005ld-1Z; Mon, 15 Apr 2024 09:23:11 +0100 Received: from linux by shell.armlinux.org.uk with local (Exim 4.94.2) (envelope-from ) id 1rwHcN-00023Y-49; Mon, 15 Apr 2024 09:23:11 +0100 Date: Mon, 15 Apr 2024 09:23:11 +0100 From: "Russell King (Oracle)" To: richard clark Cc: Adrian Hunter , ulf.hansson@linaro.org, linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH] sdhci: Fix SD card detection issue Message-ID: References: <20240415070620.133143-1-richard.xnu.clark@gmail.com> <52c08a01-8357-44dd-b727-a06438ec6c30@intel.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240415_012318_402583_E0AE0BC8 X-CRM114-Status: GOOD ( 19.82 ) 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-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org T24gTW9uLCBBcHIgMTUsIDIwMjQgYXQgMDQ6MTc6MTRQTSArMDgwMCwgcmljaGFyZCBjbGFyayB3 cm90ZToKPiBPbiBNb24sIEFwciAxNSwgMjAyNCBhdCAzOjIy4oCvUE0gUnVzc2VsbCBLaW5nIChP cmFjbGUpCj4gPGxpbnV4QGFybWxpbnV4Lm9yZy51az4gd3JvdGU6Cj4gPgo+ID4gT24gTW9uLCBB cHIgMTUsIDIwMjQgYXQgMTA6MTg6MzlBTSArMDMwMCwgQWRyaWFuIEh1bnRlciB3cm90ZToKPiA+ ID4gT24gMTUvMDQvMjQgMTA6MDYsIFJpY2hhcmQgQ2xhcmsgd3JvdGU6Cj4gPiA+ID4gVGhlIG1t Y19ncGlvX2dldF9jZCguLi4pIHdpbGwgcmV0dXJuIDAgY2FsbGVkIGZyb20gc2RoY2lfZ2V0X2Nk KC4uLiksIHdoaWNoIG1lYW5zCj4gPiA+ID4gdGhlIGNhcmQgaXMgbm90IHByZXNlbnQuIEFjdHVh bGx5LCB0aGUgY2FyZCBkZXRlY3Rpb24gcGluIGlzIGFjdGl2ZSBsb3cgYnkgZGVmYXVsdAo+ID4g PiA+IGFjY29yZGluZyB0byB0aGUgU0RIQ0kgcHNlYywgdGh1cyB0aGUgY2FyZCBkZXRlY3Rpb24g cmVzdWx0IGlzIG5vdCBjb3JyZWN0LCBtb3JlCj4gPiA+Cj4gPiA+IFNESENJIHNwZWMgY292ZXJz IHRoZSBTREhDSSBsaW5lcy4gIEdQSU8gaXMgc2VwYXJhdGUuCj4gPgo+ID4gLi4uIGFuZCB0aGUg a2V5IGJpdCBvZiBpbmZvcm1hdGlvbiB0aGF0IHNob3VsZCBiZSBtZW50aW9uZWQgaXMgaW4gdGhl Cj4gPiBjYXNlIG9mIGEgR1BJTywgdGhlIEdQSU8gbGlicmFyeSBjYW4gYmUgdG9sZCBpZiBhIEdQ SU8gaXMgYWN0aXZlLWhpZ2gKPiA+IG9yIGFjdGl2ZS1sb3cgaW4gZWl0aGVyIGZpcm13YXJlIG9y IHZpYSB0aGUgR1BJTyBsb29rdXAgZGF0YSwgYW5kIHRoaXMKPiA+IHNob3VsZCBiZSB1c2VkIGlu c3RlYWQgb2YgZHJpdmVycyBpbnZlbnRpbmcgdGhlaXIgb3duICJxdWlya2luZyIuCj4gPgo+IEFn cmVlISBCdXQgdW5mb3J0dW5hdGVseSwgaXQgc2VlbXMgSSBjYW4ndCBmaW5kIHRoZSByaWdodCBw bGFjZSB0bwo+IGhhbmRsZSB0aGlzIGZyb20gZWl0aGVyIGZpcm13YXJlIG9yIHZpYSB0aGUgR1BJ TyBsb29rdXAgZGF0YS4gV2lsbCBiZQo+IGFwcHJlY2lhdGVkIGlmIGFueSBzdWdnZXN0aW9uIGFi b3V0IHRoYXQ/IQoKSWYgeW91J3JlIHVzaW5nIERULCB0aGVuLCBmb3IgZXhhbXBsZToKCiAgICAg ICAgY2QtZ3Bpb3MgPSA8JmdwaW8xIDQgR1BJT19BQ1RJVkVfTE9XPjsKCmlzIGFsbCBpdCB0YWtl cy4gSWYgeW91IGFyZSB1c2luZyBmaXJtd2FyZSB0aGVuIEdQSU8gbG9va3VwIGRhdGEgaXNuJ3QK d2hhdCB5b3Ugc2hvdWxkIGJlIHVzaW5nLiBJJ20gYWZyYWlkIEkgZG9uJ3Qga25vdyB0aGUgQUNQ SSBiaW5kaW5ncyBmb3IKU0RIQ0kuCgotLSAKUk1LJ3MgUGF0Y2ggc3lzdGVtOiBodHRwczovL3d3 dy5hcm1saW51eC5vcmcudWsvZGV2ZWxvcGVyL3BhdGNoZXMvCkZUVFAgaXMgaGVyZSEgODBNYnBz IGRvd24gMTBNYnBzIHVwLiBEZWNlbnQgY29ubmVjdGl2aXR5IGF0IGxhc3QhCgpfX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpsaW51eC1hcm0ta2VybmVsIG1h aWxpbmcgbGlzdApsaW51eC1hcm0ta2VybmVsQGxpc3RzLmluZnJhZGVhZC5vcmcKaHR0cDovL2xp c3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1hcm0ta2VybmVsCg==