From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jon Hunter Subject: Re: [PATCH v3 09/12] arm64: dts: tegra: Add NVIDIA P3310 main board support Date: Tue, 23 Aug 2016 18:35:30 +0100 Message-ID: <9c123f91-d7ae-7cf1-f669-412aa350a0ca@nvidia.com> References: <20160819173233.13260-1-thierry.reding@gmail.com> <20160819173233.13260-10-thierry.reding@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20160819173233.13260-10-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Thierry Reding Cc: Timo Alho , Peter De Schrijver , Sivaram Nair , Joseph Lo , linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org On 19/08/16 18:32, Thierry Reding wrote: > From: Joseph Lo > > The NVIDIA P3310 is a processor module used in several reference designs > that features a Tegra186 SoC, 8 GiB of LPDDR4 RAM, 32 GiB eMMC and other > essentials such as ethernet, WiFi and a PMIC. It is typically connected > to an I/O board (such as the P2597) that provides the connecters needed > to hook it up to the outside world. > > Signed-off-by: Joseph Lo > Signed-off-by: Thierry Reding > --- > Changes in v3: > - fix compatible string > > arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi | 34 ++++++++++++++++++++++++++ > 1 file changed, 34 insertions(+) > create mode 100644 arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi > > diff --git a/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi b/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi > new file mode 100644 > index 000000000000..dc07ea618378 > --- /dev/null > +++ b/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi > @@ -0,0 +1,34 @@ > +#include "tegra186.dtsi" > + > +/ { > + model = "NVIDIA Tegra186 P3310 main Board"; > + compatible = "nvidia,p3310", "nvidia,tegra186"; > + > + aliases { > + serial0 = &uarta; > + }; > + > + chosen { > + bootargs = "earlycon console=ttyS0,115200n8"; I don't think we need 'console' in the bootargs anymore because stdout-path is enough (even for early console). Cheers Jon -- nvpublic