Hi Murali,

> +#if defined(CONFIG_VIDEO_TVP5146) || defined(CONFIG_VIDEO_TVP5146_MODULE)

As Kevin pointed out in Xilinx FPGA patches thread, this #ifdefers are
not needed. Please remove.

> +#include <linux/videodev2.h>
> +#include <media/v4l2-int-device.h>
> +#include <media/tvp5146.h>
> +#endif
>  #include <asm/setup.h>
>  #include <asm/mach-types.h>
>
> @@ -156,6 +160,22 @@ static struct platform_device davinci_fb_device = {
>  };
>  #endif
>
> +#define V4L2_CAPTURE_MASTER "vpfe-capture"
> +#if defined(CONFIG_VIDEO_VPFE_CAPTURE) || \
> +    defined(CONFIG_VIDEO_VPFE_CAPTURE_MODULE)
> +

Ditto. and other places in this patch.



-- 
---Trilok Soni
http://triloksoni.wordpress.com
http://www.linkedin.com/in/triloksoni

_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to