[PATCH v2 3/5] arm64: dts: rockchip: force PMIC reset behavior to restart PMU on RK3588 Jaguar

[Kever Yang]

On 2025/8/13 22:07, Quentin Schulz wrote:
> From: Quentin Schulz <quentin.schulz at cherry.de>
>
> The bootloader for RK3588 Jaguar currently forces the PMIC reset
> behavior (stored in RST_FUN bitfield in register SYS_CFG3 of the PMIC)
> to 0b1X which is incorrect for our devices.
>
> It is required to restart the PMU as otherwise the companion
> microcontroller cannot detect the PMIC (and by extension the full
> product and main SoC) being rebooted which is an issue as that is used
> to reset a few things like the PWM beeper and watchdogs.
>
> Let's add the new rockchip,reset-mode property to make sure the PMIC
> reset behavior is the expected one.
>
> Signed-off-by: Quentin Schulz <quentin.schulz at cherry.de>
> Link: https://lore.kernel.org/r/20250627-rk8xx-rst-fun-v4-4-ce05d041b45f@cherry.de
> Signed-off-by: Heiko Stuebner <heiko at sntech.de>
>
> [ upstream commit: ee907113430aa02a8202c91bb574c385ecc28aa2 ]
>
> (cherry picked from commit 8bd14566b75f9409de703a0d2f9a0704b71a7ebe)
Reviewed-by: Kever Yang <kever.yang at rock-chips.com>

Thanks,
- Kever
> ---
>   dts/upstream/src/arm64/rockchip/rk3588-jaguar.dts | 2 ++
>   1 file changed, 2 insertions(+)
>
> diff --git a/dts/upstream/src/arm64/rockchip/rk3588-jaguar.dts b/dts/upstream/src/arm64/rockchip/rk3588-jaguar.dts
> index ebe77cdd24e803b00fb848dc81258909472290f1..176925d0a1a809d1e2500f5e5efbbfa6a6d0bd42 100644
> --- a/dts/upstream/src/arm64/rockchip/rk3588-jaguar.dts
> +++ b/dts/upstream/src/arm64/rockchip/rk3588-jaguar.dts
> @@ -10,6 +10,7 @@
>   #include <dt-bindings/pinctrl/rockchip.h>
>   #include <dt-bindings/soc/rockchip,vop2.h>
>   #include <dt-bindings/usb/pd.h>
> +#include "rk8xx.h"
>   #include "rk3588.dtsi"
>   
>   / {
> @@ -693,6 +694,7 @@
>   		vcc13-supply = <&vcc_1v1_nldo_s3>;
>   		vcc14-supply = <&vcc_1v1_nldo_s3>;
>   		vcca-supply = <&vcc5v0_sys>;
> +		rockchip,reset-mode = <RK806_RESTART>;
>   
>   		rk806_dvs1_null: dvs1-null-pins {
>   			pins = "gpio_pwrctrl1";
>