From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Stuebner Subject: Re: [PATCH 2/2] arm64: dts: rockchip: introduce rk3399-op1 operating points Date: Tue, 23 May 2017 18:38:23 +0200 Message-ID: <9616868.DoOFoaYK71@phil> References: <20170523095109.7337-1-heiko@sntech.de> <1597629.2TFUO1J4Mt@phil> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+glpar-linux-rockchip=m.gmane.org-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org To: Doug Anderson Cc: "open list:ARM/Rockchip SoC..." , eddie.cai.linux-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, Brian Norris , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , Caesar List-Id: linux-rockchip.vger.kernel.org Am Dienstag, 23. Mai 2017, 09:35:05 CEST schrieb Doug Anderson: > Hi, > > On Tue, May 23, 2017 at 9:10 AM, Heiko Stuebner wrote: > > Am Dienstag, 23. Mai 2017, 08:57:01 CEST schrieb Doug Anderson: > >> Hi, > >> > >> On Tue, May 23, 2017 at 2:51 AM, Heiko Stuebner wrote: > >> > The OP1 is a rk3399 variant used in ChromeOS devices with a slightly > >> > higher frequency rating. Therefore introduce separate operting points, > >> > from the ChromeOS tree, for it and use it in Gru devices. > >> > > >> > Signed-off-by: Heiko Stuebner > >> > --- > >> > arch/arm64/boot/dts/rockchip/rk3399-gru.dtsi | 2 +- > >> > arch/arm64/boot/dts/rockchip/rk3399-op1-opp.dtsi | 145 +++++++++++++++++++++++ > >> > 2 files changed, 146 insertions(+), 1 deletion(-) > >> > create mode 100644 arch/arm64/boot/dts/rockchip/rk3399-op1-opp.dtsi > >> > >> The numbers here match what's in the Chrome OS tree and those were all > >> approved by Rockchip. > >> > >> One slight nit is that the descriptions these two patches together > >> make it look like this will be a no-op change for OP1 devices. This > >> is not true. After things were posted upstream there have been > >> several adjustments to the opp points in the Chrome OS tree. Since > >> what you have currently matches the ToT Chrome OS tree, that should > >> mean that there's a net difference. It would be nice to mention that > >> in the commit message. > > > > In the mainline kernel it is a no-op change, as we're still running with > > the cpu-regulators disabled on gru ;-) > > Ah, that's true. > > > > But to address your very valid point, I guess you mean something like > > > > ---- > > The OP1 is a rk3399 variant used in ChromeOS devices with a slightly > > higher frequency rating compared to the regular rk3399, but right now > > the only available operating points are somewhere in between both > > variants with both needing adjustments to actually fit their specs. > > That would be fine and would make it obvious that it's a no-op. > Ironically, the OPP points that were in mainline were actually not > totally in between. There were some numbers that were raised for > both. :) wording fun ... then we'll make that part a "[...] but right now the only available operating points don't match either variant with both needing adjustments to actually fit their specs." > > Therefore introduce separate operting points, from the ChromeOS kernel, > > for the OP1 and use it on Gru devices. > > ---- > > > > And I guess we should also swap patch ordering, first move Gru away from > > the general opps and then adust them for the regular rk3399. > > Yeah, I think swapping the order would make it slightly better. Heiko