[PATCH v6 0/6] powercap/drivers/dtpm: Create the dtpm hierarchy

From: Daniel Lezcano
Date: Wed Jan 19 2022 - 03:57:59 EST


The DTPM hierarchy is the base to build on top of it a power budget
allocator. It reflects the power consumption of the group of devices
and allows to cap their power.

The core code is there but there is no way to describe the desired
hierarchy yet.

A first proposal introduced the description through configfs [1] but
was rejected [2].

A second proposal based on the device tree with a binding similar to
the power domains [3] was proposed but finally rejected [4].

This sixth version delegates the hierarchy creation to the SoC with a
specific and self-encapsulated code using an array to describe the tree. The
SoC DTPM driver defines an array of nodes pointing to their parents. The
hierarchy description can integrate a DT node and in the future a SCMI node,
that means the description can mix different type of nodes.

As the DTPM tree depends on different devices which could be modules,
the SoC specific description must always be compiled as a module and
describe the module softdeps in order to let the userspace to handle
proper loading ordering.

In addition to the hierarchy creation, the devfreq dtpm support is also
integrated into this series.

This series was tested on a rock960 (revision B - rk3399 based) and a
db845c (Qualcomm sdm845 based).

[1] https://lore.kernel.org/all/20210401183654.27214-1-daniel.lezcano@xxxxxxxxxx/
[2] https://lore.kernel.org/all/YGYg6ZeZ1181%2FpXk@xxxxxxxxx/
[3] https://lore.kernel.org/all/20211205231558.779698-1-daniel.lezcano@xxxxxxxxxx/
[4] https://lore.kernel.org/all/YbfFapsmsjs4qnsg@xxxxxxxxxxxxxxxxxx/

Changelog:
V6:
- Switched the init table to a subsystem arrays
- Checked 'setup' function is set before calling it
- Moved out of the loop the 'of_node_put'
- Explicitely add DTPM_NODE_VIRTUAL in documentation
- Moved powercap_register_control_type() into the hierarchy creation function
- Removed the sdm845 description
- Made rk3399 always as a module and added module softdeps

V5:
- Remove DT bindings
- Added description with an array
- Added simple description for rk3399 and sdm845
- Moved dtpm table to the data section

V4:
- Added missing powerzone-cells
- Changed powerzone name to comply with the pattern property

V3:
- Remove GPU section as no power is available (yet)
- Remove '#powerzone-cells' conforming to the bindings change
- Removed required property 'compatible'
- Removed powerzone-cells from the topmost node
- Removed powerzone-cells from cpus 'consumers' in example
- Set additionnal property to false

V2:
- Added pattern properties and stick to powerzone-*
- Added required property compatible and powerzone-cells
- Added additionnal property
- Added compatible
- Renamed to 'powerzones'
- Added missing powerzone-cells to the topmost node
- Fixed errors reported by 'make DT_CHECKER_FLAGS=-m dt_binding_check'
- Move description in the SoC dtsi specific file
- Fixed missing prototype warning reported by lkp@

V1: Initial post

Daniel Lezcano (5):
powercap/drivers/dtpm: Convert the init table section to a simple
array
powercap/drivers/dtpm: Add hierarchy creation
powercap/drivers/dtpm: Add CPU DT initialization support
powercap/drivers/dtpm: Add dtpm devfreq with energy model support
rockchip/soc/drivers: Add DTPM description for rk3399

drivers/powercap/Kconfig | 8 ++
drivers/powercap/Makefile | 1 +
drivers/powercap/dtpm.c | 170 ++++++++++++++++++++++++-
drivers/powercap/dtpm_cpu.c | 41 +++++-
drivers/powercap/dtpm_devfreq.c | 204 ++++++++++++++++++++++++++++++
drivers/powercap/dtpm_subsys.h | 22 ++++
drivers/soc/rockchip/Kconfig | 8 ++
drivers/soc/rockchip/Makefile | 1 +
drivers/soc/rockchip/dtpm.c | 59 +++++++++
include/asm-generic/vmlinux.lds.h | 11 --
include/linux/dtpm.h | 33 +++--
11 files changed, 519 insertions(+), 39 deletions(-)
create mode 100644 drivers/powercap/dtpm_devfreq.c
create mode 100644 drivers/powercap/dtpm_subsys.h
create mode 100644 drivers/soc/rockchip/dtpm.c

--
2.25.1