From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933588AbcCNG6N (ORCPT ); Mon, 14 Mar 2016 02:58:13 -0400 Received: from mailout3.samsung.com ([203.254.224.33]:60223 "EHLO mailout3.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933291AbcCNG54 (ORCPT ); Mon, 14 Mar 2016 02:57:56 -0400 MIME-version: 1.0 Content-type: text/plain; charset=UTF-8 X-AuditID: cbfee68f-f793a6d000001364-ab-56e660f135ef Content-transfer-encoding: 8BIT Message-id: <56E660F1.3060407@samsung.com> Date: Mon, 14 Mar 2016 15:57:53 +0900 From: Chanwoo Choi User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 To: Krzysztof Kozlowski , kgene@kernel.org, s.nawrocki@samsung.com, tomasz.figa@gmail.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 Subject: Re: [PATCH 05/10] ARM: dts: Add exynos3250-artik5 dtsi file for ARTIK5 module References: <1457921056-21257-1-git-send-email-cw00.choi@samsung.com> <1457921056-21257-6-git-send-email-cw00.choi@samsung.com> <56E65D5D.8060702@samsung.com> In-reply-to: <56E65D5D.8060702@samsung.com> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrPIsWRmVeSWpSXmKPExsWyRsSkUPdjwrMwg61nNCy2H3nGajH/yDlW i0n3J7BY3PjVxmrx+oWhRf/j18wWmx5fY7W4vGsOm8WM8/uYLBZt/cJucfhNO6vFjMkv2SxW 7frD6MDrsXPWXXaPTas62Tw2L6n36NuyitHj8ya5ANYoLpuU1JzMstQifbsErowdX06yFPzQ qDj36BpbA2O7YhcjJ4eEgInEt4dHWCFsMYkL99azdTFycQgJrGCUWLd/CTNM0YdT11khEksZ Jc7v/s4GkuAVEJT4MfkeSxcjBwezgLzEkUvZIGFmAXWJSfMWMUPUP2CUmPX0FTtEvZZE56Kv YDaLgKrEwje3wDazAcX3v7jBBjJHVCBCovtEJUhYRKBEYu/jKewgc5gFmpkk7q5ZxQSSEAaq mXN+LSPEgkWMElPO/wc7iFNAW+LS3E6whIRAK4fEig9vWCG2CUh8m3wI7FIJAVmJTQegPpOU OLjiBssERrFZSP6ZhfDPLCT/LGBkXsUomlqQXFCclF5krFecmFtcmpeul5yfu4kRGK+n/z3r 38F494D1IUYBDkYlHl5Pu2dhQqyJZcWVuYcYTYGOmMgsJZqcD0wKeSXxhsZmRhamJqbGRuaW ZkrivAulfgYLCaQnlqRmp6YWpBbFF5XmpBYfYmTi4JRqYLT73OIxd/IEFQ2u7hm6zA6z7+6O fGhtsH3ZhNiZa3pqmrW1C42WHHY1Wbycz834mLXvEo4qjvVyW19taeb2+tT1xZvxbrXyrpnf 61/uETZc91j7lbLFEXNtk0XHDervS+hMtpZk9wtf8FtyY+8Evc5Yrlsbty4T29CxbZ5+S2Hv 8jk/Dq60WqXEUpyRaKjFXFScCAAsuoMR0gIAAA== X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFuphleLIzCtJLcpLzFFi42I5/e+xgO7HhGdhBsv6lSy2H3nGajH/yDlW i0n3J7BY3PjVxmrx+oWhRf/j18wWmx5fY7W4vGsOm8WM8/uYLBZt/cJucfhNO6vFjMkv2SxW 7frD6MDrsXPWXXaPTas62Tw2L6n36NuyitHj8ya5ANaoBkabjNTElNQihdS85PyUzLx0WyXv 4HjneFMzA0NdQ0sLcyWFvMTcVFslF58AXbfMHKAblRTKEnNKgUIBicXFSvp2mCaEhrjpWsA0 Ruj6hgTB9RgZoIGENYwZO76cZCn4oVFx7tE1tgbGdsUuRk4OCQETiQ+nrrNC2GISF+6tZ+ti 5OIQEljKKHF+93c2kASvgKDEj8n3WLoYOTiYBeQljlzKBgkzC6hLTJq3iBmi/gGjxKynr9gh 6rUkOhd9BbNZBFQlFr65BbaADSi+/8UNNpA5ogIREt0nKkHCIgIlEnsfT2EHmcMs0MwkcXfN KiaQhDBQzZzzaxkhFixilJhy/j/YQZwC2hKX5nYyTmAUmIXkvlkI981Cct8CRuZVjBKpBckF xUnpuUZ5qeV6xYm5xaV56XrJ+bmbGMEp4Zn0DsbDu9wPMQpwMCrx8P5Y8zRMiDWxrLgy9xCj BAezkgivr+OzMCHelMTKqtSi/Pii0pzU4kOMpkAPTmSWEk3OB6arvJJ4Q2MTMyNLI3NDCyNj cyVx3sf/14UJCaQnlqRmp6YWpBbB9DFxcEo1MGq/VJ9523xX+23F6HeSd3Tne1fN0107i0fY bG7GR8VX/L7fZyya7JWxP8qi9edtkZc/Rdl+LlZtfxS6YnmrfPnWl/s+Ka85fJ4ziGPvIUbl ya3LLrW+KzjAy+Rvmad7P+VaScbE67sYbwYZPL9rJcZlo32jP9VPzLNw9b6mnRyd4bxn7wnq yCixFGckGmoxFxUnAgBoP4+QHwMAAA== DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Krzysztof, On 2016년 03월 14일 15:42, Krzysztof Kozlowski wrote: > 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? I'm sorry about coding style issue. I'll fix them you mentioned. > >> + }; >> +}; >> + >> +&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 OK, I'll modify it. > >> + 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 :) ). OK. I'll remove it. Thanks for your review. Best Regards, Chanwoo Choi