On Tuesday, September 03, 2013 3:52 PM, Tushar Behera wrote: > On 3 September 2013 07:44, Jingoo Han <jg1.han@xxxxxxxxxxx> wrote: > > According to datasheet, 'lcd_en' node should use gpf0-2, > > instead of gpe3-4. gpe3-4 is used for MDM_DATA[4]; thus, > > it should be fixed. > > > > Signed-off-by: Jingoo Han <jg1.han@xxxxxxxxxxx> > > --- > > arch/arm/boot/dts/exynos4210-pinctrl.dtsi | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/arch/arm/boot/dts/exynos4210-pinctrl.dtsi b/arch/arm/boot/dts/exynos4210-pinctrl.dtsi > > index a7c2128..da5dd8d 100644 > > --- a/arch/arm/boot/dts/exynos4210-pinctrl.dtsi > > +++ b/arch/arm/boot/dts/exynos4210-pinctrl.dtsi > > @@ -374,7 +374,7 @@ > > }; > > > > lcd_en: lcd-en { > > - samsung,pins = "gpe3-4"; > > + samsung,pins = "gpf0-2"; > > samsung,pin-function = <2>; > > samsung,pin-pud = <0>; > > samsung,pin-drv = <0>; > > With this change, the LCD display stops working on Origen board. The smdkv310 board worked properly. gpe3-4 seems to be necessary for Origen board. Can you describe why gpe3-4 is necessary for Origen board? According to the datasheet, pin-function of gpe3-4 is defined as MDM_DATA[4]. So, the node name 'lcd_en' is wrong or confusing. > > gpf0-{0,1,2,3} are defined under lcd_clk. Then will you test the following on Origen board? +++ b/arch/arm/boot/dts/exynos4210-origen.dts @@ -293,7 +293,7 @@ }; fimd@11c00000 { - pinctrl-0 = <&lcd_en &lcd_clk &lcd_data24 &pwm0_out>; + pinctrl-0 = <&lcd_clk &lcd_data24 &pwm0_out>; Best regards, Jingoo Han -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html