Hi Benoit,

On 03/09/2013 16:55, Benoit Parrot wrote:
Hi,

I have not received any feedback on this patch.
It has been pending since the end of June (first post).
Can I get an estimate when it will be included/accepted?

It looks good to me beside a minor comment below.

Could you just rebase it on my lastest git://git.kernel.org/pub/scm/linux/kernel/git/bcousson/linux-omap-dt.git for_3.13/dts branch because it conflicts with the cleanup done by Javier.


On Fri, Aug 23, 2013 at 11:13:56AM -0500, Benoit Parrot wrote:
Add LCDC device node in DT for am33xx
Add LCDC and Panel info in DT for am335x-evm

Changes in v3:
- rebase to 3.11-rc6

Changes in v2:
- remove redundant/unnecessary SoC specific setting in the board dts

Signed-off-by: Benoit Parrot <bpar...@ti.com>
---
  arch/arm/boot/dts/am335x-evm.dts |   72 ++++++++++++++++++++++++++++++++++++++
  arch/arm/boot/dts/am33xx.dtsi    |    9 +++++
  2 files changed, 81 insertions(+)

diff --git a/arch/arm/boot/dts/am335x-evm.dts b/arch/arm/boot/dts/am335x-evm.dts
index 3aee1a4..b0703f1 100644
--- a/arch/arm/boot/dts/am335x-evm.dts
+++ b/arch/arm/boot/dts/am335x-evm.dts
@@ -149,6 +149,40 @@
                                0x14c (PIN_INPUT_PULLDOWN | MUX_MODE7)
                        >;
                };
+
+               lcd_pins_s0: lcd_pins_s0 {
+                       pinctrl-single,pins = <
+                               0x20 0x01       /* gpmc_ad8.lcd_data16, OUTPUT 
| MODE1 */
+                               0x24 0x01       /* gpmc_ad9.lcd_data17, OUTPUT 
| MODE1 */
+                               0x28 0x01       /* gpmc_ad10.lcd_data18, OUTPUT 
| MODE1 */
+                               0x2c 0x01       /* gpmc_ad11.lcd_data19, OUTPUT 
| MODE1 */
+                               0x30 0x01       /* gpmc_ad12.lcd_data20, OUTPUT 
| MODE1 */
+                               0x34 0x01       /* gpmc_ad13.lcd_data21, OUTPUT 
| MODE1 */
+                               0x38 0x01       /* gpmc_ad14.lcd_data22, OUTPUT 
| MODE1 */
+                               0x3c 0x01       /* gpmc_ad15.lcd_data23, OUTPUT 
| MODE1 */
+                               0xa0 0x00       /* lcd_data0.lcd_data0, OUTPUT 
| MODE0 */
+                               0xa4 0x00       /* lcd_data1.lcd_data1, OUTPUT 
| MODE0 */
+                               0xa8 0x00       /* lcd_data2.lcd_data2, OUTPUT 
| MODE0 */
+                               0xac 0x00       /* lcd_data3.lcd_data3, OUTPUT 
| MODE0 */
+                               0xb0 0x00       /* lcd_data4.lcd_data4, OUTPUT 
| MODE0 */
+                               0xb4 0x00       /* lcd_data5.lcd_data5, OUTPUT 
| MODE0 */
+                               0xb8 0x00       /* lcd_data6.lcd_data6, OUTPUT 
| MODE0 */
+                               0xbc 0x00       /* lcd_data7.lcd_data7, OUTPUT 
| MODE0 */
+                               0xc0 0x00       /* lcd_data8.lcd_data8, OUTPUT 
| MODE0 */
+                               0xc4 0x00       /* lcd_data9.lcd_data9, OUTPUT 
| MODE0 */
+                               0xc8 0x00       /* lcd_data10.lcd_data10, 
OUTPUT | MODE0 */
+                               0xcc 0x00       /* lcd_data11.lcd_data11, 
OUTPUT | MODE0 */
+                               0xd0 0x00       /* lcd_data12.lcd_data12, 
OUTPUT | MODE0 */
+                               0xd4 0x00       /* lcd_data13.lcd_data13, 
OUTPUT | MODE0 */
+                               0xd8 0x00       /* lcd_data14.lcd_data14, 
OUTPUT | MODE0 */
+                               0xdc 0x00       /* lcd_data15.lcd_data15, 
OUTPUT | MODE0 */
+                               0xe0 0x00       /* lcd_vsync.lcd_vsync, OUTPUT 
| MODE0 */
+                               0xe4 0x00       /* lcd_hsync.lcd_hsync, OUTPUT 
| MODE0 */
+                               0xe8 0x00       /* lcd_pclk.lcd_pclk, OUTPUT | 
MODE0 */
+                               0xec 0x00       /* 
lcd_ac_bias_en.lcd_ac_bias_en, OUTPUT | MODE0 */
+                       >;
+               };
+
        };

        ocp {
@@ -311,6 +345,10 @@
                                };
                        };
                };
+
+               lcdc: lcdc@4830e000 {
+                       status = "okay";
+               };
        };

        vbat: fixedregulator@0 {
@@ -374,6 +412,40 @@
                brightness-levels = <0 51 53 56 62 75 101 152 255>;
                default-brightness-level = <8>;
        };
+
+       panel {
+               compatible = "ti,tilcdc,panel";

Nit: There are a lots of TI here :-)
Could you just name it: ti,lcdc,panel ?

Thanks,
Benoit

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to