From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933326AbcCNGmz (ORCPT ); Mon, 14 Mar 2016 02:42:55 -0400 Received: from mailout4.w1.samsung.com ([210.118.77.14]:58078 "EHLO mailout4.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750925AbcCNGmq (ORCPT ); Mon, 14 Mar 2016 02:42:46 -0400 X-AuditID: cbfec7f5-f79b16d000005389-28-56e65d63a7f8 Subject: Re: [PATCH 05/10] ARM: dts: Add exynos3250-artik5 dtsi file for ARTIK5 module To: Chanwoo Choi , kgene@kernel.org, s.nawrocki@samsung.com, tomasz.figa@gmail.com References: <1457921056-21257-1-git-send-email-cw00.choi@samsung.com> <1457921056-21257-6-git-send-email-cw00.choi@samsung.com> Cc: jh80.chung@samsung.com, andi.shyti@samsung.com, inki.dae@samsung.com, sw0312.kim@samsung.com, pankaj.dubey@samsung.com, linux-samsung-soc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org From: Krzysztof Kozlowski X-Enigmail-Draft-Status: N1110 Message-id: <56E65D5D.8060702@samsung.com> Date: Mon, 14 Mar 2016 15:42:37 +0900 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-version: 1.0 In-reply-to: <1457921056-21257-6-git-send-email-cw00.choi@samsung.com> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFupkkeLIzCtJLcpLzFFi42I5/e/4Zd3k2GdhBkvncllsP/KM1eL6l+es FvOPnGO1mHR/AovFjV9trBavXxha9D9+zWyx6fE1VovLu+awWcw4v4/JYtHWL+wWh9+0s1rM mPySzWLVrj+MDnweO2fdZffYtKqTzWPzknqPvi2rGD0+b5ILYI3isklJzcksSy3St0vgyth1 6BNLwVvVijOzNzE1MJ6Q62Lk5JAQMJE4cXcNM4QtJnHh3nq2LkYuDiGBpYwSM05OY4RwnjJK TDn3mxWkSlggQmLO+bWMILaIQLbEjCUzoToaGSVa960Fc5gFmpkk7q5ZxQRSxSZgLLF5+RI2 iB1yEr3dk1hAbF4BLYmPrx6C2SwCqhI7Dk0GmyoKtOFwZxc7RI2gxI/J98BqOAXcJA6/bwC6 lQNogZ7E/YtaIGFmAXmJzWveMk9gFJyFpGMWQtUsJFULGJlXMYqmliYXFCel5xrpFSfmFpfm pesl5+duYoRE0NcdjEuPWR1iFOBgVOLh/bHmaZgQa2JZcWXuIUYJDmYlEV5fx2dhQrwpiZVV qUX58UWlOanFhxilOViUxHln7nofIiSQnliSmp2aWpBaBJNl4uCUamA0337By9ZZ9enXBW/8 rvAwr37qk3L4pcDshDtcj8UPP+OqmMMXpMf+VM/wtvz0bFvmqUfUOndcvGvC9Lbvwdao5Nr3 6qaT7SUNHLW/nS8/Ov+5203O5sqPZtkyklYOtT+YWM8IL/qScbhPLqzisOCHc4JOdWsXq694 kLnwWXiP2Jz8E05MrlOUWIozEg21mIuKEwFhsxsInAIAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 14.03.2016 11:04, Chanwoo Choi wrote: > This patch adds the support for Device Tree source for Samsung ARTIK5 module[1] > based on Exynos3250 SoC. The ARTIK5 module includes the follwoing devices: > - Application Processor (Samsung Exynos3250) > - WiFi/BT Combo chip (Broadcom4354) > - PMIC (Samsung S2MPS14) > - eMMC (4GB) > - DRAM LPDDR3 (512MB) > - Connectors pin (60 Pins x 3 set) > > Also, this patch adds the ARTIK5 development board[2] dts file which includes > the ARTIK5 module[1] and have the devices such as sound codec, sd card port, > ethernet port, uart port and so on. > > [1] https://www.artik.io/hardware/artik-5 > [2] http://www.digikey.com/product-search/en?FV=ffecca14 > > Signed-off-by: Chanwoo Choi > Signed-off-by: Andi Shyti > --- > arch/arm/boot/dts/Makefile | 1 + > arch/arm/boot/dts/exynos3250-artik5-devel.dts | 26 +++ > arch/arm/boot/dts/exynos3250-artik5.dtsi | 282 ++++++++++++++++++++++++++ Plus changes in Documentation/devicetree/bindings/arm/samsung/samsung-boards.txt. > 3 files changed, 309 insertions(+) > create mode 100644 arch/arm/boot/dts/exynos3250-artik5-devel.dts > create mode 100644 arch/arm/boot/dts/exynos3250-artik5.dtsi > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > index a4a6d70e8b26..f2de160828e8 100644 > --- a/arch/arm/boot/dts/Makefile > +++ b/arch/arm/boot/dts/Makefile > @@ -108,6 +108,7 @@ dtb-$(CONFIG_ARCH_DIGICOLOR) += \ > dtb-$(CONFIG_ARCH_EFM32) += \ > efm32gg-dk3750.dtb > dtb-$(CONFIG_ARCH_EXYNOS3) += \ > + exynos3250-artik5-devel.dtb \ > exynos3250-monk.dtb \ > exynos3250-rinato.dtb > dtb-$(CONFIG_ARCH_EXYNOS4) += \ > diff --git a/arch/arm/boot/dts/exynos3250-artik5-devel.dts b/arch/arm/boot/dts/exynos3250-artik5-devel.dts > new file mode 100644 > index 000000000000..d17c28969118 > --- /dev/null > +++ b/arch/arm/boot/dts/exynos3250-artik5-devel.dts > @@ -0,0 +1,26 @@ > +/* > + * Samsung's Exynos3250 based ARTIK5 development board device tree source > + * > + * Copyright (c) 2016 Samsung Electronics Co., Ltd. > + * http://www.samsung.com > + * > + * Device tree source file for Samsung's ARTIK5 development board > + * which is based on Samsung Exynos3250 SoC. > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + */ > + > +/dts-v1/; > +#include "exynos3250-artik5.dtsi" > + > +/ { > + model = "Samsung ARTIK5 development board"; > + compatible = "samsung,artik5-devel", "samsung,artik5", > + "samsung,exynos3250", "samsung,exynos3"; > +}; > + > +&serial_2 { > + status = "okay"; > +}; > diff --git a/arch/arm/boot/dts/exynos3250-artik5.dtsi b/arch/arm/boot/dts/exynos3250-artik5.dtsi > new file mode 100644 > index 000000000000..97d0087efb9f > --- /dev/null > +++ b/arch/arm/boot/dts/exynos3250-artik5.dtsi > @@ -0,0 +1,282 @@ > +/* > + * Samsung's Exynos3250 based ARTIK5 module device tree source > + * > + * Copyright (c) 2016 Samsung Electronics Co., Ltd. > + * http://www.samsung.com > + * > + * Device tree source file for Samsung's ARTIK5 module which is based on > + * Samsung Exynos3250 SoC. > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + */ > + > +#include "exynos3250.dtsi" > +#include > + > +/ { > + compatible = "samsung,artik5", "samsung,exynos3250", "samsung,exynos3"; > + > + Leave only one blank line. > + chosen { > + linux,stdout = &serial_2; 'stdout-path' should be used (linux,stdout is deprecated). > + }; > + > + memory { > + reg = <0x40000000 0x1ff00000>; Just one space after '='. > + }; > + > + firmware@0205F000 { > + compatible = "samsung,secure-firmware"; > + reg = <0x0205F000 0x1000>; Let stick to one convention of letter case for hex numbers. Lower-case maybe, so 0205f000? > + }; > +}; > + > +&i2c_0 { > + #address-cells = <1>; > + #size-cells = <0>; > + samsung,i2c-sda-delay = <100>; > + samsung,i2c-slave-addr = <0x10>; > + samsung,i2c-max-bus-freq = <100000>; > + status = "okay"; > + > + s2mps14_pmic@66 { > + compatible = "samsung,s2mps14-pmic"; > + interrupt-parent = <&gpx3>; > + interrupts = <5 0>; IRQ_TYPE_NONE > + reg = <0x66>; > + wakeup; This is not valid property and is ignored. Probably you meant 'wakeup_source' (which would still be ignored by sec-core driver by the way... fixing this is waiting on Exynos mainline todo list :) ). Best regards, Krzysztof