From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754654AbcHSBxW (ORCPT ); Thu, 18 Aug 2016 21:53:22 -0400 Received: from gloria.sntech.de ([95.129.55.99]:39944 "EHLO gloria.sntech.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755325AbcHSBwJ (ORCPT ); Thu, 18 Aug 2016 21:52:09 -0400 From: Heiko Stuebner To: Doug Anderson Cc: "open list:ARM/Rockchip SoC..." , zhangqing , Caesar Wang , Xing Zheng , Derek Basehore , Shawn Lin , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH] soc: rockchip: power-domain: Don't (incorrectly) set rk3399 up/down counts Date: Fri, 19 Aug 2016 00:20:01 +0200 Message-ID: <1700555.4f5dI05O1A@phil> User-Agent: KMail/5.2.3 (Linux/4.6.0-1-amd64; KDE/5.23.0; x86_64; ; ) In-Reply-To: References: <1471546561-4459-1-git-send-email-dianders@chromium.org> <86565478.t9HNV2EDZM@phil> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Am Donnerstag, 18. August 2016, 15:08:12 CEST schrieb Doug Anderson: > Hi, > > On Thu, Aug 18, 2016 at 3:03 PM, Heiko Stuebner wrote: > > Hi Doug, > > > > Am Donnerstag, 18. August 2016, 11:56:01 CEST schrieb Douglas Anderson: > >> On rk3288 it was important that powerdown and powerup counts for the > > > >> CPU/GPU in the kernel because: > > somehow this sentence seems to miss some verb or so :-) > > Sigh. I guess I can't type. > > On rk3288 it was important that powerdown and powerup counts for the > > CPU/GPU be set in the kernel because: > >> * The power on default was crazy long. > >> * We couldn't rely on the firmware to set this up because really this > >> > >> wasn't the firmware's job--the kernel was the only one that really > >> cared about bringing up / down CPUs and the GPU and doing suspend / > >> resume (which involves bringing up / down CPUs). > >> > >> On newer ARM systems (like rk3399) ARM Trusted Firmware is in charge of > >> bringing up and down the CPUs and it really should be in charge of > >> setting all these counts right. After all ATF is in charge of suspend / > >> resume and CPU up / down. Let's get out of the way and let ATF do its > >> job. > >> > >> A few other motivations for doing this: > >> * Depending on another configuration (PMU_24M_EN_CFG) these counts can > >> > >> be either in 24M or 32k cycles. Thus, though ATF isn't really so > >> involved in bringing up the GPU, ATF should probably manage the counts > >> for everything so it can also manage the 24M / 32k choice. > >> > >> * It turns out that (right now) 24M mode is broken on rk3399 and not > >> > >> being used. That means that the count the kernel was programming > >> in (24) was not 1 us (which it seems was intended) but was actually > >> .75 ms > >> > >> * On rk3399 there are actually 2 separate registers for setting CPU > >> > >> up/down time plus 1 register for GPU up/down time. The curent kernel > >> code actually was putting the register for the "little" cores in the > >> "CPU" slot and the register for the "big" cores in the "GPU" slot. It > >> was never initting the GPU counts. > >> > >> Note: this change assumes that ATF will actually set these values at > >> boot, as I'm proposing in . > > > > I'd hope to see a link to an ATF github pull request here :-) > > But I guess that simply needs some more discussion on your side. > > Caesar is going to get confirmation that the patch is OK then I think > he'll work on the ATF pull request. Once done we can update the link > here? yep and I can then also update your sentence above :-) > >> Signed-off-by: Douglas Anderson > > > > change itself looks good to me. > > > > So I guess we'll just need to wait for the counterpart to land in the ATF > > or do you know if the poweron-defaults are somewhat sane? > > Power on defaults are crappy (750 ms to turn on/off a CPU), so > non-ideal. Probably best to wait for ATF to land. ok, so we'll wait. As I might miss it when the other side gets merged into the ATF, can you ping here once that is done please? Heiko