On 26 June 2014 13:11, Javier Martinez Canillas <javier@xxxxxxxxxxxx> wrote: > Hello Rahul, > > On Thu, Jun 26, 2014 at 7:26 AM, Rahul Sharma <rahul.sharma@xxxxxxxxxxx> wrote: >> Adding regulators for HDMI for exynos5800 based Peach-pi board. >> >> Signed-off-by: Rahul Sharma <rahul.sharma@xxxxxxxxxxx> >> --- >> Based on Kukjin's for-next branch. >> Depends on: >> 1) PMIC addition to Peach-pi: http://www.spinics.net/lists/linux-samsung-soc/msg33166.html >> 2) tps65090 addition to Peach-pi: http://www.spinics.net/lists/linux-samsung-soc/msg32464.html >> >> arch/arm/boot/dts/exynos5800-peach-pi.dts | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/arch/arm/boot/dts/exynos5800-peach-pi.dts b/arch/arm/boot/dts/exynos5800-peach-pi.dts >> index 2c2c137..30d1a26 100644 >> --- a/arch/arm/boot/dts/exynos5800-peach-pi.dts >> +++ b/arch/arm/boot/dts/exynos5800-peach-pi.dts >> @@ -127,6 +127,10 @@ >> pinctrl-names = "default"; >> pinctrl-0 = <&hdmi_hpd_irq>; >> ddc = <&i2c_2>; >> + hdmi-en-supply = <&tps65090_fet7>; >> + vdd-supply = <&ldo8_reg>; >> + vdd_osc-supply = <&ldo10_reg>; >> + vdd_pll-supply = <&ldo8_reg>; > > You had a line break between the ddc and the supplies on your patch > for Peach pit, can you please add it here as well? > > I think that is more readable and also we may later factor out common > DTS fragments on these two boards in .dtsi include files so is better > to keep the diff as small as possible. > >> }; >> >> &hsi2c_7 { >> -- >> 1.7.9.5 >> > > With that trivial change: Sure. Thanks Javier. > > Reviewed-by: Javier Martinez Canillas <javier.martinez@xxxxxxxxxxxxxxx> > > Best regards, > Javier -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html