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 X-Spam-Level: X-Spam-Status: No, score=-13.8 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0BB8EC56201 for ; Fri, 20 Nov 2020 16:00:14 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 9039D22264 for ; Fri, 20 Nov 2020 16:00:13 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="oAu91dDE"; dkim=temperror (0-bit key) header.d=cerno.tech header.i=@cerno.tech header.b="gE5ULDdL"; dkim=temperror (0-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b="khT/YBqX" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9039D22264 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=cerno.tech Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=HVrDcPuSbmp7glbKbo9YzPnabxS7Cmmop3WkQgJa3mc=; b=oAu91dDEZECh4wk7TcW3Q/fTx j9Q8i127ymKFYxVkIsEgFui82XWCJzSAzHJpODEGD40GE9XCJbGcChtsHHspXtjQ3fFM5sdVPUZKP tn0koc7eBuk5w/WsVtrNWl+R+E9r7DVOOUz/elpTdl390XypdPLamaCkBILfiv80+m3VqWvoM2loA ctBzlCw2alOLpmPIVCS1x5ZLN57Z/rho4N/Pu0ahdBVbvgYkChNp6TDif++VC1LOa+phApkZzh9yy HGqNdYgQnjHr3kPrOP7oR9o5UQ96medUmAEfDALRBngubQFQV3XIpPOgfiPOb4B6XTWzqywkIA3PW pReIRbPrg==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kg8p7-0004nf-CE; Fri, 20 Nov 2020 15:59:45 +0000 Received: from out4-smtp.messagingengine.com ([66.111.4.28]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kg8p4-0004n0-0e for linux-arm-kernel@lists.infradead.org; Fri, 20 Nov 2020 15:59:42 +0000 Received: from compute6.internal (compute6.nyi.internal [10.202.2.46]) by mailout.nyi.internal (Postfix) with ESMTP id 6AFD05C00D0; Fri, 20 Nov 2020 10:59:41 -0500 (EST) Received: from mailfrontend1 ([10.202.2.162]) by compute6.internal (MEProxy); Fri, 20 Nov 2020 10:59:41 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cerno.tech; h= date:from:to:cc:subject:message-id:references:mime-version :content-type:in-reply-to; s=fm1; bh=T1SgiXg1TuJbavRTzfVKemq6SSp RHfqpbA6KSfHRZ3A=; b=gE5ULDdLWhUA+NzBMolnoMD9lgWF8gh0PfpWMoF9dYa BEEK5RJcgGrhXIEGQC/XTNwJyGUtpHa5szUh6RXT9vm9ZWnyhbrOy6m1PA4Sv/nF eSeURNX+9PkuDE06XJqTS7I5HRdZCIkN1X213ZZK4yTSdNmnzUhCkiSllbI94xO1 m2Ng2DUbaqkfD3CoxzWLopvrn22BhPMu/doL2ltf8dnOBOUfj5juMWW2ZXWpBemo EtlKMIVKZGo+1e3m6YRYiMKm5HdBCrBWwIc21ZV9pfTvpGBGgqmvKpv4J2+LldWI iQmT2yVsUZVYJsStw82KhLeKjrMoiagoiuv22XIdehw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm1; bh=T1SgiX g1TuJbavRTzfVKemq6SSpRHfqpbA6KSfHRZ3A=; b=khT/YBqXeZ4OkkoVG2ymxs N8R3KV9iauibyf3Rix7OFTwH1xv8JvlnnvtOqB6PUtNqt3etRvdGVsvEsCZWqpNX +T69G1b51OJYtRnnE7gjJEAfj6xBAYEB/Vr50wEKsR1UZQIR+yRYyDs+RErgFw3Y 7Yl+dom98iHtyXcfzfI3A3wVR3quu2NdX59M0rxi1FZS9bPdfG6cJNMOb9IWn7R8 6OGo95HzWYIXc/1XAumfKjFO4YqKA1hRgWPzxWi8jt0lwrEf3H39lgGEW1IZVhhJ 61iDZ5eNJ70lDUuR0O+fWFtyVwgwSK/+7Us49yfYnZgEYBG/Yf1vMIpAVrnbv0sw == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedujedrudegtddgkeduucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepfffhvffukfhfgggtuggjsehgtderredttdejnecuhfhrohhmpeforgigihhm vgcutfhiphgrrhguuceomhgrgihimhgvsegtvghrnhhordhtvggthheqnecuggftrfgrth htvghrnhepuedtgfejueduheevgfevvdettdduleffgfffkeeltdffkeegudekjeeuveei gedunecukfhppeeltddrkeelrdeikedrjeeinecuvehluhhsthgvrhfuihiivgeptdenuc frrghrrghmpehmrghilhhfrhhomhepmhgrgihimhgvsegtvghrnhhordhtvggthh X-ME-Proxy: Received: from localhost (lfbn-tou-1-1502-76.w90-89.abo.wanadoo.fr [90.89.68.76]) by mail.messagingengine.com (Postfix) with ESMTPA id 719ED3280060; Fri, 20 Nov 2020 10:59:40 -0500 (EST) Date: Fri, 20 Nov 2020 16:59:39 +0100 From: Maxime Ripard To: Icenowy Zheng Subject: Re: [linux-sunxi] Re: [PATCH 3/3] arm64: allwinner: dts: a64: add DT for PineTab developer sample Message-ID: <20201120155939.3ajmbny2pka2vsnf@gilmour> References: <20201107124958.2222253-1-icenowy@aosc.io> <20201107125332.2223197-1-icenowy@aosc.io> <20201110103925.rbej5ueo2fefbmlp@gilmour.lan> <6175E674-E8BC-4199-8BE8-A983065C32D5@aosc.io> <20201116155508.364dg6ycklwylswe@gilmour.lan> <8FFC1A6C-9CA4-4F94-91C4-F111A7519979@aosc.io> MIME-Version: 1.0 In-Reply-To: <8FFC1A6C-9CA4-4F94-91C4-F111A7519979@aosc.io> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201120_105942_118797_4AB77896 X-CRM114-Status: GOOD ( 25.52 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, Jernej Skrabec , linux-sunxi@googlegroups.com, linux-kernel@vger.kernel.org, Chen-Yu Tsai , Rob Herring , linux-arm-kernel@lists.infradead.org Content-Type: multipart/mixed; boundary="===============2686611807964196657==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============2686611807964196657== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="kr27hb3hbkksxdbb" Content-Disposition: inline --kr27hb3hbkksxdbb Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Nov 17, 2020 at 02:36:48AM +0800, Icenowy Zheng wrote: >=20 >=20 > =E4=BA=8E 2020=E5=B9=B411=E6=9C=8816=E6=97=A5 GMT+08:00 =E4=B8=8B=E5=8D= =8811:55:08, Maxime Ripard =E5=86=99=E5=88=B0: > >On Tue, Nov 10, 2020 at 06:41:37PM +0800, Icenowy Zheng wrote: > >>=20 > >>=20 > >> =E4=BA=8E 2020=E5=B9=B411=E6=9C=8810=E6=97=A5 GMT+08:00 =E4=B8=8B=E5= =8D=886:39:25, Maxime Ripard > >=E5=86=99=E5=88=B0: > >> >On Sat, Nov 07, 2020 at 08:53:32PM +0800, Icenowy Zheng wrote: > >> >> Some developers received PineTab samples that used an old LCD > >panel. > >> >>=20 > >> >> Add device tree for these samples. > >> >>=20 > >> >> Signed-off-by: Icenowy Zheng > >> >> --- > >> >> arch/arm64/boot/dts/allwinner/Makefile | 1 + > >> >> .../dts/allwinner/sun50i-a64-pinetab-dev.dts | 28 > >> >+++++++++++++++++++ > >> >> 2 files changed, 29 insertions(+) > >> >> create mode 100644 > >> >arch/arm64/boot/dts/allwinner/sun50i-a64-pinetab-dev.dts > >> >>=20 > >> >> diff --git a/arch/arm64/boot/dts/allwinner/Makefile > >> >b/arch/arm64/boot/dts/allwinner/Makefile > >> >> index 211d1e9d4701..a221dcebfad4 100644 > >> >> --- a/arch/arm64/boot/dts/allwinner/Makefile > >> >> +++ b/arch/arm64/boot/dts/allwinner/Makefile > >> >> @@ -13,6 +13,7 @@ dtb-$(CONFIG_ARCH_SUNXI) +=3D > >> >sun50i-a64-pinephone-1.0.dtb > >> >> dtb-$(CONFIG_ARCH_SUNXI) +=3D sun50i-a64-pinephone-1.1.dtb > >> >> dtb-$(CONFIG_ARCH_SUNXI) +=3D sun50i-a64-pinephone-1.2.dtb > >> >> dtb-$(CONFIG_ARCH_SUNXI) +=3D sun50i-a64-pinetab.dtb > >> >> +dtb-$(CONFIG_ARCH_SUNXI) +=3D sun50i-a64-pinetab-dev.dtb > >> >> dtb-$(CONFIG_ARCH_SUNXI) +=3D sun50i-a64-sopine-baseboard.dtb > >> >> dtb-$(CONFIG_ARCH_SUNXI) +=3D sun50i-a64-teres-i.dtb > >> >> dtb-$(CONFIG_ARCH_SUNXI) +=3D sun50i-a100-allwinner-perf1.dtb > >> >> diff --git > >a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinetab-dev.dts > >> >b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinetab-dev.dts > >> >> new file mode 100644 > >> >> index 000000000000..3a4153890f3e > >> >> --- /dev/null > >> >> +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinetab-dev.dts > >> >> @@ -0,0 +1,28 @@ > >> >> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > >> >> +/* > >> >> + * Copyright (C) 2020 Icenowy Zheng > >> >> + * > >> >> + */ > >> >> + > >> >> +/dts-v1/; > >> >> + > >> >> +#include "sun50i-a64-pinetab.dts" > >> >> + > >> >> +/ { > >> >> + model =3D "PineTab Developer Sample"; > >> >> + compatible =3D "pine64,pinetab-dev", "allwinner,sun50i-a64"; > >> >> +}; > >> > > >> >Changing the DT and the compatible half-way through it isn't ok. > >Please > >> >add a new DT with the newer revision like we did for the pinephone > >>=20 > >> We did this on Pine H64. > > > >What are you referring to? I couldn't find a commit where we did what > >you suggested in that patch to the pine H64. >=20 > Oh the situation is complex. On Pine H64, we didn't specify anything at > start (which is the same here), the DT is originally version-neutral > but then transitioned to model B, then reverted to model A. Here the DT i= s always > for the sample. >=20 > However, for Pine H64 there's model A/B names, but for PineTab there's no > any samples that are sold, thus except who got the samples, all PineTab > owners simply owns a "PineTab", not a "PineTab xxx version". It's fairly simple really, we can't really predict the future, so any DT submitted is for the current version of whatever board there is. This is what we (somewhat messily) did for the PineH64, for the pinephone, or really any other board that has several revisions Maxime --kr27hb3hbkksxdbb Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCX7fn6gAKCRDj7w1vZxhR xQDkAQCfC6YLcVIPzWXZLKF5hjULRGNX9s+snk1vmmRwyYTx+AD+ME8Rr9Zwu0/K hMZs3H3xTpAk0y09kQAE5aboePkK3wY= =u1dU -----END PGP SIGNATURE----- --kr27hb3hbkksxdbb-- --===============2686611807964196657== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============2686611807964196657==--