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

[Kever Yang]

On 2025/8/13 22:07, Quentin Schulz wrote:
> From: Quentin Schulz <quentin.schulz at cherry.de>
>
> The bootloader for RK3588 Tiger 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-5-ce05d041b45f@cherry.de
> Signed-off-by: Heiko Stuebner <heiko at sntech.de>
>
> [ upstream commit: e82f642b9821384045915dc30e73df7de8424827 ]
>
> (cherry picked from commit d9c568906be166834f4f977bc7f704176bac5b8a)
Reviewed-by: Kever Yang <kever.yang at rock-chips.com>

Thanks,
- Kever
> ---
>   dts/upstream/src/arm64/rockchip/rk3588-tiger.dtsi | 2 ++
>   1 file changed, 2 insertions(+)
>
> diff --git a/dts/upstream/src/arm64/rockchip/rk3588-tiger.dtsi b/dts/upstream/src/arm64/rockchip/rk3588-tiger.dtsi
> index c4933a08dd1e3c92f3e0747135faf97c5eeca906..b44e89e1bb1599ee70b921598c2eb6fd54614f55 100644
> --- a/dts/upstream/src/arm64/rockchip/rk3588-tiger.dtsi
> +++ b/dts/upstream/src/arm64/rockchip/rk3588-tiger.dtsi
> @@ -6,6 +6,7 @@
>   #include <dt-bindings/gpio/gpio.h>
>   #include <dt-bindings/leds/common.h>
>   #include <dt-bindings/pinctrl/rockchip.h>
> +#include "rk8xx.h"
>   #include "rk3588.dtsi"
>   
>   / {
> @@ -440,6 +441,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";
>