RE: [PATCH v2 5/6] davinci vpbe: platform specific additions

2010-11-29 Thread Hadli, Manjunath
On Sat, Nov 27, 2010 at 20:52:14, Muralidharan Karicheri wrote: +static int dm644x_set_if_config(enum v4l2_mbus_pixelcode pixcode) { +       unsigned int val = 0; +       int ret = 0; + +       switch (pixcode) { +       case V4L2_MBUS_FMT_FIXED: +             /* Analog out.do

RE: [PATCH v2 5/6] davinci vpbe: platform specific additions

2010-11-29 Thread Hadli, Manjunath
Resending the reply with proper indentation.. On Mon, Nov 29, 2010 at 21:27:39, Hadli, Manjunath wrote: On Sat, Nov 27, 2010 at 20:52:14, Muralidharan Karicheri wrote: +static int dm644x_set_if_config(enum v4l2_mbus_pixelcode pixcode) { +       unsigned int val = 0; +       int ret = 0;

Re: [PATCH v2 5/6] davinci vpbe: platform specific additions

2010-11-27 Thread Muralidharan Karicheri
+static int dm644x_set_if_config(enum v4l2_mbus_pixelcode pixcode) +{ +       unsigned int val = 0; +       int ret = 0; + +       switch (pixcode) { +       case V4L2_MBUS_FMT_FIXED: +             /* Analog out.do nothing */ +             break; +       case V4L2_MBUS_FMT_YUYV8_2X8: +  

Re: [PATCH v2 5/6] davinci vpbe: platform specific additions

2010-11-27 Thread Muralidharan Karicheri
Just want to add one more point. Probably it is a good idea to keep this function in venc. Why this function has to be platform specific? On Sat, Nov 27, 2010 at 10:22 AM, Muralidharan Karicheri mkarich...@gmail.com wrote: +static int dm644x_set_if_config(enum v4l2_mbus_pixelcode pixcode) +{ +

[PATCH v2 5/6] davinci vpbe: platform specific additions

2010-11-24 Thread Manjunath Hadli
This patch implements the overall device creation for the Video display driver, and addition of tables for the mode and output list. Signed-off-by: Manjunath Hadli manjunath.ha...@ti.com Signed-off-by: Muralidharan Karicheri m-kariche...@ti.com --- arch/arm/mach-davinci/board-dm644x-evm.c|