Message ID | 20240502154455.3427793-1-treapking@chromium.org (mailing list archive) |
---|---|
State | New |
Headers | show |
Series | [v2] arm64: dts: mediatek: mt8192-asurada: Add off-on-delay-us for pp3300_mipibrdg | expand |
On Thu, 02 May 2024 23:39:51 +0800, Pin-yen Lin wrote: > Set off-on-delay-us to 500000 us for pp3300_mipibrdg to make sure it > complies with the panel's unprepare delay (the time to power down > completely) of the power sequence. Explicit configuration on the > regulator node is required because mt8192-asurada uses the same power > supply for the panel and the anx7625 DP bridge. > > For example, the power sequence could be violated in this sequence: > 1. Bridge on: panel goes off, but regulator doesn't turn off (refcount=1). > 2. Bridge off: regulator turns off (refcount=0). > 3. Bridge resume -> regulator turns on but the bridge driver doesn't > check the delay. > > [...] Applied to v6.10-next/dts64, thanks! [1/1] arm64: dts: mediatek: mt8192-asurada: Add off-on-delay-us for pp3300_mipibrdg commit: 476d7adf84deb884e0f8f74417bf5249ab9a95b3 Cheers, Angelo
diff --git a/arch/arm64/boot/dts/mediatek/mt8192-asurada.dtsi b/arch/arm64/boot/dts/mediatek/mt8192-asurada.dtsi index 7a704246678f..08d71ddf3668 100644 --- a/arch/arm64/boot/dts/mediatek/mt8192-asurada.dtsi +++ b/arch/arm64/boot/dts/mediatek/mt8192-asurada.dtsi @@ -147,6 +147,7 @@ pp3300_mipibrdg: regulator-3v3-mipibrdg { regulator-boot-on; gpio = <&pio 127 GPIO_ACTIVE_HIGH>; vin-supply = <&pp3300_g>; + off-on-delay-us = <500000>; }; /* separately switched 3.3V power rail */