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=-3.8 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no 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 D1008C4361B for ; Mon, 7 Dec 2020 10:08:30 +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 821082310B for ; Mon, 7 Dec 2020 10:08:30 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 821082310B 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=W5NbYKSlCMpeIf1+w3EzJ8xcAxCBaGF0z0XI3rgiyEU=; b=erP4hXXRYa14XQzLZxJpnAcqY 6cBRPXSR0a/i3nJNBed1YaYGmNfIXfRpkXa2APfFsuwE/yztqCsWnOELoA2i+Eu+RJUBv3K5/2E3l fVmpSO7pJ3Aeo5HnRklla4RDKxukCNiXfaCdxdwgtWlk80XmrV5EewWJ+Cu4/fPmVyA8QudJP0QXp +w6bhWvqgq8TJeCfIW997bdkdrLPEmvyGStLxn/NFhk6JkrFLjpRCYmhTDUqFxkFHiM/1HtsRnWfQ pwJv21laj/gFCjDw7FqdkzyluXuOmBUBR4grMi+IHNqV+BnwwnYbnbszzWmauNLLF0oKEGPwbLvgW 71cMyUo4g==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kmDQF-0003zK-1d; Mon, 07 Dec 2020 10:07:11 +0000 Received: from wout4-smtp.messagingengine.com ([64.147.123.20]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kmDQC-0003yi-Of for linux-arm-kernel@lists.infradead.org; Mon, 07 Dec 2020 10:07:09 +0000 Received: from compute6.internal (compute6.nyi.internal [10.202.2.46]) by mailout.west.internal (Postfix) with ESMTP id 8C623C5E; Mon, 7 Dec 2020 05:07:06 -0500 (EST) Received: from mailfrontend1 ([10.202.2.162]) by compute6.internal (MEProxy); Mon, 07 Dec 2020 05:07:07 -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=NbC2rEyqxYKunHDEfwzu0AKzVpe iws1VQtlVFzi38L4=; b=VqgVpTxNvPLk2cQTzwO3Txv1qM8ENYD2ADo3vR4JqCs waIWbiUxFblI+r3kyhykZhjihHR4miHM8HYprjdn6VLs4C5JvU+omiE0Vu1B8F3w TIouqejq6CZFllExC2XkAjoe6YkwITlJ6YdxWVszj+f1Vj+oyS50XMEB2giYJE+5 tqpso+B5nWpQ39wDuqSsU12yoho+QIj7y3THsA60vcc/N7d4Z4Ft+n4sT6HXBjcm imPH13uSKLxnJ4lodG8pMOVSjWOOH/eeMEEzQq9bIXw6ACBolwjTbf0zwsOlXAqC hSyXkVhx9Hcd9Ja0LbQv+xkLfZXYfLVazNZK/Gv4/hA== 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=NbC2rE yqxYKunHDEfwzu0AKzVpeiws1VQtlVFzi38L4=; b=HQsjYMEMqp/I4of0A4Hq76 sREvII8R2Q/vxtwwv5cNOK0lcjSImpDkM3A2O49wJzALl/JxSUJTiizhkRnP1hFp T6pFxcnS0lxKmdN+fvGQpbFK2SvLpEj+7xGHkDviohNFBgK/kbBnEldnE2qb7JEX KMR2jZCpARA/w//z8wwcgpA5j0pft0cNLnRbYssEWf3DkteXd9Xydmw0O/b2g0Dq XxP9czrV+QxswrUAv7Km8hhY4Hx6R8urp3zUl9ty/ux+EV3J/S/SOdbmrQCheT0O CHqQGZRxpiDCa3tatOWI3QNj5jUag94tVgbSSDhSJkRBHctKKauO/V9xdd8jr8Ug == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedujedrudejgedgudduucetufdoteggodetrfdotf 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 C3E68240057; Mon, 7 Dec 2020 05:07:04 -0500 (EST) Date: Mon, 7 Dec 2020 11:07:03 +0100 From: Maxime Ripard To: =?utf-8?B?Q2zDqW1lbnQgUMOpcm9u?= Subject: Re: [linux-sunxi] Re: [PATCH 3/3] arm64: allwinner: dts: a64: add DT for PineTab developer sample Message-ID: <20201207100703.tfavhmexbtrzucry@gilmour> References: <97E2037C-3C3C-4B0B-8462-39B9E38CB3BB@aosc.io> <20201123125332.2p5z3ew7svszvyfs@gilmour> <009A22D9-AF20-45C4-9674-13334B3EFFBA@aosc.io> <20201128103827.d6sfc2eumli2betx@gilmour> <1666a61f6ea3e7d573795f9000a0b096c7b7dee0.camel@aosc.io> <20201201103519.b7rbkmjssmvy6qeo@gilmour> MIME-Version: 1.0 In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201207_050709_041269_DF0B13A2 X-CRM114-Status: GOOD ( 38.33 ) 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 , Jernej Skrabec , Samuel Holland , linux-kernel , linux-sunxi , Rob Herring , Chen-Yu Tsai , linux-arm-kernel , Icenowy Zheng Content-Type: multipart/mixed; boundary="===============1113653182890200437==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============1113653182890200437== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="7z2tcet5tgoj6v36" Content-Disposition: inline --7z2tcet5tgoj6v36 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sun, Dec 06, 2020 at 10:03:16PM +0100, Cl=C3=A9ment P=C3=A9ron wrote: > Hi Maxime >=20 > On Tue, 1 Dec 2020 at 11:35, Maxime Ripard wrote: > > > > On Sat, Nov 28, 2020 at 10:07:27PM +0100, Cl=C3=A9ment P=C3=A9ron wrote: > > > Hi Maxime, Icenowy, > > > > > > On Sat, 28 Nov 2020 at 12:59, Icenowy Zheng wrote: > > > > > > > > > > > > > > > > =E4=BA=8E 2020=E5=B9=B411=E6=9C=8828=E6=97=A5 GMT+08:00 =E4=B8=8B= =E5=8D=887:54:04, "Cl=C3=A9ment P=C3=A9ron" =E5=86= =99=E5=88=B0: > > > > >Hi Icenowy, > > > > > > > > > >On Sat, 28 Nov 2020 at 12:28, Icenowy Zheng wrot= e: > > > > >> > > > > >> =E5=9C=A8 2020-11-28=E6=98=9F=E6=9C=9F=E5=85=AD=E7=9A=84 11:38 += 0100=EF=BC=8CMaxime Ripard=E5=86=99=E9=81=93=EF=BC=9A > > > > >> > On Mon, Nov 23, 2020 at 09:10:38PM +0800, Icenowy Zheng wrote: > > > > >> > > > > > > > Okay. But I'm not satisfied with a non-public samp= le > > > > >> > > > > > > > occupies > > > > >> > > > > > > > the pinetab name. Is rename it to pinetab-dev and = add a > > > > >> > > > > > > > pinetab-retail okay? > > > > >> > > > > > > > > > > >> > > > > > > To me, naming the production version anything but > > > > >"pinetab" > > > > >> > > > > > > isn't > > > > >> > > > > > > satisfying either. > > > > >> > > > > > > > > > >> > > > > > I understand where you're coming from, but the point I= was > > > > >> > > > > > making my > > > > >> > > > > > previous mail is precisely that it's not really possib= le. > > > > >> > > > > > > > > > >> > > > > > You want to name the early adopter version _the_ produ= ction > > > > >> > > > > > version. Let's assume the hardware changes again betwe= en > > > > >the > > > > >> > > > > > early > > > > >> > > > > > adopter and mass-production version. Which one will be > > > > >_the_ > > > > >> > > > > > production version? The early-adopter or the mass-prod= uced > > > > >> > > > > > one? > > > > >> > > > > > > > > > >> > > > > > There's really no good answer here, and both would suc= k in > > > > >> > > > > > their > > > > >> > > > > > own way. The only way to deal with this is to simply a= void > > > > >> > > > > > defining one version as the one true board, and just > > > > >loading > > > > >> > > > > > the > > > > >> > > > > > proper DT in u-boot based on whatever clue we have of = the > > > > >> > > > > > hardware > > > > >> > > > > > revision. > > > > >> > > > > Then will it be okay to rename current pinetab DT to > > > > >> > > > > pinetab-sample and then introduce new DTs all with suffi= xes? > > > > >> > > > > > > > >> > > > No. From my previous mail: > > > > >> > > > > > > >> > > It can be seen as dropping the PineTab DT and introduce new = DTs > > > > >> > > with > > > > >> > > suffix. > > > > >> > > > > > > >> > > Do we have rule that we cannot drop boards? > > > > >> > > > > > >> > Are you really arguing that removing a DT and then adding an > > > > >> > identical > > > > >> > one under a different name is not renaming it? > > > > >> > > > > >> Then we can just keep confusing name? > > > > > > > > > >Sorry maybe I missed some information > > > > >But why don't you do like pinephone? > > > > > > > > They're the same board revision with different LCD panels. > > > > > > I just ask Pine64 about this and here is the reply : > > > "The PineTab LCD panel change was a last minutes before production > > > starts that vendor advise us switch over to new LCD controller due to > > > EoL concern". > > > > > > "Pine64 communication" is not so bad we just need to ask and they rep= ly :) > > > > > > So the issue is not that the Product was not finalized but that one > > > component arrives in EoL. > > > This could also happens during a running production. > > > > Like you said, it can happen pretty much any time, for any reason, so > > the intent doesn't really matter here. >=20 > Agree, that was more to support Pin64 guys here. >=20 > As pinetab compatible can't be reused maybe somethings like this : > sun50i-a64-pinetab.dtsi > sun50i-a64-pinetab-1.0-early-adopter.dtb > sun50i-a64-pinetab-1.0.dtb or sun50i-a64-pinetab-retail.dtb. But > retail is like prod it's not explicit. >=20 > What do you think ? I already said what I think multiple times in this thread: the DT that has been merged for the early adopter, devs, whatever, won't be renamed. As far as I'm concerned, I don't care about the name that is picked for the new version as long as everyone is clear on the fact that that name won't change ever either. Maxime --7z2tcet5tgoj6v36 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCX83+xwAKCRDj7w1vZxhR xdWMAQD0dT1OLbOP8Y4Rt1xw421wx928drzSaTET8HnvuRI9CQD/YOVfccphlNsN sP7QKx7HJq+niBWklfJ9I8z8cdODLgQ= =j1fo -----END PGP SIGNATURE----- --7z2tcet5tgoj6v36-- --===============1113653182890200437== 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 --===============1113653182890200437==--