Re: [PATCH 1/2] ARM: Tegra: dt: Split out separate Tegra SoC DT

On Fri, Apr 29, 2011 at 10:12:30PM -0600, Stephen Warren wrote:

tegra250.dts describes everything within the Tegra SoC, but leaves all
optional devices disabled.

tegra-harmony.dts is now the configuration of the SoC for the board,
i.e. it enables the optional devices that are required for the board,
and provides any required platform data for those devics.

The prevents every Tegra board having to repeat all the basic internal
SoC wiring.

Signed-off-by: Stephen Warren <redacted>

Acked-by: Olof Johansson <redacted>

A mostly unrelated question below.

[...]

+	serial@70006000 {
+		compatible = "nvidia,tegra250-uart";

I know this is how Grant specified it, but shouldn't these also have
a compat for ns16550?

+		reg = <0x70006000 0x40>;
+		reg-shift = <2>;
+		interrupts = < 68 >;
+		status = "disabled";
+	};

-Olof