[PATCH v5 0/8] RK3588 and Rock 5B dts additions: thermal, OPP and fan
Dragan Simic
dsimic at manjaro.org
Mon Jun 17 11:56:45 PDT 2024
Hello Alexey,
On 2024-06-17 20:28, Alexey Charkov wrote:
> This enables thermal monitoring and CPU DVFS on RK3588(s), as well as
> active cooling on Radxa Rock 5B via the provided PWM fan.
>
> Some RK3588 boards use separate regulators to supply CPUs and their
> respective memory interfaces, so this is handled by coupling those
> regulators in affected boards' device trees to ensure that their
> voltage is adjusted in step.
>
> This also enables the built-in thermal sensor (TSADC) for all boards
> that don't currently have it enabled, using the default CRU based
> emergency thermal reset. This default configuration only uses on-SoC
> devices and doesn't rely on any external wiring, thus it should work
> for all devices (tested only on Rock 5B though).
>
> The boards that have TSADC_SHUT signal wired to the PMIC reset line
> can choose to override the default reset logic in favour of GPIO
> driven (PMIC assisted) reset, but in my testing it didn't work on
> Radxa Rock 5B - maybe I'm reading the schematic wrong and it doesn't
> support PMIC assisted reset after all.
>
> Fan control on Rock 5B has been split into two intervals: let it spin
> at the minimum cooling state between 55C and 65C, and then accelerate
> if the system crosses the 65C mark - thanks to Dragan for suggesting.
> This lets some cooling setups with beefier heatsinks and/or larger
> fan fins to stay in the quietest non-zero fan state while still
> gaining potential benefits from the airflow it generates, and
> possibly avoiding noisy speeds altogether for some workloads.
I should be able to test an RK3588 cooling setup with a really beefy
heatsink in the near future, so we'll see how well the "silent zone"
performs in practice and not just in theory. :)
> OPPs help actually scale CPU frequencies up and down for both cooling
> and performance - tested on Rock 5B under varied loads. I've dropped
> those OPPs that cause frequency reductions without accompanying
> decrease
> in CPU voltage, as they don't seem to be adding much benefit in day to
> day use, while the kernel log gets a number of "OPP is inefficient"
> lines.
>
> Note that this submission doesn't touch the SRAM read margin updates or
> the OPP calibration based on silicon quality which the downstream
> driver
> does and which were mentioned in [1]. It works as it is (also confirmed
> by
> Sebastian in his follow-up message [2]), and it is stable in my testing
> on
> Rock 5B, so it sounds better to merge a simple version first and then
> extend when/if required.
>
> This patch series has been rebased on top of Heiko's recent for-next
> branch
> with Dragan's patch [3] which rearranges the .dtsi files for
> per-variant OPPs.
> As a result, it now includes separate CPU OPP tables for RK3588(s) and
> RK3588j.
Thanks for the current iteration of this series! I'll review it
in detail and perform thorough stress testing on my ROCK 5B as soon
as possible.
> GPU OPPs have also been split out to accommodate for the difference in
> RK3588j.
>
> [1]
> https://lore.kernel.org/linux-rockchip/CABjd4YzTL=5S7cS8ACNAYVa730WA3iGd5L_wP1Vn9=f83RCORA@mail.gmail.com/
> [2]
> https://lore.kernel.org/linux-rockchip/pkyne4g2cln27dcdu3jm7bqdqpmd2kwkbguiolmozntjuiajrb@gvq4nupzna4o/
> [3]
> https://lore.kernel.org/linux-rockchip/9ffedc0e2ca7f167d9d795b2a8f43cb9f56a653b.1717923308.git.dsimic@manjaro.org/
>
> Signed-off-by: Alexey Charkov <alchark at gmail.com>
> ---
> Changes in v5:
> - Rebased against linux-rockchip/for-next with Dragan's .dtsi
> reshuffling on top
> - Added separate OPP values for RK3588j (these also apply to RK3588m)
> - Separated GPU OPP values for RK3588j (RK3588m ones differ slightly,
> not included here)
> - Dragan's patch:
> https://lore.kernel.org/linux-rockchip/9ffedc0e2ca7f167d9d795b2a8f43cb9f56a653b.1717923308.git.dsimic@manjaro.org/
> - Link to v4:
> https://lore.kernel.org/r/20240506-rk-dts-additions-v4-0-271023ddfd40@gmail.com
>
> Changes in v4:
> - Rebased against linux-rockchip/for-next
> - Reordered DT nodes alphabetically as pointed out by Diederik
> - Moved the TSADC enablement to per-board .dts/.dtsi files
> - Dropped extra "inefficient" OPPs (same voltage - lower frequencies)
> - Dropped second passive cooling trips altogether to keep things simple
> - Added a cooling map for passive GPU cooling (in a separate patch)
> - Link to v3:
> https://lore.kernel.org/r/20240229-rk-dts-additions-v3-0-6afe8473a631@gmail.com
>
> Changes in v3:
> - Added regulator coupling for EVB1 and QuartzPro64
> - Enabled the TSADC for all boards in .dtsi, not just Rock 5B (thanks
> ChenYu)
> - Added comments regarding two passive cooling trips in each zone
> (thanks Dragan)
> - Fixed active cooling map numbering for Radxa Rock 5B (thanks Dragan)
> - Dropped Daniel's Acked-by tag from the Rock 5B fan patch, as there's
> been quite some
> churn there since the version he acknowledged
> - Link to v2:
> https://lore.kernel.org/r/20240130-rk-dts-additions-v2-0-c6222c4c78df@gmail.com
>
> Changes in v2:
> - Dropped the rfkill patch which Heiko has already applied
> - Set higher 'polling-delay-passive' (100 instead of 20)
> - Name all cooling maps starting from map0 in each respective zone
> - Drop 'contribution' properties from passive cooling maps
> - Link to v1:
> https://lore.kernel.org/r/20240125-rk-dts-additions-v1-0-5879275db36f@gmail.com
>
> ---
> Alexey Charkov (8):
> arm64: dts: rockchip: add thermal zones information on RK3588
> arm64: dts: rockchip: enable thermal management on all RK3588
> boards
> arm64: dts: rockchip: add passive GPU cooling on RK3588
> arm64: dts: rockchip: enable automatic fan control on Rock 5B
> arm64: dts: rockchip: Add CPU/memory regulator coupling for
> RK3588
> arm64: dts: rockchip: Add OPP data for CPU cores on RK3588
> arm64: dts: rockchip: Add OPP data for CPU cores on RK3588j
> arm64: dts: rockchip: Split GPU OPPs of RK3588 and RK3588j
>
> .../boot/dts/rockchip/rk3588-armsom-sige7.dts | 4 +
> arch/arm64/boot/dts/rockchip/rk3588-base.dtsi | 197
> +++++++++++++++++----
> .../dts/rockchip/rk3588-edgeble-neu6a-common.dtsi | 4 +
> arch/arm64/boot/dts/rockchip/rk3588-evb1-v10.dts | 16 ++
> arch/arm64/boot/dts/rockchip/rk3588-ok3588-c.dts | 4 +
> arch/arm64/boot/dts/rockchip/rk3588-opp.dtsi | 190
> ++++++++++++++++++++
> .../arm64/boot/dts/rockchip/rk3588-quartzpro64.dts | 12 ++
> arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 34 +++-
> .../arm64/boot/dts/rockchip/rk3588-toybrick-x0.dts | 4 +
> .../arm64/boot/dts/rockchip/rk3588-turing-rk1.dtsi | 4 +
> arch/arm64/boot/dts/rockchip/rk3588.dtsi | 1 +
> arch/arm64/boot/dts/rockchip/rk3588j.dtsi | 141
> +++++++++++++++
> arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts | 4 +
> arch/arm64/boot/dts/rockchip/rk3588s.dtsi | 1 +
> 14 files changed, 577 insertions(+), 39 deletions(-)
> ---
> base-commit: 5cc74606bf40a2bbaccd3e3bb2781f637baebde5
> change-id: 20240124-rk-dts-additions-a6d7b52787b9
>
> Best regards,
More information about the linux-arm-kernel
mailing list