lkml.org 
[lkml]   [2014]   [Oct]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH v3 6/6] ARM: dts: rockchip: add intmem node for rk3288 smp support
Date
Am Freitag, 10. Oktober 2014, 14:26:10 schrieb Kever Yang:
> This patch add intmem node des which is needed by platsmp.c
>
> Signed-off-by: Heiko Stuebner <heiko@sntech.de>

again without this Signed-off please (also true for patch 5/6 where I forgot to
mention it)

> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
> ---
>
> Changes in v3:
> - remove 'enable-method' from this patch
> - add compitable name "rockchip,rk3288-pmu-sram" for pmu-intmem
>
> Changes in v2:
> - adjust the alignment
>
> arch/arm/boot/dts/rk3288.dtsi | 17 +++++++++++++++++
> 1 file changed, 17 insertions(+)
>
> diff --git a/arch/arm/boot/dts/rk3288.dtsi b/arch/arm/boot/dts/rk3288.dtsi
> index 44108fe..5e9c56d 100644
> --- a/arch/arm/boot/dts/rk3288.dtsi
> +++ b/arch/arm/boot/dts/rk3288.dtsi
> @@ -441,6 +441,23 @@
> status = "disabled";
> };
>
> + bus_intmem@ff700000 {
> + compatible = "mmio-sram";
> + reg = <0xff700000 0x18000>;
> + #address-cells = <1>;
> + #size-cells = <1>;
> + ranges = <0 0xff700000 0x18000>;
> + smp-sram@0 {
> + compatible = "rockchip,rk3066-smp-sram";
> + reg = <0x00 0x10>;
> + };
> + };
> +
> + pmu_intmem@ff720000 {
> + compatible = "mmio-sram", "rockchip,rk3288-pmu-sram";

general rule, the more specific compatible should be in front, so
compatible = "rockchip,rk3288-pmu-sram", "mmio-sram";

This way, if there is a driver for "rockchip,rk3288-pmu-sram" at some point in
the future, it will grab the node before the more generic sram driver does.

But in general, what is the pmu sram necessary for, as no part of this series
uses it?

> + reg = <0xff720000 0x4000>;
> + };
> +
> pmu: power-management@ff730000 {
> compatible = "rockchip,rk3066-pmu", "syscon";
> reg = <0xff730000 0x100>;



\
 
 \ /
  Last update: 2014-10-11 20:41    [W:0.378 / U:0.856 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site