[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20250908-acpm-clk-v4-0-633350c0c0b1@linaro.org>
Date: Mon, 08 Sep 2025 13:12:41 +0000
From: Tudor Ambarus <tudor.ambarus@...aro.org>
To: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
Peter Griffin <peter.griffin@...aro.org>,
André Draszik <andre.draszik@...aro.org>,
Michael Turquette <mturquette@...libre.com>,
Stephen Boyd <sboyd@...nel.org>, Krzysztof Kozlowski <krzk@...nel.org>,
Alim Akhtar <alim.akhtar@...sung.com>,
Sylwester Nawrocki <s.nawrocki@...sung.com>,
Chanwoo Choi <cw00.choi@...sung.com>,
Catalin Marinas <catalin.marinas@....com>, Will Deacon <will@...nel.org>
Cc: linux-kernel@...r.kernel.org, linux-samsung-soc@...r.kernel.org,
devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-clk@...r.kernel.org, willmcvicker@...gle.com, kernel-team@...roid.com,
Tudor Ambarus <tudor.ambarus@...aro.org>
Subject: [PATCH v4 0/5] exynos-acpm: add DVFS protocol and clock driver
Dependencies description:
All patches should go through the Samsung SoC tree.
The acpm-clk driver (#4) depends on the ACPM DVFS ops (#2).
If the clock subsystem needs to merge the new clock driver, it will
need an immutable tag with the 2 patches.
No dependecies for #1, #3, #5.
The Alive CLock and Power Manager (ACPM) firmware exposes clocks that
are variable and index based. These clocks don't provide an entire range
of values between the limits but only discrete points within the range.
The firmware also manages the voltage scaling appropriately with the
clock scaling. Make the ACPM node a clock provider.
Add support for the ACPM DVFS protocol. It translates clock frequency
requests to messages that can be interpreted by the ACPM firmware.
Add an ACPM clock driver to model the clocks exposed by the ACPM firmware.
Thanks,
ta
Signed-off-by: Tudor Ambarus <tudor.ambarus@...aro.org>
---
Changes in v4:
- clk-acpm:
- remove GS101_ACPM_CLK_ID type handling. Dead code, it should be
introduced with next devices.
- remove runtime check on clock IDs. Instead add a comment about the
assumptions the code is making: the clock IDs start from zero, are
sequential and do not have gaps. Slight changes based on this
assumption: s/hws[id]/hws[i], remove the inclusion of
dt-bindings/clock/google,gs101-acpm.h and the use of the clock IDs.
The clocks are defined solely by name in the driver.
- move firmware patches close to each other, in between the bindings and
the clock driver
- update the description of dependencies in the cover letter.
- Link to v3: https://lore.kernel.org/r/20250903-acpm-clk-v3-0-65ecd42d88c7@linaro.org
Changes in v3:
- dt-bindings:
- move clock bindings to a new bindings header
- update commit's subject, s/add #clock-cells/add ACPM clocks.
It also suggests that the bindings are added.
- prepend "GS101_" on clocks binding name. The bindings name are the
same for GS201 and the acpm-clk driver will likely include both.
- collect Rob's R-b
- clk-acpm:
- move clock definitions here instead of keeping them into the
ACPM protocol driver
- use platform_driver.id_table to differentiate device type
- fix Kconfig dependency, || (COMPILE_TEST && !EXYNOS_ACPM_PROTOCOL)
- update commit subject, s/dev/pdev
- exynos-acpm:
- move clock definitions to clk-acpm
- use devm-action to unregister clk-acpm platform device
- Link to v2: https://lore.kernel.org/r/20250827-acpm-clk-v2-0-de5c86b49b64@linaro.org
Changes in v2:
- dt-bindings: clocks are not longer a child of ACPM protocol. Instead
make Alive Clock and Power Manager (ACPM) node a clock provider.
Update commit message.
- firmware: exynos-acpm: register by hand the ACPM clocks dev (new
patch)
- firmware: exynos-acpm: use defines intead of enum
- acpm-clk:
- switch to determine_rate
- drop __init, __refdata, __initconst, this is a module, we need those
methods and data, after boot as well.
- fix the assumption that the clocks are defined by ID in ascending order.
There's still an assumption that the clk_id < nr_clks, but this is
now covered by a sanity check in the clock driver.
- arm64: defconfig: enable Exynos ACPM clocks (add patch together with
this patch set)
- Link to v1: https://lore.kernel.org/r/20250819-acpm-clk-v1-0-6bbd97474671@linaro.org
---
Tudor Ambarus (5):
dt-bindings: firmware: google,gs101-acpm-ipc: add ACPM clocks
firmware: exynos-acpm: add DVFS protocol
firmware: exynos-acpm: register ACPM clocks pdev
clk: samsung: add Exynos ACPM clock driver
arm64: defconfig: enable Exynos ACPM clocks
.../bindings/firmware/google,gs101-acpm-ipc.yaml | 11 ++
arch/arm64/configs/defconfig | 1 +
drivers/clk/samsung/Kconfig | 10 ++
drivers/clk/samsung/Makefile | 1 +
drivers/clk/samsung/clk-acpm.c | 184 +++++++++++++++++++++
drivers/firmware/samsung/Makefile | 4 +-
drivers/firmware/samsung/exynos-acpm-dvfs.c | 83 ++++++++++
drivers/firmware/samsung/exynos-acpm-dvfs.h | 21 +++
drivers/firmware/samsung/exynos-acpm.c | 26 +++
include/dt-bindings/clock/google,gs101-acpm.h | 26 +++
.../linux/firmware/samsung/exynos-acpm-protocol.h | 10 ++
11 files changed, 376 insertions(+), 1 deletion(-)
---
base-commit: c17b750b3ad9f45f2b6f7e6f7f4679844244f0b9
change-id: 20250819-acpm-clk-28d2a78e0307
Best regards,
--
Tudor Ambarus <tudor.ambarus@...aro.org>
Powered by blists - more mailing lists