Patch "arm64: dts: mediatek: mt8192-asurada: Add off-on-delay-us for pp3300_mipibrdg" has been added to the 6.10-stable tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This is a note to let you know that I've just added the patch titled

    arm64: dts: mediatek: mt8192-asurada: Add off-on-delay-us for pp3300_mipibrdg

to the 6.10-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     arm64-dts-mediatek-mt8192-asurada-add-off-on-delay-u.patch
and it can be found in the queue-6.10 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.



commit e679f80d9e90c1ec10dfbbb8a2a8583c91f9c810
Author: Pin-yen Lin <treapking@xxxxxxxxxxxx>
Date:   Thu May 2 23:39:51 2024 +0800

    arm64: dts: mediatek: mt8192-asurada: Add off-on-delay-us for pp3300_mipibrdg
    
    [ Upstream commit 897a7edba9330974726c564dfdbf4fb5e203b9ac ]
    
    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.
    
    Or in this sequence:
    1. Bridge on: panel goes off. The regulator doesn't turn off (refcount=1),
       but the .unprepared_time in panel_edp is still updated.
    2. Bridge off, regulator goes off (refcount=0).
    3. Panel on, but the panel driver uses the wrong .unprepared_time to check
       the unprepare delay.
    
    Fixes: f9f00b1f6b9b ("arm64: dts: mediatek: asurada: Add display regulators")
    Signed-off-by: Pin-yen Lin <treapking@xxxxxxxxxxxx>
    Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxx>
    Link: https://lore.kernel.org/r/20240502154455.3427793-1-treapking@xxxxxxxxxxxx
    Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxx>
    Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>

diff --git a/arch/arm64/boot/dts/mediatek/mt8192-asurada.dtsi b/arch/arm64/boot/dts/mediatek/mt8192-asurada.dtsi
index 7a704246678f0..08d71ddf36683 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 */




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux