From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1753534AbcBOXEo (ORCPT <rfc822;w@1wt.eu>);
	Mon, 15 Feb 2016 18:04:44 -0500
Received: from mailout2.w1.samsung.com ([210.118.77.12]:46457 "EHLO
	mailout2.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1753355AbcBOXEk (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Mon, 15 Feb 2016 18:04:40 -0500
X-AuditID: cbfec7f4-f79026d00000418a-27-56c25986a661
Subject: Re: [PATCH] ARM: dts: Move syscon reboot/poweroff to common dtsi for
 Exynos
To: Javier Martinez Canillas <javier@osg.samsung.com>,
        linux-kernel@vger.kernel.org
References: <1455042606-10075-1-git-send-email-javier@osg.samsung.com>
 <56C02265.40103@samsung.com> <56C1D4D1.2050108@osg.samsung.com>
Cc: k.kozlowski.k@gmail.com, devicetree@vger.kernel.org,
        linux-samsung-soc@vger.kernel.org, Andi Shyti <andi.shyti@samsung.com>,
        Alim Akhtar <alim.akhtar@samsung.com>, Kukjin Kim <kgene@kernel.org>,
        linux-arm-kernel@lists.infradead.org
From: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Message-id: <56C25982.4020301@samsung.com>
Date: Tue, 16 Feb 2016 08:04:34 +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: <56C1D4D1.2050108@osg.samsung.com>
Content-type: text/plain; charset=iso-8859-2
Content-transfer-encoding: 7bit
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrLLMWRmVeSWpSXmKPExsVy+t/xa7ptkYfCDDZtYrd4MG8bm8X2I89Y
	LeYfOcdq8ebtGiaL5/9+sFu8fmFo0f/4NbPFpsfXWC0u75rDZjHj/D4mBy6PnbPusntsWtXJ
	5rF5Sb3Hln4gr2/LKkaPz5vkAtiiuGxSUnMyy1KL9O0SuDLmr7vKWDCRreLq1u3MDYw3WLoY
	OTkkBEwk5j58wgRhi0lcuLeerYuRi0NIYCmjxN/2y6wQzlNGiRVz1rCDVAkLhEl83N7ICGKL
	CIRK/Lt4mxGiqINR4vabXWAOs8BHRolL8++AVbEJGEtsXr6EDcTmFdCSmPX+ANBuDg4WAVWJ
	I02aIGFRgQiJw51d7BAlghI/Jt8DO49TQF9i98Z1YOXMAjoSXydFgISZBeQlNq95yzyBUWAW
	ko5ZCFWzkFQtYGRexSiaWppcUJyUnmuoV5yYW1yal66XnJ+7iRESCV92MC4+ZnWIUYCDUYmH
	l8PjUJgQa2JZcWXuIUYJDmYlEd6rYUAh3pTEyqrUovz4otKc1OJDjNIcLErivHN3vQ8REkhP
	LEnNTk0tSC2CyTJxcEo1MLZoGondWso8Zc3d+bEn96QcuHD44dUkdscplpNFOP0WSE71+DJJ
	KER69waR1NTXa7kbN06LnrXZaOuiGXOCj118viqx61OSMMPLNNVTP6bzPNRzrObR4zrx+otQ
	9GvRySy10nzu76tXbmrMZlf9dbZDtPh6Ib+8hsuyhfpnI7pzdazkvJzYPZVYijMSDbWYi4oT
	AbFzMCSAAgAA
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 15.02.2016 22:38, Javier Martinez Canillas wrote:
>>
>> In some of the DTSI this was under "soc" node, not at top-level.
>> Unfortunately we do not have consistency here - some DTSI have "soc",
> 
> Yes, I noticed this but as you said not all DTSI have a soc node
> and that's why I made it top level in the DTSI.
> 
>> some not. Anyway I think we should move to "soc" version.
>>
> 
> Not sure I'm following, did you mean to do it as a follow up or
> to add a soc node for the missing DTSI as a part of this series?

I meant to send a v2 with this under "soc" node even though on some DTS
we do not have the "soc". Optional follow up would be to make DTS
consistent and move nodes under "soc"... but that would be also a lot of
churn.

Best regards,
Krzysztof