From mboxrd@z Thu Jan 1 00:00:00 1970 From: xuejiancheng Subject: Re: [PATCH v2 4/9] ARM: dts: add dts files for hi3519-demb board Date: Thu, 10 Dec 2015 14:32:05 +0800 Message-ID: <56691C65.6090808@huawei.com> References: <1449110668-23647-1-git-send-email-xuejiancheng@huawei.com> <56652912.80308@huawei.com> <5666548B.90502@huawei.com> <5533669.m0KDLq8XOc@wuerfel> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <5533669.m0KDLq8XOc@wuerfel> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Arnd Bergmann Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, pawel.moll-5wv7dgnIgG8@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org, galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org, khilman-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org, xuwei5-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org, haojian.zhuang-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, zhangfei.gao-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, bintian.wang-hv44wF8Li93QT0dZR+AlfA@public.gmane.org, suwenping-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, yanhaifeng-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org, gaofei-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org, ml.yang-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org, yanghongwei-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org List-Id: devicetree@vger.kernel.org On 2015/12/9 23:31, Arnd Bergmann wrote: > On Tuesday 08 December 2015 11:54:51 xuejiancheng wrote: >> On 2015/12/7 14:37, xuejiancheng wrote: >>> >>> On 2015/12/4 18:49, Arnd Bergmann wrote: >>>> On Friday 04 December 2015 10:27:58 xuejiancheng wrote: >>>>>> >>>> Maybe split out the sysctrl binding from >>>> Documentation/devicetree/bindings/arm/hisilicon/hisilicon.txt, as it has >>>> you already have a couple of those, and it's not clear how they relate >>>> to one another. >>>> >>>> If we introduce a string for all hip04 compatible sysctrl devices, we should >>>> document that and use it consistently, so hi3519 becomes >>>> >>>> compatible = "hisilicon,hi3519-sysctrl", "hisilicon,hip04-sysctrl", "hisilicon,sysctrl"; >>>> >>>> but I'd clarify in the binding documentation that "hisilicon,sysctrl" should >>>> only be used for hip04 and hi3519 but not the others. >>>> >>>> As this seems to be a standard part, we can also think about making a >>>> high-level driver for in in drivers/soc rather than relying on the syscon >>>> driver which we tend to use more for one-off devices with random register >>>> layouts. >>>> >>> Sorry. I didn't understand your meaning well and maybe I gave you a wrong description. >>> Please allow me to clarify it again. >>> The "sysctrl" nodes here is just used for the "reboot" function. It is corresponding to >>> the driver "drivers/power/reset/hisi-reboot.c". The compatible string in the driver is >>> "hisilicon,sysctrl". >>> The layout of this block is also different from the one in HiP04. >> >> I'll use "syscon" as the compatible value for sysctrl node and "syscon-reboot" for a new reboot node. >> >> > > This is not what I meant. You have to use "syscon" as the most generic > "compatible" value here, but should add a machine specific string > as a more specific one. "hisilicon,sysctrl" is not appropriate because > it does not identify the IP block uniquely, you can only use that > in combination with another more specific string. OK. I will use "hisilicon,hi3519-syscon" and "syscon" as the compatible value for the sysctrl node in hi3519.dtsi. Thank you! > > That way, we have to option to create a high-level driver for the IP > block later if it turns out that we need some more generic functionality > that is provided by those registers. > > Arnd > > . > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html