Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1285039

Re: [PATCH v2 4/9] ARM: dts: add dts files for hi3519-demb board

Show all headers | View raw

On 2015/12/4 18:49, Arnd Bergmann wrote:
> On Friday 04 December 2015 10:27:58 xuejiancheng wrote:
>>>
>>>> +            sysctrl: system-controller@12020000 {
>>>> +                    compatible = "hisilicon,sysctrl";
>>>> +                    reg = <0x12020000 0x1000>;
>>>> +                    reboot-offset = <0x4>;
>>>> +            };
>>>
>>> Is this one identical to the one in hip04?
>>>
>>> If not, pick a new unique compatible string
>>
>> Yes. It's compatible with the one in hip04.
> 
> Ok, we should add a compatible string for that then, as the hip04 apparently
> has a slightly different layout from hip01.
> 
> Can you add a separate patch to clarify the existing hisilicon,sysctrl nodes?
> 
> It look like hi3620 accidentally uses the same string as hip04 already
> but is actually a bit different.
> 
> 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.

>>>> +
>>>> +            crg: crg@12010000 {
>>>> +                    compatible = "hisilicon,hi3519-crg";
>>>
>>>
>>> what is a "crg"? Is there a standard name for these?
>>
>> The "crg" means Clock and Reset Generator. It's a block which supplies clock
>> and reset signals for other modules in the chip. I used "hi3519-clock"
>> in last version patch. Rob Herring suggested that it's better to use "hi3519-crg"
>> as the compatible string if it's a whole block.
>>
>> what about writing like this？
>>
>> crg: clock-reset-controller@12010000 {
>>         compatible = "hisilicon,hi3519-crg";
>>         }
>>
> 
> Ok, that's better.
> 
> 	Arnd
> 
> .
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2 4/9] ARM: dts: add dts files for hi3519-demb board Jiancheng Xue <xuejiancheng@huawei.com> - 2015-12-03 04:30 +0100
  Re: [PATCH v2 4/9] ARM: dts: add dts files for hi3519-demb board Arnd Bergmann <arnd@arndb.de> - 2015-12-03 10:40 +0100
    Re: [PATCH v2 4/9] ARM: dts: add dts files for hi3519-demb board xuejiancheng <xuejiancheng@huawei.com> - 2015-12-04 03:40 +0100
      Re: [PATCH v2 4/9] ARM: dts: add dts files for hi3519-demb board Arnd Bergmann <arnd@arndb.de> - 2015-12-04 12:00 +0100
        Re: [PATCH v2 4/9] ARM: dts: add dts files for hi3519-demb board xuejiancheng <xuejiancheng@huawei.com> - 2015-12-07 07:50 +0100
          Re: [PATCH v2 4/9] ARM: dts: add dts files for hi3519-demb board zhangfei <zhangfei.gao@linaro.org> - 2015-12-08 04:40 +0100
          Re: [PATCH v2 4/9] ARM: dts: add dts files for hi3519-demb board xuejiancheng <xuejiancheng@huawei.com> - 2015-12-08 05:00 +0100
            Re: [PATCH v2 4/9] ARM: dts: add dts files for hi3519-demb board Arnd Bergmann <arnd@arndb.de> - 2015-12-09 16:40 +0100
              Re: [PATCH v2 4/9] ARM: dts: add dts files for hi3519-demb board xuejiancheng <xuejiancheng@huawei.com> - 2015-12-10 07:40 +0100
                Re: [PATCH v2 4/9] ARM: dts: add dts files for hi3519-demb board Arnd Bergmann <arnd@arndb.de> - 2015-12-10 09:10 +0100
                Re: [PATCH v2 4/9] ARM: dts: add dts files for hi3519-demb board xuejiancheng <xuejiancheng@huawei.com> - 2015-12-10 10:10 +0100
                Re: [PATCH v2 4/9] ARM: dts: add dts files for hi3519-demb board Arnd Bergmann <arnd@arndb.de> - 2015-12-10 10:20 +0100

csiph-web