Re: [PATCH] ARM: dts: Fix gpio pin of lcd_en node for exynos4210

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

 



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




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux