Received: by 2002:a25:1506:0:0:0:0:0 with SMTP id 6csp1832801ybv; Fri, 14 Feb 2020 06:49:42 -0800 (PST) X-Google-Smtp-Source: APXvYqzvQd02ThFRaDJnuBdm0zyu0KG2nTOutXCqOkg5x446BshQzhuPhx9T+UCEMW6DKhBeewng X-Received: by 2002:a9d:aea:: with SMTP id 97mr2547362otq.51.1581691782203; Fri, 14 Feb 2020 06:49:42 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1581691782; cv=none; d=google.com; s=arc-20160816; b=MlGEouJpUAEDwyi5pZpB8aE31Bue9gVH63IUeVogJCWH/p/WfK7hRfYtYmSnvTzNm4 +/V6VUt6BrfVl0qHmNqQ5bYGNaeXZRBTk61loCIJMxqq65OxbHvldroEPfD7G2mptjkr ABoI1ndU3B1cvW0RJDUUy1uoBwQZJ6+m/m2SiBJGHgQs6+WjEb1xRE9h2U0h2vUWzjV4 6ffFKIyx0iPG1VFKZD8n/sxTAkDPDA/ACOBbyiOUZg/tstVJ+tJ9ECmRRJaGmg7a3Vlo WB6S2/Z9VToZOGfm5ke5/mkl1hvgA1Zwej1EUOy+15XFoxvwENEtvRD12SSW5k7a5iiT qDFA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature; bh=aQmPj/He60oqx22LSZ9uL16ZlUIptiUj/aXAEEJHvoY=; b=nIPJ7Khf5imCKGlG4qHNwS8eATPgrGFbAczES3REbKH87d4TnGAQtNU6EC9khOUihn FSEUL8OwAG1g4koa9CchMxeVZL9/EMyV4eLGzUiy72G018ryFSczsO2wdGv2Zp7A1Zc6 0jCMp2pIDaA/IRfr14P5vkP8cB+/m8Z8/RMrKTwRq/wKEZ1fFj0HlaUXceu/rqRg2AF2 NcyA+ixLC5qxRkxueko/7iJn4CS56EoUUQaBzY/2SmyUjYIMu0utJGHZ+b18Cjuj8C8G BlICZ8TIFaScT79DKlsgq6rS8T83Oup9AdeK4dA1ZyDngWB9TJNvUNZMpaTr6QyJSJ3m P1/Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=Pbhb2Fem; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id u79si3149107oia.11.2020.02.14.06.49.30; Fri, 14 Feb 2020 06:49:42 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=Pbhb2Fem; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729435AbgBNOtS (ORCPT + 99 others); Fri, 14 Feb 2020 09:49:18 -0500 Received: from mail.kernel.org ([198.145.29.99]:44322 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729294AbgBNOtS (ORCPT ); Fri, 14 Feb 2020 09:49:18 -0500 Received: from earth.universe (dyndsl-095-033-170-229.ewe-ip-backbone.de [95.33.170.229]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 013BA222C2; Fri, 14 Feb 2020 14:49:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1581691757; bh=+U/C5/WoFlL7gJcY7WhJ5ANkFVe5DyWAcqpTE17f8M4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Pbhb2FemxY1A3p5lgEEfOHh7xg70hLcmPLizP+Ub11LjzSTJdxDUGrPw3K32Jz5Nq 6ubz8Cc12RC8+F3LFx5zlHkBcQ+4dL68aQcVPxDW2Zzy+ymWpPnekCGPLEzGJlXWOg ZVUEDrWVEe44hgH9LiZhAjqNJuIyieUvFWHke7Lo= Received: by earth.universe (Postfix, from userid 1000) id 410453C0C83; Fri, 14 Feb 2020 15:49:15 +0100 (CET) Date: Fri, 14 Feb 2020 15:49:15 +0100 From: Sebastian Reichel To: Merlijn Wajer Cc: =?utf-8?Q?Beno=C3=AEt?= Cousson , Tony Lindgren , Rob Herring , Mark Rutland , Dmitry Torokhov , Mattias Jacobsson <2pi@mok.nu>, "Darren Hart (VMware)" , linux-omap@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-input@vger.kernel.org Subject: Re: [RFC PATCH 2/2] ARM: dts: n900: remove mmc1 card detect gpio Message-ID: <20200214144915.jnmnsqennc4jvvsv@earth.universe> References: <20200214130249.6845-1-merlijn@wizzup.org> <20200214130249.6845-3-merlijn@wizzup.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="lsufgzzbafems3q2" Content-Disposition: inline In-Reply-To: <20200214130249.6845-3-merlijn@wizzup.org> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --lsufgzzbafems3q2 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Fri, Feb 14, 2020 at 02:02:48PM +0100, Merlijn Wajer wrote: > Instead, expose the key via the input framework, as SW_MACHINE_COVER How about: The chip-detect GPIO is actually detecting if the cover is closed. Technically it's possible to use the SD card with open cover. The only downside is risk of battery falling out and user being able to physically remove the card. The behaviour of SD card not being available when the device is open is unexpected and creates more problems than it solves. There is a high chance, that more people accidently break their rootfs by opening the case without physically removing the card. > --- Missing SoB. Patch itself is Reviewed-by: Sebastian Reichel -- Sebastian > arch/arm/boot/dts/omap3-n900.dts | 12 ++++++++---- > 1 file changed, 8 insertions(+), 4 deletions(-) >=20 > diff --git a/arch/arm/boot/dts/omap3-n900.dts b/arch/arm/boot/dts/omap3-n= 900.dts > index 7028a7cb2849..ed773e1609a5 100644 > --- a/arch/arm/boot/dts/omap3-n900.dts > +++ b/arch/arm/boot/dts/omap3-n900.dts > @@ -108,6 +108,14 @@ > linux,code =3D ; > linux,can-disable; > }; > + > + machine_cover { > + label =3D "Machine Cover"; > + gpios =3D <&gpio6 0 GPIO_ACTIVE_LOW>; /* 160 */ > + linux,input-type =3D ; > + linux,code =3D ; > + linux,can-disable; > + }; > }; > =20 > isp1707: isp1707 { > @@ -805,10 +813,6 @@ > pinctrl-0 =3D <&mmc1_pins>; > vmmc-supply =3D <&vmmc1>; > bus-width =3D <4>; > - /* For debugging, it is often good idea to remove this GPIO. > - It means you can remove back cover (to reboot by removing > - battery) and still use the MMC card. */ > - cd-gpios =3D <&gpio6 0 GPIO_ACTIVE_LOW>; /* 160 */ > }; > =20 > /* most boards use vaux3, only some old versions use vmmc2 instead */ > --=20 > 2.23.0 >=20 --lsufgzzbafems3q2 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAl5Gs2UACgkQ2O7X88g7 +pocChAAohM6F5rZSxC2jdWHmrT/fWK7ws4C0hdiAaN5T48NxDbgOTEIt2FSpJiL Pz/fAFkzRp5C7XQMnS0kgmKhpSeGf1BTuqUHP1+zveCVVugzbpmBF88eFMxhEzFX ifR6qNrIraksoPPygP9kVg8Z4tM0efv4thiPsYDnQ6EeVUAfD/nzPV4DOCyuU9s+ wtrTll+4dJCk6RH042LSd2lcU16eWQKMNgZeEezUCnJxxv2U5Htf8HEAjLnwMM/T 48lV38WxGw7EH+XS1RXKIkeh9wubbBRZt8gsjBr4LNp264Hm3MY3vp2z5YSq3GPQ RFFXjF7ewPvgVbx7RgvqqnLpc0aozGooUJYQEpRiIElsBs1OOXOstp+xtvrkofwT 4RelJkkvsxtLGoBaUkDKJePErDC+dkGkEEDZ9XBsI3QEY10QCJUc/MUzDjUASJcb BrxKAh32jfzWG4xAU+KtI4hCWJQjlr7rjkqsj26Ao4nLRpGwE5ZjjiGBIPtWO/Hc cRrqEDk2eAq9PPNzdYo2s4KRJZ+UPPUUiKjkGku/JM6tWXwSW8bmTvwYXsWFY4S8 66j7U07LDNtsQq5KevhRHYwuRFfX1RGd8rDUEI0t/JJxmgrSWAHNsBz40z3uFDzF bgPD0U07aeZ4qbk28xFChIy12Df1vDahLDeXBrYA8vCHgoeu72g= =0Z0i -----END PGP SIGNATURE----- --lsufgzzbafems3q2--