Messages in this thread | | | Subject | Re: [PATCH v2 4/9] ARM: dts: add dts files for hi3519-demb board | From | xuejiancheng <> | Date | Fri, 4 Dec 2015 10:27:58 +0800 |
| |
Hi Arnd, Thank you very much for all your comments.
On 2015/12/3 17:36, Arnd Bergmann wrote: > On Thursday 03 December 2015 10:44:28 Jiancheng Xue wrote: > >> + >> +/dts-v1/; >> +#include "hi3519.dtsi" >> + >> +/ { >> + model = "HiSilicon HI3519 DEMO Board"; >> + compatible = "hisilicon,hi3519"; >> + >> + chosen { >> + bootargs = "mem=64M console=ttyAMA0,115200 early_printk \ >> +root=/dev/mtdblock2 rootfstype=jffs2 \ >> +mtdparts=hi_sfc:1M(boot),4M(kernel),11M(rootfs)"; >> + }; > > Most of the arguments should be dropped and replaced with the respective > DT properties in this file: > > mem: /memory (you have that already, but the size seems wrong) > console: /chosen/stdout-path > early_printk: just drop this, maybe use "earlycon") > root: this one is fine > rootfstype: should not be needed > mtdparts: use nodes below the MTD device >
This chosen node is just for debug. The real parameters will be set at boot stage. I will drop it.
>> + >> +#include "skeleton.dtsi" >> +#include <dt-bindings/clock/hi3519-clock.h> >> +/ { >> + aliases { >> + serial0 = &uart0; >> + }; > > Move this into the .dts file.
OK. Thank you.
> >> + >> + uart0: uart@12100000 { > > rename to serial@12100000
OK.
> >> + dual_timer1: dual_timer@12001000 { >> + compatible = "arm,sp804", "arm,primecell"; >> + interrupts = <0 66 4>, <0 67 4>; >> + reg = <0x12001000 0x1000>; >> + clocks = <&crg HI3519_FIXED_3M>; >> + status = "disable"; >> + }; > > rename to timer@12001000
OK.
> >> + 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.
> >> + >> + 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"; }
> > Arnd > > . >
Jiancheng .
| |