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 6156CC77B7D for ; Mon, 15 May 2023 16:15:25 +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-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-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=O/RD2xMG2O33yakvvpzqHsv7W/BQkczCuuXlqi6230E=; b=umjzwSwCz+5Gyc4VUrBmIZbtp9 DhVAGniC7jZ2KX+J3sX/coVGN4PrkNkISTAKAgNddx3QzELJNutQTDjr7qSYSYM7aP9tp+0cruQmb auCW5f3s6F55Zk5j+UhRWc2GViXnH3YSGw3kxRWszaxxK+RrZpQqr6uYdaJiha4cXTyicmZmKqvQ8 KFKzBzlHpVzCAv5Zpot/qCKbmW18CouqI04+TzxjhH88Qpg84RjdiUOO6tQIQYfPh2XfeejTXDXFi j6tLqsT4BJgvoXcB19kkO143OW+BTYEtvOf14pRfpYGXQS/wAWNjRxPiElvffLi3JYLCcT4e5Rg6d bHcgk1IQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1pyaqz-002iPl-08; Mon, 15 May 2023 16:15:17 +0000 Received: from mail-ej1-x62a.google.com ([2a00:1450:4864:20::62a]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1pyZmj-002VnQ-2z for linux-arm-kernel@lists.infradead.org; Mon, 15 May 2023 15:06:51 +0000 Received: by mail-ej1-x62a.google.com with SMTP id a640c23a62f3a-9659e9bbff5so2386491166b.1 for ; Mon, 15 May 2023 08:06:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1684163207; x=1686755207; h=user-agent:in-reply-to:content-disposition:mime-version:references :message-id:subject:cc:to:from:date:from:to:cc:subject:date :message-id:reply-to; bh=ClsAc+MQWEbajrlHlPJBJiVswUOmZH90ORl3dfNFYBQ=; b=O8zV0zQwfKIr5rH7o5lHONZT52pDSOBEsASAlgUsXYOwKSUCuOPOTKazqC7mndtM4K 86ZF26CFTysSxAJJnmuNKuUwQTDHoCR//VRc0lW3M69IUV8HLyshViBuA9Ksjae7psSz 34cDjjhRvuA2cqxuTR3Rc1fJMgwV3t6vw/8Mmdenf7kMCRt+122BkjExd2/OVO0rtv+k N5s3aORmbyjfQ7WvrMF/V4HAU9Gy+Xt4SlBzUH9HceVYxDer8GMlisKRcx8jIEV5e9Sf dDnP0uz4Un4SsiLmXBCD1u6cbblcczkIgfkAqfqTvHC+zeLvlAz6H1n83QFIGYkKyqEX 7hTA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1684163207; x=1686755207; h=user-agent:in-reply-to:content-disposition:mime-version:references :message-id:subject:cc:to:from:date:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=ClsAc+MQWEbajrlHlPJBJiVswUOmZH90ORl3dfNFYBQ=; b=b3pdnwcRj/4I+M8dxmuFHOS7WVmvC4djmUnZ3V5ZJL8E0d+PTyWMIDOkMw+3213dw4 R2MttDWIoQ7e/0WJzrNEMfn8ZZROI92NsvKOQg18aJEHk1SI2oGyMKO2PZ2MlKM1CF0u Dq6BHQ7k3ikEOv65X30yF+GWcE/KLn8dwmLE5efN5iDokZXBIbGLo+D79KvFXxEbdaQR sJHp8zxP8Qb3uDCrnXlnIbeNAyixjW08JKzpvyKkMSgeIN/k41U9+7YAQGhhvsV2NxB0 qjM6O8D41c0EbYovf6UCWecVtbbFg9lqhQjWQkFFWirdTzLIvmDqCkcT+jkw7Jtl2Iz2 HLjg== X-Gm-Message-State: AC+VfDy8r1gaTlqI5Bz3klj+qePDviVuBmK5bGO9f0ZWJMV+usgFb2WI jP/2fJpCX0Mhb4BH4OaMsos= X-Google-Smtp-Source: ACHHUZ7L/BMseVSQpyf69tujmTDgJlIjLXeqvkKh5mCCAWwCg9c7cxmQWtCE5GyuRsGBWMPUWkZ+Lg== X-Received: by 2002:a17:906:7312:b0:961:800b:3f5e with SMTP id di18-20020a170906731200b00961800b3f5emr31396103ejc.72.1684163207327; Mon, 15 May 2023 08:06:47 -0700 (PDT) Received: from orome (p200300e41f305300f22f74fffe1f3a53.dip0.t-ipconnect.de. [2003:e4:1f30:5300:f22f:74ff:fe1f:3a53]) by smtp.gmail.com with ESMTPSA id lv4-20020a170906bc8400b00966293c06e9sm9499977ejb.126.2023.05.15.08.06.46 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 15 May 2023 08:06:46 -0700 (PDT) Date: Mon, 15 May 2023 17:06:44 +0200 From: Thierry Reding To: Krzysztof Kozlowski Cc: Rob Herring , Krzysztof Kozlowski , Jon Hunter , devicetree@vger.kernel.org, linux-tegra@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 1/4] dt-bindings: Document additional Jetson Orin NX SKUs Message-ID: References: <20230331163159.17145-1-thierry.reding@gmail.com> <3a8724bc-fb44-0080-fd24-c04e3841385e@linaro.org> <4129f3d5-1df8-a767-6783-1d7123b2e9d4@linaro.org> MIME-Version: 1.0 In-Reply-To: <4129f3d5-1df8-a767-6783-1d7123b2e9d4@linaro.org> User-Agent: Mutt/2.2.10 (2023-03-25) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230515_080649_966014_CDA62846 X-CRM114-Status: GOOD ( 27.53 ) 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: multipart/mixed; boundary="===============5586354913875033021==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============5586354913875033021== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="UzM2R6eGT/Bqb/nB" Content-Disposition: inline --UzM2R6eGT/Bqb/nB Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Apr 07, 2023 at 10:52:58AM +0200, Krzysztof Kozlowski wrote: > On 04/04/2023 12:49, Thierry Reding wrote: > > On Fri, Mar 31, 2023 at 10:19:00PM +0200, Krzysztof Kozlowski wrote: > >> On 31/03/2023 18:31, Thierry Reding wrote: > >>> From: Thierry Reding > >>> > >>> Beyond the original 16 GiB SKU (0), additional SKUs exist, such as th= e 8 > >>> GiB SKU (1) and an internal-only SKU (2) that comes with an equipeed = SD > >> > >> typo: equipped > >> > >>> card slot. > >> > >> Is there a point in documenting all of them if there is no DTS? Also, > >> size of storage (eMMC?) pretty often is runtime-detectable, so you do = no > >> need a new DTS and new compatible. > >=20 > > This is for the sake of completeness since these compatible strings > > correspond to the part numbers that will show up on stickers on these > > modules. In practice, yes, most of the differences will be runtime- > > detected and the DT updated to reflect the SKU differences by UEFI. >=20 > Just because there is some sticker, it does not mean we need a > compatible. We actually omit dozen of versions per device - all PMICs, > I2C IIO and others have some packaging bins and revision numbers. >=20 > Although here if I understand correctly, UEFI firmware will add these > compatibles? That's the idea. UEFI does some probing of the hardware and currently writes information about the detected SKUs into the /chosen node. I think we could achieve the same effect in a more standard way by writing out the compatible strings instead. So while we likely won't have these compatible strings in the DTS files, we may very well end up having these in the DTB that is passed to the kernel. So we don't need these to be documented for validation within the kernel repository, but I'm concerned it could lead to confusion if people end up with undocumented compatible strings in the DTB. Perhaps that's not as big a deal as I think it is, so I'll drop this for now. We'll go with the "standard" SKU compatible strings for now and can revisit if this becomes an actual problem. Sorry for the delay, I hadn't seen your replies before. Thierry --UzM2R6eGT/Bqb/nB Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAmRiSoIACgkQ3SOs138+ s6El6w//Xg/pAHxUCf+hHGn4aycKajfi76NRWhI3+Lm99cGQ1/aX0/Ke8VT7Tdwn JpxSFF8k0wlUI83EmJF/4iIW7YWW4L/b5R1iti3emm53BdWCvoJk4ODydTpOA+sV YuX+PhPFKmXUUhz7Wu2wsLoGvcDvRampPA8NWWw4nCxGI3T8F/nea7CLIT+rWMiI gFjXRGk8oNdnFSkGRXnZN8mmfZj2Jvpx6yqEkuYL9st3YlCaYJd720F7GeloTvsY mkltj2tma7ly7ltYJ3FsguGPhU8PtuZ/J9+ZTsSScuopctDiHdK4egKJpEJqe5Kg DSIEu3HqC00MX0Bl9ZumrQikKrJ6WcCpuF4mXQV+iwukK7iNsvpXTgf9bw6o1Gl9 j1oHNeVSH+bqX87GVdIKtTgZpRXYvH+cFsmv/4wvmdwYuOP2ccIm46QseokgyZUz i6EPtUDGWfAPIJ+k3uZvMhdur/qD6t2LfAATI925BS58QcyqkYx6cSF2wpyFkY1c zppAp2kp0GOo8t+EVj4PoC7S+aYgmcVDO1PkTjAt+onok7i6t3/Zb67+dRl0iQAj 2qQLXEpU8D+leZT3Vcg96hmi1QS+Rc/8iPY7nEOEggpzCEhwZblEMAE/2qWjCSJj 4bw3i1j6/h0EpzdekNy1w0MDHK4x/KaDJOT25mEO5vMqdqCWJEQ= =yUiS -----END PGP SIGNATURE----- --UzM2R6eGT/Bqb/nB-- --===============5586354913875033021== 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 --===============5586354913875033021==--