[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20250618-rk8xx-rst-fun-v3-0-081f02d3d348@cherry.de>
Date: Wed, 18 Jun 2025 12:32:39 +0200
From: Quentin Schulz <foss+kernel@...il.net>
To: Lee Jones <lee@...nel.org>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Heiko Stuebner <heiko@...ech.de>,
Sebastian Reichel <sebastian.reichel@...labora.com>
Cc: Lukasz Czechowski <lukasz.czechowski@...umatec.com>,
Daniel Semkowicz <dse@...umatec.com>,
Nicolas Frattaroli <nicolas.frattaroli@...labora.com>,
devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-rockchip@...ts.infradead.org, linux-kernel@...r.kernel.org,
Quentin Schulz <quentin.schulz@...rry.de>
Subject: [PATCH v3 0/5] rockchip: rk8xx: allow to customize PMIC reset mode
on RK806
This allows to customize the PMIC reset method (also called RST_FUN) on
RK806 PMIC from Rockchip, mainly found on RK3588 devices but also on
RK3576.
Finally, this is required on the two RK3588 devices from Theobroma as
U-Boot changes the silicon-default (which is suitable for us) to
something that breaks our companion microcontroller's reboot detection
which breaks a bunch of assumptions in the MCU FW code.
To validate this works on those devices do the following:
On Tiger:
i2cset -y -f 6 0x6f 0x9 0x62
On Jaguar:
i2cset -y -f 0 0x6f 0x9 0x62
You hear a nice (loud :) ) beep, then reboot and it should stop right
before entering U-Boot TPL again.
Signed-off-by: Quentin Schulz <quentin.schulz@...rry.de>
---
Changes in v3:
- (hopefully) fixed missing bitfield.h include in driver as reported by
Intel's kernel test robot,
- removed dt-binding header file,
- removed mentions to constants that used to be in dt-binding header
file since they aren't anymore,
- added (patch 3) header file in arch/arm64/boot/dts/rockchip to make
the value of rockchip,reset-mode easier to understand when reading the
device tree,
- Link to v2: https://lore.kernel.org/r/20250605-rk8xx-rst-fun-v2-0-143d190596dd@cherry.de
Changes in v2:
- moved rst_fun variable declaration out of the switch-case,
- initialized rst_fun variable to make kernel test robot happy even
though the variable wouldn't be used uninitialized due to breaking
before using it,
- renamed rockchip,rst-fun to rockchip,reset-mode
- rewrote rockchip,reset-mode binding description to not mention the
relation to registers or register values,
- added binding header file to make it easier to understand what the
mode is when reading a Device Tree without having to read the binding,
- Link to v1: https://lore.kernel.org/r/20250526-rk8xx-rst-fun-v1-0-ea894d9474e0@cherry.de
---
Quentin Schulz (5):
dt-bindings: mfd: rk806: allow to customize PMIC reset mode
mfd: rk8xx-core: allow to customize RK806 reset mode
arm64: dts: rockchip: add header for RK8XX PMIC constants
arm64: dts: rockchip: force PMIC reset behavior to restart PMU on RK3588 Jaguar
arm64: dts: rockchip: force PMIC reset behavior to restart PMU on RK3588 Tiger
.../devicetree/bindings/mfd/rockchip,rk806.yaml | 21 +++++++++++++++++++++
arch/arm64/boot/dts/rockchip/rk3588-jaguar.dts | 2 ++
arch/arm64/boot/dts/rockchip/rk3588-tiger.dtsi | 2 ++
arch/arm64/boot/dts/rockchip/rk8xx.h | 18 ++++++++++++++++++
drivers/mfd/rk8xx-core.c | 15 +++++++++++++++
include/linux/mfd/rk808.h | 2 ++
6 files changed, 60 insertions(+)
---
base-commit: 52da431bf03b5506203bca27fe14a97895c80faf
change-id: 20250526-rk8xx-rst-fun-f261c40b6d0c
Best regards,
--
Quentin Schulz <quentin.schulz@...rry.de>
Powered by blists - more mailing lists