diff mbox

[v3,10/12] arm64: dts: tegra: Add NVIDIA P2771 board support

Message ID 20160819173233.13260-11-thierry.reding@gmail.com
State Deferred
Headers show

Commit Message

Thierry Reding Aug. 19, 2016, 5:32 p.m. UTC
From: Joseph Lo <josephl@nvidia.com>

Add NVIDIA P2771 is composed of a P3310 processor module that connects
to the P2597 I/O board. It comes with a 1200x1920 MIPI DSI panel
connected via the P2597's display connector and has several connectors
such as HDMI, USB 3.0, PCIe and ethernet.

Signed-off-by: Joseph Lo <josephl@nvidia.com>
Signed-off-by: Thierry Reding <treding@nvidia.com>
---
 arch/arm64/boot/dts/nvidia/Makefile                | 1 +
 arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts | 8 ++++++++
 2 files changed, 9 insertions(+)
 create mode 100644 arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts

Comments

Stephen Warren Aug. 22, 2016, 7:11 p.m. UTC | #1
On 08/19/2016 11:32 AM, Thierry Reding wrote:
> From: Joseph Lo <josephl@nvidia.com>
>
> Add NVIDIA P2771 is composed of a P3310 processor module that connects
> to the P2597 I/O board. It comes with a 1200x1920 MIPI DSI panel
> connected via the P2597's display connector and has several connectors
> such as HDMI, USB 3.0, PCIe and ethernet.

>  arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts | 8 ++++++++

The DT will be different for the different HW versions. So far, there 
are at least 3 different versions with 3 combinations of 2 different HW 
changes (PCIe lane layout, and SD card GPIO assignments). I suspect 
we'll want this file to be tegra186-p2771-0000-NNN.dts where NNN is the 
version number from the EEPROM (which is 000 for the first version, 300 
for the version that introduces the first change, and 500 for the 
version that introduces the second change).

> diff --git a/arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts b/arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts

> +#include "tegra186-p3310.dtsi"
> +
> +/ {
> +	model = "NVIDIA Tegra186 P2771-0000 Board";
> +	compatible = "nvidia,p2771-0000", "nvidia,tegra186";

Do we need to propagate the "nvidia,p3310" entry from the included CVM 
DT? It'd be nice if we could write:

/prepend/ compatible = "nvidia,p2771-0000";

... and have that happen automatically.
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm64/boot/dts/nvidia/Makefile b/arch/arm64/boot/dts/nvidia/Makefile
index 0f7cdf3e05c1..67234f3dc795 100644
--- a/arch/arm64/boot/dts/nvidia/Makefile
+++ b/arch/arm64/boot/dts/nvidia/Makefile
@@ -1,4 +1,5 @@ 
 dtb-$(CONFIG_ARCH_TEGRA_132_SOC) += tegra132-norrin.dtb
+dtb-$(CONFIG_ARCH_TEGRA_186_SOC) += tegra186-p2771-0000.dtb
 dtb-$(CONFIG_ARCH_TEGRA_210_SOC) += tegra210-p2371-0000.dtb
 dtb-$(CONFIG_ARCH_TEGRA_210_SOC) += tegra210-p2371-2180.dtb
 dtb-$(CONFIG_ARCH_TEGRA_210_SOC) += tegra210-p2571.dtb
diff --git a/arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts b/arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts
new file mode 100644
index 000000000000..66b936389fa7
--- /dev/null
+++ b/arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts
@@ -0,0 +1,8 @@ 
+/dts-v1/;
+
+#include "tegra186-p3310.dtsi"
+
+/ {
+	model = "NVIDIA Tegra186 P2771-0000 Board";
+	compatible = "nvidia,p2771-0000", "nvidia,tegra186";
+};