All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thierry Reding <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Jon Hunter <jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [RESEND PATCH] arm64: tegra: Add support for Jetson Xavier NX
Date: Tue, 14 Jul 2020 11:17:20 +0200	[thread overview]
Message-ID: <20200714091720.GC141356@ulmo> (raw)
In-Reply-To: <20200713161701.30449-1-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

[-- Attachment #1: Type: text/plain, Size: 2339 bytes --]

On Mon, Jul 13, 2020 at 05:17:01PM +0100, Jon Hunter wrote:
> Add the device-tree source files for the Tegra194 Jetson Xavier NX
> Developer Kit. The Xavier NX Developer Kit consists of a small form
> factor system-on-module (SOM) board (part number p3668-0000) and a
> carrier board (part number p3509-0000).
> 
> The Xavier NX Developer Kit SOM features a micro-SD card slot,
> however, there is also a variant of the SOM available that features a
> 16GB eMMC. Given that the carrier board can be used with the different
> SOM variants, that have different part numbers, both the compatible
> string and file name of the device-tree source file for the Developer
> Kit is a concatenation of the SOM and carrier board part numbers.
> 
> This is based upon some initial work by Thierry Reding
> <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>.
> 
> Signed-off-by: Jon Hunter <jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> ---
> 
> I have updated the author and given credit to Thierry for doing the
> initial work on the change.
> 
>  arch/arm64/boot/dts/nvidia/Makefile           |   1 +
>  .../nvidia/tegra194-p3509-0000+p3668-0000.dts | 343 ++++++++++++++++++
>  .../boot/dts/nvidia/tegra194-p3668-0000.dtsi  | 296 +++++++++++++++
>  3 files changed, 640 insertions(+)
>  create mode 100644 arch/arm64/boot/dts/nvidia/tegra194-p3509-0000+p3668-0000.dts
>  create mode 100644 arch/arm64/boot/dts/nvidia/tegra194-p3668-0000.dtsi

I need this patch to avoid an error being flagged by checkpatch:

--- >8 ---
diff --git a/Documentation/devicetree/bindings/arm/tegra.yaml b/Documentation/devicetree/bindings/arm/tegra.yaml
index 60b38eb5c61a..53845db64df1 100644
--- a/Documentation/devicetree/bindings/arm/tegra.yaml
+++ b/Documentation/devicetree/bindings/arm/tegra.yaml
@@ -101,3 +101,11 @@ properties:
           - enum:
               - nvidia,p2972-0000
           - const: nvidia,tegra194
+      - description: Jetson Xavier NX
+        items:
+          - const: nvidia,p3668-0000
+          - const: nvidia,tegra194
+      - description: Jetson Xavier NX Developer Kit
+        items:
+          - const: nvidia,p3509-0000+p3668-0000
+          - const: nvidia,tegra194
--- >8---

I'll submit that separately and have applied this one in the meantime.

Thanks,
Thierry

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

WARNING: multiple messages have this Message-ID (diff)
From: Thierry Reding <thierry.reding@gmail.com>
To: Jon Hunter <jonathanh@nvidia.com>
Cc: linux-tegra@vger.kernel.org, devicetree@vger.kernel.org,
	Rob Herring <robh+dt@kernel.org>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [RESEND PATCH] arm64: tegra: Add support for Jetson Xavier NX
Date: Tue, 14 Jul 2020 11:17:20 +0200	[thread overview]
Message-ID: <20200714091720.GC141356@ulmo> (raw)
In-Reply-To: <20200713161701.30449-1-jonathanh@nvidia.com>


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

On Mon, Jul 13, 2020 at 05:17:01PM +0100, Jon Hunter wrote:
> Add the device-tree source files for the Tegra194 Jetson Xavier NX
> Developer Kit. The Xavier NX Developer Kit consists of a small form
> factor system-on-module (SOM) board (part number p3668-0000) and a
> carrier board (part number p3509-0000).
> 
> The Xavier NX Developer Kit SOM features a micro-SD card slot,
> however, there is also a variant of the SOM available that features a
> 16GB eMMC. Given that the carrier board can be used with the different
> SOM variants, that have different part numbers, both the compatible
> string and file name of the device-tree source file for the Developer
> Kit is a concatenation of the SOM and carrier board part numbers.
> 
> This is based upon some initial work by Thierry Reding
> <treding@nvidia.com>.
> 
> Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
> ---
> 
> I have updated the author and given credit to Thierry for doing the
> initial work on the change.
> 
>  arch/arm64/boot/dts/nvidia/Makefile           |   1 +
>  .../nvidia/tegra194-p3509-0000+p3668-0000.dts | 343 ++++++++++++++++++
>  .../boot/dts/nvidia/tegra194-p3668-0000.dtsi  | 296 +++++++++++++++
>  3 files changed, 640 insertions(+)
>  create mode 100644 arch/arm64/boot/dts/nvidia/tegra194-p3509-0000+p3668-0000.dts
>  create mode 100644 arch/arm64/boot/dts/nvidia/tegra194-p3668-0000.dtsi

I need this patch to avoid an error being flagged by checkpatch:

--- >8 ---
diff --git a/Documentation/devicetree/bindings/arm/tegra.yaml b/Documentation/devicetree/bindings/arm/tegra.yaml
index 60b38eb5c61a..53845db64df1 100644
--- a/Documentation/devicetree/bindings/arm/tegra.yaml
+++ b/Documentation/devicetree/bindings/arm/tegra.yaml
@@ -101,3 +101,11 @@ properties:
           - enum:
               - nvidia,p2972-0000
           - const: nvidia,tegra194
+      - description: Jetson Xavier NX
+        items:
+          - const: nvidia,p3668-0000
+          - const: nvidia,tegra194
+      - description: Jetson Xavier NX Developer Kit
+        items:
+          - const: nvidia,p3509-0000+p3668-0000
+          - const: nvidia,tegra194
--- >8---

I'll submit that separately and have applied this one in the meantime.

Thanks,
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

WARNING: multiple messages have this Message-ID (diff)
From: Thierry Reding <thierry.reding@gmail.com>
To: Jon Hunter <jonathanh@nvidia.com>
Cc: linux-arm-kernel@lists.infradead.org,
	linux-tegra@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
	devicetree@vger.kernel.org
Subject: Re: [RESEND PATCH] arm64: tegra: Add support for Jetson Xavier NX
Date: Tue, 14 Jul 2020 11:17:20 +0200	[thread overview]
Message-ID: <20200714091720.GC141356@ulmo> (raw)
In-Reply-To: <20200713161701.30449-1-jonathanh@nvidia.com>

[-- Attachment #1: Type: text/plain, Size: 2281 bytes --]

On Mon, Jul 13, 2020 at 05:17:01PM +0100, Jon Hunter wrote:
> Add the device-tree source files for the Tegra194 Jetson Xavier NX
> Developer Kit. The Xavier NX Developer Kit consists of a small form
> factor system-on-module (SOM) board (part number p3668-0000) and a
> carrier board (part number p3509-0000).
> 
> The Xavier NX Developer Kit SOM features a micro-SD card slot,
> however, there is also a variant of the SOM available that features a
> 16GB eMMC. Given that the carrier board can be used with the different
> SOM variants, that have different part numbers, both the compatible
> string and file name of the device-tree source file for the Developer
> Kit is a concatenation of the SOM and carrier board part numbers.
> 
> This is based upon some initial work by Thierry Reding
> <treding@nvidia.com>.
> 
> Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
> ---
> 
> I have updated the author and given credit to Thierry for doing the
> initial work on the change.
> 
>  arch/arm64/boot/dts/nvidia/Makefile           |   1 +
>  .../nvidia/tegra194-p3509-0000+p3668-0000.dts | 343 ++++++++++++++++++
>  .../boot/dts/nvidia/tegra194-p3668-0000.dtsi  | 296 +++++++++++++++
>  3 files changed, 640 insertions(+)
>  create mode 100644 arch/arm64/boot/dts/nvidia/tegra194-p3509-0000+p3668-0000.dts
>  create mode 100644 arch/arm64/boot/dts/nvidia/tegra194-p3668-0000.dtsi

I need this patch to avoid an error being flagged by checkpatch:

--- >8 ---
diff --git a/Documentation/devicetree/bindings/arm/tegra.yaml b/Documentation/devicetree/bindings/arm/tegra.yaml
index 60b38eb5c61a..53845db64df1 100644
--- a/Documentation/devicetree/bindings/arm/tegra.yaml
+++ b/Documentation/devicetree/bindings/arm/tegra.yaml
@@ -101,3 +101,11 @@ properties:
           - enum:
               - nvidia,p2972-0000
           - const: nvidia,tegra194
+      - description: Jetson Xavier NX
+        items:
+          - const: nvidia,p3668-0000
+          - const: nvidia,tegra194
+      - description: Jetson Xavier NX Developer Kit
+        items:
+          - const: nvidia,p3509-0000+p3668-0000
+          - const: nvidia,tegra194
--- >8---

I'll submit that separately and have applied this one in the meantime.

Thanks,
Thierry

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

  parent reply	other threads:[~2020-07-14  9:17 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-12 10:25 [PATCH 0/3] arm64: tegra: A few DT updates for Tegra Jon Hunter
2020-07-12 10:25 ` Jon Hunter
     [not found] ` <20200712102506.23686-1-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2020-07-12 10:25   ` [PATCH 1/3] arm64: tegra: Add support for Jetson Xavier NX Jon Hunter
2020-07-12 10:25     ` Jon Hunter
     [not found]     ` <20200712102506.23686-2-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2020-07-13 16:17       ` [RESEND PATCH] " Jon Hunter
2020-07-13 16:17         ` Jon Hunter
2020-07-13 16:17         ` Jon Hunter
     [not found]         ` <20200713161701.30449-1-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2020-07-14  9:17           ` Thierry Reding [this message]
2020-07-14  9:17             ` Thierry Reding
2020-07-14  9:17             ` Thierry Reding
2020-07-12 10:25   ` [PATCH 2/3] arm64: tegra: Enable DFLL support on Jetson Nano Jon Hunter
2020-07-12 10:25     ` Jon Hunter
     [not found]     ` <20200712102506.23686-3-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2020-07-14  9:18       ` Thierry Reding
2020-07-14  9:18         ` Thierry Reding
2020-07-12 10:25   ` [PATCH 3/3] arm64: tegra: Populate VBUS for USB3 on Jetson TX2 Jon Hunter
2020-07-12 10:25     ` Jon Hunter
     [not found]     ` <20200712102506.23686-4-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2020-07-14  9:19       ` Thierry Reding
2020-07-14  9:19         ` Thierry Reding

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200714091720.GC141356@ulmo \
    --to=thierry.reding-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.