On 11/19/14 16:47, Kukjin Kim wrote: > On 11/18/14 15:36, Chanwoo Choi wrote: >> Dear Kukjin, >> > Hi, > >> Could you please review or pick this patchset? >> >> Best Regards, >> Chanwoo Choi >> >> On 10/08/2014 11:11 AM, YoungJun Cho wrote: >>> From: Youngjun Cho <yj44.cho@xxxxxxxxxxx> >>> >>> This patch adds new board dts file to support Samsung Monk board which is >>> based on Exynos3250 SoC and has different H/W configuration from Rinato. >>> >>> This patch is based on linux-samsung.git for-next branch and depends on >>> [PATCHv2 0/2] ARM: dts: Add new board dts file for Exynos3250-based Rinato board >>> >>> Changes from v1: >>> - The 'memory' entries coalesce into one entry >>> - Removes useless space >>> - Puts voltage-regulator-0 under the root node >>> >>> This dts file support following features: >>> - eMMC >>> - Main PMIC (Samsung S2MPS14) >>> - Interface PMIC (Maxim MAX77836, MUIC, fuel-gauge, charger) >>> - RTC of Exynos3250 >>> - ADC of Exynos3250 with NTC thermistor >>> - I2S of Exynos3250 >>> - TMU of Exynos3250 >>> - Secure firmware for Exynos3250 secondary cpu boot >>> - Serial ports of Exynos3250 >>> - gpio-key for power key >>> >>> Signed-off-by: Youngjun Cho <yj44.cho@xxxxxxxxxxx> >>> Signed-off-by: Chanwoo Choi <cw00.choi@xxxxxxxxxxx> >>> Signed-off-by: Inki Dae <inki.dae@xxxxxxxxxxx> >>> Signed-off-by: Seung-Woo Kim <sw0312.kim@xxxxxxxxxxx> >>> Signed-off-by: Jaehoon Chung <jh80.chung@xxxxxxxxxxx> >>> Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> >>> --- >>> arch/arm/boot/dts/Makefile | 3 +- >>> arch/arm/boot/dts/exynos3250-monk.dts | 583 ++++++++++++++++++++++++++++++++++ >>> 2 files changed, 585 insertions(+), 1 deletion(-) >>> create mode 100644 arch/arm/boot/dts/exynos3250-monk.dts >>> >>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile >>> index 5728918..0c8ae64 100644 >>> --- a/arch/arm/boot/dts/Makefile >>> +++ b/arch/arm/boot/dts/Makefile >>> @@ -64,7 +64,8 @@ dtb-$(CONFIG_ARCH_BRCMSTB) += \ >>> dtb-$(CONFIG_ARCH_DAVINCI) += da850-enbw-cmc.dtb \ >>> da850-evm.dtb >>> dtb-$(CONFIG_ARCH_EFM32) += efm32gg-dk3750.dtb >>> -dtb-$(CONFIG_ARCH_EXYNOS) += exynos3250-rinato.dtb \ >>> +dtb-$(CONFIG_ARCH_EXYNOS) += exynos3250-monk.dtb \ >>> + exynos3250-rinato.dtb \ >>> exynos4210-origen.dtb \ >>> exynos4210-smdkv310.dtb \ >>> exynos4210-trats.dtb \ >>> diff --git a/arch/arm/boot/dts/exynos3250-monk.dts b/arch/arm/boot/dts/exynos3250-monk.dts >>> new file mode 100644 >>> index 0000000..615672f >>> --- /dev/null >>> +++ b/arch/arm/boot/dts/exynos3250-monk.dts >>> @@ -0,0 +1,583 @@ >>> +/* >>> + * Samsung's Exynos3250 based Monk board device tree source >>> + * >>> + * Copyright (c) 2014 Samsung Electronics Co., Ltd. >>> + * http://www.samsung.com >>> + * >>> + * Device tree source file for Samsung's Monk 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.dtsi" >>> +#include <dt-bindings/input/input.h> >>> + >>> +/ { >>> + model = "Samsung Monk board"; >>> + compatible = "samsung,monk", "samsung,exynos3250", "samsung,exynos3"; >>> + >>> + aliases { >>> + i2c7 = &i2c_max77836; >>> + }; >>> + >>> + memory { >>> + reg = <0x40000000 0x1ff00000>; >>> + }; >>> + >>> + chosen { >>> + bootargs = "console=ttySAC1,115200N8 root=/dev/mmcblk0p15 rootwait earlyprintk panic=5"; > > Actually, the bootargs can be supplied from bootloader and can be > removed from here and it's a recent requirement from mailinglist. But > I've already applied similar one in rinato.dts. So please cleanup them > together later...? :) > > [snip] > > Will apply, thanks. > > - Kukjin I think, you need adding board file description in the Doc? WARNING: DT compatible string "samsung,monk" appears un-documented -- check ./Documentation/devicetree/bindings/ #79: FILE: arch/arm/boot/dts/exynos3250-monk.dts:21: + compatible = "samsung,monk", "samsung,exynos3250", "samsung,exynos3"; Thanks, Kukjin -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html