Re: [PATCH v2 4/6] arm64: dts: rockchip: add core dtsi file for RK3328 SoCs

From: 陈亮 <cl@rock-chips.com>
To: Sudeep Holla <sudeep.holla@arm.com>, heiko@sntech.de
Cc: mark.rutland@arm.com, wsa@the-dreams.de,
	linux-iio@vger.kernel.org, catalin.marinas@arm.com,
	shawn.lin@rock-chips.com, will.deacon@arm.com,
	kever.yang@rock-chips.com, dianders@chromium.org,
	yamada.masahiro@socionext.com, tony.xie@rock-chips.com,
	linux-i2c@vger.kernel.org, pmeerw@pmeerw.net, lars@metafoo.de,
	zhengxing@rock-chips.com, khilman@baylibre.com,
	linux-rockchip@lists.infradead.org, jay.xu@rock-chips.com,
	wxt@rock-chips.com, huangtao@rock-chips.com,
	devicetree@vger.kernel.org, zhangqing@rock-chips.com,
	paweljarosz3691@gmail.com, arnd@arndb.de, yhx@rock-chips.com,
	knaack.h@gmx.de, robh+dt@kernel.org, matthias.bgg@gmail.com,
	rocky.hao@rock-chips.com, linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, david.wu@rock-chips.com,
	fabio.estevam@nxp.com, andy.yan@rock-chips.com,
	akpm@linux-foundation.org, shawnguo@kernel.org, afaerber@suse.de,
	jic23@kernel.org
Subject: Re: [PATCH v2 4/6] arm64: dts: rockchip: add core dtsi file for RK3328 SoCs
Date: Thu, 23 Mar 2017 10:15:20 +0800	[thread overview]
Message-ID: <58D32FB8.9070909@rock-chips.com> (raw)
In-Reply-To: <7240249b-690a-5800-0b46-fb93a49e2af1@arm.com>

Hi,

在 2017年03月23日 02:09, Sudeep Holla 写道:
>
> On 16/03/17 13:17, cl@rock-chips.com wrote:
>> From: Liang Chen <cl@rock-chips.com>
>>
>> This patch adds core dtsi file for Rockchip RK3328 SoCs.
>>
>> Signed-off-by: Liang Chen <cl@rock-chips.com>
>> ---
>>   arch/arm64/boot/dts/rockchip/rk3328.dtsi | 1362 ++++++++++++++++++++++++++++++
>>   1 file changed, 1362 insertions(+)
>>   create mode 100644 arch/arm64/boot/dts/rockchip/rk3328.dtsi
>>
>> diff --git a/arch/arm64/boot/dts/rockchip/rk3328.dtsi b/arch/arm64/boot/dts/rockchip/rk3328.dtsi
>> new file mode 100644
>> index 0000000..a92955c
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/rockchip/rk3328.dtsi
> [...]
>
>> +	cpus {
>> +		#address-cells = <2>;
>> +		#size-cells = <0>;
>> +
>> +		cpu0: cpu@0 {
>> +			device_type = "cpu";
>> +			compatible = "arm,cortex-a53", "arm,armv8";
>> +			reg = <0x0 0x0>;
>> +			enable-method = "psci";
>> +			clocks = <&cru ARMCLK>;
> Why "clocks" property is present only for cpu0 ?
>
I will add "clocks" property for each cpu node next version, thanks for 
the comment.