Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Message-ID: <2b8f0e62-ab3c-23bd-f9e8-590df3af12be@gmail.com>
Date:   Mon, 9 May 2022 03:40:08 +0900
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.8.1
Subject: Re: [RFC PATCH 0/2] rockchip / devfreq: Coordinate DRAM controller
 resources between ATF and kernel
Content-Language: en-US
To:     Brian Norris <briannorris@chromium.org>,
        MyungJoo Ham <myungjoo.ham@samsung.com>,
        Kyungmin Park <kyungmin.park@samsung.com>,
        Chanwoo Choi <cw00.choi@samsung.com>,
        Heiko Stuebner <heiko@sntech.de>
Cc:     linux-kernel@vger.kernel.org,
        Elaine Zhang <zhangqing@rock-chips.com>,
        linux-pm@vger.kernel.org, Doug Anderson <dianders@chromium.org>,
        linux-arm-kernel@lists.infradead.org,
        linux-rockchip@lists.infradead.org
References: <20220406014842.2771799-1-briannorris@chromium.org>
From:   Chanwoo Choi <cwchoi00@gmail.com>
In-Reply-To: <20220406014842.2771799-1-briannorris@chromium.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

Hi Brian,

On 22. 4. 6. 10:48, Brian Norris wrote:
> On Rockchip RK3399, there are a few hardware resources that are shared
> between firmware (ARM Trusted Firmware) and kernel (power domain
> driver) that need to be coordinated properly for DRAM DVFS to work
> reliably. See patch 1 for plenty more description.
> 
> These fixes are based in part on the specification in the RK3399, and in
> part based on extrapolation and observation. Any confirmation about the
> behavior of PMU_CRU_GATEDIS_CON0, etc., is welcome.
> 
> Otherwise, see the patches.
> 
> Regards,
> Brian
> 
> 
> Brian Norris (2):
>    soc: rockchip: power-domain: Manage resource conflicts with firmware
>    PM / devfreq: rk3399_dmc: Block PMU during transitions
> 
>   drivers/devfreq/rk3399_dmc.c      |  13 ++++
>   drivers/soc/rockchip/pm_domains.c | 118 ++++++++++++++++++++++++++++++
>   include/soc/rockchip/pm_domains.h |  25 +++++++
>   3 files changed, 156 insertions(+)
>   create mode 100644 include/soc/rockchip/pm_domains.h
> 

Applied them. with Heiko reviewed-by tag for patch1.

-- 
Best Regards,
Samsung Electronics
Chanwoo Choi