Re: [PATCH] ARM: dts: omap5-igep0050: Correct hdmi regulator

[Nishanth Menon]

On 05/12/2016 11:57 AM, Tony Lindgren wrote:
> * Eduard Gavin <egavin@xxxxxxxxxxx> [160509 07:43]:
>> Dear Nishanth,
>>   In igepV5 the LDO7 is connected to:
>>     VDDA_DSIPORTA - ball AA33
>>     VDDA_DSIPORTC - ball AE33
>>     VDDA_HDMI - ball AN25
>>   LDO4 is connected to:
>>     VPP1 - ball AD9
> 
> Thanks for the info, here's an updated patch with comments and now
> using 1.8V instead of the 2.0V omap5-board-common.dtsi.
> 
> Regards,
> 
> Tony
> 
> 8< -----------------
> From: Tony Lindgren <tony@xxxxxxxxxxx>
> Date: Mon, 9 May 2016 07:51:00 -0700
> Subject: [PATCH] ARM: dts: Fix ldo7 source for HDMI on igepv5
> 
> Fix ldo7 source for HDMI on igepv5.
> 
> Suggested-by: Peter Ujfalusi <peter.ujfalusi@xxxxxx>
> Signed-off-by: Tony Lindgren <tony@xxxxxxxxxxx>
> 
> --- a/arch/arm/boot/dts/omap5-igep0050.dts
> +++ b/arch/arm/boot/dts/omap5-igep0050.dts
> @@ -35,6 +35,16 @@
>  	};
>  };
>  
> +/*
> + * LDO7 is used for HDMI: VDDA_DSIPORTA - ball AA33, VDDA_DSIPORTC - ball AE33
> + * VDDA_HDMI - ball AN25. LDO4 is: VPP1 - ball AD9.

We should set ldo4_reg as 2v as well (default in omap5-board-common is
1.8v). VPP1 should be 2v to prevent efuse from being messed up.

> + */
> +&ldo7_reg {
> +	status = "okay";
> +	regulator-min-microvolt = <1800000>;
> +	regulator-max-microvolt = <1800000>;
> +};
> +
>  &omap5_pmx_core {
>  	i2c4_pins: pinmux_i2c4_pins {
>  		pinctrl-single,pins = <
> 


-- 
Regards,
Nishanth Menon