Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755589AbbDKWdU (ORCPT ); Sat, 11 Apr 2015 18:33:20 -0400 Received: from forward4l.mail.yandex.net ([84.201.143.137]:51046 "EHLO forward4l.mail.yandex.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753013AbbDKWdQ (ORCPT ); Sat, 11 Apr 2015 18:33:16 -0400 To: Andrew Lunn , Jason Cooper , Gregory Clement , Sebastian Hesselbarth Subject: Re: [PATCH 2/2] ARM: mvebu: dts: Add dts file for DLink DNS-327L X-PHP-Originating-Script: 501:rcube.php MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Date: Sun, 12 Apr 2015 01:33:04 +0300 From: Andrew Cc: Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , Linus Walleij , Wolfram Sang , devicetree@vger.kernel.org, linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org In-Reply-To: <20150411214005.GJ5244@lunn.ch> References: <1428784160-19431-1-git-send-email-andrew@ncrmnt.org> <1428784160-19431-3-git-send-email-andrew@ncrmnt.org> <20150411214005.GJ5244@lunn.ch> Message-ID: <926c05317f677c9e2562e1fe07b4a550@mail.ncrmnt.org> User-Agent: Roundcube Webmail/1.0.2 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3981 Lines: 144 Andrew Lunn писал 12.04.2015 00:40: > On Sat, Apr 11, 2015 at 11:29:20PM +0300, Andrew Andrianov wrote: >> Signed-off-by: Andrew Andrianov >> --- >> arch/arm/boot/dts/Makefile | 1 + >> arch/arm/boot/dts/armada-370-dlink-dns327l.dts | 309 >> ++++++++++++++++++++++++ >> 2 files changed, 310 insertions(+) >> create mode 100644 arch/arm/boot/dts/armada-370-dlink-dns327l.dts >> >> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile >> index a1c776b..8535e4e 100644 >> --- a/arch/arm/boot/dts/Makefile >> +++ b/arch/arm/boot/dts/Makefile >> @@ -612,6 +612,7 @@ dtb-$(CONFIG_ARCH_ZYNQ) += \ >> zynq-zybo.dtb >> dtb-$(CONFIG_MACH_ARMADA_370) += \ >> armada-370-db.dtb \ >> + armada-370-dlink-dns327l.dtb \ >> armada-370-mirabox.dtb \ >> armada-370-netgear-rn102.dtb \ >> armada-370-netgear-rn104.dtb \ >> diff --git a/arch/arm/boot/dts/armada-370-dlink-dns327l.dts >> b/arch/arm/boot/dts/armada-370-dlink-dns327l.dts >> new file mode 100644 >> index 0000000..12bc072 >> --- /dev/null >> +++ b/arch/arm/boot/dts/armada-370-dlink-dns327l.dts >> @@ -0,0 +1,309 @@ >> +/* >> + * Device Tree file for DLINK DNS-327L >> + * >> + * Copyright (C) 2014, Andrew Andrianov >> + * >> + * This program is free software; you can redistribute it and/or >> + * modify it under the terms of the GNU General Public License >> + * as published by the Free Software Foundation; either version >> + * 2 of the License, or (at your option) any later version. >> + */ > > Hi Andrew > > There has been a move to license all the 370 and XP DTS files under > dual GPL and X11. Are you willing to add X11? No problem, will fix. > >> + >> +/* Remaining mysteries: >> + * >> + * There's still something unknown on i2c address 0x13 >> + * CONFIG_ARM_MVEBU_V7_CPUIDLE=y causes hard freezes every 1-8 hours >> + * >> + */ >> + >> +/dts-v1/; >> + >> +#include >> +#include >> +#include "armada-370.dtsi" >> + >> +/ { >> + model = "DLINK DNS-327L"; >> + compatible = "dlink,dns327l", >> + "marvell,armada370", >> + "marvell,armada-370-xp"; >> + >> + chosen { >> + bootargs = "console=ttyS0,115200 earlyprintk"; > > It would be nice to have > > stdout-path = &uart0; > > here. That will help with boot loaders, using this DT file, and it a > step towards removing the console=ttyS0,115200. > >> + }; >> + >> + memory { >> + device_type = "memory"; >> + reg = <0x00000000 0x20000000>; /* 512 MiB */ >> + }; >> + >> + soc { >> + ranges = > + MBUS_ID(0x01, 0xe0) 0 0xfff00000 0x100000>; >> + >> + pcie-controller { >> + status = "okay"; >> + >> + /* Connected to Marvell SATA controller */ >> + pcie@1,0 { >> + /* Port 0, Lane 0 */ >> + status = "okay"; >> + }; >> + >> + /* Connected to NEC USB 3.0 controller */ >> + pcie@2,0 { >> + /* Port 1, Lane 0 */ >> + status = "okay"; >> + }; >> + }; >> + >> + internal-regs { >> + serial@12000 { >> + status = "okay"; >> + }; >> + >> + serial@12100 { >> + status = "okay"; >> + }; > > Are both serial ports usable? > Could you document the pinout for the console? > > Thanks > Andrew The serial console pinout is : [RXD|(gap)|3.3v|GND|TXD] I've been documenting the hardware extensively in my blog here: http://ncrmnt.org/wp/2015/02/15/debian-dlink-dns-327l-the-guide/ Pictures of the PCB are all there. The second UART is used to interface with weltrend mcu only. I'll resubmit the patches with fixes sometime tomorrow evening (GMT+4). Hopefully I'll also find out is hardware freezes with cpuidle enabled are still there on 4.0-rc7. -- Regards, Andrew -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/