From 6013ab707048d8eefdf91da18319d29609e91004 Mon Sep 17 00:00:00 2001 From: Cody Xie Date: Sat, 16 Sep 2023 10:13:26 +0800 Subject: [PATCH] arm64: dts: rockchip: rk3588-vehicle-evb-v22: Add delay for all camera related regulators It will be more safer for regulator consumer devices to power on after their power suppliers are stable. And regulator consumer devices need to completely shut down, then power up for a reset cycle. Therefor, regulator providers need to be make sure shutdown completely. - startup-delay-us can be used to wait regulator providers be powered on good, according their power-up time. - off-on-delay-us can be used to wait regulator providers be shut down completely. 1. cameraN_vcc12v_buck have a approximate ~1ms power on rise time and ~13-~14ms power off fall time 2. vcc5v0_buck uses MPQ4317 which have a approximate ~1.5ms EN(on)->PG(high) delay and ~0.8ms EN(off)->PG(low) Change-Id: I8f23a6a4b53c8760f62c64758a481ad86be4da16 Signed-off-by: Cody Xie --- .../boot/dts/rockchip/rk3588-vehicle-evb-v22.dts | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/arch/arm64/boot/dts/rockchip/rk3588-vehicle-evb-v22.dts b/arch/arm64/boot/dts/rockchip/rk3588-vehicle-evb-v22.dts index c2986168a6c3..a214764f4d6c 100644 --- a/arch/arm64/boot/dts/rockchip/rk3588-vehicle-evb-v22.dts +++ b/arch/arm64/boot/dts/rockchip/rk3588-vehicle-evb-v22.dts @@ -27,6 +27,8 @@ vin-supply = <&vcc12v_dcin>; pinctrl-names = "default"; pinctrl-0 = <&vcc5v0_buck_en>; + startup-delay-us = <2500>; + off-on-delay-us = <1500>; regulator-state-mem { regulator-on-in-suspend; regulator-suspend-microvolt = <5000000>; @@ -156,6 +158,8 @@ regulator-max-microvolt = <12000000>; enable-active-high; gpio = <&i2c5_nca9539_gpio 6 GPIO_ACTIVE_HIGH>; + startup-delay-us = <2000>; + off-on-delay-us = <16000>; vin-supply = <&vcc12v_dcin>; regulator-state-mem { regulator-off-in-suspend; @@ -172,6 +176,8 @@ regulator-max-microvolt = <12000000>; enable-active-high; gpio = <&i2c5_nca9539_gpio 7 GPIO_ACTIVE_HIGH>; + startup-delay-us = <2000>; + off-on-delay-us = <16000>; vin-supply = <&vcc12v_dcin>; regulator-state-mem { regulator-off-in-suspend; @@ -188,6 +194,8 @@ regulator-max-microvolt = <12000000>; enable-active-high; gpio = <&i2c5_nca9539_gpio 8 GPIO_ACTIVE_HIGH>; + startup-delay-us = <2000>; + off-on-delay-us = <16000>; vin-supply = <&vcc12v_dcin>; regulator-state-mem { regulator-off-in-suspend; @@ -204,6 +212,8 @@ regulator-max-microvolt = <12000000>; enable-active-high; gpio = <&i2c5_nca9539_gpio 9 GPIO_ACTIVE_HIGH>; + startup-delay-us = <2000>; + off-on-delay-us = <16000>; vin-supply = <&vcc12v_dcin>; regulator-state-mem { regulator-off-in-suspend; @@ -220,6 +230,8 @@ regulator-max-microvolt = <5000000>; enable-active-high; gpio = <&i2c5_nca9539_gpio 10 GPIO_ACTIVE_HIGH>; + startup-delay-us = <2000>; + off-on-delay-us = <16000>; vin-supply = <&vcc5v0_usb>; }; @@ -232,6 +244,8 @@ regulator-max-microvolt = <5000000>; enable-active-high; gpio = <&i2c5_nca9539_gpio 11 GPIO_ACTIVE_HIGH>; + startup-delay-us = <2000>; + off-on-delay-us = <16000>; vin-supply = <&vcc5v0_usb>; };