Re: [PATCH 2/4] dt-bindings: tegra: Document Jetson Orin Nano

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Mar 31, 2023 at 10:20:16PM +0200, Krzysztof Kozlowski wrote:
> On 31/03/2023 18:31, Thierry Reding wrote:
> > From: Thierry Reding <treding@xxxxxxxxxx>
> > 
> > The Jetson Orin Nano is the little sibling of the Jetson Orin NX.
> > Document the corresponding compatible strings for these devices.
> > 
> > Signed-off-by: Thierry Reding <treding@xxxxxxxxxx>
> > ---
> >  Documentation/devicetree/bindings/arm/tegra.yaml | 7 +++++++
> >  1 file changed, 7 insertions(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/arm/tegra.yaml b/Documentation/devicetree/bindings/arm/tegra.yaml
> > index 61e638c9cad7..60c151da5e06 100644
> > --- a/Documentation/devicetree/bindings/arm/tegra.yaml
> > +++ b/Documentation/devicetree/bindings/arm/tegra.yaml
> > @@ -220,6 +220,13 @@ properties:
> >                - nvidia,p3767-0001
> >                - nvidia,p3767-0002
> >            - const: nvidia,tegra234
> > +      - description: Jetson Orin Nano
> > +        items:
> > +          - enum:
> > +              - nvidia,p3767-0003
> > +              - nvidia,p3767-0004
> > +              - nvidia,p3767-0005
> 
> Similar questions as for patch #1. Where are the DTSes? Where are the
> differences? If we keep documenting every SKU which is the same from
> user/OS perspective, this list would grow crazy.

As for the concern of this list growing crazy: as far as I know it's
complete after this. I'm not aware of any other SKUs being planned for
the future. Obviously that could always change, but again, I think there
is some advantage in being as complete as possible when documenting
these, otherwise someone will eventually come across a SKU that's not
documented and be easily confused.

Thierry

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux