On 08/19/2016 11:32 AM, Thierry Reding wrote:

From: Joseph Lo <redacted>

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).

+#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.