Re: [PATCH 1/4] dt-bindings: Document additional Jetson Orin NX SKUs

[Thierry Reding <thierry.reding@gmail.com>]

[-- Attachment #1.1: Type: text/plain, Size: 1447 bytes --]

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 <treding@nvidia.com>
> > 
> > Beyond the original 16 GiB SKU (0), additional SKUs exist, such as the 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.

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.

As far as I know, UEFI doesn't actually do anything with the compatible
strings themselves, but that's potentially something that could happen
at some point. The SKU numbers also show up in EEPROMs, so I think
having one place where these are documented might be helpful to people.

The 16 GiB in this case is actually DRAM, but it's also detected at
runtime. We don't actually plan on upstreaming DTS files for all of
these, since we don't expect all SKUs to be widely used (the internal
one, for example) so we should be able to cover pretty much all variants
with just two DTS files.

Thierry

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[-- Attachment #2: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel