ARM: tegra: mark panel regulator as enabled on boot
diff mbox

Message ID 1d6749b2aa21112cb73e162a73e9e0e36b32a677.1481273581.git.marvin24@gmx.de
State Accepted
Headers show

Commit Message

Marc Dietrich Dec. 9, 2016, 9:20 a.m. UTC
Current u-boot enables the display already. Fixes sporadic panel
initialization failure with tegra_defconfig.

Signed-off-by: Marc Dietrich <marvin24@gmx.de>
---
 arch/arm/boot/dts/tegra20-paz00.dts | 1 +
 1 file changed, 1 insertion(+)

Comments

Misha Komarovskiy Dec. 19, 2016, 8:20 p.m. UTC | #1
Hi,

On Fri, Dec 9, 2016 at 12:20 PM, Marc Dietrich <marvin24@gmx.de> wrote:
> Current u-boot enables the display already. Fixes sporadic panel
> initialization failure with tegra_defconfig.
>
> Signed-off-by: Marc Dietrich <marvin24@gmx.de>
> ---

Tested on 4.9.0, 4.8.13 and 4.4.39, display starts every time with this patch.
Thank you.

Tested-by: Misha Komarovskiy <zombah@gmail.com>

>  arch/arm/boot/dts/tegra20-paz00.dts | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/boot/dts/tegra20-paz00.dts b/arch/arm/boot/dts/tegra20-paz00.dts
> index 4e361a8c167e..1be690cce9aa 100644
> --- a/arch/arm/boot/dts/tegra20-paz00.dts
> +++ b/arch/arm/boot/dts/tegra20-paz00.dts
> @@ -570,6 +570,7 @@
>                         regulator-min-microvolt = <3300000>;
>                         regulator-max-microvolt = <3300000>;
>                         gpio = <&gpio TEGRA_GPIO(A, 4) GPIO_ACTIVE_HIGH>;
> +                       regulator-boot-on;
>                         enable-active-high;
>                 };
>         };
> --
> 2.11.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
Thierry Reding Jan. 25, 2017, 8:15 a.m. UTC | #2
On Fri, Dec 09, 2016 at 10:20:38AM +0100, Marc Dietrich wrote:
> Current u-boot enables the display already. Fixes sporadic panel
> initialization failure with tegra_defconfig.
> 
> Signed-off-by: Marc Dietrich <marvin24@gmx.de>
> ---
>  arch/arm/boot/dts/tegra20-paz00.dts | 1 +
>  1 file changed, 1 insertion(+)

Applied, thanks. I did move the regulator-boot-on up a line so that the
regulator- and GPIO-related properties remain grouped together.

Thierry

Patch
diff mbox

diff --git a/arch/arm/boot/dts/tegra20-paz00.dts b/arch/arm/boot/dts/tegra20-paz00.dts
index 4e361a8c167e..1be690cce9aa 100644
--- a/arch/arm/boot/dts/tegra20-paz00.dts
+++ b/arch/arm/boot/dts/tegra20-paz00.dts
@@ -570,6 +570,7 @@ 
 			regulator-min-microvolt = <3300000>;
 			regulator-max-microvolt = <3300000>;
 			gpio = <&gpio TEGRA_GPIO(A, 4) GPIO_ACTIVE_HIGH>;
+			regulator-boot-on;
 			enable-active-high;
 		};
 	};