2022-04-06 15:54:09

by Brian Norris

[permalink] [raw]
Subject: [RFC PATCH 0/2] rockchip / devfreq: Coordinate DRAM controller resources between ATF and kernel

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

--
2.35.1.1094.g7c7d902a7c-goog


2022-05-09 07:38:22

by Chanwoo Choi

[permalink] [raw]
Subject: Re: [RFC PATCH 0/2] rockchip / devfreq: Coordinate DRAM controller resources between ATF and kernel

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