From: Heiko Stuebner <heiko@sntech.de>
To: Andy Yan <andy.yan@rock-chips.com>
Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-rockchip@lists.infradead.org, robh+dt@kernel.org
Subject: Re: [PATCH 1/2] ARM: dts: rockchip: add cpu opp table for rv1108
Date: Mon, 21 Aug 2017 17:30:05 +0200
Message-ID: <1530970.y6TTxhzjmZ@phil>
User-Agent: KMail/5.2.3 (Linux/4.11.0-1-amd64; KDE/5.28.0; x86_64; ; )
In-Reply-To: <1503306404-28876-1-git-send-email-andy.yan@rock-chips.com>
References: <1503306404-28876-1-git-send-email-andy.yan@rock-chips.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 754
Lines: 32

Hi Andy,

Am Montag, 21. August 2017, 17:06:44 CEST schrieb Andy Yan:
> Add cpu opp table for rv1108 to support frequency
> from 408MHZ to 1008MHZ.
> 
> Signed-off-by: Andy Yan <andy.yan@rock-chips.com>
> ---
> 
>  arch/arm/boot/dts/rv1108-evb.dts | 4 ++++
>  arch/arm/boot/dts/rv1108.dtsi    | 9 +++++++++
>  2 files changed, 13 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/rv1108-evb.dts b/arch/arm/boot/dts/rv1108-evb.dts
> index 32fb09a..e517aa3 100644
> --- a/arch/arm/boot/dts/rv1108-evb.dts
> +++ b/arch/arm/boot/dts/rv1108-evb.dts
> @@ -104,6 +104,10 @@
>  	};
>  };
>  
> +&cpu0 {
> +	cpu-supply = <&vdd_core>;
> +};
> +

please split into two patches (operating points + cpu-supply)
and use the operating-points-v2 binding.


Thanks
Heiko