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 20BF2CCD192 for ; Wed, 18 Sep 2024 10:03: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:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:MIME-Version:References:In-Reply-To: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=h7bPQY3ZhK/kSQzRiOpOJJWq1Mly/YzpU+Xm9AUcnLA=; b=UmZbEUBrA6d7htRjuiUfuP504b acOrl2WaI3dQ6MIj20ebVrPi15pzCkwx56pegRnQ2nGmmhRD0p79AZn3xCwwIyqWiVIi8KrgcdLT9 bARXWcOorKTRL8oRJw5+om7YkhikN6SZ1jQCBkbKOG6kvZBiXRUjkqBlARWw0BbHJaRfmR8CsS6T5 3qE5x3tUWWu0ByuDRcxD1Jx2GaYP5ec0eohE3j9tNVPyrHTwaZL5TcYHAqzxft6JZ3Ppyk6vYBK/U qi/q5ynmBSkU8/V6V/V8APTKaGKzx2VpBdH89lgizAQPfMTMvE+Q47AdCPz1YmPYd+Z5xVcPPjMm9 pN6F8ujw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1sqrWo-00000007ypf-2FXb; Wed, 18 Sep 2024 10:03:18 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1sqrVi-00000007ykY-0gh3 for linux-arm-kernel@lists.infradead.org; Wed, 18 Sep 2024 10:02:12 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 9FE12FEC; Wed, 18 Sep 2024 03:02:35 -0700 (PDT) Received: from donnerap.manchester.arm.com (usa-sjc-imap-foss1.foss.arm.com [10.121.207.14]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id ABC7B3F64C; Wed, 18 Sep 2024 03:02:04 -0700 (PDT) Date: Wed, 18 Sep 2024 11:02:00 +0100 From: Andre Przywara To: =?UTF-8?B?S3J5xaF0b2YgxIxlcm7DvQ==?= Cc: devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev Subject: Re: [PATCH] arm64: dts: allwinner: h5: NanoPi NEO Plus2: Add disable-wp to mmc0 Message-ID: <20240918110200.4f6be49c@donnerap.manchester.arm.com> In-Reply-To: References: <20240914-b4-nanopineoplus2-fix-mmc0-wp-v1-1-12f54f0d6620@gmail.com> <20240916150809.21889537@donnerap.manchester.arm.com> Organization: ARM X-Mailer: Claws Mail 3.18.0 (GTK+ 2.24.32; aarch64-unknown-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240918_030210_269758_BA3D7A19 X-CRM114-Status: GOOD ( 20.13 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, 17 Sep 2024 17:51:11 +0200 Kry=C5=A1tof =C4=8Cern=C3=BD wrote: Hi Kry=C5=A1tof, (putting back the lists, please keep the discussion public, for everyone's benefit and for having this archived) > 16. z=C3=A1=C5=99=C3=AD 2024 16:08:09 SEL=C4=8C, Andre Przywara napsal: > >On Sat, 14 Sep 2024 12:07:00 +0200 > >Kry=C5=A1tof =C4=8Cern=C3=BD wrote: > > =20 > >> The board does not have wp pin/switch for micro SD card (mmc0). =20 > > > >That is true, but for *every* microSD card. So I wonder if we should add > >this property to all of the boards? I believe none of the arm64 boards h= ave > >full size SD card slots? =20 >=20 > I believe this would be a correct thing to do for micro SD cards, few boa= rds do have this property set already, but most don't. However, I am not 10= 0% certain (I have had a lot of SBCs, in my hands, but definitely not all o= f them), so I added it for this board only. So do you think it should be ad= ded for other boards too, or every user should add it for their SBC on thei= r own? Yes, I would like you to add this for the other arm64 boards, too. I appreciate your concerns about not knowing, but we can help out here. Maybe you can quickly go over the wiki and check each board's picture for having a microSD slot, just to be on the safe side. But I am pretty sure that full-size SD slots were only used on the first generation (A10) boards, and any arm64 board always uses micro-SD slots. I would double check this on my side during review. If we would wait for individual board owners to submit patches, we would wait forever. So thank you for spotting this, and sending a fix, but it would reduce churn and improve the situation for everyone if you would cover the other boards as well. Thanks! Andre